Passed
Push — master ( 22c42a...1e3b7d )
by Jan
08:26 queued 03:52
created
src/Controller/RedirectController.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -61,7 +61,7 @@  discard block
 block discarded – undo
61 61
 
62 62
         //Check if a user has set a preferred language setting:
63 63
         $user = $this->getUser();
64
-        if (($user instanceof User) && ! empty($user->getLanguage())) {
64
+        if (($user instanceof User) && !empty($user->getLanguage())) {
65 65
             $locale = $user->getLanguage();
66 66
         }
67 67
 
@@ -69,7 +69,7 @@  discard block
 block discarded – undo
69 69
         $new_url = $request->getUriForPath('/'.$locale.$request->getPathInfo());
70 70
 
71 71
         //If either mod_rewrite is not enabled or the index.php version is enforced, add index.php to the string
72
-        if (($this->enforce_index_php || ! $this->checkIfModRewriteAvailable())
72
+        if (($this->enforce_index_php || !$this->checkIfModRewriteAvailable())
73 73
             && false === strpos($new_url, 'index.php')) {
74 74
             //Like Request::getUriForPath only with index.php
75 75
             $new_url = $request->getSchemeAndHttpHost().$request->getBaseUrl().'/index.php/'.$locale.$request->getPathInfo();
@@ -87,7 +87,7 @@  discard block
 block discarded – undo
87 87
      */
88 88
     public function checkIfModRewriteAvailable()
89 89
     {
90
-        if (! function_exists('apache_get_modules')) {
90
+        if (!function_exists('apache_get_modules')) {
91 91
             //If we can not check for apache modules, we just hope for the best and assume url rewriting is available
92 92
             //If you want to enforce index.php versions of the url, you can override this via ENV vars.
93 93
             return true;
Please login to merge, or discard this patch.
src/Security/EntityListeners/ElementPermissionListener.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -98,7 +98,7 @@  discard block
 block discarded – undo
98 98
             );
99 99
 
100 100
             //Check if user is allowed to read info, otherwise apply placeholder
101
-            if ((null !== $annotation) && ! $this->isGranted('read', $annotation, $element)) {
101
+            if ((null !== $annotation) && !$this->isGranted('read', $annotation, $element)) {
102 102
                 $property->setAccessible(true);
103 103
                 $value = $annotation->getPlaceholder();
104 104
 
@@ -146,8 +146,8 @@  discard block
 block discarded – undo
146 146
                 $property->setAccessible(true);
147 147
 
148 148
                 //If the user is not allowed to edit or read this property, reset all values.
149
-                if ((! $this->isGranted('read', $annotation, $element)
150
-                    || ! $this->isGranted('edit', $annotation, $element))) {
149
+                if ((!$this->isGranted('read', $annotation, $element)
150
+                    || !$this->isGranted('edit', $annotation, $element))) {
151 151
                     //Set value to old value, so that there a no change to this property
152 152
                     if (isset($old_data[$property->getName()])) {
153 153
                         $property->setValue($element, $old_data[$property->getName()]);
@@ -170,7 +170,7 @@  discard block
 block discarded – undo
170 170
      */
171 171
     protected function isRunningFromCLI()
172 172
     {
173
-        if (empty($_SERVER['REMOTE_ADDR']) && ! isset($_SERVER['HTTP_USER_AGENT']) && count($_SERVER['argv']) > 0) {
173
+        if (empty($_SERVER['REMOTE_ADDR']) && !isset($_SERVER['HTTP_USER_AGENT']) && count($_SERVER['argv']) > 0) {
174 174
             return true;
175 175
         }
176 176
 
@@ -203,7 +203,7 @@  discard block
 block discarded – undo
203 203
         }
204 204
 
205 205
         //Check if we have already have saved the permission, otherwise save it to cache
206
-        if (! isset($this->perm_cache[$mode][get_class($element)][$operation])) {
206
+        if (!isset($this->perm_cache[$mode][get_class($element)][$operation])) {
207 207
             $this->perm_cache[$mode][get_class($element)][$operation] = $this->security->isGranted($operation, $element);
208 208
         }
209 209
 
Please login to merge, or discard this patch.
src/Form/AttachmentFormType.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -112,7 +112,7 @@  discard block
 block discarded – undo
112 112
             'required' => false,
113 113
             'label' => 'attachment.edit.download_url',
114 114
             'mapped' => false,
115
-            'disabled' => ! $this->allow_attachments_download,
115
+            'disabled' => !$this->allow_attachments_download,
116 116
             'attr' => [
117 117
                 'class' => 'form-control-sm',
118 118
             ],
@@ -141,7 +141,7 @@  discard block
 block discarded – undo
141 141
         //Check the secure file checkbox, if file is in securefile location
142 142
         $builder->get('secureFile')->addEventListener(
143 143
             FormEvents::PRE_SET_DATA,
144
-            function (FormEvent $event): void {
144
+            function(FormEvent $event): void {
145 145
                 $attachment = $event->getForm()->getParent()->getData();
146 146
                 if ($attachment instanceof Attachment) {
147 147
                     $event->setData($attachment->isSecure());
Please login to merge, or discard this patch.
src/Form/UserAdminForm.php 1 patch
Spacing   +16 added lines, -16 removed lines patch added patch discarded remove patch
@@ -76,7 +76,7 @@  discard block
 block discarded – undo
76 76
                 'attr' => [
77 77
                     'placeholder' => 'user.username.placeholder',
78 78
                 ],
79
-                'disabled' => ! $this->security->isGranted('edit_username', $entity),
79
+                'disabled' => !$this->security->isGranted('edit_username', $entity),
80 80
             ])
81 81
 
82 82
             ->add('group', StructuralEntityType::class, [
@@ -84,7 +84,7 @@  discard block
 block discarded – undo
84 84
                 'required' => false,
85 85
                 'label' => 'group.label',
86 86
                 'disable_not_selectable' => true,
87
-                'disabled' => ! $this->security->isGranted('change_group', $entity),
87
+                'disabled' => !$this->security->isGranted('change_group', $entity),
88 88
             ])
89 89
 
90 90
             ->add('first_name', TextType::class, [
@@ -94,7 +94,7 @@  discard block
 block discarded – undo
94 94
                     'placeholder' => 'user.firstName.placeholder',
95 95
                 ],
96 96
                 'required' => false,
97
-                'disabled' => ! $this->security->isGranted('edit_infos', $entity),
97
+                'disabled' => !$this->security->isGranted('edit_infos', $entity),
98 98
             ])
99 99
 
100 100
             ->add('last_name', TextType::class, [
@@ -104,7 +104,7 @@  discard block
 block discarded – undo
104 104
                     'placeholder' => 'user.lastName.placeholder',
105 105
                 ],
106 106
                 'required' => false,
107
-                'disabled' => ! $this->security->isGranted('edit_infos', $entity),
107
+                'disabled' => !$this->security->isGranted('edit_infos', $entity),
108 108
             ])
109 109
 
110 110
             ->add('email', TextType::class, [
@@ -114,7 +114,7 @@  discard block
 block discarded – undo
114 114
                     'placeholder' => 'user.email.placeholder',
115 115
                 ],
116 116
                 'required' => false,
117
-                'disabled' => ! $this->security->isGranted('edit_infos', $entity),
117
+                'disabled' => !$this->security->isGranted('edit_infos', $entity),
118 118
             ])
119 119
 
120 120
             ->add('department', TextType::class, [
@@ -124,7 +124,7 @@  discard block
 block discarded – undo
124 124
                     'placeholder' => 'user.department.placeholder',
125 125
                 ],
126 126
                 'required' => false,
127
-                'disabled' => ! $this->security->isGranted('edit_infos', $entity),
127
+                'disabled' => !$this->security->isGranted('edit_infos', $entity),
128 128
             ])
129 129
 
130 130
             //Config section
@@ -137,7 +137,7 @@  discard block
 block discarded – undo
137 137
                 'placeholder' => 'user_settings.language.placeholder',
138 138
                 'label' => 'user.language_select',
139 139
                 'preferred_choices' => ['en', 'de'],
140
-                'disabled' => ! $this->security->isGranted('change_user_settings', $entity),
140
+                'disabled' => !$this->security->isGranted('change_user_settings', $entity),
141 141
             ])
142 142
             ->add('timezone', TimezoneType::class, [
143 143
                 'required' => false,
@@ -148,12 +148,12 @@  discard block
 block discarded – undo
148 148
                 'placeholder' => 'user_settings.timezone.placeholder',
149 149
                 'label' => 'user.timezone.label',
150 150
                 'preferred_choices' => ['Europe/Berlin'],
151
-                'disabled' => ! $this->security->isGranted('change_user_settings', $entity),
151
+                'disabled' => !$this->security->isGranted('change_user_settings', $entity),
152 152
             ])
153 153
             ->add('theme', ChoiceType::class, [
154 154
                 'required' => false,
155 155
                 'choices' => User::AVAILABLE_THEMES,
156
-                'choice_label' => function ($entity, $key, $value) {
156
+                'choice_label' => function($entity, $key, $value) {
157 157
                     return $value;
158 158
                 },
159 159
                 'attr' => [
@@ -161,12 +161,12 @@  discard block
 block discarded – undo
161 161
                 ],
162 162
                 'placeholder' => 'user_settings.theme.placeholder',
163 163
                 'label' => 'user.theme.label',
164
-                'disabled' => ! $this->security->isGranted('change_user_settings', $entity),
164
+                'disabled' => !$this->security->isGranted('change_user_settings', $entity),
165 165
             ])
166 166
             ->add('currency', CurrencyEntityType::class, [
167 167
                 'required' => false,
168 168
                 'label' => 'user.currency.label',
169
-                'disabled' => ! $this->security->isGranted('change_user_settings', $entity),
169
+                'disabled' => !$this->security->isGranted('change_user_settings', $entity),
170 170
             ])
171 171
 
172 172
             ->add('new_password', RepeatedType::class, [
@@ -180,7 +180,7 @@  discard block
 block discarded – undo
180 180
                 'invalid_message' => 'password_must_match',
181 181
                 'required' => false,
182 182
                 'mapped' => false,
183
-                'disabled' => ! $this->security->isGranted('set_password', $entity),
183
+                'disabled' => !$this->security->isGranted('set_password', $entity),
184 184
                 'constraints' => [new Length([
185 185
                     'min' => 6,
186 186
                     'max' => 128,
@@ -193,7 +193,7 @@  discard block
 block discarded – undo
193 193
                     'class' => 'checkbox-custom',
194 194
                 ],
195 195
                 'label' => 'user.edit.needs_pw_change',
196
-                'disabled' => ! $this->security->isGranted('set_password', $entity),
196
+                'disabled' => !$this->security->isGranted('set_password', $entity),
197 197
             ])
198 198
 
199 199
             ->add('disabled', CheckboxType::class, [
@@ -202,7 +202,7 @@  discard block
 block discarded – undo
202 202
                     'class' => 'checkbox-custom',
203 203
                 ],
204 204
                 'label' => 'user.edit.user_disabled',
205
-                'disabled' => ! $this->security->isGranted('set_password', $entity)
205
+                'disabled' => !$this->security->isGranted('set_password', $entity)
206 206
                     || $entity === $this->security->getUser(),
207 207
             ])
208 208
 
@@ -210,7 +210,7 @@  discard block
 block discarded – undo
210 210
             ->add('permissions', PermissionsType::class, [
211 211
                 'mapped' => false,
212 212
                 'data' => $builder->getData(),
213
-                'disabled' => ! $this->security->isGranted('edit_permissions', $entity),
213
+                'disabled' => !$this->security->isGranted('edit_permissions', $entity),
214 214
             ])
215 215
         ;
216 216
         /*->add('comment', CKEditorType::class, ['required' => false,
@@ -229,7 +229,7 @@  discard block
 block discarded – undo
229 229
                 'data_class' => $options['attachment_class'],
230 230
             ],
231 231
             'by_reference' => false,
232
-            'disabled' => ! $this->security->isGranted($is_new ? 'create' : 'edit', $entity),
232
+            'disabled' => !$this->security->isGranted($is_new ? 'create' : 'edit', $entity),
233 233
         ]);
234 234
 
235 235
         //Buttons
Please login to merge, or discard this patch.
src/Form/AdminPages/CategoryAdminForm.php 1 patch
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -42,7 +42,7 @@  discard block
 block discarded – undo
42 42
             'label_attr' => [
43 43
                 'class' => 'checkbox-custom',
44 44
             ],
45
-            'disabled' => ! $this->security->isGranted($is_new ? 'create' : 'edit', $entity),
45
+            'disabled' => !$this->security->isGranted($is_new ? 'create' : 'edit', $entity),
46 46
         ]);
47 47
 
48 48
         $builder->add('disable_manufacturers', CheckboxType::class, [
@@ -52,7 +52,7 @@  discard block
 block discarded – undo
52 52
             'label_attr' => [
53 53
                 'class' => 'checkbox-custom',
54 54
             ],
55
-            'disabled' => ! $this->security->isGranted($is_new ? 'create' : 'edit', $entity),
55
+            'disabled' => !$this->security->isGranted($is_new ? 'create' : 'edit', $entity),
56 56
         ]);
57 57
 
58 58
         $builder->add('disable_autodatasheets', CheckboxType::class, [
@@ -62,7 +62,7 @@  discard block
 block discarded – undo
62 62
             'label_attr' => [
63 63
                 'class' => 'checkbox-custom',
64 64
             ],
65
-            'disabled' => ! $this->security->isGranted($is_new ? 'create' : 'edit', $entity),
65
+            'disabled' => !$this->security->isGranted($is_new ? 'create' : 'edit', $entity),
66 66
         ]);
67 67
 
68 68
         $builder->add('disable_properties', CheckboxType::class, [
@@ -72,7 +72,7 @@  discard block
 block discarded – undo
72 72
             'label_attr' => [
73 73
                 'class' => 'checkbox-custom',
74 74
             ],
75
-            'disabled' => ! $this->security->isGranted($is_new ? 'create' : 'edit', $entity),
75
+            'disabled' => !$this->security->isGranted($is_new ? 'create' : 'edit', $entity),
76 76
         ]);
77 77
 
78 78
         $builder->add('partname_hint', TextType::class, [
@@ -82,7 +82,7 @@  discard block
 block discarded – undo
82 82
             'attr' => [
83 83
                 'placeholder' => 'category.edit.partname_hint.placeholder',
84 84
             ],
85
-            'disabled' => ! $this->security->isGranted($is_new ? 'create' : 'edit', $entity),
85
+            'disabled' => !$this->security->isGranted($is_new ? 'create' : 'edit', $entity),
86 86
         ]);
87 87
 
88 88
         $builder->add('partname_regex', TextType::class, [
@@ -92,7 +92,7 @@  discard block
 block discarded – undo
92 92
             'attr' => [
93 93
                 'placeholder' => 'category.edit.partname_regex.placeholder',
94 94
             ],
95
-            'disabled' => ! $this->security->isGranted($is_new ? 'create' : 'edit', $entity),
95
+            'disabled' => !$this->security->isGranted($is_new ? 'create' : 'edit', $entity),
96 96
         ]);
97 97
 
98 98
         $builder->add('default_description', TextType::class, [
@@ -102,7 +102,7 @@  discard block
 block discarded – undo
102 102
             'attr' => [
103 103
                 'placeholder' => 'category.edit.default_description.placeholder',
104 104
             ],
105
-            'disabled' => ! $this->security->isGranted($is_new ? 'create' : 'edit', $entity),
105
+            'disabled' => !$this->security->isGranted($is_new ? 'create' : 'edit', $entity),
106 106
         ]);
107 107
 
108 108
         $builder->add('default_comment', TextType::class, [
@@ -112,7 +112,7 @@  discard block
 block discarded – undo
112 112
             'attr' => [
113 113
                 'placeholder' => 'category.edit.default_comment.placeholder',
114 114
             ],
115
-            'disabled' => ! $this->security->isGranted($is_new ? 'create' : 'edit', $entity),
115
+            'disabled' => !$this->security->isGranted($is_new ? 'create' : 'edit', $entity),
116 116
         ]);
117 117
     }
118 118
 }
Please login to merge, or discard this patch.
src/Form/AdminPages/SupplierForm.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -43,7 +43,7 @@  discard block
 block discarded – undo
43 43
             'required' => false,
44 44
             'label' => 'supplier.edit.default_currency',
45 45
             'disable_not_selectable' => true,
46
-            'disabled' => ! $this->security->isGranted($is_new ? 'create' : 'move', $entity),
46
+            'disabled' => !$this->security->isGranted($is_new ? 'create' : 'move', $entity),
47 47
         ]);
48 48
 
49 49
         $builder->add('shipping_costs', MoneyType::class, [
@@ -51,7 +51,7 @@  discard block
 block discarded – undo
51 51
             'currency' => $this->params->get('default_currency'),
52 52
             'scale' => 3,
53 53
             'label' => 'supplier.shipping_costs.label',
54
-            'disabled' => ! $this->security->isGranted($is_new ? 'create' : 'move', $entity),
54
+            'disabled' => !$this->security->isGranted($is_new ? 'create' : 'move', $entity),
55 55
         ]);
56 56
     }
57 57
 }
Please login to merge, or discard this patch.
src/Form/AdminPages/AttachmentTypeAdminForm.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -54,15 +54,15 @@
 block discarded – undo
54 54
                 'placeholder' => 'attachment_type.edit.filetype_filter.placeholder',
55 55
             ],
56 56
             'empty_data' => '',
57
-            'disabled' => ! $this->security->isGranted($is_new ? 'create' : 'edit', $entity),
57
+            'disabled' => !$this->security->isGranted($is_new ? 'create' : 'edit', $entity),
58 58
         ]);
59 59
 
60 60
         //Normalize data before writing it to database
61 61
         $builder->get('filetype_filter')->addViewTransformer(new CallbackTransformer(
62
-            function ($value) {
62
+            function($value) {
63 63
                 return $value;
64 64
             },
65
-            function ($value) {
65
+            function($value) {
66 66
                 return $this->filterTools->normalizeFilterString($value);
67 67
             }
68 68
         ));
Please login to merge, or discard this patch.
src/Form/AdminPages/BaseEntityAdminForm.php 1 patch
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -73,14 +73,14 @@  discard block
 block discarded – undo
73 73
                 'attr' => [
74 74
                     'placeholder' => 'part.name.placeholder',
75 75
                 ],
76
-                'disabled' => ! $this->security->isGranted($is_new ? 'create' : 'edit', $entity),
76
+                'disabled' => !$this->security->isGranted($is_new ? 'create' : 'edit', $entity),
77 77
             ])
78 78
 
79 79
             ->add('parent', StructuralEntityType::class, [
80 80
                 'class' => get_class($entity),
81 81
                 'required' => false,
82 82
                 'label' => 'parent.label',
83
-                'disabled' => ! $this->security->isGranted($is_new ? 'create' : 'move', $entity),
83
+                'disabled' => !$this->security->isGranted($is_new ? 'create' : 'move', $entity),
84 84
             ])
85 85
 
86 86
             ->add('not_selectable', CheckboxType::class, [
@@ -90,7 +90,7 @@  discard block
 block discarded – undo
90 90
                 'label_attr' => [
91 91
                     'class' => 'checkbox-custom',
92 92
                 ],
93
-                'disabled' => ! $this->security->isGranted($is_new ? 'create' : 'edit', $entity),
93
+                'disabled' => !$this->security->isGranted($is_new ? 'create' : 'edit', $entity),
94 94
             ])
95 95
 
96 96
             ->add('comment', CKEditorType::class, [
@@ -101,7 +101,7 @@  discard block
 block discarded – undo
101 101
                     'rows' => 4,
102 102
                 ],
103 103
                 'help' => 'bbcode.hint',
104
-                'disabled' => ! $this->security->isGranted($is_new ? 'create' : 'edit', $entity),
104
+                'disabled' => !$this->security->isGranted($is_new ? 'create' : 'edit', $entity),
105 105
             ]);
106 106
 
107 107
         $this->additionalFormElements($builder, $options, $entity);
@@ -112,7 +112,7 @@  discard block
 block discarded – undo
112 112
             'allow_add' => true,
113 113
             'allow_delete' => true,
114 114
             'label' => false,
115
-            'disabled' => ! $this->security->isGranted($is_new ? 'create' : 'edit', $entity),
115
+            'disabled' => !$this->security->isGranted($is_new ? 'create' : 'edit', $entity),
116 116
             'entry_options' => [
117 117
                 'data_class' => $options['attachment_class'],
118 118
             ],
@@ -121,7 +121,7 @@  discard block
 block discarded – undo
121 121
 
122 122
         $builder->add('master_picture_attachment', MasterPictureAttachmentType::class, [
123 123
             'required' => false,
124
-            'disabled' => ! $this->security->isGranted($is_new ? 'create' : 'edit', $entity),
124
+            'disabled' => !$this->security->isGranted($is_new ? 'create' : 'edit', $entity),
125 125
             'label' => 'part.edit.master_attachment',
126 126
             'entity' => $entity,
127 127
         ]);
@@ -132,11 +132,11 @@  discard block
 block discarded – undo
132 132
             'attr' => [
133 133
                 'class' => $is_new ? 'btn-success' : '',
134 134
             ],
135
-            'disabled' => ! $this->security->isGranted($is_new ? 'create' : 'edit', $entity),
135
+            'disabled' => !$this->security->isGranted($is_new ? 'create' : 'edit', $entity),
136 136
         ])
137 137
             ->add('reset', ResetType::class, [
138 138
                 'label' => 'entity.edit.reset',
139
-                'disabled' => ! $this->security->isGranted($is_new ? 'create' : 'edit', $entity),
139
+                'disabled' => !$this->security->isGranted($is_new ? 'create' : 'edit', $entity),
140 140
             ]);
141 141
     }
142 142
 
Please login to merge, or discard this patch.
src/Form/AdminPages/StorelocationAdminForm.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -43,7 +43,7 @@  discard block
 block discarded – undo
43 43
             'label_attr' => [
44 44
                 'class' => 'checkbox-custom',
45 45
             ],
46
-            'disabled' => ! $this->security->isGranted($is_new ? 'create' : 'move', $entity),
46
+            'disabled' => !$this->security->isGranted($is_new ? 'create' : 'move', $entity),
47 47
         ]);
48 48
 
49 49
         $builder->add('limit_to_existing_parts', CheckboxType::class, [
@@ -53,7 +53,7 @@  discard block
 block discarded – undo
53 53
             'label_attr' => [
54 54
                 'class' => 'checkbox-custom',
55 55
             ],
56
-            'disabled' => ! $this->security->isGranted($is_new ? 'create' : 'move', $entity),
56
+            'disabled' => !$this->security->isGranted($is_new ? 'create' : 'move', $entity),
57 57
         ]);
58 58
 
59 59
         $builder->add('only_single_part', CheckboxType::class, [
@@ -63,7 +63,7 @@  discard block
 block discarded – undo
63 63
             'label_attr' => [
64 64
                 'class' => 'checkbox-custom',
65 65
             ],
66
-            'disabled' => ! $this->security->isGranted($is_new ? 'create' : 'move', $entity),
66
+            'disabled' => !$this->security->isGranted($is_new ? 'create' : 'move', $entity),
67 67
         ]);
68 68
 
69 69
         $builder->add('storage_type', StructuralEntityType::class, [
@@ -72,7 +72,7 @@  discard block
 block discarded – undo
72 72
             'help' => 'storelocation.storage_type.help',
73 73
             'class' => MeasurementUnit::class,
74 74
             'disable_not_selectable' => true,
75
-            'disabled' => ! $this->security->isGranted($is_new ? 'create' : 'move', $entity),
75
+            'disabled' => !$this->security->isGranted($is_new ? 'create' : 'move', $entity),
76 76
         ]);
77 77
     }
78 78
 }
Please login to merge, or discard this patch.