Reemplazada la función everyFirstDatOfMonthAt($hour) por everyDay($da…

…y, $hour) que si pones $day = 1, es lo mesmo que la anterior, pero te da opción a usar otros días.
NeoRazorX · Jun 17, 2023 · 6eb25f0 · 6eb25f0
1 parent 8cdd602
commit 6eb25f0
Show file tree

Hide file tree

Showing 2 changed files with 30 additions and 8 deletions.
diff --git a/Core/Model/CronJob.php b/Core/Model/CronJob.php
@@ -88,14 +88,15 @@ public function every(string $period): bool
         return strtotime($this->date) <= strtotime('-' . $period);
     }
 
-    public function everyDayAt(int $hour, bool $strict = false): bool
+    public function everyDay(int $day, int $hour, bool $strict = false): bool
     {
-        return $this->everyDayAux('today', $hour, $strict);
+        $date = date('Y-m-' . $day);
+        return $this->everyDayAux($date, $hour, $strict);
     }
 
-    public function everyFirstDayOfMonthAt(int $hour, bool $strict): bool
+    public function everyDayAt(int $hour, bool $strict = false): bool
     {
-        return $this->everyDayAux('first day of this month', $hour, $strict);
+        return $this->everyDayAux('today', $hour, $strict);
     }
 
     public function everyFridayAt(int $hour, bool $strict): bool
@@ -171,15 +172,14 @@ private function everyDayAux(string $day, int $hour, bool $strict): bool
             return true;
         }
 
-        // devolvemos true si la última ejecución es anterior a hoy a la hora indicada
-        $last = strtotime($this->date);
-        $start = strtotime($day . ' +' . $hour . ' hours');
-
         // si strict es true, solamente devolvemos true si es la hora exacta
         $end = $strict ?
             strtotime($day . ' +' . $hour . ' hours +59 minutes') :
             strtotime($day . ' +23 hours +59 minutes');
 
+        // devolvemos true si la última ejecución es anterior a hoy a la hora indicada
+        $last = strtotime($this->date);
+        $start = strtotime($day . ' +' . $hour . ' hours');
         $this->start = microtime(true);
         return $last <= $start && $this->start >= $start && $this->start <= $end;
     }

diff --git a/Test/Core/Model/CronJobTest.php b/Test/Core/Model/CronJobTest.php
@@ -110,6 +110,28 @@ public function testEveryFunction(): void
         $this->assertTrue($job->delete());
     }
 
+    public function testEveryDayFunction(): void
+    {
+        $currentDay = date('d') < 28 ? (int)date('d') : 10;
+
+        $job = new CronJob();
+        $job->jobname = 'TestName';
+        $job->pluginname = 'TestPlugin';
+
+        // como nunca se ha ejecutado, si decimos de ejecutar hoy, se ejecutará
+        $this->assertTrue($job->everyDay($currentDay, 1));
+        $this->assertTrue($job->save());
+
+        // como ya se ha ejecutado, si decimos de ejecutar hoy, no se ejecutará
+        $this->assertFalse($job->everyDayAt($currentDay, 1));
+
+        // como ya se ha ejecutado, si decimos de ejecutar mañana, no se ejecutará
+        $this->assertFalse($job->everyDayAt($currentDay + 1, 1));
+
+        // eliminamos
+        $this->assertTrue($job->delete());
+    }
+
     public function testEveryDayAtFunction(): void
     {
         $job = new CronJob();