| @@ -61,7 +61,7 @@ | ||
| 61 | 61 | |
| 62 | 62 | /** | 
| 63 | 63 | * @param array $config | 
| 64 | - * @param $selfCheck | |
| 64 | + * @param string $selfCheck | |
| 65 | 65 | * | 
| 66 | 66 | * @return array|bool | 
| 67 | 67 | */ | 
| @@ -39,7 +39,7 @@ discard block | ||
| 39 | 39 |              ->addEventListener(FormEvents::POST_SET_DATA, function(FormEvent $event) use ($configs) { | 
| 40 | 40 | $form = $event->getForm(); | 
| 41 | 41 | |
| 42 | -                foreach($configs as $property => $config) { | |
| 42 | +                foreach ($configs as $property => $config) { | |
| 43 | 43 |                      if (true === $depended = $this->prepare($config, 'self')) { | 
| 44 | 44 | $this->resizeBySelf($event, $property, $config); | 
| 45 | 45 | continue; | 
| @@ -51,7 +51,7 @@ discard block | ||
| 51 | 51 | |
| 52 | 52 |                      if ($data = $form->get($property)->getData()) { | 
| 53 | 53 | $form->get($partnerName) | 
| 54 | - ->setData(call_user_func_array(array($data, 'get' . $partnerName), array())) | |
| 54 | + ->setData(call_user_func_array(array($data, 'get'.$partnerName), array())) | |
| 55 | 55 | ; | 
| 56 | 56 | } | 
| 57 | 57 | } | 
| @@ -86,7 +86,7 @@ discard block | ||
| 86 | 86 | */ | 
| 87 | 87 | private function build(FormEvent $event, array $configs) | 
| 88 | 88 |      { | 
| 89 | -        foreach($configs as $property => $config) { | |
| 89 | +        foreach ($configs as $property => $config) { | |
| 90 | 90 | $this->resize($event, $property, $config); | 
| 91 | 91 | $this->resizeBySelf($event, $property, $config); | 
| 92 | 92 | } | 
| @@ -121,7 +121,7 @@ discard block | ||
| 121 | 121 |          } else { | 
| 122 | 122 |              if (!$partnerData = $partner->getData()) { | 
| 123 | 123 | $accessor = PropertyAccess::createPropertyAccessor(); | 
| 124 | -                if($fieldData = $accessor->getValue($data, $property)) { | |
| 124 | +                if ($fieldData = $accessor->getValue($data, $property)) { | |
| 125 | 125 | $partnerData = $accessor->getValue($fieldData, $partnerName); | 
| 126 | 126 | } | 
| 127 | 127 | } | 
| @@ -166,7 +166,7 @@ | ||
| 166 | 166 | * @param FormBuilderInterface $builder | 
| 167 | 167 | * @param UserInterface|null $user | 
| 168 | 168 | * | 
| 169 | - * @return callable | |
| 169 | + * @return \Closure | |
| 170 | 170 | */ | 
| 171 | 171 | public static function getUserAwareChoiceBuilder(FormBuilderInterface $builder, UserInterface $user = null) | 
| 172 | 172 |      { | 
| @@ -170,7 +170,7 @@ | ||
| 170 | 170 | */ | 
| 171 | 171 | public static function getUserAwareChoiceBuilder(FormBuilderInterface $builder, UserInterface $user = null) | 
| 172 | 172 |      { | 
| 173 | -        return function (EntityRepository $objectRepository) use ($builder, $user) { | |
| 173 | +        return function(EntityRepository $objectRepository) use ($builder, $user) { | |
| 174 | 174 |              return $objectRepository->createQueryBuilder('o') | 
| 175 | 175 |                  ->andWhere('o.user = :user') | 
| 176 | 176 |                  ->setParameter('user', $user ?: $builder->getAttribute('user_aware')) | 
| @@ -91,7 +91,7 @@ | ||
| 91 | 91 | } | 
| 92 | 92 | |
| 93 | 93 | /** | 
| 94 | - * @param $name | |
| 94 | + * @param string $name | |
| 95 | 95 | * @param null $default | 
| 96 | 96 | * | 
| 97 | 97 | * @return null|string | 
| @@ -79,6 +79,9 @@ | ||
| 79 | 79 | ); | 
| 80 | 80 | } | 
| 81 | 81 | |
| 82 | + /** | |
| 83 | + * @param string $name | |
| 84 | + */ | |
| 82 | 85 | private function getFilterDimension($name) | 
| 83 | 86 |      { | 
| 84 | 87 | $filter = $this->filter->get($name); | 
| @@ -88,7 +88,7 @@ discard block | ||
| 88 | 88 | * @param \Twig_Template $scope | 
| 89 | 89 | * @param null|string $name | 
| 90 | 90 | * | 
| 91 | - * @return mixed|string | |
| 91 | + * @return string | |
| 92 | 92 | */ | 
| 93 | 93 | public function getUiPageActions(\Twig_Template $scope, $name = null) | 
| 94 | 94 |      { | 
| @@ -108,7 +108,7 @@ discard block | ||
| 108 | 108 | * @param \Twig_Template $scope | 
| 109 | 109 | * @param string $name | 
| 110 | 110 | * | 
| 111 | - * @return mixed | |
| 111 | + * @return string | |
| 112 | 112 | */ | 
| 113 | 113 | public function getUiPageSelf(\Twig_Template $scope, $name) | 
| 114 | 114 |      { | 
| @@ -178,8 +178,8 @@ discard block | ||
| 178 | 178 | /** | 
| 179 | 179 | * Check color is negative? | 
| 180 | 180 | * | 
| 181 | - * @param $color | |
| 182 | - * @param $type | |
| 181 | + * @param string $color | |
| 182 | + * @param string $type | |
| 183 | 183 | * | 
| 184 | 184 | * @return bool | 
| 185 | 185 | */ | 
| @@ -195,8 +195,8 @@ discard block | ||
| 195 | 195 | /** | 
| 196 | 196 | * Check color is positive? | 
| 197 | 197 | * | 
| 198 | - * @param $color | |
| 199 | - * @param $type | |
| 198 | + * @param string $color | |
| 199 | + * @param string $type | |
| 200 | 200 | * | 
| 201 | 201 | * @return bool | 
| 202 | 202 | */ | 
| @@ -288,7 +288,7 @@ | ||
| 288 | 288 | * | 
| 289 | 289 | * @throws \SM\SMException | 
| 290 | 290 | */ | 
| 291 | - public function getPosibleTransitions(StatableInterface $object = null, $objectIdentifier = 'id') | |
| 291 | + public function getPosibleTransitions(StatableInterface $object = null, $objectIdentifier = 'id') | |
| 292 | 292 |      { | 
| 293 | 293 |          if (empty($object)) { | 
| 294 | 294 | return; | 
| @@ -50,7 +50,7 @@ discard block | ||
| 50 | 50 | $xor = array(); | 
| 51 | 51 | $properties = is_array($properties) ? $properties : array($properties); | 
| 52 | 52 | |
| 53 | -        foreach($properties as $field) { | |
| 53 | +        foreach ($properties as $field) { | |
| 54 | 54 |              if ($assoc = $this->addAssociation($queryBuilder, $field, false)) { | 
| 55 | 55 |                  $fieldPath = sprintf('%s.%s', $assoc[0], $holder = $assoc[1]); | 
| 56 | 56 |              } else { | 
| @@ -156,7 +156,7 @@ discard block | ||
| 156 | 156 | ; | 
| 157 | 157 | } | 
| 158 | 158 | |
| 159 | - return array($alias, $field , $hidden); | |
| 159 | + return array($alias, $field, $hidden); | |
| 160 | 160 | } | 
| 161 | 161 | } | 
| 162 | 162 | |
| @@ -22,9 +22,9 @@ | ||
| 22 | 22 | |
| 23 | 23 | $em->createQueryBuilder() | 
| 24 | 24 | ->update(get_class($object), 'o') | 
| 25 | -                ->set('o.' . $object->getUniqueStateField(), ':update') | |
| 26 | -                ->where('o.' . $object->getUniqueStateField() . ' = :where') | |
| 27 | -                ->andWhere('o.' . $id . ' <> :self') | |
| 25 | +                ->set('o.'.$object->getUniqueStateField(), ':update') | |
| 26 | +                ->where('o.'.$object->getUniqueStateField().' = :where') | |
| 27 | +                ->andWhere('o.'.$id.' <> :self') | |
| 28 | 28 |                  ->setParameter('update', false) | 
| 29 | 29 |                  ->setParameter('where', true) | 
| 30 | 30 |                  ->setParameter('self', $object) | 
| @@ -15,20 +15,20 @@ | ||
| 15 | 15 | |
| 16 | 16 | $this->register( | 
| 17 | 17 | 'getCurrentUser', | 
| 18 | -            function ($arg) { | |
| 18 | +            function($arg) { | |
| 19 | 19 |                  return sprintf('$this->get("security.token_storage")->getToken()->getUser()', $arg); | 
| 20 | 20 | }, | 
| 21 | -            function (array $variables) { | |
| 21 | +            function(array $variables) { | |
| 22 | 22 |                  return $variables['container']->get('security.token_storage')->getToken()->getUser(); | 
| 23 | 23 | } | 
| 24 | 24 | ); | 
| 25 | 25 | |
| 26 | 26 | $this->register( | 
| 27 | 27 | 'getQueryParam', | 
| 28 | -            function ($limitKey, $default) { | |
| 28 | +            function($limitKey, $default) { | |
| 29 | 29 |                  return sprintf('$this->get("request")->get(%s, %s)', $limitKey, $default); | 
| 30 | 30 | }, | 
| 31 | -            function (array $variables, $limitKey, $default) { | |
| 31 | +            function(array $variables, $limitKey, $default) { | |
| 32 | 32 |                  return $variables['container']->get('request')->get($limitKey, $default); | 
| 33 | 33 | } | 
| 34 | 34 | ); |