Completed
Push — master ( a923e4...fe6df8 )
by Craig
05:56
created
src/system/RoutesModule/Helper/Base/AbstractWorkflowHelper.php 1 patch
Indentation   +24 added lines, -24 removed lines patch added patch discarded remove patch
@@ -93,31 +93,31 @@
 block discarded – undo
93 93
     }
94 94
 
95 95
     /**
96
-      * This method returns a list of possible object states.
97
-      *
98
-      * @return array List of collected state information
99
-      */
100
-     public function getObjectStates()
101
-     {
102
-         $states = [];
103
-         $states[] = [
104
-             'value' => 'initial',
105
-             'text' => $this->translator->__('Initial'),
106
-             'ui' => 'danger'
107
-         ];
108
-         $states[] = [
109
-             'value' => 'approved',
110
-             'text' => $this->translator->__('Approved'),
111
-             'ui' => 'success'
112
-         ];
113
-         $states[] = [
114
-             'value' => 'deleted',
115
-             'text' => $this->translator->__('Deleted'),
116
-             'ui' => 'danger'
117
-         ];
96
+     * This method returns a list of possible object states.
97
+     *
98
+     * @return array List of collected state information
99
+     */
100
+        public function getObjectStates()
101
+        {
102
+            $states = [];
103
+            $states[] = [
104
+                'value' => 'initial',
105
+                'text' => $this->translator->__('Initial'),
106
+                'ui' => 'danger'
107
+            ];
108
+            $states[] = [
109
+                'value' => 'approved',
110
+                'text' => $this->translator->__('Approved'),
111
+                'ui' => 'success'
112
+            ];
113
+            $states[] = [
114
+                'value' => 'deleted',
115
+                'text' => $this->translator->__('Deleted'),
116
+                'ui' => 'danger'
117
+            ];
118 118
     
119
-         return $states;
120
-     }
119
+            return $states;
120
+        }
121 121
     
122 122
     /**
123 123
      * This method returns information about a certain state.
Please login to merge, or discard this patch.
system/CategoriesModule/DependencyInjection/ZikulaCategoriesExtension.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -26,7 +26,7 @@
 block discarded – undo
26 26
      */
27 27
     public function load(array $configs, ContainerBuilder $container)
28 28
     {
29
-        $loader = new YamlFileLoader($container, new FileLocator(realpath(__DIR__.'/../Resources/config')));
29
+        $loader = new YamlFileLoader($container, new FileLocator(realpath(__DIR__ . '/../Resources/config')));
30 30
 
31 31
         $loader->load('services.yml');
32 32
         $loader->load('helpers.yml');
Please login to merge, or discard this patch.
src/system/CategoriesModule/Form/Type/CategoryTreeType.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -88,7 +88,7 @@  discard block
 block discarded – undo
88 88
         $resolver->setAllowedTypes('includeLeaf', 'bool');
89 89
         $resolver->setAllowedTypes('all', 'bool');
90 90
 
91
-        $resolver->setNormalizer('label', function (Options $options, $label) {
91
+        $resolver->setNormalizer('label', function(Options $options, $label) {
92 92
             if (null === $label || empty($label)) {
93 93
                 $isMultiple = $options['multiple'];
94 94
                 $translator = $options['translator'];
@@ -98,7 +98,7 @@  discard block
 block discarded – undo
98 98
 
99 99
             return $label;
100 100
         });
101
-        $resolver->setNormalizer('placeholder', function (Options $options, $placeholder) {
101
+        $resolver->setNormalizer('placeholder', function(Options $options, $placeholder) {
102 102
             if (!$options['required']) {
103 103
                 if (null == $placeholder || empty($placeholder)) {
104 104
                     $isMultiple = $options['multiple'];
@@ -110,7 +110,7 @@  discard block
 block discarded – undo
110 110
 
111 111
             return $placeholder;
112 112
         });
113
-        $resolver->setNormalizer('choices', function (Options $options, $choices) {
113
+        $resolver->setNormalizer('choices', function(Options $options, $choices) {
114 114
             if (empty($choices)) {
115 115
                 $choices = $this->getCategoryChoices($options);
116 116
             }
Please login to merge, or discard this patch.
src/system/CategoriesModule/Form/Type/CategoryType.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -91,7 +91,7 @@  discard block
 block discarded – undo
91 91
                 'mapped' => false,
92 92
                 'required' => false
93 93
             ])
94
-            ->addEventListener(FormEvents::SUBMIT, function (FormEvent $event) use ($options) {
94
+            ->addEventListener(FormEvents::SUBMIT, function(FormEvent $event) use ($options) {
95 95
                 // ensure all locales have a display name
96 96
                 /** @var CategoryEntity $category */
97 97
                 $category = $event->getData();
@@ -109,10 +109,10 @@  discard block
 block discarded – undo
109 109
         $builder->get('name')
110 110
             ->addModelTransformer(new CallbackTransformer(
111 111
                 // remove slash from name before persistence to prevent issues with path
112
-                function ($string) {
112
+                function($string) {
113 113
                     return $string;
114 114
                 },
115
-                function ($string) {
115
+                function($string) {
116 116
                     return str_replace('/', '/', $string);
117 117
                 }
118 118
             ))
Please login to merge, or discard this patch.
src/system/CategoriesModule/Form/Type/CategoriesType.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -57,10 +57,10 @@  discard block
 block discarded – undo
57 57
 
58 58
         /** @var CategoryRegistryEntity[] $registries */
59 59
         foreach ($registries as $registry) {
60
-            $queryBuilderClosure = function (CategoryRepositoryInterface $repo) use ($registry, $options) {
60
+            $queryBuilderClosure = function(CategoryRepositoryInterface $repo) use ($registry, $options) {
61 61
                 return $repo->getChildrenQueryBuilder($registry->getCategory(), $options['direct']);
62 62
             };
63
-            $choiceLabelClosure = function (CategoryEntity $category) use ($registry) {
63
+            $choiceLabelClosure = function(CategoryEntity $category) use ($registry) {
64 64
                 $indent = str_repeat('--', $category->getLvl() - $registry->getCategory()->getLvl() - 1);
65 65
 
66 66
                 return (!empty($indent) ? '|' : '') . $indent . $category->getName();
@@ -119,7 +119,7 @@  discard block
 block discarded – undo
119 119
         $resolver->setAllowedTypes('entityCategoryClass', 'string');
120 120
         $resolver->setAllowedTypes('em', [ObjectManager::class, 'null']);
121 121
 
122
-        $resolver->addAllowedValues('entityCategoryClass', function ($value) {
122
+        $resolver->addAllowedValues('entityCategoryClass', function($value) {
123 123
             return is_subclass_of($value, AbstractCategoryAssignment::class);
124 124
         });
125 125
     }
Please login to merge, or discard this patch.
src/system/CategoriesModule/Form/Type/CategoryRegistryType.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -74,7 +74,7 @@  discard block
 block discarded – undo
74 74
             ])
75 75
         ;
76 76
 
77
-        $formModifier = function (FormInterface $form, $modName = null) use ($options) {
77
+        $formModifier = function(FormInterface $form, $modName = null) use ($options) {
78 78
             $entities = null === $modName ? [] : $this->entitySelectionBuilder->buildFor($modName);
79 79
             $form->add('entityname', 'Symfony\Component\Form\Extension\Core\Type\ChoiceType', [
80 80
                 'label' => $options['translator']->__('Entity'),
@@ -85,7 +85,7 @@  discard block
 block discarded – undo
85 85
 
86 86
         $builder->addEventListener(
87 87
             FormEvents::PRE_SET_DATA,
88
-            function (FormEvent $event) use ($formModifier) {
88
+            function(FormEvent $event) use ($formModifier) {
89 89
                 /** @var CategoryRegistryEntity $data */
90 90
                 $data = $event->getData();
91 91
                 $formModifier($event->getForm(), $data->getModname());
@@ -94,7 +94,7 @@  discard block
 block discarded – undo
94 94
 
95 95
         $builder->get('modname')->addEventListener(
96 96
             FormEvents::POST_SUBMIT,
97
-            function (FormEvent $event) use ($formModifier) {
97
+            function(FormEvent $event) use ($formModifier) {
98 98
                 $modName = $event->getForm()->getData();
99 99
                 $formModifier($event->getForm()->getParent(), $modName);
100 100
             }
Please login to merge, or discard this patch.
src/system/CategoriesModule/Controller/NodeController.php 2 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -54,7 +54,7 @@
 block discarded – undo
54 54
                 $newCategory->setName($category->getName() . 'copy');
55 55
                 $displayNames = [];
56 56
                 foreach ($newCategory->getDisplay_name() as $locale => $displayName) {
57
-                    $displayNames[$locale] = $displayName . ' ' .$this->__('copy');
57
+                    $displayNames[$locale] = $displayName . ' ' . $this->__('copy');
58 58
                 }
59 59
                 $newCategory->setDisplay_name($displayNames);
60 60
                 $action = 'edit';
Please login to merge, or discard this patch.
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -70,7 +70,8 @@
 block discarded – undo
70 70
                         $parent = $repo->find($parentId);
71 71
                         $category->setParent($parent);
72 72
                         $category->setRoot($parent->getRoot());
73
-                    } elseif (empty($parent) && $request->request->has('after')) { // sibling of top-level child
73
+                    } elseif (empty($parent) && $request->request->has('after')) {
74
+// sibling of top-level child
74 75
                         $sibling = $repo->find($request->request->get('after'));
75 76
                         $category->setParent($sibling->getParent());
76 77
                         $category->setRoot($sibling->getRoot());
Please login to merge, or discard this patch.
src/system/CategoriesModule/Controller/CategoryController.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -79,7 +79,7 @@  discard block
 block discarded – undo
79 79
         return [
80 80
             'decorate' => true,
81 81
             'html' => true,
82
-            'childOpen' => function ($node) {
82
+            'childOpen' => function($node) {
83 83
                 $jsTreeData = [];
84 84
                 $jsTreeData['disabled'] = $node['status'] != 'A';
85 85
                 $jsTreeData['type'] = $node['is_leaf'] ? 'leaf' : 'default';
@@ -87,7 +87,7 @@  discard block
 block discarded – undo
87 87
 
88 88
                 return '<li ' . $jsTreeData . 'class="jstree-open" id="' . $this->domTreeNodePrefix . $node['id'] . '">';
89 89
             },
90
-            'nodeDecorator' => function ($node) use ($locale) {
90
+            'nodeDecorator' => function($node) use ($locale) {
91 91
                 $displayName = isset($node['display_name'][$locale]) ? $node['display_name'][$locale] : $node['name'];
92 92
                 $title = ' title="' . $this->createTitleAttribute($node, $displayName, $locale) . '"';
93 93
                 $classes = [];
Please login to merge, or discard this patch.
src/lib/Zikula/Bundle/HookBundle/Controller/HookController.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -208,7 +208,7 @@  discard block
 block discarded – undo
208 208
                 }
209 209
 
210 210
                 // does the user have admin permissions on the provider module?
211
-                if (!$this->get('zikula_permissions_module.api.permission')->hasPermission($hookproviders[$i]['name']."::", '::', ACCESS_ADMIN)) {
211
+                if (!$this->get('zikula_permissions_module.api.permission')->hasPermission($hookproviders[$i]['name'] . "::", '::', ACCESS_ADMIN)) {
212 212
                     unset($hookproviders[$i]);
213 213
                     continue;
214 214
                 }
@@ -292,7 +292,7 @@  discard block
 block discarded – undo
292 292
         if (!$this->get('kernel')->isBundle($subscriber)) {
293 293
             throw new \RuntimeException($this->get('translator.default')->__f('Subscriber module "%s" is not available.', $subscriber));
294 294
         }
295
-        if (!$this->get('zikula_permissions_module.api.permission')->hasPermission($subscriber.'::', '::', ACCESS_ADMIN)) {
295
+        if (!$this->get('zikula_permissions_module.api.permission')->hasPermission($subscriber . '::', '::', ACCESS_ADMIN)) {
296 296
             throw new AccessDeniedException();
297 297
         }
298 298
 
@@ -310,7 +310,7 @@  discard block
 block discarded – undo
310 310
         if (!$this->get('kernel')->isBundle($provider)) {
311 311
             throw new \RuntimeException($this->get('translator.default')->__f('Provider module "%s" is not available.', $provider));
312 312
         }
313
-        if (!$this->get('zikula_permissions_module.api.permission')->hasPermission($provider.'::', '::', ACCESS_ADMIN)) {
313
+        if (!$this->get('zikula_permissions_module.api.permission')->hasPermission($provider . '::', '::', ACCESS_ADMIN)) {
314 314
             throw new AccessDeniedException();
315 315
         }
316 316
 
@@ -372,7 +372,7 @@  discard block
 block discarded – undo
372 372
         if (!$this->get('kernel')->isBundle($subscriber)) {
373 373
             throw new \RuntimeException($this->get('translator.default')->__f('Subscriber module "%s" is not available.', $subscriber));
374 374
         }
375
-        if (!$this->get('zikula_permissions_module.api.permission')->hasPermission($subscriber.'::', '::', ACCESS_ADMIN)) {
375
+        if (!$this->get('zikula_permissions_module.api.permission')->hasPermission($subscriber . '::', '::', ACCESS_ADMIN)) {
376 376
             throw new AccessDeniedException();
377 377
         }
378 378
 
Please login to merge, or discard this patch.