refactor: simplify unique and required validation rule checks and optimize cleaning logic

Replace `toCollection()` calls with `has()` for validation rule checks, streamlining the logic. Simplify cleaning invalid rules using `keys()->intersect` and `except`.

Author: ManukMinasyan

resources/views/livewire/manage-fields-table.blade.php

@@ -88,7 +88,7 @@ class="h-4 w-4 text-gray-400 shrink-0"/>
                             <div class="px-3 py-3 text-sm text-gray-600 dark:text-gray-400 truncate">
                                 @if($field->settings?->unique_per_entity_type)
                                     {{ __('custom-fields::custom-fields.common.unique') }}
-                                @elseif($field->validation_rules?->toCollection()->first(fn($rule) => $rule->name === 'required'))
+                                @elseif($field->validation_rules?->has('required'))
                                     {{ __('custom-fields::custom-fields.common.required') }}
                                 @endif
                             </div>
@@ -178,7 +178,7 @@ class="h-4 w-4 text-gray-400 shrink-0"/>
                                     <div class="px-3 py-3 text-sm text-gray-600 dark:text-gray-400 opacity-60 truncate">
                                         @if($field->settings?->unique_per_entity_type)
                                             {{ __('custom-fields::custom-fields.common.unique') }}
-                                        @elseif($field->validation_rules?->toCollection()->first(fn($rule) => $rule->name === 'required'))
+                                        @elseif($field->validation_rules?->has('required'))
                                             {{ __('custom-fields::custom-fields.common.required') }}
                                         @endif
                                     </div>

src/Console/Commands/Upgrade/Steps/CleanMultiValueValidationRulesStep.php

@@ -69,23 +69,19 @@ public function execute(bool $dryRun, Command $command): UpgradeStepResult
         $failed = 0;
 
         foreach ($affectedFields as $field) {
-            $rules = $field->validation_rules?->toCollection() ?? collect();
+            $rules = $field->validation_rules ?? collect();
 
-            $invalidRules = $rules->filter(
-                fn ($rule): bool => in_array($rule->name, self::STRING_ONLY_RULES, true)
-            );
+            $invalidKeys = $rules->keys()->intersect(self::STRING_ONLY_RULES);
 
-            if ($invalidRules->isEmpty()) {
+            if ($invalidKeys->isEmpty()) {
                 continue;
             }
 
-            $ruleNames = $invalidRules->pluck('name')->implode(', ');
+            $ruleNames = $invalidKeys->implode(', ');
             $command->line(sprintf("  Processing field '%s' (type: %s, id: %s): removing [%s]", $field->name, $field->type, $field->id, $ruleNames));
 
             if (! $dryRun) {
-                $cleanedRules = $rules->reject(
-                    fn ($rule): bool => in_array($rule->name, self::STRING_ONLY_RULES, true)
-                )->values()->toArray();
+                $cleanedRules = $rules->except(self::STRING_ONLY_RULES)->toArray();
 
                 try {
                     $field->update([