refactor ajaxOverlay in CardWidget

Author: co0lc0der

src/CardWidget.php

@@ -10,6 +10,8 @@
  */
 class CardWidget extends \yii\base\Widget
 {
+	const OVERLAY_TYPES = ['overlay', 'dark'];
+
 	/**
 	 * title of a card
 	 * @var string
@@ -106,10 +108,7 @@ public function run(): string
 		$html .= $this->getCardBody();
 		$html .= $this->getCardFooter();
 
-		if ($this->ajaxLoad) {
-			$overlay = ($this->ajaxOverlay == 'dark') ? 'overlay dark' : 'overlay';
-			$html .= ($this->ajaxLoad) ? Html::tag('div', '<i class="fas fa-2x fa-sync-alt fa-spin"></i>', ['class' => $overlay, 'data-ajax-load-url' => $this->ajaxLoad]) : '';
-		}
+		$html .= (!empty($this->ajaxLoad)) ? Html::tag('div', '<i class="fas fa-2x fa-sync-alt fa-spin"></i>', ['class' => $this->getOverlayClass(), 'data-ajax-load-url' => $this->ajaxLoad]) : '';
 
 		$html .= Html::endTag('div'); // the end of a card
 
@@ -217,6 +216,16 @@ protected function getCardFooterClass(): string
 		return $class;
 	}
 
+	/**
+	 * @return string
+	 */
+	protected function getOverlayClass(): string
+	{
+		if (empty($this->ajaxOverlay) || !in_array($this->ajaxOverlay, self::OVERLAY_TYPES)) return '';
+
+		return ($this->ajaxOverlay == 'dark') ? 'overlay dark' : 'overlay';
+	}
+
 	/**
 	 * @return void
 	 */