flack / openpsa

lib/org/openpsa/mail/message.php

Spacing +1 added lines, -1 removed lines

@@ -102,7 +102,7 @@
             if (is_string($value)) {
                 $this->_headers[$header] = trim($value);
             }
-            if (   strtolower($header) == 'from'
+            if (strtolower($header) == 'from'
                 || strtolower($header) == 'reply-to'
                 || strtolower($header) == 'to') {
                 $this->_headers[$header] = $this->_encode_address_field($value);

lib/org/openpsa/slideshow/image.php

Braces +4 added lines, -2 removed lines

@@ -41,7 +41,8 @@ 
 
         try {
             $folder = midcom_db_topic::get_cached($this->topic);
-        } catch (midcom_error $e) {
+        }
+        catch (midcom_error $e) {
             $e->log();
         }
         $folder->delete_attachment(self::FOLDER_THUMBNAIL);
@@ -51,7 +52,8 @@ 
     {
         try {
             return new midcom_db_attachment($this->$type);
-        } catch (midcom_error $e) {
+        }
+        catch (midcom_error $e) {
             $e->log();
             return false;
         }

Spacing +1 added lines, -1 removed lines

@@ -150,7 +150,7 @@
         $rows = $mc->get_rows(['id', 'name', 'guid'], 'id');
 
         foreach ($images as $image) {
-            if (   !isset($rows[$image->attachment])
+            if (!isset($rows[$image->attachment])
                 || !isset($rows[$image->image])
                 || !isset($rows[$image->thumbnail])) {
                 continue;

lib/org/openpsa/core/queryfilter.php

Spacing +2 added lines, -2 removed lines

@@ -77,9 +77,9 @@
         $filter_id = $this->_identifier . '_' . $filtername;
         $user = midcom::get()->auth->user->get_storage();
 
-        if (   isset($_POST['unset_filter'])
+        if (isset($_POST['unset_filter'])
             && $_POST['unset_filter'] == $filter_id) {
-            if (   $user->get_parameter("org_openpsa_core_filter", $filter_id)
+            if ($user->get_parameter("org_openpsa_core_filter", $filter_id)
                 && !$user->delete_parameter("org_openpsa_core_filter", $filter_id)) {
                 $message_content = sprintf(
                     $l10n->get('the handed filter for %s could not be set as parameter'),

lib/org/openpsa/helpers/main.php

Braces +2 added lines, -1 removed lines

@@ -55,7 +55,8 @@
              $guid = $parts[1];
              try {
                  $attachments[$parts[0]] = midcom_db_attachment::get_cached($guid);
-             } catch (midcom_error $e) {
+             }
+             catch (midcom_error $e) {
                  $e->log();
              }
          }

Indentation +26 added lines, -26 removed lines

@@ -40,31 +40,31 @@
         return $output;
     }
 
-     /**
-      * @param midcom_core_dbaobject $object The object we're working on
-      * @param string $field The schema field name
-      * @return midcom_db_attachment[] List of attachments, indexed by identifier
-      */
-     public static function get_dm2_attachments($object, $field) : array
-     {
-         $attachments = [];
-         $identifiers = explode(',', $object->get_parameter('midcom.helper.datamanager2.type.blobs', 'guids_' . $field));
-         if (empty($identifiers)) {
-             return $attachments;
-         }
-         foreach ($identifiers as $identifier) {
-             $parts = explode(':', $identifier);
-             if (count($parts) != 2) {
-                 continue;
-             }
-             $guid = $parts[1];
-             try {
-                 $attachments[$parts[0]] = midcom_db_attachment::get_cached($guid);
-             } catch (midcom_error $e) {
-                 $e->log();
-             }
-         }
+        /**
+         * @param midcom_core_dbaobject $object The object we're working on
+         * @param string $field The schema field name
+         * @return midcom_db_attachment[] List of attachments, indexed by identifier
+         */
+        public static function get_dm2_attachments($object, $field) : array
+        {
+            $attachments = [];
+            $identifiers = explode(',', $object->get_parameter('midcom.helper.datamanager2.type.blobs', 'guids_' . $field));
+            if (empty($identifiers)) {
+                return $attachments;
+            }
+            foreach ($identifiers as $identifier) {
+                $parts = explode(':', $identifier);
+                if (count($parts) != 2) {
+                    continue;
+                }
+                $guid = $parts[1];
+                try {
+                    $attachments[$parts[0]] = midcom_db_attachment::get_cached($guid);
+                } catch (midcom_error $e) {
+                    $e->log();
+                }
+            }
 
-         return $attachments;
-     }
+            return $attachments;
+        }
 }

lib/org/openpsa/contacts/cron/duplicates/clean.php

Braces +2 added lines, -1 removed lines

@@ -35,7 +35,8 @@
                 try {
                     midcom::get()->dbfactory->get_object_by_guid($param->name);
                     $tried[$param->name] = true;
-                } catch (midcom_error $e) {
+                }
+                catch (midcom_error $e) {
                     $tried[$param->name] = false;
                 }
             }

lib/org/openpsa/contacts/duplicates/merge.php

Spacing +6 added lines, -6 removed lines

@@ -45,7 +45,7 @@ 
      */
     public function merge($obj1, $obj2, $merge_mode)
     {
-        if (   $merge_mode !== 'all'
+        if ($merge_mode !== 'all'
             && $merge_mode !== 'future') {
             debug_add("invalid mode {$merge_mode}", MIDCOM_LOG_ERROR);
             throw new midcom_error('invalid merge mode');
@@ -87,9 +87,9 @@ 
                         if (!empty($conf['duplicate_check'])) {
                             $dup = $this->check_duplicate($results, $result, $conf['duplicate_check']);
 
-                            if (   is_object($dup)
+                            if (is_object($dup)
                                 || $dup === false) {
-                                if (   !is_object($dup)
+                                if (!is_object($dup)
                                     || !array_key_exists($dup->id, $todelete)) {
                                     $todelete[$result->id] = $result;
                                 }
@@ -98,7 +98,7 @@ 
                         }
                     }
                 }
-                if (   $needs_update
+                if ($needs_update
                     && !$result->update()) {
                     throw new midcom_error("Failed to update {$classname} #{$result->id}, errstr: " . midcom_connection::get_error_string());
                 }
@@ -147,7 +147,7 @@ 
         }
 
         foreach ($results as $result) {
-            if (   $result->$field === $object->$field
+            if ($result->$field === $object->$field
                 && $result->id !== $object->id) {
                 return $result;
             }
@@ -165,7 +165,7 @@ 
         $changed = false;
         foreach ($person2 as $property => $value) {
             // Copy only simple properties not marked to be skipped missing from person1
-            if (   empty($person2->$property)
+            if (empty($person2->$property)
                 || !empty($person1->$property)
                 || isset($skip_properties[$property])
                 || !is_scalar($value)) {

lib/org/openpsa/user/midcom/interfaces.php

Braces +4 added lines, -2 removed lines

@@ -25,7 +25,8 @@ 
         midcom::get()->auth->request_sudo($this->_component);
         try {
             $person = new midcom_db_person($args['guid']);
-        } catch (midcom_error $e) {
+        }
+        catch (midcom_error $e) {
             $handler->print_error('Person with guid #' . $args['guid'] . ' does not exist');
             midcom::get()->auth->drop_sudo();
             return false;
@@ -33,7 +34,8 @@ 
         $accounthelper = new org_openpsa_user_accounthelper($person);
         try {
             $accounthelper->reopen_account();
-        } catch (midcom_error $e) {
+        }
+        catch (midcom_error $e) {
             $handler->print_error($e->getMessage());
             midcom::get()->auth->drop_sudo();
             return false;

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();