small fixes and don't allow 0.0.0.0 or :: as ip

Author: Boy132

subdomains/src/Filament/Server/Resources/Subdomains/SubdomainResource.php

@@ -37,7 +37,7 @@ public static function canAccess(): bool
         /** @var Server $server */
         $server = Filament::getTenant();
 
-        return parent::canAccess() && $server->allocation && CloudflareDomain::count() > 0;
+        return parent::canAccess() && $server->allocation && $server->allocation->ip !== '0.0.0.0' && $server->allocation->ip !== '::' && CloudflareDomain::count() > 0;
     }
 
     public static function getNavigationLabel(): string

subdomains/src/Models/CloudflareDomain.php

@@ -15,6 +15,7 @@ class CloudflareDomain extends Model
 {
     protected $fillable = [
         'name',
+        'cloudflare_id',
     ];
 
     protected static function boot(): void
@@ -42,7 +43,7 @@ public function fetchCloudflareId(): void
 
             if (count($zones) > 0) {
                 $this->update([
-                    'cloudflare_id' => $zones[0]->id,
+                    'cloudflare_id' => $zones[0]['id'],
                 ]);
             }
         }

subdomains/src/Models/Subdomain.php

@@ -24,6 +24,7 @@ class Subdomain extends Model implements HasLabel
     protected $fillable = [
         'name',
         'record_type',
+        'cloudflare_id',
         'domain_id',
         'server_id',
     ];
@@ -47,12 +48,12 @@ protected static function boot(): void
 
     public function domain(): BelongsTo
     {
-        return $this->BelongsTo(CloudflareDomain::class, 'domain_id');
+        return $this->belongsTo(CloudflareDomain::class, 'domain_id');
     }
 
     public function server(): BelongsTo
     {
-        return $this->BelongsTo(Server::class);
+        return $this->belongsTo(Server::class);
     }
 
     public function getLabel(): string|Htmlable|null
@@ -62,42 +63,46 @@ public function getLabel(): string|Htmlable|null
 
     protected function createOnCloudflare(): void
     {
-        if (!$this->server->allocation || $this->cloudflare_id) {
+        if (!$this->server->allocation || $this->server->allocation->ip === '0.0.0.0' || $this->server->allocation->ip === '::') {
             return;
         }
 
-        $response = Http::cloudflare()->post("zones/{$this->domain->cloudflare_id}/dns_records", [
-            'name' => $this->name,
-            'ttl' => 120,
-            'type' => $this->record_type,
-            'comment' => 'Created by Pelican Subdomains plugin',
-            'content' => $this->server->allocation->ip,
-            'proxied' => false,
-        ])->json();
-
-        if ($response['success']) {
-            $dnsRecord = $response['result'];
-
-            $this->updateQuietly([
-                'cloudflare_id' => $dnsRecord->id,
-            ]);
+        if (!$this->cloudflare_id) {
+            $response = Http::cloudflare()->post("zones/{$this->domain->cloudflare_id}/dns_records", [
+                'name' => $this->name,
+                'ttl' => 120,
+                'type' => $this->record_type,
+                'comment' => 'Created by Pelican Subdomains plugin',
+                'content' => $this->server->allocation->ip,
+                'proxied' => false,
+            ])->json();
+
+            if ($response['success']) {
+                $dnsRecord = $response['result'];
+
+                $this->updateQuietly([
+                    'cloudflare_id' => $dnsRecord['id'],
+                ]);
+            }
         }
     }
 
     protected function updateOnCloudflare(): void
     {
-        if (!$this->server->allocation || !$this->cloudflare_id) {
+        if (!$this->server->allocation || $this->server->allocation->ip === '0.0.0.0' || $this->server->allocation->ip === '::') {
             return;
         }
 
-        Http::cloudflare()->patch("zones/{$this->domain->cloudflare_id}/dns_records/{$this->cloudflare_id}", [
-            'name' => $this->name,
-            'ttl' => 120,
-            'type' => $this->record_type,
-            'comment' => 'Created by Pelican Subdomains plugin',
-            'content' => $this->server->allocation->ip,
-            'proxied' => false,
-        ]);
+        if ($this->cloudflare_id) {
+            Http::cloudflare()->patch("zones/{$this->domain->cloudflare_id}/dns_records/{$this->cloudflare_id}", [
+                'name' => $this->name,
+                'ttl' => 120,
+                'type' => $this->record_type,
+                'comment' => 'Created by Pelican Subdomains plugin',
+                'content' => $this->server->allocation->ip,
+                'proxied' => false,
+            ]);
+        }
     }
 
     protected function deleteOnCloudflare(): void