Fix Undefined index: year in vendor/lavary/crunz/src/Event.php (#118)

lavary · Jun 15, 2018 · 634be46 · 634be46
1 parent bae0601
commit 634be46
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/src/Event.php b/src/Event.php
@@ -349,8 +349,8 @@ public function on($date)
         $segments = \array_only($date, \array_flip($this->fieldsPosition));
 
         if ($date['year']) {
-            $this->skip(function () use ($segments) {
-                return (int) date('Y') != $segments['year'];
+            $this->skip(function () use ($date) {
+                return (int) date('Y') != $date['year'];
             });
         }
 

diff --git a/tests/Unit/EventTest.php b/tests/Unit/EventTest.php
@@ -216,6 +216,10 @@ public function testIsDue()
         $e = new Event($this->id, 'php bar');
         $this->assertEquals('0 19 * * 6 *', $e->saturdays()->at('19:00')->timezone('EST')->getExpression());
         $this->assertTrue($e->isDue($timezone));
+
+        $e = new Event($this->id, 'php bar');
+        Carbon::setTestNow(Carbon::create(\date('Y'), 4, 12, 0, 0, 0));
+        $this->assertTrue($e->on('00:00 ' . \date('Y') . '-04-12')->isDue($timezone));
     }
 
     public function testName()