refactor: Code refactor [IntervieweeData] #2

src/QUI/Interview/Controls/IntervieweeData.php

@@ -24,6 +24,14 @@ class IntervieweeData extends QUI\Control
      */
     public function __construct($attributes = [])
     {
+        // default options
+        $this->setAttributes([
+            'class'    => 'quiqqer-interview-control-interviewee',
+            'template' => 'largeImageTop' // template
+        ]);
+
+        $this->addCSSFile(dirname(__FILE__).'/IntervieweeData.css');
+
         parent::__construct($attributes);
     }
 
@@ -41,11 +49,6 @@ public function getBody()
             return '';
         }
 
-        $intervieweeName           = $Site->getAttribute('quiqqer.interview.settings.intervieweeData.name');
-        $intervieweeAdditionalInfo = $Site->getAttribute('quiqqer.interview.settings.intervieweeData.additionalInfo');
-        $intervieweeAvatar         = $Site->getAttribute('quiqqer.interview.settings.intervieweeData.avatar');
-        $intervieweeDescription    = $Site->getAttribute('quiqqer.interview.settings.intervieweeData.description');
-
         switch ($this->getAttribute('interviewee-template')) {
             case 'largeImageLeft':
                 $html = '/IntervieweeData.largeImageLeft.html';
@@ -62,11 +65,22 @@ public function getBody()
                 break;
         }
 
+        try {
+            $intervieweeData = $this->getIntervieweeData();
+        } catch (Exception $Exception) {
+            QUI\System\Log::addInfo($Exception->getMessage());
+
+            $intervieweeData = [
+                'name' => false,
+                'Image' => false
+            ];
+        }
+
         $Engine->assign([
-            'intervieweeName'           => $intervieweeName,
-            'intervieweeAdditionalInfo' => $intervieweeAdditionalInfo,
-            'intervieweeAvatar'         => $intervieweeAvatar,
-            'intervieweeDescription'    => $intervieweeDescription,
+            'intervieweeName'           => $intervieweeData['intervieweeName'],
+            'intervieweeAdditionalInfo' => $intervieweeData['intervieweeAdditionalInfo'],
+            'intervieweeAvatar'         => $intervieweeData['intervieweeAvatar'],
+            'intervieweeDescription'    => $intervieweeData['intervieweeDescription']
         ]);
 
         $this->addCSSFile(
@@ -76,6 +90,23 @@ public function getBody()
         return $Engine->fetch(dirname(__FILE__) . $html);
     }
 
+    private function getIntervieweeData()
+    {
+        $Site = $this->getSite();
+
+        $intervieweeName           = $Site->getAttribute('quiqqer.interview.settings.intervieweeData.name');
+        $intervieweeAdditionalInfo = $Site->getAttribute('quiqqer.interview.settings.intervieweeData.additionalInfo');
+        $intervieweeAvatar         = $Site->getAttribute('quiqqer.interview.settings.intervieweeData.avatar');
+        $intervieweeDescription    = $Site->getAttribute('quiqqer.interview.settings.intervieweeData.description');
+
+        return [
+            'intervieweeName'           => $intervieweeName,
+            'intervieweeAdditionalInfo' => $intervieweeAdditionalInfo,
+            'intervieweeAvatar'         => $intervieweeAvatar,
+            'intervieweeDescription'    => $intervieweeDescription,
+        ];
+    }
+
     /**
      * @return mixed|QUI\Projects\Site
      *