Refactoring

Dominus77 · May 2, 2018 · 9b1a267 · 9b1a267
1 parent ac8e8d2
commit 9b1a267
Showing 1 changed file with 19 additions and 11 deletions.
diff --git a/SocialLikes.php b/SocialLikes.php
@@ -83,26 +83,34 @@ public function init()
      */
     public function run()
     {
-        $this->renderWidget($this->items);
+        if (!empty($this->items)) {
+            echo Html::beginTag('div', $this->containerOptions) . PHP_EOL;
+            $this->renderItems($this->items);
+            echo Html::endTag('div') . PHP_EOL;
+        }
     }
 
     /**
-     * Render widget
+     * Render items
      * @param array $items
      */
-    public function renderWidget($items = [])
+    public function renderItems($items = [])
     {
-        if (!empty($items)) {
-            echo Html::beginTag('div', $this->containerOptions) . PHP_EOL;
-            foreach ($items as $key => $options) {
-                $title = (isset($options['title'])) ? $options['title'] : $this->title;
-                if (isset($options['serviceOptions']))
-                    echo Html::tag('div', $title ? $key : '', $options['serviceOptions']) . PHP_EOL;
-            }
-            echo Html::endTag('div') . PHP_EOL;
+        foreach ($items as $key => $options) {
+            if (isset($options['serviceOptions']))
+                echo Html::tag('div', $this->getContentTitle($options) ? $key : '', $options['serviceOptions']) . PHP_EOL;
         }
     }
 
+    /**
+     * @param array $options
+     * @return bool|mixed
+     */
+    protected function getContentTitle($options = [])
+    {
+        return (isset($options['title'])) ? $options['title'] : $this->title;
+    }
+
     /**
      * Register client assets
      */