flack / openpsa

lib/org/openpsa/invoices/handler/projects.php

Braces +6 added lines, -3 removed lines

@@ -51,7 +51,8 @@ 
             try {
                 $deliverable = org_openpsa_sales_salesproject_deliverable_dba::get_cached($task->get_agreement());
                 $item->deliverable = $deliverable->id;
-            } catch (midcom_error $e) {
+            }
+            catch (midcom_error $e) {
                 $e->log();
             }
             $item->invoice = $invoice->id;
@@ -133,7 +134,8 @@ 
                 $customer = org_openpsa_contacts_group_dba::get_cached($customer_id);
                 $data['customer_label'] = $customer->official;
                 $data['disabled'] = '';
-            } catch (midcom_error $e) {
+            }
+            catch (midcom_error $e) {
                 $data['customer_label'] = $this->_l10n->get('no customer');
                 $data['disabled'] = ' disabled="disabled"';
             }
@@ -156,7 +158,8 @@ 
                     } else {
                         $data['invoiceable_units'] = $task->plannedHours;
                     }
-                } catch (midcom_error $e) {
+                }
+                catch (midcom_error $e) {
                     $e->log();
                     if ($this->_config->get('default_hourly_price')) {
                         $data['default_price'] = $this->_config->get('default_hourly_price');

lib/org/openpsa/invoices/invoice.php

Spacing +1 added lines, -1 removed lines

@@ -237,7 +237,7 @@
                 $invoice_item->pricePerUnit = $deliverable->pricePerUnit;
                 $invoice_item->deliverable = $deliverable->id;
                 //calculate price
-                if (   $deliverable->invoiceByActualUnits
+                if ($deliverable->invoiceByActualUnits
                     || $deliverable->plannedUnits == 0) {
                     $invoice_item->units = $hours;
                 } else {

Braces +6 added lines, -3 removed lines

@@ -128,7 +128,8 @@ 
             try {
                 $task = new org_openpsa_projects_task_dba($id);
                 $task->update_cache();
-            } catch (midcom_error $e) {
+            }
+            catch (midcom_error $e) {
             }
         }
 
@@ -293,14 +294,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/connection.php

Braces +2 added lines, -1 removed lines

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

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;

lib/midcom/core/dbaobject.php

Braces +2 added lines, -1 removed lines

@@ -93,7 +93,8 @@
             try {
                 $mgdschemaclass = $this->__mgdschema_class_name__;
                 $this->__object = new $mgdschemaclass($id);
-            } catch (mgd_exception $e) {
+            }
+            catch (mgd_exception $e) {
                 debug_add('Constructing ' . $this->__mgdschema_class_name__ . ' object ' . $id . ' failed, reason: ' . $e->getMessage(), MIDCOM_LOG_WARN);
                 throw new midcom_error_midgard($e, $id);
             }

Spacing +2 added lines, -2 removed lines

@@ -106,7 +106,7 @@ 
         if (is_object($id)) {
             $this->__object = midcom::get()->dbfactory->convert_midcom_to_midgard($id);
         } else {
-            if (   is_int($id)
+            if (is_int($id)
                 && $id < 1) {
                 throw new midcom_error($id . ' is not a valid database ID');
             }
@@ -120,7 +120,7 @@ 
             }
 
             //Some useful information for performance tuning
-            if (   midcom::get()->config->get('log_level') >= MIDCOM_LOG_DEBUG
+            if (midcom::get()->config->get('log_level') >= MIDCOM_LOG_DEBUG
                 && $this->__object->guid) {
                 static $guids = [];
                 static $total = 0;

lib/midcom/helper/_dbfactory.php

Braces +8 added lines, -4 removed lines

@@ -44,7 +44,8 @@ 
     {
         try {
             $tmp = midgard_object_class::get_object_by_guid($guid);
-        } catch (mgd_exception $e) {
+        }
+        catch (mgd_exception $e) {
             debug_add('Loading object by GUID ' . $guid . ' failed, reason: ' . $e->getMessage(), MIDCOM_LOG_INFO);
 
             throw new midcom_error_midgard($e, $guid);
@@ -237,7 +238,8 @@ 
                 $parent_data = [
                     $parent->__midcom_class_name__ => $parent_guid
                 ];
-            } catch (midcom_error $e) {
+            }
+            catch (midcom_error $e) {
                 $parent_data = [
                     '' => null
                 ];
@@ -251,7 +253,8 @@ 
 
         try {
             return $this->get_cached($classname, $parent_guid);
-        } catch (midcom_error $e) {
+        }
+        catch (midcom_error $e) {
             return null;
         }
     }
@@ -310,7 +313,8 @@ 
                 if ($the_object === null) {
                     try {
                         $the_object = $this->get_object_by_guid($object_guid);
-                    } catch (midcom_error $e) {
+                    }
+                    catch (midcom_error $e) {
                         return ['' => null];
                     }
                 }

Spacing +6 added lines, -6 removed lines

@@ -48,7 +48,7 @@ 
             throw new midcom_error_midgard($e, $guid);
         }
         $person_class = midcom::get()->config->get('person_class');
-        if (   get_class($tmp) == 'midgard_person'
+        if (get_class($tmp) == 'midgard_person'
             && $person_class != 'midgard_person') {
             $tmp = new $person_class($guid);
         }
@@ -79,7 +79,7 @@ 
         }
         $object = new $classname($src);
         $cache[$classname][$object->guid] = $object;
-        $cache[$classname][$object->id] =& $cache[$classname][$object->guid];
+        $cache[$classname][$object->id] = & $cache[$classname][$object->guid];
         return $cache[$classname][$object->guid];
     }
 
@@ -182,14 +182,14 @@ 
             return true;
         }
 
-        if (   isset($object->__object)
+        if (isset($object->__object)
             && is_object($object->__object)
             && $object->__object instanceof $class) {
             // Decorator whose MgdSchema object matches
             return true;
         }
 
-        if (   isset($object->__mgdschema_class_name__)
+        if (isset($object->__mgdschema_class_name__)
             && $object->__mgdschema_class_name__ == $class) {
             // Decorator without object instantiated, check class match
             return true;
@@ -212,7 +212,7 @@ 
         $parent_data = $this->_get_parent_guid_cached($object->guid, $object);
 
         $parent_guid = current($parent_data);
-        if (   empty($parent_guid)
+        if (empty($parent_guid)
             || $parent_guid === $object->guid) {
             return null;
         }
@@ -415,7 +415,7 @@ 
                 ];
             }
 
-            if (   $parent_property
+            if ($parent_property
                 && $reflector->get_link_target($parent_property)) {
                 $target_class = $reflector->get_link_name($parent_property);
                 if ($target_class == 'midgard_person') {

lib/midcom/core/group.php

Braces +2 added lines, -1 removed lines

@@ -94,7 +94,8 @@
             }
             try {
                 $this->_storage = new midgard_group($id);
-            } catch (Exception $e) {
+            }
+            catch (Exception $e) {
                 debug_add('Tried to load a midcom_core_group, but got error ' . $e->getMessage(), MIDCOM_LOG_ERROR);
                 debug_print_r('Passed argument was:', $id);
                 throw new midcom_error($e->getMessage());

lib/org/openpsa/relatedto/exec/delete_dangling_links.php

Braces +2 added lines, -1 removed lines

@@ -27,7 +27,8 @@
     try {
         midcom::get()->dbfactory->get_object_by_guid($result->fromGuid);
         midcom::get()->dbfactory->get_object_by_guid($result->toGuid);
-    } catch (midcom_error $e) {
+    }
+    catch (midcom_error $e) {
         echo $i . "/" . $total . ": Deleting relatedto #" . $result->id . "\n";
         $result->delete();
     }

lib/net/nemein/rss/cron/fetchfeeds.php

Braces +2 added lines, -1 removed lines

@@ -28,7 +28,8 @@
         foreach ($qb->execute() as $feed) {
             try {
                 midcom_db_topic::get_cached($feed->node);
-            } catch (midcom_error $e) {
+            }
+            catch (midcom_error $e) {
                 debug_add("Node #{$feed->node} does not exist, skipping feed #{$feed->id}", MIDCOM_LOG_ERROR);
                 continue;
             }

lib/net/nehmer/blog/midcom/interfaces.php

Braces +2 added lines, -1 removed lines

@@ -32,7 +32,8 @@
             foreach ($qb->execute() as $article) {
                 try {
                     $dm->set_storage($article);
-                } catch (midcom_error $e) {
+                }
+                catch (midcom_error $e) {
                     $e->log(MIDCOM_LOG_WARN);
                     continue;
                 }