dbup icon indicating copy to clipboard operation
dbup copied to clipboard

statusで表示される日時について

Open uya opened this issue 8 years ago • 1 comments

status時に表示される日時が、applied内のタイムスタンプではなく、sql内のタイムスタンプになっている為、「クエリ実行日時」ではなく「クエリ作成日時」になっていました。

作成日時はsqlディレクトリ内をls -lする事で見られると思うので、下記の修正にてクエリの実行日時が表示されるようにしました。

余計な事とは思いますが、よろしければご反映下さい。

diff --git a/src/Dbup/Application.php b/src/Dbup/Application.php
index 349e1b7..82807ab 100644
--- a/src/Dbup/Application.php
+++ b/src/Dbup/Application.php
@@ -192,16 +192,17 @@ EOL;
         $isApplied = function($file) use ($appliedFiles){
             foreach ($appliedFiles as $appliedFile) {
                 if ($appliedFile->getFileName() === $file->getFileName()){
-                    return true;
+                    return $appliedFile->getMTime();
                 }
             }
-            return false;
+            return null;
         };

         $statuses = [];

         foreach($files as $file){
-            $appliedAt = $isApplied($file)? date('Y-m-d H:i:s', $file->getMTime()): "";
+            $_appliedAt = $isApplied($file);
+            $appliedAt = !empty($_appliedAt) ? date('Y-m-d H:i:s', $_appliedAt): "";
             $statuses[] = new Status($appliedAt, $file);
         }

uya avatar Jul 20 '17 08:07 uya

返事がおそくなりすいません。issueありがとうございます。手元でも確認してみます。

I'm sorry for my replay late. Thank you for this report and code. I will check and fix this bug :)

brtriver avatar Aug 30 '17 07:08 brtriver