used a trait to avoid code duplication on frontend pages

Author: degami

app/base/abstracts/FrontendPageWithObject.php

@@ -15,31 +15,14 @@
 use \App\App;
 use \App\Site\Routing\RouteInfo;
 use \Exception;
+use \App\Base\Traits\FrontendTrait;
 
 /**
  * Base for a page diplaying a model
  */
 abstract class FrontendPageWithObject extends FrontendPage
 {
-    /**
-     * @var array template data
-     */
-    protected $templateData = [];
-
-    /**
-     * @var RouteInfo route info object
-     */
-    protected $route_info = null;
-
-    /**
-     * gets route group
-     *
-     * @return string
-     */
-    public static function getRouteGroup()
-    {
-        return '';
-    }
+    use FrontendTrait;
 
     /**
      * {@inheritdocs}
@@ -65,68 +48,6 @@ public function process(RouteInfo $route_info = null, $route_data = [])
         return $return;
     }
 
-    /**
-     * {@inheritdocs}
-     *
-     * @return string
-     */
-    public function getCurrentLocale()
-    {
-        if (isset($this->templateData['object']) && ($this->templateData['object'] instanceof Model) && $this->templateData['object']->isLoaded()) {
-            if ($this->templateData['object']->getLocale()) {
-                return $this->getApp()->setCurrentLocale($this->templateData['object']->getLocale())->getCurrentLocale();
-            }
-        }
-
-        return $this->getApp()->setCurrentLocale(parent::getCurrentLocale())->getCurrentLocale();
-    }
-
-    /**
-     * {@inheritdocs}
-     *
-     * @return array
-     */
-    protected function getTemplateData()
-    {
-        return $this->templateData;
-    }
-
-    /**
-     * sets object to show
-     *
-     * @param Model $object
-     */
-    protected function setObject(Model $object)
-    {
-        $this->templateData['object'] = $object;
-        return $this;
-    }
-
-    /**
-     * gets object to show
-     *
-     * @return Model|null
-     */
-    protected function getObject()
-    {
-        return $this->templateData['object'] ?? null;
-    }
-
-    /**
-     * {@inheritdocs}
-     *
-     * @return array
-     */
-    public function getTranslations()
-    {
-        return array_map(
-            function ($el) {
-                return $this->getRouting()->getBaseUrl() . $el;
-            },
-            $this->getContainer()->call([$this->getObject(), 'getTranslations'])
-        );
-    }
-
     /**
      * {@inheritdocs}
      *

app/base/traits/PageTrait.php

@@ -25,6 +25,11 @@ trait PageTrait
      */
     protected $current_user = null;
 
+    /**
+     * @var RouteInfo route info object
+     */
+    protected $route_info = null;
+
     /**
      * calculates JWT token id
      *
@@ -79,7 +84,7 @@ protected function getTokenData()
             $data = new \Lcobucci\JWT\ValidationData();
             $data->setIssuer($container->get('jwt_issuer'));
             $data->setAudience($container->get('jwt_audience'));
-            
+
             $claimUID = (string) $token->getClaim('uid');
             $claimUserName = (string) $token->getClaim('username');
 

app/site/controllers/Frontend/ContactForm.php

@@ -22,21 +22,14 @@
 use \Symfony\Component\HttpFoundation\Response;
 use \DateTime;
 use \Exception;
+use \App\Base\Traits\FrontendTrait;
 
 /**
  * Contact Form Page
  */
 class ContactForm extends FormPage // and and is similar to FrontendPageWithObject
 {
-    /**
-     * @var array template data
-     */
-    protected $templateData = [];
-
-    /**
-     * @var RouteInfo route info object
-     */
-    protected $route_info = null;
+    use FrontendTrait;
 
     /**
      * {@inheritdocs}
@@ -52,16 +45,6 @@ protected function getTemplateName()
         return 'contact_form';
     }
 
-    /**
-     * gets route group
-     *
-     * @return string
-     */
-    public static function getRouteGroup()
-    {
-        return '';
-    }
-
     /**
      * return route path
      *
@@ -91,22 +74,6 @@ public function process(RouteInfo $route_info = null, $route_data = [])
         return parent::process($route_info);
     }
 
-    /**
-     * {@inheritdocs}
-     *
-     * @return string
-     */
-    public function getCurrentLocale()
-    {
-        if ($this->templateData['object'] instanceof Model && $this->templateData['object']->isLoaded()) {
-            if ($this->templateData['object']->getLocale()) {
-                return $this->getApp()->setCurrentLocale($this->templateData['object']->getLocale())->getCurrentLocale();
-            }
-        }
-
-        return $this->getApp()->setCurrentLocale(parent::getCurrentLocale())->getCurrentLocale();
-    }
-
     /**
      * {@inheritdocs}
      *
@@ -119,16 +86,6 @@ protected function getBaseTemplateData()
         return $out;
     }
 
-    /**
-     * {@inheritdocs}
-     *
-     * @return array
-     */
-    protected function getTemplateData()
-    {
-        return $this->templateData;
-    }
-
     /**
      * {@inheritdocs}
      *
@@ -266,19 +223,4 @@ public static function getObjectClass()
     {
         return Contact::class;
     }
-
-    /**
-     * {@inheritdocs}
-     *
-     * @return array
-     */
-    public function getTranslations()
-    {
-        return array_map(
-            function ($el) {
-                return $this->getRouting()->getBaseUrl() . $el;
-            },
-            $this->getContainer()->call([$this->templateData['object'], 'getTranslations'])
-        );
-    }
 }