gregorybesson / PlaygroundCore

src/PlaygroundCore/Controller/ElfinderController.php

Doc Comments +1 added lines, -1 removed lines

@@ -10,7 +10,7 @@
     protected $Config;
 
     /**
-     * @return array|\Zend\View\Model\ViewModel
+     * @return ViewModel
      */
     public function indexAction()
     {

Spacing +2 added lines, -2 removed lines

@@ -29,7 +29,7 @@ 
     {
         $view = new ViewModel();
         $this->getConfig();
-        $view->BasePath    = $this->Config['BasePath'];
+        $view->BasePath = $this->Config['BasePath'];
         $view->ConnectorPath = '/admin/elfinder/connector';
         $view->setTerminal(true);
 
@@ -83,7 +83,7 @@ 
     public function getConfig()
     {
         if (!$this->Config) {
-            $config       = $this->getServiceLocator()->get('config');
+            $config = $this->getServiceLocator()->get('config');
             if (isset($config['playgroundcore']) && isset($config['playgroundcore']['QuConfig']) && isset($config['playgroundcore']['QuConfig']['QuElFinder'])) {
                 $this->Config = $config['playgroundcore']['QuConfig']['QuElFinder'];
             } else {

src/PlaygroundCore/Controller/Frontend/SwitchLocaleController.php

Doc Comments +2 added lines, -2 removed lines

@@ -26,7 +26,7 @@ 
     * context : (back/front)
     * referer : retour à la page
     *
-    * @return Redirect $redirect redirect to referer
+    * @return \Zend\Http\Response $redirect redirect to referer
     */
     public function switchAction()
     {
@@ -58,7 +58,7 @@ 
 
     /**
     * getServiceLocator : Recuperer le service locator
-    * @return ServiceLocator $serviceLocator
+    * @return ServiceLocatorInterface $serviceLocator
     */
     public function getServiceLocator()
     {

Indentation +25 added lines, -25 removed lines

@@ -1,11 +1,11 @@ 
 <?php
 /**
- * Zend Framework (http://framework.zend.com/)
- *
- * @link      http://github.com/zendframework/ZendSkeletonApplication for the canonical source repository
- * @copyright Copyright (c) 2005-2013 Zend Technologies USA Inc. (http://www.zend.com)
- * @license   http://framework.zend.com/license/new-bsd New BSD License
- */
+     * Zend Framework (http://framework.zend.com/)
+     *
+     * @link      http://github.com/zendframework/ZendSkeletonApplication for the canonical source repository
+     * @copyright Copyright (c) 2005-2013 Zend Technologies USA Inc. (http://www.zend.com)
+     * @license   http://framework.zend.com/license/new-bsd New BSD License
+     */
 
 namespace PlaygroundCore\Controller\Frontend;
 
@@ -16,18 +16,18 @@ 
 class SwitchLocaleController extends AbstractActionController implements ServiceLocatorAwareInterface
 {
     /**
-    * @var $localeService : Service des locales
-    */
+     * @var $localeService : Service des locales
+     */
     protected $localeService;
 
     /**
-    * switchAction : permet de switcher de langue en fonction d'un context (back/front)
-    * locale : locale pour switch
-    * context : (back/front)
-    * referer : retour à la page
-    *
-    * @return Redirect $redirect redirect to referer
-    */
+     * switchAction : permet de switcher de langue en fonction d'un context (back/front)
+     * locale : locale pour switch
+     * context : (back/front)
+     * referer : retour à la page
+     *
+     * @return Redirect $redirect redirect to referer
+     */
     public function switchAction()
     {
         $locale = $this->getEvent()->getRouteMatch()->getParam('locale');
@@ -57,27 +57,27 @@ 
 
 
     /**
-    * getServiceLocator : Recuperer le service locator
-    * @return ServiceLocator $serviceLocator
-    */
+     * getServiceLocator : Recuperer le service locator
+     * @return ServiceLocator $serviceLocator
+     */
     public function getServiceLocator()
     {
         return $this->serviceLocator;
     }
 
     /**
-    * setServiceLocator : set le service locator
-    */
+     * setServiceLocator : set le service locator
+     */
     public function setServiceLocator(ServiceLocatorInterface $serviceLocator)
     {
         $this->serviceLocator = $serviceLocator;
     }
 
-      /**
-    * getLocaleService : Recuperer le service des locales
-    *
-    * @return Service/Locale $localeService
-    */
+        /**
+         * getLocaleService : Recuperer le service des locales
+         *
+         * @return Service/Locale $localeService
+         */
     public function getLocaleService()
     {
         if ($this->localeService === null) {

src/PlaygroundCore/Controller/Plugin/ShortenUrl.php

Doc Comments +1 added lines, -1 removed lines

@@ -32,7 +32,7 @@
      * set service
      *
      * @param  $service
-     * @return String
+     * @return ShortenUrl
      */
     public function setService($service)
     {

src/PlaygroundCore/Entity/Formgen.php

Doc Comments +2 added lines, -2 removed lines

@@ -72,7 +72,7 @@ 
 
     /**
      * @param string $id
-     * @return Locale
+     * @return Formgen
      */
     public function setId($id)
     {
@@ -179,7 +179,7 @@ 
     }
 
     /**
-     * @return string $active
+     * @return integer $active
      */
     public function getActive()
     {

Indentation +14 added lines, -14 removed lines

@@ -49,15 +49,15 @@ 
      */
     protected $formtemplate;
 
-     /**
-     * @ORM\ManyToOne(targetEntity="PlaygroundCore\Entity\Locale", inversedBy="locale")
-     */
+        /**
+         * @ORM\ManyToOne(targetEntity="PlaygroundCore\Entity\Locale", inversedBy="locale")
+         */
     protected $locale;
 
-     /**
-     * active
-     * @ORM\Column(type="boolean")
-     */
+        /**
+         * active
+         * @ORM\Column(type="boolean")
+         */
     protected $active = 0;
 
     /**
@@ -274,19 +274,19 @@ 
 
 
     /**
-    * setInputFilter
-    * @param InputFilterInterface $inputFilter
-    */
+     * setInputFilter
+     * @param InputFilterInterface $inputFilter
+     */
     public function setInputFilter(InputFilterInterface $inputFilter)
     {
         throw new \Exception("Not used");
     }
 
     /**
-    * getInputFilter
-    *
-    * @return  InputFilter $inputFilter
-    */
+     * getInputFilter
+     *
+     * @return  InputFilter $inputFilter
+     */
     public function getInputFilter()
     {
         if (!$this->inputFilter) {

src/PlaygroundCore/Entity/Locale.php

Doc Comments -1 removed lines

@@ -130,7 +130,6 @@
 
 
     /**
-     * @param int $id
      * @return Locale
      */
     public function setActiveFront($activeFront)

Indentation +15 added lines, -15 removed lines

@@ -37,15 +37,15 @@ 
      */
     protected $locale;
 
-     /**
-     * active_back
-     * @ORM\Column(type="boolean", nullable=false)
-     */
+        /**
+         * active_back
+         * @ORM\Column(type="boolean", nullable=false)
+         */
     protected $active_back = 0;
-     /**
-     * active_front
-     * @ORM\Column(type="boolean", nullable=false)
-     */
+        /**
+         * active_front
+         * @ORM\Column(type="boolean", nullable=false)
+         */
     protected $active_front = 0;
 
     /**
@@ -232,19 +232,19 @@ 
 
 
     /**
-    * setInputFilter
-    * @param InputFilterInterface $inputFilter
-    */
+     * setInputFilter
+     * @param InputFilterInterface $inputFilter
+     */
     public function setInputFilter(InputFilterInterface $inputFilter)
     {
         throw new \Exception("Not used");
     }
 
     /**
-    * getInputFilter
-    *
-    * @return  InputFilter $inputFilter
-    */
+     * getInputFilter
+     *
+     * @return  InputFilter $inputFilter
+     */
     public function getInputFilter()
     {
         if (!$this->inputFilter) {

src/PlaygroundCore/Entity/Website.php

Doc Comments +3 added lines, -4 removed lines

@@ -143,7 +143,7 @@ 
     }
 
     /**
-     * @return string $active
+     * @return integer $active
      */
     public function getActive()
     {
@@ -162,7 +162,7 @@ 
     }
 
      /**
-     * @return string $default
+     * @return integer $default
      */
     public function getDefault()
     {
@@ -181,7 +181,7 @@ 
     }
 
     /**
-     * @return PlaygroundCore\Entity\Locale $locales
+     * @return \Doctrine\Common\Collections\Collection $locales
      */
     public function getLocales()
     {
@@ -249,7 +249,6 @@ 
     /**
     * getFlag : get path for flag associate to the country
     *
-    * @param string $flag
     */
     public function getFlag()
     {

Indentation +14 added lines, -14 removed lines

@@ -161,9 +161,9 @@ 
         return $this;
     }
 
-     /**
-     * @return string $default
-     */
+        /**
+         * @return string $default
+         */
     public function getDefault()
     {
         return $this->byDefault;
@@ -247,10 +247,10 @@ 
     }
 
     /**
-    * getFlag : get path for flag associate to the country
-    *
-    * @param string $flag
-    */
+     * getFlag : get path for flag associate to the country
+     *
+     * @param string $flag
+     */
     public function getFlag()
     {
         return "/lib/images/flag/".strtolower($this->getCode());
@@ -281,19 +281,19 @@ 
 
 
     /**
-    * setInputFilter
-    * @param InputFilterInterface $inputFilter
-    */
+     * setInputFilter
+     * @param InputFilterInterface $inputFilter
+     */
     public function setInputFilter(InputFilterInterface $inputFilter)
     {
         throw new \Exception("Not used");
     }
 
     /**
-    * getInputFilter
-    *
-    * @return  InputFilter $inputFilter
-    */
+     * getInputFilter
+     *
+     * @return  InputFilter $inputFilter
+     */
     public function getInputFilter()
     {
         if (!$this->inputFilter) {

src/PlaygroundCore/Filter/Sanitize.php

Doc Comments +4 added lines, -5 removed lines

@@ -85,7 +85,6 @@ 
      *
      * Returns $value filtered to valid url
      *
-     * @param  string $value
      * @return string
      */
     public function filterSubstring($s)
@@ -137,7 +136,7 @@ 
      * Remove not replaced character
      *
      * @param string|array $notReplaced
-     * @return Zend_Filter_Sanitize
+     * @return Sanitize
      */
     public function removeNotReplacedChar($notReplaced)
     {
@@ -177,7 +176,7 @@ 
      * Sets the delimiterReplacement option
      *
      * @param  string $delimiterReplacement
-     * @return Zend_Filter_Sanitize Provides a fluent interface
+     * @return Sanitize Provides a fluent interface
      */
     public function setDelimiterReplacement($delimiterReplacement)
     {
@@ -199,7 +198,7 @@ 
      * Add word delimiter
      *
      * @param string|array $delimiter
-     * @return Zend_Filter_Sanitize
+     * @return Sanitize
      */
     public function addWordDelimiter($delimiter)
     {
@@ -221,7 +220,7 @@ 
      * Remove word delimiter
      *
      * @param string|array $delimiter
-     * @return Zend_Filter_Sanitize
+     * @return Sanitize
      */
     public function removeWordDelimiter($delimiter)
     {

Spacing +5 added lines, -5 removed lines

@@ -77,7 +77,7 @@ 
     {
         $filename = pathinfo($s, PATHINFO_FILENAME);
         $extension = pathinfo($s, PATHINFO_EXTENSION);
-        return $this->filterSubstring($filename).($extension  ? '.'.$this->filterSubstring($extension) : '');
+        return $this->filterSubstring($filename).($extension ? '.'.$this->filterSubstring($extension) : '');
     }
 
     /**
@@ -149,7 +149,7 @@ 
                 $this->removeNotReplacedChar($n);
             }
         } else {
-            if (! in_array($notReplaced, $this->getNotReplacedChars())) {
+            if (!in_array($notReplaced, $this->getNotReplacedChars())) {
                 throw new \Exception("Not replaced character '$notReplaced' is not in array.");
             }
             $newArray = array();
@@ -233,7 +233,7 @@ 
                 $this->removeWordDelimiter($delim);
             }
         } else {
-            if (! in_array($delimiter, $this->getWordDelimiters())) {
+            if (!in_array($delimiter, $this->getWordDelimiters())) {
                 throw new \Exception("Word delimiter '$delimiter' is not in delimiters array.");
             }
             $newArray = array();
@@ -316,7 +316,7 @@ 
         if (count($this->getNotReplacedChars()) == 0) {
             $reg = '~[^-a-z0-9_]+~';
         } else {
-            $reg = '~[^-a-z0-9_' . implode('', $this->getNotReplacedChars()) .']+~';
+            $reg = '~[^-a-z0-9_'.implode('', $this->getNotReplacedChars()).']+~';
         }
         return preg_replace($reg, '', $s);
     }
@@ -329,7 +329,7 @@ 
      */
     private function replaceDoubleDelimiterReplacementWithSingle($s)
     {
-        $doubleDelimiterReplacement = $this->getDelimiterReplacement() . $this->getDelimiterReplacement();
+        $doubleDelimiterReplacement = $this->getDelimiterReplacement().$this->getDelimiterReplacement();
         while (strpos($s, $doubleDelimiterReplacement) !== false) {
             $s = str_replace($doubleDelimiterReplacement, $this->getDelimiterReplacement(), $s);
         }

src/PlaygroundCore/Mail/Service/Message.php

Doc Comments +7 added lines, -5 removed lines

@@ -18,7 +18,7 @@ 
     /**
      *
      * @param  ServiceManager  $serviceManager
-     * @return AbstractService
+     * @return Message
      */
     public function setServiceManager(ServiceManager $serviceManager)
     {
@@ -61,7 +61,7 @@ 
      *            Either the template to use, or a ViewModel
      * @param null|array $values
      *            Values to use when the template is rendered
-     * @return Message
+     * @return MailMessage
      */
     public function createHtmlMessage($from, $to, $subject, $nameOrModel, $values = array())
     {
@@ -102,7 +102,7 @@ 
      *            Either the template to use, or a ViewModel
      * @param null|array $values
      *            Values to use when the template is rendered
-     * @return Message
+     * @return MailMessage
      */
     public function createTextMessage($from, $to, $subject, $nameOrModel, $values = array())
     {
@@ -115,7 +115,7 @@ 
     /**
      * Send the message
      *
-     * @param Message $message
+     * @param MailMessage $message
      */
     public function send(MailMessage $message)
     {
@@ -155,7 +155,9 @@ 
 
     /**
      *
-     * @return Message
+     * @param string $encoding
+     * @param string $subject
+     * @return MailMessage
      */
     protected function getDefaultMessage($from, $encoding, $to, $subject, $body)
     {

Indentation +1 added lines, -1 removed lines

@@ -72,7 +72,7 @@
         $html->type = "text/html";
 
         $resolver = $this->getServiceManager()
-                         ->get('Zend\View\Resolver\TemplatePathStack');
+                            ->get('Zend\View\Resolver\TemplatePathStack');
 
         // check if plain text email template exist
         if ($resolver->resolve($nameOrModel . '-plain')) {

Spacing +2 added lines, -2 removed lines

@@ -75,8 +75,8 @@
                          ->get('Zend\View\Resolver\TemplatePathStack');
 
         // check if plain text email template exist
-        if ($resolver->resolve($nameOrModel . '-plain')) {
-            $contentText = $renderer->render($nameOrModel . '-plain', $values);
+        if ($resolver->resolve($nameOrModel.'-plain')) {
+            $contentText = $renderer->render($nameOrModel.'-plain', $values);
             $text = new MimePart($contentText);
         } else {
             $text = new MimePart('');

src/PlaygroundCore/Service/Cron.php

Doc Comments +1 added lines, -1 removed lines

@@ -399,7 +399,7 @@
     /**
      * determine whether a given time falls within the given cron expr
      *
-     * @param string|numeric $time
+     * @param integer $time
      *      timestamp or strtotime()-compatible string
      * @param string $expr
      *      any valid cron expression, in addition supporting:

Spacing +8 added lines, -8 removed lines

@@ -162,7 +162,7 @@ 
         $em = $this->getEm();
         $cronRegistry = $this->getCronjobs();
         $pending = $this->getPending();
-        $scheduleLifetime = $this->scheduleLifetime * 60; //convert min to sec
+        $scheduleLifetime = $this->scheduleLifetime*60; //convert min to sec
 
         $now = new \DateTime;
         foreach ($pending as $job) {
@@ -243,7 +243,7 @@ 
             $exists[$identifier] = true;
         }
 
-        $scheduleAhead = $this->getScheduleAhead() * 60;
+        $scheduleAhead = $this->getScheduleAhead()*60;
 
         $cronRegistry = $this->getCronjobs();
 
@@ -260,7 +260,7 @@ 
                 );
                 $scheduleTimestamp = $scheduleTime->getTimestamp();
 
-                $identifier = $code . $scheduleTimestamp;
+                $identifier = $code.$scheduleTimestamp;
                 if (isset($exists[$identifier])) {
                     //already scheduled
                     continue;
@@ -306,7 +306,7 @@ 
         $em = $this->getEm();
         $running = $em->getRepository('PlaygroundCore\Entity\Cronjob')
             ->getRunning();
-        $expiryTime = time() - $this->getMaxRunningTime() * 60;
+        $expiryTime = time() - $this->getMaxRunningTime()*60;
 
         foreach ($running as $job) {
             if ($job->getExecuteTime()
@@ -335,11 +335,11 @@ 
         $em = $this->getEm();
         $lifetime = array(
             Mapper\Cronjob::STATUS_SUCCESS  =>
-                $this->getSuccessLogLifetime() * 60,
+                $this->getSuccessLogLifetime()*60,
             Mapper\Cronjob::STATUS_MISSED   =>
-                $this->getFailureLogLifetime() * 60,
+                $this->getFailureLogLifetime()*60,
             Mapper\Cronjob::STATUS_ERROR    =>
-                $this->getFailureLogLifetime() * 60,
+                $this->getFailureLogLifetime()*60,
         );
 
         $history = $em->getRepository('PlaygroundCore\Entity\Cronjob')
@@ -545,7 +545,7 @@ 
             ));
         }
 
-        return ($num >= $from) && ($num <= $to) && ($num % $mod === 0);
+        return ($num >= $from) && ($num <= $to) && ($num%$mod === 0);
     }
 
     /**