cross-solution / YAWIK

module/Organizations/src/Organizations/Entity/OrganizationReference.php

Doc Comments +1 added lines, -1 removed lines

@@ -150,7 +150,7 @@
      *   $this->proxy($method[, $arg1[, $arg2[, ...]]]);
      * </pre>
      *
-     * @param $method
+     * @param string $method
      *
      * @return self|mixed
      */

Spacing +1 added lines, -1 removed lines

@@ -195,7 +195,7 @@
         return $this->proxy('__isset', $property);
     }
 
-    public function notEmpty($property, array $args=[])
+    public function notEmpty($property, array $args = [])
     {
         return $this->proxy('notEmpty', $args);
     }

module/Organizations/src/Organizations/Repository/Filter/PaginationQueryFactory.php

Doc Comments +1 added lines, -1 removed lines

@@ -27,7 +27,7 @@
      *
      * @see \Zend\ServiceManager\FactoryInterface::createService()
      * @param ServiceLocatorInterface $serviceLocator
-     * @return \Organizations\Repository\Filter\PaginationQuery|mixed
+     * @return PaginationQuery
      */
     public function createService(ServiceLocatorInterface $serviceLocator)
     {

Spacing +1 added lines, -1 removed lines

@@ -31,7 +31,7 @@
      */
     public function createService(ServiceLocatorInterface $serviceLocator)
     {
-        $auth  = $serviceLocator->getServiceLocator()->get('AuthenticationService');
+        $auth = $serviceLocator->getServiceLocator()->get('AuthenticationService');
         $filter = new PaginationQuery($auth);
         return $filter;
     }

module/Organizations/src/Organizations/Repository/OrganizationName.php

Doc Comments +3 added lines

@@ -14,6 +14,9 @@
 
 class OrganizationName extends AbstractRepository
 {
+    /**
+     * @param string $name
+     */
     public function findbyName($name, $create = true)
     {
         $entity = $this->findOneBy(array('name' => $name));

module/Pdf/Module.php

Doc Comments +2 added lines, -2 removed lines

@@ -164,7 +164,7 @@ 
     /**
      * give a summary of all inserted Files,
      * this is for having access to those files in the post-process
-     * @param \Core\View\Helper\InsertFile\FileEvent|\Zend\View\ViewEvent $e
+     * @param FileEvent $e
      * @return NULL
      */
     public function collectFiles(FileEvent $e)
@@ -313,7 +313,7 @@ 
      *
      * @param string $name
      * @param \Zend\View\Renderer\RendererInterface $renderer
-     * @return string|boolean
+     * @return string|false
      */
     public function resolve($name, Renderer $renderer = null)
     {

Indentation +5 added lines, -5 removed lines

@@ -30,11 +30,11 @@
  */
 class Module
 {
-     /**
-     * Loads module specific configuration.
-     *
-     * @return array
-     */
+        /**
+         * Loads module specific configuration.
+         *
+         * @return array
+         */
     public function getConfig()
     {
         return ModuleConfigLoader::load(__DIR__ . '/config');

Spacing +12 added lines, -12 removed lines

@@ -49,7 +49,7 @@ 
      */
     public function getConfig()
     {
-        return ModuleConfigLoader::load(__DIR__ . '/config');
+        return ModuleConfigLoader::load(__DIR__.'/config');
     }
     
     public function setServiceManager(ServiceManager $serviceManager)
@@ -64,7 +64,7 @@ 
         $eventManager->getSharedManager()->attach(
             'Applications',
             'application.detail.actionbuttons',
-            function ($event) {
+            function($event) {
                 return 'pdf/application/details/button';
             }
         );
@@ -144,9 +144,9 @@ 
     {
         $file = $e->getLastFileObject();
         // assume it is of the class Core\Entity\FileEntity
-        $return = '<div class="col-md-3"><a href="#attachment_' . $file->getId() . '">' . $file->getName() . '</a></div>' . PHP_EOL
-                . '<div class="col-md-3">' . $file->getType() . '</div>'
-                . '<div class="col-md-3">' . $file->prettySize . '</div>';
+        $return = '<div class="col-md-3"><a href="#attachment_'.$file->getId().'">'.$file->getName().'</a></div>'.PHP_EOL
+                . '<div class="col-md-3">'.$file->getType().'</div>'
+                . '<div class="col-md-3">'.$file->prettySize.'</div>';
         /*
          * this snippet was for direct inserting an image into the PDF
         if ($file && $file instanceOf FileEntity && 0 === strpos($file->getType(), 'image')) {
@@ -246,7 +246,7 @@ 
         
         // the handles are for temporary files
         error_reporting(0);
-        foreach (array(self::RENDER_FULL, self::RENDER_WITHOUT_PDF, self::RENDER_WITHOUT_ATTACHMENTS ) as $render) {
+        foreach (array(self::RENDER_FULL, self::RENDER_WITHOUT_PDF, self::RENDER_WITHOUT_ATTACHMENTS) as $render) {
             $handles = array();
             try {
                 $pdf = new extern\mPDFderive();
@@ -260,8 +260,8 @@ 
                     if (is_array($this->appendImage) && !empty($this->appendImage)) {
                         foreach ($this->appendImage as $imageAttachment) {
                             $content = $imageAttachment->getContent();
-                            $url = 'data:image/' . $imageAttachment->getType() . ';base64,' . base64_encode($content);
-                            $html = '<a name="attachment_' . $imageAttachment->getId() . '"><img src="' . $url . '" /><br /></a>';
+                            $url = 'data:image/'.$imageAttachment->getType().';base64,'.base64_encode($content);
+                            $html = '<a name="attachment_'.$imageAttachment->getId().'"><img src="'.$url.'" /><br /></a>';
                             $pdf->WriteHTML($html);
                         }
                     }
@@ -288,7 +288,7 @@ 
                     $pagecount = $pdf->SetSourceFile($filename);
                     for ($pages = 0; $pages < $pagecount; $pages++) {
                         $pdf->AddPage();
-                        $pdf->WriteHTML(' pages: ' . $pagecount);
+                        $pdf->WriteHTML(' pages: '.$pagecount);
                         $tx = $pdf->ImportPage($pages + 1);
                         $pdf->UseTemplate($tx);
                     }
@@ -325,18 +325,18 @@ 
             $defaultSuffix = $viewTemplatePathStack->getDefaultSuffix();
             if (pathinfo($name, PATHINFO_EXTENSION) != $defaultSuffix) {
                 ;
-                $name .= '.pdf.' . $defaultSuffix;
+                $name .= '.pdf.'.$defaultSuffix;
             } else {
                 // TODO: replace Filename by Filename for PDF
             }
 
             foreach ($paths as $path) {
-                $file = new SplFileInfo($path . $name);
+                $file = new SplFileInfo($path.$name);
                 if ($file->isReadable()) {
                     // Found! Return it.
                     if (($filePath = $file->getRealPath()) === false && substr($path, 0, 7) === 'phar://') {
                         // Do not try to expand phar paths (realpath + phars == fail)
-                        $filePath = $path . $name;
+                        $filePath = $path.$name;
                         if (!file_exists($filePath)) {
                             break;
                         }

module/Settings/src/Settings/Form/FormAbstract.php

Doc Comments +8 added lines

@@ -50,6 +50,11 @@ 
         return $this->hydrator;
     }
     
+    /**
+     * @param string $name
+     *
+     * @return callable
+     */
     protected function getPlugin($name)
     {
         $plugin = null;
@@ -88,5 +93,8 @@ 
         $this->add($this->forms->get('DefaultButtonsFieldset'));
     }
     
+    /**
+     * @return string
+     */
     abstract public function getCoreFieldset();
 }

module/Settings/src/Settings/Repository/Settings.php

Doc Comments -1 removed lines

@@ -44,7 +44,6 @@
     
     /**
      * the $id for an entity 'setting' is the same as for the entity 'user'
-     * @param type $id
      */
     public function getSettingsByUser($user)
     {

config/config.php

Braces +1 added lines, -2 removed lines

@@ -40,8 +40,7 @@
             if (strpos($addModule, '-') === 0) {
                 $remove = substr($addModule,1);
                 $modules = array_filter($modules, function ($elem) use ($remove) { return strcasecmp($elem,$remove); });
-            }
-            else {
+            } else {
                 if (!in_array($addModule, $modules)) {
                     $modules[] = $addModule;
                 }

Spacing +5 added lines, -5 removed lines

@@ -12,7 +12,7 @@ 
 
 $env = getenv('APPLICATION_ENV') ?: 'production';
 
-if (!file_exists(__DIR__ . '/autoload/yawik.config.global.php')) {
+if (!file_exists(__DIR__.'/autoload/yawik.config.global.php')) {
     $modules = array('Install');
 } else {
     $modules = array(
@@ -34,12 +34,12 @@ 
         // kann und über Server-Variablen oder ähnlichen steuern kann
         $allModules = False;
     }
-    foreach (glob(__DIR__ . '/autoload/*.module.php') as $moduleFile) {
+    foreach (glob(__DIR__.'/autoload/*.module.php') as $moduleFile) {
         $addModules = require $moduleFile;
         foreach ($addModules as $addModule) {
             if (strpos($addModule, '-') === 0) {
-                $remove = substr($addModule,1);
-                $modules = array_filter($modules, function ($elem) use ($remove) { return strcasecmp($elem,$remove); });
+                $remove = substr($addModule, 1);
+                $modules = array_filter($modules, function($elem) use ($remove) { return strcasecmp($elem, $remove); });
             }
             else {
                 if (!in_array($addModule, $modules)) {
@@ -92,7 +92,7 @@ 
     ),
 );
 
-$envConfigFile = __DIR__ . '/config.' . $env . '.php';
+$envConfigFile = __DIR__.'/config.'.$env.'.php';
 if (file_exists($envConfigFile)) {
     if (is_readable($envConfigFile)) {
         $envConfig = include $envConfigFile;

module/Applications/Module.php

Indentation +4 added lines, -4 removed lines

@@ -35,10 +35,10 @@
             'applications list'             => 'list view scripts.',
             'applications reset-files-permissions [--filter=]' => 'Resets (means: Set again) the permissions of attachments and contact images',
             array('--filter=JSON', "available keys:\n"
-                                   . "- before    ISODate   only applications before the given date\n"
-                                   . "- after     ISODate   only applications after the given date\n"
-                                   . "- id        String    Mongo ID of the application\n"
-                                   . "- isDraft   Boolean   "),
+                                    . "- before    ISODate   only applications before the given date\n"
+                                    . "- after     ISODate   only applications after the given date\n"
+                                    . "- id        String    Mongo ID of the application\n"
+                                    . "- isDraft   Boolean   "),
         );
     }
     

Spacing +3 added lines, -3 removed lines

@@ -49,7 +49,7 @@ 
      */
     public function getConfig()
     {
-        return ModuleConfigLoader::load(__DIR__ . '/config');
+        return ModuleConfigLoader::load(__DIR__.'/config');
     }
 
     /**
@@ -62,8 +62,8 @@ 
         return array(
             'Zend\Loader\StandardAutoloader' => array(
                 'namespaces' => array(
-                    __NAMESPACE__ => __DIR__ . '/src/' . __NAMESPACE__,
-                    __NAMESPACE__ . 'Test' => __DIR__ . '/test/' . __NAMESPACE__ .'Test',
+                    __NAMESPACE__ => __DIR__.'/src/'.__NAMESPACE__,
+                    __NAMESPACE__.'Test' => __DIR__.'/test/'.__NAMESPACE__.'Test',
                 ),
             ),
         );

module/Applications/src/Applications/Controller/ConsoleController.php

Indentation +5 added lines, -5 removed lines

@@ -157,7 +157,7 @@ 
 
         $table = new Table(
             array('columnWidths' => array(40, 40, 40),
-                                 'decorator' => 'ascii')
+                                    'decorator' => 'ascii')
         );
 
         $table->appendRow(array('Module', 'Name', 'Description'));
@@ -238,8 +238,8 @@ 
             $attachments = $app->getAttachments();
             foreach ($attachments as $attachment) {
                 $attachment->getPermissions()
-                           ->clear()
-                           ->inherit($permissions);
+                            ->clear()
+                            ->inherit($permissions);
             }
             $contact = $app->getContact();
             if ($contact) {
@@ -247,8 +247,8 @@ 
 
                 if ($image) {
                     $image->getPermissions()
-                          ->clear()
-                          ->inherit($permissions);
+                            ->clear()
+                            ->inherit($permissions);
                 }
             }
         }

Spacing +22 added lines, -22 removed lines

@@ -63,7 +63,7 @@ 
                 
         echo "Generate keywords for $count applications ...\n";
         
-        $progress     = new ProgressBar($count);
+        $progress = new ProgressBar($count);
 
         /** @var PropertyToKeywords $filter */
         $filter = $services->get('filtermanager')->get('Core/Repository/PropertyToKeywords');
@@ -71,7 +71,7 @@ 
 
         /** @var \Applications\Entity\Application $application */
         foreach ($applications as $application) {
-            $progress->update($i++, 'Application ' . $i . ' / ' . $count);
+            $progress->update($i++, 'Application '.$i.' / '.$count);
             $keywords = $filter->filter($application);
             
             $application->setKeywords($keywords);
@@ -97,13 +97,13 @@ 
     {
         $applications = $this->fetchApplications();
         $count = count($applications);
-        $i=0;
-        echo "Calculate rating for " . $count . " applications ...\n";
+        $i = 0;
+        echo "Calculate rating for ".$count." applications ...\n";
         
         $progress = new ProgressBar($count);
         /** @var  \Applications\Entity\Application $application */
         foreach ($applications as $application) {
-            $progress->update($i++, 'Application ' . $i . ' / ' . $count);
+            $progress->update($i++, 'Application '.$i.' / '.$count);
             $application->getRating(/* recalculate */ true);
         }
         $progress->update($i, 'Write to database...');
@@ -131,14 +131,14 @@ 
         $documentManager = $services->get('Core/DocumentManager');
 
         $count = count($applications);
-        $i=0;
+        $i = 0;
 
-        echo  $count . " applications in Draft Mode and older than " . $days . " days will be deleted\n";
+        echo  $count." applications in Draft Mode and older than ".$days." days will be deleted\n";
 
         $progress = new ProgressBar($count);
 
         foreach ($applications as $application) {
-            $progress->update($i++, 'Application ' . $i . ' / ' . $count);
+            $progress->update($i++, 'Application '.$i.' / '.$count);
             $documentManager->remove($application);
         }
         $progress->update($i, 'clean up database');
@@ -160,17 +160,17 @@ 
 
         $table->appendRow(array('Module', 'Name', 'Description'));
 
-        $offset=strlen(getcwd())+1;
-        $links="";
-        $github='https://github.com/cross-solution/YAWIK/blob/master/';
+        $offset = strlen(getcwd()) + 1;
+        $links = "";
+        $github = 'https://github.com/cross-solution/YAWIK/blob/master/';
 
         foreach ($config['view_manager']['template_map'] as $key => $absolute_filename) {
             // strip the application_root plus an additional slash
-            $filename=substr(realpath($absolute_filename), $offset);
+            $filename = substr(realpath($absolute_filename), $offset);
             if (preg_match('~module/([^/]+)~', $filename, $match)) {
-                $module=$match[1];
+                $module = $match[1];
             } else {
-                $module="not found ($key)";
+                $module = "not found ($key)";
             }
 
             $viewModel = new ViewModel();
@@ -179,17 +179,17 @@ 
             $row = new Row();
             $row->appendColumn(new Column($module));
             if ($filename) {
-                $row->appendColumn(new Column('`' . $key . '`_'));
-                $links.='.. _'. $key .': '. $github.$filename .PHP_EOL;
+                $row->appendColumn(new Column('`'.$key.'`_'));
+                $links .= '.. _'.$key.': '.$github.$filename.PHP_EOL;
             } else {
                 $row->appendColumn(new Column("WRONG CONFIGURATION"));
             }
-            $comment="";
+            $comment = "";
             if (file_exists($absolute_filename)) {
-                $src=file_get_contents($absolute_filename);
-                $comment="file exists";
+                $src = file_get_contents($absolute_filename);
+                $comment = "file exists";
                 if (preg_match("/{{rtd:\s*(.*)}}/", $src, $match)) {
-                    $comment=$match['1'];
+                    $comment = $match['1'];
                 }
             }
             $row->appendColumn(new Column($comment));
@@ -206,7 +206,7 @@ 
     {
         echo "Loading applications... ";
 
-        $filter       = \Zend\Json\Json::decode($this->params('filter', '{}'), \Zend\Json\Json::TYPE_ARRAY);
+        $filter = \Zend\Json\Json::decode($this->params('filter', '{}'), \Zend\Json\Json::TYPE_ARRAY);
         $filter['$or'] = array(
             array('attachments' => array('$exists' => 1)),
             array('contact.image' => array('$exists' => 1)),
@@ -220,7 +220,7 @@ 
             return;
         }
         $progress = new ProgressBar($count);
-        $i=0;
+        $i = 0;
 
         foreach ($applications as $app) {
             $progress->update($i++, "Process $i / $count");