flack / openpsa

lib/org/openpsa/projects/task/status.php

Spacing +1 added lines, -1 removed lines

@@ -63,7 +63,7 @@
     public function _on_created()
     {
         //Remove the resource if necessary
-        if (   $this->type == self::DECLINED
+        if ($this->type == self::DECLINED
             && $this->targetPerson) {
             $qb = org_openpsa_projects_task_resource_dba::new_query_builder();
             $qb->add_constraint('task', '=', $this->task);

Braces +2 added lines, -1 removed lines

@@ -95,7 +95,8 @@
                     org_openpsa_projects_workflow::accept($task, 0, $this->comment);
                     return;
                 }
-            } catch (midcom_error $e) {
+            }
+            catch (midcom_error $e) {
                 $e->log();
             }
         }

lib/org/openpsa/projects/status.php

Spacing +1 added lines, -1 removed lines

@@ -68,7 +68,7 @@
             $status_changer_label = $this->l10n->get('system');
             $target_person_label = $this->l10n->get('system');
 
-            if (    $status_change->metadata->creator
+            if ($status_change->metadata->creator
                  && $status_change->metadata->creator != $fallback_creator->guid) {
                 $status_changer = org_openpsa_widgets_contact::get($status_change->metadata->creator);
                 $status_changer_label = $status_changer->show_inline();

lib/org/openpsa/sales/salesproject.php

Spacing +3 added lines, -3 removed lines

@@ -90,7 +90,7 @@ 
             }
         }
         $profit = $value - $cost;
-        if (   $this->value != $value
+        if ($this->value != $value
             || $this->profit != $profit) {
             $this->value = $value;
             $this->profit = $value - $cost;
@@ -220,12 +220,12 @@ 
 
     public function _on_updating()
     {
-        if (   $this->state != self::STATE_ACTIVE
+        if ($this->state != self::STATE_ACTIVE
             && !$this->end) {
             //Not active anymore and end not set, set it to now
             $this->end = time();
         }
-        if (   $this->end
+        if ($this->end
             && $this->state == self::STATE_ACTIVE) {
             //Returned to active state, clear the end marker.
             $this->end = 0;

Braces +4 added lines, -2 removed lines

@@ -120,14 +120,16 @@
         if (!empty($this->customer)) {
             try {
                 return org_openpsa_contacts_group_dba::get_cached($this->customer);
-            } catch (midcom_error $e) {
+            }
+            catch (midcom_error $e) {
                 $e->log();
             }
         }
         if (!empty($this->customerContact)) {
             try {
                 return org_openpsa_contacts_person_dba::get_cached($this->customerContact);
-            } catch (midcom_error $e) {
+            }
+            catch (midcom_error $e) {
                 $e->log();
             }
         }

lib/midcom/db/member.php

Braces +4 added lines, -2 removed lines

@@ -23,7 +23,8 @@ 
         try {
             $person = new midcom_db_person($this->uid);
             $grp = new midcom_db_group($this->gid);
-        } catch (midcom_error $e) {
+        }
+        catch (midcom_error $e) {
             $e->log();
             return 'Invalid membership record';
         }
@@ -40,7 +41,8 @@ 
         }
         try {
             $person = new midcom_db_person($this->uid);
-        } catch (midcom_error $e) {
+        }
+        catch (midcom_error $e) {
             return;
         }
         midcom::get()->cache->invalidate($person->guid);

lib/midcom/helper/xsspreventer.php

Spacing +1 added lines, -1 removed lines

@@ -37,7 +37,7 @@
     {
         return preg_replace_callback(
             "%(<\s*)+(/\s*)+{$element}%i",
-            function ($matches) {
+            function($matches) {
                 return htmlentities($matches[0]);
             },
             $input

lib/midcom/helper/toolbar/node.php

Braces +2 added lines, -1 removed lines

@@ -88,7 +88,8 @@
                         $style = midcom_db_style::get_cached($style_id);
                         $styleeditor_url = midcom_connection::get_url('self') . "__mfa/asgard/object/view/{$style->guid}/";
                         $enabled = true;
-                    } catch (midcom_error $e) {
+                    }
+                    catch (midcom_error $e) {
                         $e->log();
                     }
                 }

Spacing +3 added lines, -3 removed lines

@@ -57,7 +57,7 @@ 
         }
         $buttons = [];
         $workflow = new midcom\workflow\datamanager;
-        if (   $this->topic->can_do('midgard:update')
+        if ($this->topic->can_do('midgard:update')
             && $this->topic->can_do('midcom.admin.folder:topic_management')) {
             $buttons[] = $workflow->get_button("__ais/folder/edit/", [
                 MIDCOM_TOOLBAR_LABEL => midcom::get()->i18n->get_string('edit folder', 'midcom.admin.folder'),
@@ -91,7 +91,7 @@ 
             ];
         }
         $buttons = array_merge($buttons, $this->get_approval_controls($this->topic, false));
-        if (   $this->topic->can_do('midcom.admin.folder:template_management')
+        if ($this->topic->can_do('midcom.admin.folder:template_management')
             && midcom::get()->auth->can_user_do('midgard.admin.asgard:manage_objects', null, 'midgard_admin_asgard_plugin')) {
             $enabled = false;
             $styleeditor_url = '';
@@ -124,7 +124,7 @@ 
                     MIDCOM_TOOLBAR_ACCESSKEY => 'f',
                 ]);
             }
-            if (   $this->topic->guid !== midcom::get()->config->get('midcom_root_topic_guid')
+            if ($this->topic->guid !== midcom::get()->config->get('midcom_root_topic_guid')
                 && $this->topic->can_do('midgard:delete')) {
                 $workflow = new midcom\workflow\delete(['object' => $this->topic, 'recursive' => true]);
                 $buttons[] = $workflow->get_button("__ais/folder/delete/", [

lib/midcom/style/midcom_helper_datamanager2_unlock.php

Braces +2 added lines, -1 removed lines

@@ -4,7 +4,8 @@
 try {
     $person = new midcom_db_person($metadata->locker);
     $name = $person->name;
-} catch (midcom_error $e) {
+}
+catch (midcom_error $e) {
     $name = $this->data['handler']->_l10n_midcom->get('unknown user');
     $e->log();
 }

lib/net/nemein/wiki/parser.php

Braces +2 added lines, -1 removed lines

@@ -63,7 +63,8 @@
 
         try {
             $attachment = new midcom_db_attachment($guid);
-        } catch (midcom_error $e) {
+        }
+        catch (midcom_error $e) {
             $e->log();
             return "<span class=\"missing_photo\" title=\"{$guid}\">{$fulltag}</span>{$after}";
         }

Spacing +3 added lines, -3 removed lines

@@ -223,7 +223,7 @@ 
                 return $this->render_link($parts[1], $parts[2]) . $after;
 
             // Macro [something: <data>] (for example [abbr: BOFH - Bastard Operator From Hell] or [photo: <GUID>])
-            case (   preg_match('/^(.*?): (.*)/', $fulltext, $macro_parts)
+            case (preg_match('/^(.*?): (.*)/', $fulltext, $macro_parts)
                   && method_exists($this, "_run_macro_{$macro_parts[1]}")):
                 $method = "_run_macro_{$macro_parts[1]}";
                 return $this->$method($macro_parts[2], $match[0], $after);
@@ -256,7 +256,7 @@ 
                 $folder = $wikipage_match['latest_parent'];
             }
 
-            if (   isset($folder[MIDCOM_NAV_OBJECT])
+            if (isset($folder[MIDCOM_NAV_OBJECT])
                 && $folder[MIDCOM_NAV_OBJECT]->can_do('midgard:create')) {
                 $workflow = $this->get_workflow('datamanager');
                 return "<a href=\"{$folder[MIDCOM_NAV_ABSOLUTEURL]}create/?wikiword={$wikipage_match['remaining_path']}\" " . $workflow->render_attributes() . " class=\"wiki_missing\" title=\"" . $this->_l10n->get('click to create') . "\">{$text}</a>";
@@ -291,7 +291,7 @@ 
                     $links[$parts[1]] = $parts[2];
                 }
                 // Ignore macros [something: <data>] (for example [abbr: BOFH - Bastard Operator From Hell] or [photo: <GUID>])
-                elseif (   !preg_match('/^(.*?): (.*)/', $fulltext, $macro_parts)
+                elseif (!preg_match('/^(.*?): (.*)/', $fulltext, $macro_parts)
                         || !method_exists($this, "_run_macro_{$macro_parts[1]}")) {
                     // MediaWiki-style link [wikipage] (no text)
                     $links[$fulltext] = $fulltext;

lib/net/nehmer/static/navigation.php

Spacing +1 added lines, -1 removed lines

@@ -46,7 +46,7 @@
         $qb->add_constraint('name', '<>', '');
 
         // Unless in Auto-Index mode or the index article is hidden, we skip the index article.
-        if (   !$this->_config->get('autoindex')
+        if (!$this->_config->get('autoindex')
             && !$this->_config->get('indexinnav')) {
             $qb->add_constraint('name', '<>', 'index');
         }