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/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/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;
     }
 

module/Cv/src/Cv/Form/LanguageSkillFieldset.php

Indentation +3 added lines, -3 removed lines

@@ -11,9 +11,9 @@
     public function init()
     {
         $this->setName('language')
-             ->setHydrator(new EntityHydrator())
-             ->setObject(new LanguageEntity())
-             ->setLabel('Language');
+                ->setHydrator(new EntityHydrator())
+                ->setObject(new LanguageEntity())
+                ->setLabel('Language');
         
         $this->add(
             array(

module/Core/src/Core/Entity/LocationInterface.php

Doc Comments +7 added lines, -6 removed lines

@@ -20,14 +20,14 @@ 
      * Sets the Postal Code of a location
      *
      * @param   string $postalCode
-     * @return mixed
+     * @return AbstractLocation
      */
     public function setPostalCode($postalCode);
 
     /**
      * Gets the Postal Code of a location
      *
-     * @return mixed
+     * @return string
      */
     public function getPostalCode();
 
@@ -44,6 +44,7 @@ 
      *
      * @param GeoJson $coordinates
      * @internal param $point
+     * @return AbstractLocation
      */
     public function setCoordinates(GeoJson $coordinates);
 
@@ -51,7 +52,7 @@ 
      * Sets the city name of a Location
      *
      * @param $city
-     * @return mixed
+     * @return AbstractLocation
      */
     public function setCity($city);
 
@@ -66,14 +67,14 @@ 
      * Sets the country of a location
      *
      * @param $country
-     * @return mixed
+     * @return AbstractLocation
      */
     public function setCountry($country);
 
     /**
      * Gets the country of a location
      *
-     * @return mixed
+     * @return string
      */
     public function getCountry();
 
@@ -81,7 +82,7 @@ 
      * Sets the region of a location. Eg. "Hessen" is a region in "Germany"
      *
      * @param $region
-     * @return mixed
+     * @return AbstractLocation
      */
     public function setRegion($region);