View GitHub Repository AdminBundle

Schedule Inspection
Subscribe
Completed
Pull Request — dev (#50)
by Arnaud
02:55
created
Status
Category
Toggle Patches
Field/Field/Link.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch 
@@ -84,7 +84,7 @@  discard block
 
 block discarded – undo
84 84 
         ]);
85 85 
         $resolver->setNormalizer('route', function(Options $options, $value) {
86 86 
             // route or url should be defined
87 
-            if (!$value && !$options->offsetGet('url') &&!$options->offsetGet('admin')) {
87 
+            if (!$value && !$options->offsetGet('url') && !$options->offsetGet('admin')) {
88 88 
                 throw new InvalidOptionsException(
89 89 
                     'You must set either an url or a route for the property "'.$this->name.'"'
90 90 
                 );
 
@@ -92,7 +92,7 @@  discard block
 
 block discarded – undo
92 92
93 93 
             return $value;
94 94 
         });
95 
-        $resolver->setNormalizer('admin', function (Options $options, $value) {
95 
+        $resolver->setNormalizer('admin', function(Options $options, $value) {
96 96 
             // if a Admin is defined, an Action should be defined too
97 97 
             if ($value && !$options->offsetGet('action')) {
98 98 
                 throw new InvalidOptionsException(
Please login to merge, or discard this patch.