cross-solution / YAWIK

module/Core/src/Core/Collection/IdentityWrapper.php

Doc Comments -1 removed lines

@@ -381,7 +381,6 @@
 	}
 
 	/**
-	 * @param mixed $element
 	 * @return mixed
 	 */
 	protected function getElement($key)

Indentation +31 added lines, -31 removed lines

@@ -25,7 +25,7 @@ 
      */
     protected $identityExtractor;
     
-	/**
+    /**
      * @param Collection $collection
      */
     public function __construct(Collection $collection)
@@ -81,7 +81,7 @@ 
         $element = $this->getElement($key);
         
         if ($element !== false && $this->collection->removeElement($element)) {
-    		return $element;
+            return $element;
         }
     }
     
@@ -108,7 +108,7 @@ 
     {
         $element = $this->getElement($key);
         
-		return $element !== false ? $element : null;
+        return $element !== false ? $element : null;
     }
     
     /**
@@ -184,7 +184,7 @@ 
         
         if ($element !== false)
         {
-    		return $this->getKey($element);
+            return $this->getKey($element);
         }
     }
     
@@ -252,7 +252,7 @@ 
         // validate mapped elements
         array_map($this->getIdentityExtractor(), $mapped->toArray());
         
-		return new static($mapped);
+        return new static($mapped);
     }
     
     /**
@@ -341,15 +341,15 @@ 
     }
     
     /**
-	 * @param callable $identityExtractor
-	 * @return IdentityWrapper
-	 */
-	public function setIdentityExtractor(callable $identityExtractor)
-	{
-		$this->identityExtractor = $identityExtractor;
+     * @param callable $identityExtractor
+     * @return IdentityWrapper
+     */
+    public function setIdentityExtractor(callable $identityExtractor)
+    {
+        $this->identityExtractor = $identityExtractor;
 		
-		return $this;
-	}
+        return $this;
+    }
 
     /**
      *
@@ -371,23 +371,23 @@ 
         return $this->identityExtractor;
     }
 
-	/**
-	 * @param mixed $element
-	 * @return mixed
-	 */
-	protected function getKey($element)
-	{
-	    return call_user_func($this->getIdentityExtractor(), $element);
-	}
+    /**
+     * @param mixed $element
+     * @return mixed
+     */
+    protected function getKey($element)
+    {
+        return call_user_func($this->getIdentityExtractor(), $element);
+    }
 
-	/**
-	 * @param mixed $element
-	 * @return mixed
-	 */
-	protected function getElement($key)
-	{
-	    return $this->collection->filter(function ($element) use ($key) {
-	        return $this->getKey($element) == $key;
-	    })->first();
-	}
+    /**
+     * @param mixed $element
+     * @return mixed
+     */
+    protected function getElement($key)
+    {
+        return $this->collection->filter(function ($element) use ($key) {
+            return $this->getKey($element) == $key;
+        })->first();
+    }
 }

Spacing +4 added lines, -4 removed lines

@@ -116,7 +116,7 @@ 
      */
     public function getKeys()
     {
-        return $this->collection->map(function ($element) {
+        return $this->collection->map(function($element) {
             return $this->getKey($element);
         })->toArray();
     }
@@ -309,7 +309,7 @@ 
      */
     public function offsetExists($offset)
     {
-        return (bool)$this->getElement($offset);
+        return (bool) $this->getElement($offset);
     }
     
     /**
@@ -359,7 +359,7 @@ 
     {
         if (!isset($this->identityExtractor)) {
             // default identity extractor
-            $this->identityExtractor = function ($element) {
+            $this->identityExtractor = function($element) {
                 if (!is_callable([$element, 'getId'])) {
                     throw new \LogicException('$element must have getId() method');
                 }
@@ -386,7 +386,7 @@ 
 	 */
 	protected function getElement($key)
 	{
-	    return $this->collection->filter(function ($element) use ($key) {
+	    return $this->collection->filter(function($element) use ($key) {
 	        return $this->getKey($element) == $key;
 	    })->first();
 	}

Braces +1 added lines, -2 removed lines

@@ -137,8 +137,7 @@
         if ($this->getElement($key) !== false)
         {
             $this->collection->set($this->collection->indexOf($value), $value);
-        }
-        else
+        } else
         {
             $this->collection->add($value);
         }

module/Core/src/Core/Form/CollectionContainer.php

Doc Comments +1 added lines, -1 removed lines

@@ -179,7 +179,7 @@
     
     /**
      * @throws \RuntimeException
-     * @return Collection
+     * @return \Core\Entity\EntityInterface
      */
     protected function getCollection()
     {

Spacing +2 added lines, -2 removed lines

@@ -80,12 +80,12 @@
             $collection[] = $this->newEntry;
             $form = $this->buildForm($key, $this->newEntry);
             $eventManager = $form->getEventManager();
-			$eventManager->attach(CoreForm::EVENT_IS_VALID, function (Event $event) use ($collection) {
+			$eventManager->attach(CoreForm::EVENT_IS_VALID, function(Event $event) use ($collection) {
                 if (!$event->getParam('isValid')) {
                     $collection->removeElement($this->newEntry);
                 }
             });
-			$eventManager->attach(CoreForm::EVENT_PREPARE, function (Event $event) use ($collection) {
+			$eventManager->attach(CoreForm::EVENT_PREPARE, function(Event $event) use ($collection) {
                 $this->setupForm($event->getTarget(), $collection->indexOf($this->newEntry));
             });
             

Indentation +4 added lines, -4 removed lines

@@ -80,12 +80,12 @@ 
             $collection[] = $this->newEntry;
             $form = $this->buildForm($key, $this->newEntry);
             $eventManager = $form->getEventManager();
-			$eventManager->attach(CoreForm::EVENT_IS_VALID, function (Event $event) use ($collection) {
+            $eventManager->attach(CoreForm::EVENT_IS_VALID, function (Event $event) use ($collection) {
                 if (!$event->getParam('isValid')) {
                     $collection->removeElement($this->newEntry);
                 }
             });
-			$eventManager->attach(CoreForm::EVENT_PREPARE, function (Event $event) use ($collection) {
+            $eventManager->attach(CoreForm::EVENT_PREPARE, function (Event $event) use ($collection) {
                 $this->setupForm($event->getTarget(), $collection->indexOf($this->newEntry));
             });
             
@@ -117,7 +117,7 @@ 
         }
     }
     
-	/**
+    /**
      * @see \Core\Form\Element\ViewHelperProviderInterface::getViewHelper()
      */
     public function getViewHelper()
@@ -221,7 +221,7 @@ 
      */
     protected function setupForm(CoreForm $form, $key)
     {
-         $form->setAttribute('action', sprintf('?form=%s', $this->formatAction($key)))
+            $form->setAttribute('action', sprintf('?form=%s', $this->formatAction($key)))
             ->setAttribute('data-entry-key', $key)
             ->setOption('control_buttons', [[
                 'class' => 'btn-danger form-collection-container-remove-button',

module/Auth/src/Auth/Form/UserImageFactory.php

Indentation +3 added lines, -3 removed lines

@@ -55,8 +55,8 @@
     {
         /* @var $options \Applications\Options\ModuleOptions */
         $form->get($this->fileName)->setViewHelper('FormImageUpload')
-           ->setMaxSize($options->getContactImageMaxSize())
-           ->setAllowedTypes($options->getContactImageMimeType())
-           ->setForm($form);
+            ->setMaxSize($options->getContactImageMaxSize())
+            ->setAllowedTypes($options->getContactImageMimeType())
+            ->setForm($form);
     }
 }

module/Cv/src/Cv/Entity/ContactImage.php

Spacing +1 added lines, -1 removed lines

@@ -33,7 +33,7 @@
      */
     public function getUri()
     {
-        return "/file/Cv.ContactImage/" . $this->id . "/" .urlencode($this->name);
+        return "/file/Cv.ContactImage/".$this->id."/".urlencode($this->name);
     }
     
     /**

Indentation +9 added lines, -9 removed lines

@@ -44,16 +44,16 @@
         $this->contact->setImage(null);
     }
     
-	/**
-	 * @param Contact $contact
-	 * @return ContactImage
-	 */
-	public function setContact(Contact $contact)
-	{
-		$this->contact = $contact;
+    /**
+     * @param Contact $contact
+     * @return ContactImage
+     */
+    public function setContact(Contact $contact)
+    {
+        $this->contact = $contact;
 		
-		return $this;
-	}
+        return $this;
+    }
 
     /**
      * @return Contact

module/Applications/config/module.config.php

Spacing +10 added lines, -10 removed lines

@@ -23,7 +23,7 @@ 
                 * for multiple paths.
                 * example https://github.com/doctrine/DoctrineORMModule
                 */
-               'paths' => array( __DIR__ . '/../src/Applications/Entity')
+               'paths' => array(__DIR__.'/../src/Applications/Entity')
             ),
         ),
         'eventmanager' => array(
@@ -138,7 +138,7 @@ 
         'translation_file_patterns' => array(
             array(
                 'type' => 'gettext',
-                'base_dir' => __DIR__ . '/../language',
+                'base_dir' => __DIR__.'/../language',
                 'pattern' => '%s.mo',
             ),
         ),
@@ -146,16 +146,16 @@ 
     // Configure the view service manager
     'view_manager' => array(
         'template_path_stack' => array(
-            'Applications' => __DIR__ . '/../view',
+            'Applications' => __DIR__.'/../view',
         ),
         'template_map' => array(
-            'applications/error/not-found' => __DIR__ . '/../view/error/not-found.phtml',
-            'layout/apply' => __DIR__ . '/../view/layout/layout.phtml',
-            'applications/sidebar/manage' => __DIR__ . '/../view/sidebar/manage.phtml',
-            'applications/mail/forward' => __DIR__ . '/../view/mail/forward.phtml',
-            'applications/detail/pdf' => __DIR__ . '/../view/applications/manage/detail.pdf.phtml',
-            'applications/index/disclaimer' => __DIR__ . '/../view/applications/index/disclaimer.phtml',
-            'content/applications-privacy-policy' => __DIR__ . '/../view/applications/index/disclaimer.phtml',
+            'applications/error/not-found' => __DIR__.'/../view/error/not-found.phtml',
+            'layout/apply' => __DIR__.'/../view/layout/layout.phtml',
+            'applications/sidebar/manage' => __DIR__.'/../view/sidebar/manage.phtml',
+            'applications/mail/forward' => __DIR__.'/../view/mail/forward.phtml',
+            'applications/detail/pdf' => __DIR__.'/../view/applications/manage/detail.pdf.phtml',
+            'applications/index/disclaimer' => __DIR__.'/../view/applications/index/disclaimer.phtml',
+            'content/applications-privacy-policy' => __DIR__.'/../view/applications/index/disclaimer.phtml',
         )
     ),
     'view_helpers' => array(

Indentation +26 added lines, -26 removed lines

@@ -10,20 +10,20 @@ 
 
 return array(
     'doctrine' => array(
-       'driver' => array(
+        'driver' => array(
             'odm_default' => array(
                 'drivers' => array(
                     'Applications\Entity' => 'annotation',
                 ),
             ),
             'annotation' => array(
-               /*
+                /*
                 * All drivers (except DriverChain) require paths to work on. You
                 * may set this value as a string (for a single path) or an array
                 * for multiple paths.
                 * example https://github.com/doctrine/DoctrineORMModule
                 */
-               'paths' => array( __DIR__ . '/../src/Applications/Entity')
+                'paths' => array( __DIR__ . '/../src/Applications/Entity')
             ),
         ),
         'eventmanager' => array(
@@ -63,16 +63,16 @@ 
             'Applications/Options/ModuleOptions' => 'Applications\Options\ModuleOptions',
         ),
         'factories' => array(
-           'Applications/Options' => 'Applications\Factory\ModuleOptionsFactory',
-           'ApplicationRepository' => 'Applications\Repository\Service\ApplicationRepositoryFactory',
-           'ApplicationMapper' => 'Applications\Repository\Service\ApplicationMapperFactory',
-           'EducationMapper'   => 'Applications\Repository\Service\EducationMapperFactory',
-           'Applications/Listener/ApplicationCreated' => 'Applications\Factory\Listener\EventApplicationCreatedFactory',
-           'Applications/Listener/ApplicationStatusChangePre' => 'Applications\Factory\Listener\StatusChangeFactory',
-           'Applications\Auth\Dependency\ListListener' => 'Applications\Factory\Auth\Dependency\ListListenerFactory'
+            'Applications/Options' => 'Applications\Factory\ModuleOptionsFactory',
+            'ApplicationRepository' => 'Applications\Repository\Service\ApplicationRepositoryFactory',
+            'ApplicationMapper' => 'Applications\Repository\Service\ApplicationMapperFactory',
+            'EducationMapper'   => 'Applications\Repository\Service\EducationMapperFactory',
+            'Applications/Listener/ApplicationCreated' => 'Applications\Factory\Listener\EventApplicationCreatedFactory',
+            'Applications/Listener/ApplicationStatusChangePre' => 'Applications\Factory\Listener\StatusChangeFactory',
+            'Applications\Auth\Dependency\ListListener' => 'Applications\Factory\Auth\Dependency\ListListenerFactory'
         ),
         'aliases' => [
-           'Applications/Listener/ApplicationStatusChangePost' => 'Applications/Listener/ApplicationStatusChangePre'
+            'Applications/Listener/ApplicationStatusChangePost' => 'Applications/Listener/ApplicationStatusChangePre'
         ]
     ),
     'controllers' => array(
@@ -170,25 +170,25 @@ 
     ),
     'form_elements' => array(
         'invokables' => array(
-             'Applications/Mail' => 'Applications\Form\Mail',
-             'Applications/BaseFieldset' => 'Applications\Form\BaseFieldset',
-             'Applications/SettingsFieldset' => 'Applications\Form\SettingsFieldset',
-             'Applications/CommentForm' => 'Applications\Form\CommentForm',
-             'Applications/CommentFieldset' => 'Applications\Form\CommentFieldset',
-             'Applications/Apply' => 'Applications\Form\Apply',
-             'Applications/Contact' => 'Applications\Form\ContactContainer',
-             'Applications/Base'  => 'Applications\Form\Base',
-             'Applications/Facts' => 'Applications\Form\Facts',
-             'Applications/FactsFieldset' => 'Applications\Form\FactsFieldset',
-             'Applications/Attributes' => 'Applications\Form\Attributes',
-             'Applications/Filter' => 'Applications\Form\FilterApplication',
-             'href' => 'Applications\Form\Element\Ref',
-         ),
+                'Applications/Mail' => 'Applications\Form\Mail',
+                'Applications/BaseFieldset' => 'Applications\Form\BaseFieldset',
+                'Applications/SettingsFieldset' => 'Applications\Form\SettingsFieldset',
+                'Applications/CommentForm' => 'Applications\Form\CommentForm',
+                'Applications/CommentFieldset' => 'Applications\Form\CommentFieldset',
+                'Applications/Apply' => 'Applications\Form\Apply',
+                'Applications/Contact' => 'Applications\Form\ContactContainer',
+                'Applications/Base'  => 'Applications\Form\Base',
+                'Applications/Facts' => 'Applications\Form\Facts',
+                'Applications/FactsFieldset' => 'Applications\Form\FactsFieldset',
+                'Applications/Attributes' => 'Applications\Form\Attributes',
+                'Applications/Filter' => 'Applications\Form\FilterApplication',
+                'href' => 'Applications\Form\Element\Ref',
+            ),
         'factories' => array(
             'Applications/ContactImage' => 'Applications\Factory\Form\ContactImageFactory',
             'Applications/Attachments' => 'Applications\Factory\Form\AttachmentsFactory',
         ),
-     ),
+        ),
 
     'form_elements_config' => array(
         'Applications/Apply' => array(

module/Core/src/Core/Form/View/Helper/FormCollectionContainer.php

Braces +3 added lines, -1 removed lines

@@ -60,10 +60,12 @@
             %s
         </div>';
         
-        foreach ($container as $form) /* @var $form \Zend\Form\Form */
+        foreach ($container as $form) {
+            /* @var $form \Zend\Form\Form */
         {
             $formsMarkup .= sprintf($formTemplateWrapper, $formContainerHelper->renderElement($form, $layout, $parameter));
         }
+        }
         
         $templateForm = $container->getTemplateForm();
 		$templateMarkup = sprintf(

Indentation +12 added lines, -12 removed lines

@@ -60,7 +60,7 @@ 
         $translator = $this->getTranslator();
         $formContainerHelper = $view->formContainer();
         $formsMarkup = '';
-		$formTemplateWrapper = '<div class="form-collection-container-form">
+        $formTemplateWrapper = '<div class="form-collection-container-form">
             '. ($this->displayRemoveButton ? '<button type="button" class="btn btn-sm btn-danger pull-right form-collection-container-remove-button">' . $translator->translate('Remove') . '</button>' : '') . '
             %s
         </div>';
@@ -71,16 +71,16 @@ 
         }
         
         $templateForm = $container->getTemplateForm();
-		$templateMarkup = sprintf(
+        $templateMarkup = sprintf(
             $view->formCollection()->getTemplateWrapper(),
             $view->escapeHtmlAttr(sprintf($formTemplateWrapper, $formContainerHelper->renderElement($templateForm, $layout, $parameter)))
         );
         
-		return sprintf('<div class="form-collection-container" data-new-entry-key="%s" data-remove-action="%s" data-remove-question="%s">
+        return sprintf('<div class="form-collection-container" data-new-entry-key="%s" data-remove-action="%s" data-remove-question="%s">
                 <h3>%s</h3>
                 %s%s%s
             </div>',
-		    CollectionContainer::NEW_ENTRY,
+            CollectionContainer::NEW_ENTRY,
             $container->formatAction('remove'),
             $translator->translate('Really remove?'),
             $container->getLabel(),
@@ -90,14 +90,14 @@ 
         );
     }
     /**
-	 * @param boolean $displayRemoveButton
-	 * @return FormCollectionContainer
+     * @param boolean $displayRemoveButton
+     * @return FormCollectionContainer
      * @since 0.26
-	 */
-	public function setDisplayRemoveButton($displayRemoveButton)
-	{
-		$this->displayRemoveButton = (bool)$displayRemoveButton;
+     */
+    public function setDisplayRemoveButton($displayRemoveButton)
+    {
+        $this->displayRemoveButton = (bool)$displayRemoveButton;
 		
-		return $this;
-	}
+        return $this;
+    }
 }

Spacing +3 added lines, -3 removed lines

@@ -61,7 +61,7 @@ 
         $formContainerHelper = $view->formContainer();
         $formsMarkup = '';
 		$formTemplateWrapper = '<div class="form-collection-container-form">
-            '. ($this->displayRemoveButton ? '<button type="button" class="btn btn-sm btn-danger pull-right form-collection-container-remove-button">' . $translator->translate('Remove') . '</button>' : '') . '
+            '. ($this->displayRemoveButton ? '<button type="button" class="btn btn-sm btn-danger pull-right form-collection-container-remove-button">'.$translator->translate('Remove').'</button>' : '').'
             %s
         </div>';
         
@@ -86,7 +86,7 @@ 
             $container->getLabel(),
             $formsMarkup,
             $templateMarkup,
-            '<div class="form-collection-container-add-wrapper"><button type="button" class="btn btn-success form-collection-container-add-button"><span class="yk-icon yk-icon-plus"></span> ' . sprintf($translator->translate('Add %s'), $container->getLabel()) . '</button></div>'
+            '<div class="form-collection-container-add-wrapper"><button type="button" class="btn btn-success form-collection-container-add-button"><span class="yk-icon yk-icon-plus"></span> '.sprintf($translator->translate('Add %s'), $container->getLabel()).'</button></div>'
         );
     }
     /**
@@ -96,7 +96,7 @@ 
 	 */
 	public function setDisplayRemoveButton($displayRemoveButton)
 	{
-		$this->displayRemoveButton = (bool)$displayRemoveButton;
+		$this->displayRemoveButton = (bool) $displayRemoveButton;
 		
 		return $this;
 	}

module/Cv/src/Cv/Form/PreferredJobForm.php

Indentation +1 added lines, -1 removed lines

@@ -11,7 +11,7 @@
 
     public function init()
     {
-      //  $this->setDescription(/*@translate*/' Where do you want to work tomorrow. This heading gives an immediate overview of your desired next job.');
+        //  $this->setDescription(/*@translate*/' Where do you want to work tomorrow. This heading gives an immediate overview of your desired next job.');
         $this->setIsDescriptionsEnabled(true);
         parent::init();
     }

module/Cv/src/Cv/Form/NativeLanguageFieldset.php

Indentation +3 added lines, -3 removed lines

@@ -124,9 +124,9 @@
     public function init()
     {
         $this->setName('nativeLanguages')
-             ->setHydrator(new EntityHydrator())
-             ->setObject(new NativeLanguageEntity())
-             ->setLabel('Native Language');
+                ->setHydrator(new EntityHydrator())
+                ->setObject(new NativeLanguageEntity())
+                ->setLabel('Native Language');
 
 
         $this->add(

module/Cv/src/Cv/Entity/Cv.php

Indentation +5 added lines, -5 removed lines

@@ -60,11 +60,11 @@
     protected $skills;
 
     /**
-    * Skills
-    *
-    * @var ArrayCollection
-    * @ODM\EmbedMany(targetDocument="\Cv\Entity\Language")
-    */
+     * Skills
+     *
+     * @var ArrayCollection
+     * @ODM\EmbedMany(targetDocument="\Cv\Entity\Language")
+     */
     protected $languageSkills;
 
     /**

Doc Comments +8 added lines, -5 removed lines

@@ -186,7 +186,7 @@ 
     }
     
     /**
-     * @return ArrayCollection
+     * @return IdentityWrapper
      */
     public function getEducationsIndexedById()
     {
@@ -215,7 +215,7 @@ 
     }
     
     /**
-     * @return ArrayCollection
+     * @return IdentityWrapper
      */
     public function getEmploymentsIndexedById()
     {
@@ -244,7 +244,7 @@ 
     }
     
     /**
-     * @return ArrayCollection
+     * @return IdentityWrapper
      */
     public function getSkillsIndexedById()
     {
@@ -322,7 +322,7 @@ 
     }
 
     /**
-     * @return ArrayCollection
+     * @return IdentityWrapper
      */
     public function getLanguageSkillsIndexedById()
     {
@@ -360,7 +360,7 @@ 
     }
     
     /**
-     * @param Status|string $status
+     * @param string $status
      */
     public function setStatus($status)
     {
@@ -407,6 +407,9 @@ 
         }
     }
 
+    /**
+     * @param UserInterface $oldUser
+     */
     private function updatePermissions($oldUser = null)
     {
         $hasPermissions = (bool) $this->permissions;

Spacing +3 added lines, -3 removed lines

@@ -82,7 +82,7 @@ 
      * @var array
      * @ODM\Collection
      */
-    protected $nativeLanguages=[];
+    protected $nativeLanguages = [];
 
 
     /**
@@ -267,7 +267,7 @@ 
      */
     public function setIsDraft($isDraft)
     {
-        $this->isDraft=$isDraft;
+        $this->isDraft = $isDraft;
         return $this;
     }
 
@@ -337,7 +337,7 @@ 
      */
     public function setNativeLanguages($nativeLanguages)
     {
-        $this->nativeLanguages=$nativeLanguages;
+        $this->nativeLanguages = $nativeLanguages;
         return $this;
     }