flack / openpsa

lib/org/openpsa/calendar/conflictmanager.php

Spacing +4 added lines, -4 removed lines

@@ -125,7 +125,7 @@ 
         }
         // TODO: Shared tasks need a separate check (different member object)
 
-        if (   !empty($this->busy_members)
+        if (!empty($this->busy_members)
             || !empty($this->busy_resources)) {
             //Unresolved conflicts (note return value is for conflicts not lack of them)
             midcom::get()->auth->drop_sudo();
@@ -188,7 +188,7 @@ 
         }
 
         if (array_key_exists($member->event, $modified_events)) {
-            $event =& $modified_events[$member->event];
+            $event = & $modified_events[$member->event];
             $set_as_modified = false;
         } else {
             try {
@@ -202,7 +202,7 @@ 
         }
         debug_add("overlap found in event {$event->title} (#{$event->id})");
 
-        if (   $event->tentative
+        if ($event->tentative
             && $rob_tentative) {
             debug_add('event is tentative, robbing resources');
             $event->resources = array_diff_key($event->resources, $this->_event->resources);
@@ -229,7 +229,7 @@ 
         }
         debug_add("overlap found in event {$event->title} (#{$event->id})");
 
-        if (   $event->tentative
+        if ($event->tentative
             && $rob_tentative) {
             debug_add('event is tentative, robbing participants');
             $event->participants = array_diff_key($event->participants, $this->_event->participants);

Braces +4 added lines, -2 removed lines

@@ -142,7 +142,8 @@ 
             try {
                 $event = new org_openpsa_calendar_event_dba($member->event);
                 $set_as_modified = true;
-            } catch (midcom_error) {
+            }
+            catch (midcom_error) {
                 debug_add("event_resource #{$member->id} links to bogus event #{$member->event}, skipping and removing", MIDCOM_LOG_WARN);
                 $member->delete();
                 return;
@@ -170,7 +171,8 @@ 
 
         try {
             $event = new org_openpsa_calendar_event_dba($member->eid);
-        } catch (midcom_error) {
+        }
+        catch (midcom_error) {
             debug_add("eventmember #{$member->id} links to bogus event #{$member->eid}, skipping and removing", MIDCOM_LOG_WARN);
             $member->delete();
             return;

lib/org/openpsa/notifications/main.php

Spacing +1 added lines, -1 removed lines

@@ -102,7 +102,7 @@
 
         // If user has preference for this message, we use that
         $personal_preferences = $recipient->list_parameters('org.openpsa.notifications');
-        if (   !empty($personal_preferences)
+        if (!empty($personal_preferences)
             && array_key_exists("{$component}:{$action}", $personal_preferences)) {
             return $personal_preferences[$action];
         }

Braces +2 added lines, -1 removed lines

@@ -58,7 +58,8 @@
         // TODO: Should we sudo here to ensure getting correct prefs regardless of ACLs?
         try {
             $recipient = midcom_db_person::get_cached($recipient);
-        } catch (midcom_error) {
+        }
+        catch (midcom_error) {
             return false;
         }
 

lib/org/openpsa/projects/project.php

Spacing +1 added lines, -1 removed lines

@@ -249,7 +249,7 @@
             $new_status = $this->_find_status($this->_status_map[$this->status_type], $status_types);
         }
 
-        if (   $new_status !== null
+        if ($new_status !== null
             && $this->status != $new_status) {
             $this->status = $new_status;
             $update_required = true;

lib/midcom/connection.php

Spacing +1 added lines, -1 removed lines

@@ -261,7 +261,7 @@
             if ($part === '') {
                 continue;
             }
-            if (    midcom::get()->config->get('theme')
+            if (midcom::get()->config->get('theme')
                  && !$args_started
                  && self::check_page_exists($part)) {
                 $page_style .= '/' . $part;

Braces +2 added lines, -1 removed lines

@@ -84,7 +84,8 @@
 
         try {
             $user = new midgard_user($login_tokens);
-        } catch (mgd_exception) {
+        }
+        catch (mgd_exception) {
             return null;
         }
         if (!$trusted && !self::verify_password($password, $user->password)) {

lib/midcom/admin/help/style/midcom_admin_help_mgdschemas.php

Spacing +1 added lines, -1 removed lines

@@ -15,7 +15,7 @@
 
         foreach ($properties as $propname => $val) {
             $description = preg_replace('/ *\n */', "\n", $val['value']);
-            if (   $val['link']
+            if ($val['link']
                 && $linked_component = midcom::get()->dbclassloader->get_component_for_class($val['link_name'])) {
                 $proplink = $data['router']->generate('help', ['component' => $linked_component, 'help_id' => 'mgdschemas']);
                 $proplink = "<a href='{$proplink}#{$val['link_name']}' title='{$linked_component}/{$val['link_name']}::{$val['link_target']}'>{$val['link_name']}:{$val['link_target']}</a>";

lib/org/openpsa/projects/task.php

Spacing +1 added lines, -1 removed lines

@@ -110,7 +110,7 @@
     public function _on_updated()
     {
         // Sync the object's ACL properties into MidCOM ACL system
-        if (   !$this->_skip_acl_refresh) {
+        if (!$this->_skip_acl_refresh) {
             if ($this->orgOpenpsaAccesstype && $this->orgOpenpsaOwnerWg) {
                 debug_add("Synchronizing task ACLs to MidCOM");
                 $sync = new org_openpsa_core_acl_synchronizer();

lib/net/nehmer/blog/handler/admin.php

Spacing +1 added lines, -1 removed lines

@@ -51,7 +51,7 @@
         $this->article->require_do('midgard:update');
 
         $schemadb = $data['schemadb'];
-        if (   $this->_config->get('simple_name_handling')
+        if ($this->_config->get('simple_name_handling')
             && !midcom::get()->auth->can_user_do('midcom:urlname')) {
             foreach ($schemadb->all() as $schema) {
                 $schema->get_field('name')['readonly'] = true;

lib/org/openpsa/invoices/handler/invoice/view.php

Spacing +1 added lines, -1 removed lines

@@ -107,7 +107,7 @@
 
             // sending per email enabled in billing data?
             $billing_data = org_openpsa_invoices_billing_data_dba::get_by_object($this->invoice);
-            if (    !$this->invoice->sent
+            if (!$this->invoice->sent
                  && intval($billing_data->sendingoption) == 2) {
                 $buttons[] = $this->build_button('send_by_mail', 'paper-plane');
             }

src/midcom/datamanager/extension/type/imageType.php

Spacing +2 added lines, -2 removed lines

@@ -46,7 +46,7 @@ 
             ]
         ]);
 
-        $resolver->setNormalizer('constraints', function (Options $options, $value) {
+        $resolver->setNormalizer('constraints', function(Options $options, $value) {
             if ($options['required']) {
                 return [new constraint(['config' => $options['type_config']])];
             }
@@ -78,7 +78,7 @@ 
 
         $builder->add('delete', CheckboxType::class, ['attr' => [
             "class" => "midcom_datamanager_photo_checkbox"
-        ], "required" => false ]);
+        ], "required" => false]);
         $builder->add('identifier', HiddenType::class);
     }