Prefer empty() instead of null coalescing operator

vaersaagod · Apr 3, 2024 · e4ab25c · e4ab25c
1 parent 4b80725
commit e4ab25c
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/src/helpers/SEOMateHelper.php b/src/helpers/SEOMateHelper.php
@@ -20,6 +20,7 @@
 
 use Illuminate\Support\Collection;
 
+use Symfony\Component\DomCrawler\Field\ChoiceFormField;
 use vaersaagod\seomate\models\Settings;
 use vaersaagod\seomate\SEOMate;
 
@@ -146,7 +147,7 @@ public static function getPropertyDataByScopeAndHandle(ElementInterface|array $s
             }
         }
 
-        if (\str_contains(trim($handle), '{')) {
+        if (str_contains(trim($handle), '{')) {
             try {
                 return Craft::$app->getView()->renderObjectTemplate($handle, $scope);
             } catch (\Throwable $throwable) {
@@ -155,7 +156,7 @@ public static function getPropertyDataByScopeAndHandle(ElementInterface|array $s
             }
         }
 
-        if ($scope[$handle] ?? null) {
+        if (!empty($scope[$handle])) {
             if ($type === 'text') {
                 return static::getStringPropertyValue($scope[$handle]);
             }