fix(shops): only check date if shop is timed in active check (lk-arpg#1408)

* fix(shops): shop being displayed as inactive when is_timed_shop is false but date data still exists

* refactor: fix PHP styling

---------

Co-authored-by: perappu <perappu@users.noreply.github.com>

Author: perappu

app/Models/Shop/Shop.php

@@ -200,20 +200,22 @@ public function getMonthsAttribute() {
      * We dont account for is_visible here, as this is used for checking both visible and invisible shop.
      */
     public function getIsActiveCheckAttribute() {
-        if ($this->start_at && $this->start_at > Carbon::now()) {
-            return false;
-        }
-
-        if ($this->end_at && $this->end_at < Carbon::now()) {
-            return false;
-        }
-
-        if ($this->days && !in_array(Carbon::now()->format('l'), $this->days)) {
-            return false;
-        }
-
-        if ($this->months && !in_array(Carbon::now()->format('F'), $this->months)) {
-            return false;
+        if ($this->is_timed_shop) {
+            if ($this->start_at && $this->start_at > Carbon::now()) {
+                return false;
+            }
+
+            if ($this->end_at && $this->end_at < Carbon::now()) {
+                return false;
+            }
+
+            if ($this->days && !in_array(Carbon::now()->format('l'), $this->days)) {
+                return false;
+            }
+
+            if ($this->months && !in_array(Carbon::now()->format('F'), $this->months)) {
+                return false;
+            }
         }
 
         if (!$this->is_timed_shop && !$this->is_active) {