Passed
Push — master ( 1a9f82...722134 )
by Andreas
12:17
created
lib/org/openpsa/mail/main.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -39,7 +39,7 @@
 block discarded – undo
39 39
  */
40 40
 class org_openpsa_mail
41 41
 {
42
-    use midcom_baseclasses_components_base {__get as base__get;}
42
+    use midcom_baseclasses_components_base {__get as base__get; }
43 43
 
44 44
     /**
45 45
      * Text body
Please login to merge, or discard this patch.
lib/net/nehmer/blog/handler.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -15,7 +15,7 @@  discard block
 block discarded – undo
15 15
 {
16 16
     public function get_url(midcom_db_article $article, bool $allow_external = false) : string
17 17
     {
18
-        if (   $allow_external
18
+        if ($allow_external
19 19
             && $this->_config->get('link_to_external_url')
20 20
             && !empty($article->url)) {
21 21
             return $article->url;
@@ -67,7 +67,7 @@  discard block
 block discarded – undo
67 67
 
68 68
         $qb->add_constraint('topic.guid', 'IN', $topic_guids);
69 69
 
70
-        if (   count($topic_guids) > 1
70
+        if (count($topic_guids) > 1
71 71
             && $list_from_folders_categories = $this->_config->get('list_from_folders_categories')) {
72 72
             $list_from_folders_categories = explode(',', $list_from_folders_categories);
73 73
 
Please login to merge, or discard this patch.
lib/net/nemein/tag/handler.php 1 patch
Braces   +4 added lines, -2 removed lines patch added patch discarded remove patch
@@ -381,7 +381,8 @@  discard block
 block discarded – undo
381 381
             try {
382 382
                 $tag = net_nemein_tag_tag_dba::get_cached($link->tag);
383 383
                 $link_object_map[$link->fromGuid][$tag->tag] = $link;
384
-            } catch (midcom_error $e) {
384
+            }
385
+            catch (midcom_error $e) {
385 386
                 $e->log();
386 387
             }
387 388
         }
@@ -410,7 +411,8 @@  discard block
 block discarded – undo
410 411
                     $tmpobject = midcom::get()->dbfactory->convert_midgard_to_midcom($tmpobject);
411 412
                 }
412 413
                 $return[] = $tmpobject;
413
-            } catch (midcom_error $e) {
414
+            }
415
+            catch (midcom_error $e) {
414 416
                 $e->log();
415 417
             }
416 418
         }
Please login to merge, or discard this patch.
lib/net/nehmer/blog/navigation.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -64,14 +64,14 @@  discard block
 block discarded – undo
64 64
 
65 65
     private function _add_pseudo_leaves(array &$leaves)
66 66
     {
67
-        if (   $this->_config->get('archive_enable')
67
+        if ($this->_config->get('archive_enable')
68 68
             && $this->_config->get('archive_in_navigation')) {
69 69
             $leaves["{$this->_topic->id}_ARCHIVE"] = [
70 70
                 MIDCOM_NAV_URL => "archive/",
71 71
                 MIDCOM_NAV_NAME => $this->_l10n->get('archive'),
72 72
             ];
73 73
         }
74
-        if (   $this->_config->get('rss_enable')
74
+        if ($this->_config->get('rss_enable')
75 75
             && $this->_config->get('feeds_in_navigation')) {
76 76
             $leaves[self::LEAFID_FEEDS] = [
77 77
                 MIDCOM_NAV_URL => "feeds/",
@@ -79,7 +79,7 @@  discard block
 block discarded – undo
79 79
             ];
80 80
         }
81 81
 
82
-        if (   $this->_config->get('categories_in_navigation')
82
+        if ($this->_config->get('categories_in_navigation')
83 83
             && $this->_config->get('categories') != '') {
84 84
             $categories = explode(',', $this->_config->get('categories'));
85 85
             foreach ($categories as $category) {
@@ -90,7 +90,7 @@  discard block
 block discarded – undo
90 90
             }
91 91
         }
92 92
 
93
-        if (   $this->_config->get('archive_years_in_navigation')
93
+        if ($this->_config->get('archive_years_in_navigation')
94 94
             && $this->_config->get('archive_years_enable')) {
95 95
             $qb = midcom_db_article::new_query_builder();
96 96
             $qb->add_constraint('topic', '=', $this->_topic->id);
@@ -121,7 +121,7 @@  discard block
 block discarded – undo
121 121
      */
122 122
     public static function add_scheduling_constraints(midcom_core_querybuilder $qb, midcom_helper_configuration $config)
123 123
     {
124
-        if (   $config->get('enable_scheduled_publishing')
124
+        if ($config->get('enable_scheduled_publishing')
125 125
             && !midcom::get()->auth->admin) {
126 126
             // Show the article only if the publishing time has passed or the viewer
127 127
             // is the author
Please login to merge, or discard this patch.
lib/org/openpsa/invoices/midcom/indexer.php 1 patch
Indentation   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -9,11 +9,11 @@
 block discarded – undo
9 9
 use midcom\datamanager\indexer\client;
10 10
 use midcom\datamanager\datamanager;
11 11
 
12
- /**
13
- * Indexer client class
14
- *
15
- * @package org.openpsa.invoices
16
- */
12
+    /**
13
+     * Indexer client class
14
+     *
15
+     * @package org.openpsa.invoices
16
+     */
17 17
 class org_openpsa_invoices_midcom_indexer extends client
18 18
 {
19 19
     public function prepare_document(midcom_services_indexer_document $document, datamanager $dm)
Please login to merge, or discard this patch.
lib/midcom/helper/formatter.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -50,12 +50,12 @@
 block discarded – undo
50 50
      */
51 51
     public static function compile(string $content) : string
52 52
     {
53
-        return preg_replace_callback("%&\(([^)]*)\);%i", function ($variable)
53
+        return preg_replace_callback("%&\(([^)]*)\);%i", function($variable)
54 54
         {
55 55
             $parts = explode(':', $variable[1]);
56 56
             $variable = '$' . str_replace('.', '->', $parts[0]);
57 57
 
58
-            if (   isset($parts[1])
58
+            if (isset($parts[1])
59 59
                 && array_key_exists($parts[1], self::$_filters)) {
60 60
                 if ($parts[1] == 'p') {
61 61
                     $command = 'eval(\'?>\' . ' . $variable . ')';
Please login to merge, or discard this patch.
lib/midcom/admin/folder/style/midcom-admin-folder-order-type.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -4,7 +4,7 @@  discard block
 block discarded – undo
4 4
 $count = count($data['navigation_items']);
5 5
 
6 6
 foreach ($data['navigation_items'] as $i => $item) {
7
-    if (   isset($item[MIDCOM_NAV_SORTABLE])
7
+    if (isset($item[MIDCOM_NAV_SORTABLE])
8 8
         && !$item[MIDCOM_NAV_SORTABLE]) {
9 9
         continue;
10 10
     }
@@ -15,7 +15,7 @@  discard block
 block discarded – undo
15 15
     $style = '';
16 16
 
17 17
     // Skip all components that return the default icon
18
-    if (   isset($item[MIDCOM_NAV_COMPONENT])
18
+    if (isset($item[MIDCOM_NAV_COMPONENT])
19 19
         && ($tmp = midcom::get()->componentloader->get_component_icon($item[MIDCOM_NAV_COMPONENT], false))) {
20 20
         $icon = '<i class="fa fa-' . $tmp . '"></i>';
21 21
     } elseif (!$item[MIDCOM_NAV_GUID]) {
Please login to merge, or discard this patch.
lib/org/openpsa/contacts/duplicates/check.php 2 patches
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -113,7 +113,7 @@  discard block
 block discarded – undo
113 113
 
114 114
     protected function match(string $property, array $data1, array $data2) : bool
115 115
     {
116
-        if (   !empty($data1[$property])
116
+        if (!empty($data1[$property])
117 117
             && $data1[$property] == $data2[$property]) {
118 118
             return true;
119 119
         }
@@ -195,7 +195,7 @@  discard block
 block discarded – undo
195 195
                 continue;
196 196
             }
197 197
 
198
-            if (   $obj1->get_parameter('org.openpsa.contacts.duplicates:not_duplicate', $obj2->guid)
198
+            if ($obj1->get_parameter('org.openpsa.contacts.duplicates:not_duplicate', $obj2->guid)
199 199
                 || $obj2->get_parameter('org.openpsa.contacts.duplicates:not_duplicate', $obj1->guid)) {
200 200
                 // Not-duplicate parameter found, returning zero probability
201 201
                 continue;
Please login to merge, or discard this patch.
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -190,7 +190,8 @@
 block discarded – undo
190 190
             try {
191 191
                 $obj1 = $this->load($arr1['guid']);
192 192
                 $obj2 = $this->load($arr2['guid']);
193
-            } catch (midcom_error $e) {
193
+            }
194
+            catch (midcom_error $e) {
194 195
                 $e->log();
195 196
                 continue;
196 197
             }
Please login to merge, or discard this patch.
lib/org/openpsa/contacts/duplicates/check/person.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -42,7 +42,7 @@
 block discarded – undo
42 42
             // if user's lastname is part of the email address, check to see if the difference is only in the domain part
43 43
             $email1 = preg_replace('/@.+/', '', $person1['email']);
44 44
             $email2 = preg_replace('/@.+/', '', $person2['email']);
45
-            if (   str_contains($email1, $person1['lastname'])
45
+            if (str_contains($email1, $person1['lastname'])
46 46
                 && $email1 == $email2) {
47 47
                 $p += 0.5;
48 48
             }
Please login to merge, or discard this patch.