Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
uya opened this issue Jul 20, 2017 · 1 comment
Open

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

uya opened this issue Jul 20, 2017 · 1 comment

Comments

@uya
Copy link

uya commented Jul 20, 2017

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);
         }
@brtriver
Copy link
Owner

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

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants