Passed
Push — master ( 9da2d4...b22c21 )
by Andreas
09:56
created
lib/org/openpsa/contacts/handler/search.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -82,7 +82,7 @@
 block discarded – undo
82 82
 
83 83
         // Handle automatic wildcards
84 84
         $auto_wildcards = $this->_config->get('auto_wildcards');
85
-        if (   $auto_wildcards
85
+        if ($auto_wildcards
86 86
             && !str_contains($this->_query_string_processed, '%')) {
87 87
             switch ($auto_wildcards) {
88 88
                 case 'both':
Please login to merge, or discard this patch.
lib/org/openpsa/directmarketing/support/bounce_detector.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -45,7 +45,7 @@  discard block
 block discarded – undo
45 45
 }
46 46
 $recipient = $_ENV["RECIPIENT"];
47 47
 $token_length = strlen($recipient) - strlen($prefix) - strlen($suffix);
48
-if (   $token_length <= 0
48
+if ($token_length <= 0
49 49
     || !str_starts_with($recipient, $prefix)
50 50
     || !str_ends_with($recipient, $suffix)) {
51 51
     error_log("Recipient address does not match address template: $recipient");
@@ -61,7 +61,7 @@  discard block
 block discarded – undo
61 61
         fclose($fh);
62 62
         exit(0);
63 63
     }
64
-} elseif (   str_starts_with($BOUNCE_LOGGER, "http://")
64
+} elseif (str_starts_with($BOUNCE_LOGGER, "http://")
65 65
           || str_starts_with($BOUNCE_LOGGER, "https://")) {
66 66
     $client = new org_openpsa_httplib();
67 67
     $client->post($BOUNCE_LOGGER, ["token" => $token]);
Please login to merge, or discard this patch.
src/midcom/routing/parser.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -46,7 +46,7 @@  discard block
 block discarded – undo
46 46
         $url = $this->context->get_key(MIDCOM_CONTEXT_URI);
47 47
 
48 48
         $prefix_length = strlen(midcom_connection::get_url('prefix'));
49
-        if (   $prefix_length > 1
49
+        if ($prefix_length > 1
50 50
             && str_starts_with($url, midcom_connection::get_url('prefix'))) {
51 51
             $url = substr($url, $prefix_length);
52 52
         }
@@ -71,7 +71,7 @@  discard block
 block discarded – undo
71 71
      */
72 72
     public function find_urlmethod() : ?string
73 73
     {
74
-        if (   empty($this->argv)
74
+        if (empty($this->argv)
75 75
             || !str_starts_with($this->argv[0], 'midcom-')) {
76 76
             return null;
77 77
         }
Please login to merge, or discard this patch.
src/midcom/datamanager/extension/transformer/imageTransformer.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -47,7 +47,7 @@
 block discarded – undo
47 47
         }
48 48
 
49 49
         $result = [];
50
-        if (   !empty($array['file'])
50
+        if (!empty($array['file'])
51 51
             || !empty($array['identifier']) && str_starts_with($array['identifier'], 'tmpfile-')) {
52 52
             $result['file'] = parent::reverseTransform($array);
53 53
         } elseif (!empty($array['objects'])) {
Please login to merge, or discard this patch.
lib/midgard/admin/asgard/handler/welcome.php 2 patches
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -33,7 +33,7 @@  discard block
 block discarded – undo
33 33
                 $qb = new midcom_core_querybuilder($midcom_dba_classname);
34 34
                 $qb->add_constraint('metadata.revised', '>=', $since);
35 35
 
36
-                if (   $only_mine
36
+                if ($only_mine
37 37
                     && midcom::get()->auth->user) {
38 38
                     $qb->add_constraint('metadata.authors', 'LIKE', '|' . midcom::get()->auth->user->guid . '|');
39 39
                 }
@@ -198,7 +198,7 @@  discard block
 block discarded – undo
198 198
                 continue;
199 199
             }
200 200
 
201
-            if (   $object->can_do('midgard:update')
201
+            if ($object->can_do('midgard:update')
202 202
                 && $object->can_do('midcom:approve')) {
203 203
                 $object->metadata->approve();
204 204
                 midcom::get()->uimessages->add($this->_l10n->get('midgard.admin.asgard'), sprintf($this->_l10n->get('object %s approved'), $object->guid));
Please login to merge, or discard this patch.
Braces   +6 added lines, -3 removed lines patch added patch discarded remove patch
@@ -112,7 +112,8 @@  discard block
 block discarded – undo
112 112
             $row['approved'] = $object->is_approved() ? $object->metadata->approved : '';
113 113
             try {
114 114
                 $row['index_title'] = $reflector->get_object_label($object) ?: '[' . $this->_l10n->get('no title') . ']';
115
-            } catch (Exception $e) {
115
+            }
116
+            catch (Exception $e) {
116 117
                 $row['index_title'] = 'ERROR: ' . $e->getMessage();
117 118
             }
118 119
             $link = $this->router->generate('object_' . $this->_request_data['default_mode'], ['guid' => $object->guid]);
@@ -177,7 +178,8 @@  discard block
 block discarded – undo
177 178
         foreach ($guids as $guid) {
178 179
             try {
179 180
                 $object = midcom::get()->dbfactory->get_object_by_guid($guid);
180
-            } catch (midcom_error $e) {
181
+            }
182
+            catch (midcom_error $e) {
181 183
                 continue;
182 184
             }
183 185
 
@@ -192,7 +194,8 @@  discard block
 block discarded – undo
192 194
         foreach ($guids as $guid) {
193 195
             try {
194 196
                 $object = midcom::get()->dbfactory->get_object_by_guid($guid);
195
-            } catch (midcom_error $e) {
197
+            }
198
+            catch (midcom_error $e) {
196 199
                 continue;
197 200
             }
198 201
 
Please login to merge, or discard this patch.
lib/midgard/admin/user/style/midgard-admin-user-group-folders.php 1 patch
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -17,7 +17,8 @@
 block discarded – undo
17 17
         foreach ($data['objects'] as $guid => $privs) {
18 18
             try {
19 19
                 $object = new midcom_db_topic($guid);
20
-            } catch (midcom_error $e) {
20
+            }
21
+            catch (midcom_error $e) {
21 22
                 continue;
22 23
             }
23 24
             echo "<tr>\n";
Please login to merge, or discard this patch.
lib/org/openpsa/invoices/handler/billingdata.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -30,7 +30,7 @@  discard block
 block discarded – undo
30 30
     private function load_controller(org_openpsa_invoices_billing_data_dba $bd) : controller
31 31
     {
32 32
         $schemadb = schemadb::from_path($this->_config->get('schemadb_billing_data'));
33
-        $vat =& $schemadb->get('default')->get_field('vat');
33
+        $vat = & $schemadb->get('default')->get_field('vat');
34 34
         // Fill VAT select
35 35
         if ($options = $this->get_vat_options($this->_config->get('vat_percentages'))) {
36 36
             $vat['type_config']['options'] = $options;
@@ -68,7 +68,7 @@  discard block
 block discarded – undo
68 68
         $data['controller'] = $this->load_controller($bd);
69 69
 
70 70
         $workflow = $this->get_workflow('datamanager', ['controller' => $data['controller']]);
71
-        if (   $mode == 'edit'
71
+        if ($mode == 'edit'
72 72
             && $bd->can_do('midgard:delete')) {
73 73
             $delete = $this->get_workflow('delete', [
74 74
                 'object' => $bd,
Please login to merge, or discard this patch.
lib/org/openpsa/products/midcom/interfaces.php 2 patches
Braces   +4 added lines, -2 removed lines patch added patch discarded remove patch
@@ -79,7 +79,8 @@  discard block
 block discarded – undo
79 79
                 try {
80 80
                     $dms['group']->set_storage($group);
81 81
                     org_openpsa_products_viewer::index($dms['group'], $indexer, $topic, $config);
82
-                } catch (midcom_error $e) {
82
+                }
83
+                catch (midcom_error $e) {
83 84
                     $e->log(MIDCOM_LOG_WARN);
84 85
                 }
85 86
             }
@@ -91,7 +92,8 @@  discard block
 block discarded – undo
91 92
                     try {
92 93
                         $dms['product']->set_storage($product);
93 94
                         org_openpsa_products_viewer::index($dms['product'], $indexer, $topic, $config);
94
-                    } catch (midcom_error $e) {
95
+                    }
96
+                    catch (midcom_error $e) {
95 97
                         $e->log(MIDCOM_LOG_WARN);
96 98
                     }
97 99
                 }
Please login to merge, or discard this patch.
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -52,7 +52,7 @@
 block discarded – undo
52 52
      */
53 53
     public function _on_reindex($topic, midcom_helper_configuration $config, midcom_services_indexer $indexer)
54 54
     {
55
-        if (   !$config->get('index_products')
55
+        if (!$config->get('index_products')
56 56
             && !$config->get('index_groups')) {
57 57
             debug_add("No indexing to groups and products, skipping", MIDCOM_LOG_WARN);
58 58
             return true;
Please login to merge, or discard this patch.
lib/org/openpsa/notifications/notifier/email.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -38,7 +38,7 @@
 block discarded – undo
38 38
             $mail->from = $config->get('default_sender') ?: '"OpenPSA Notifier" <noreply@' . $_SERVER['SERVER_NAME'] . '>';
39 39
         }
40 40
 
41
-        $mail->subject = $message['title'] ??  'org.openpsa.notifications message (no title provided)';
41
+        $mail->subject = $message['title'] ?? 'org.openpsa.notifications message (no title provided)';
42 42
         if (array_key_exists('attachments', $message)) {
43 43
             $mail->attachments = $message['attachments'];
44 44
         }
Please login to merge, or discard this patch.