flack / openpsa

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

Braces +6 added lines, -3 removed lines

@@ -49,7 +49,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;
@@ -121,7 +122,8 @@ 
                 $customer = org_openpsa_contacts_group_dba::get_cached($customer_id);
                 $data['customer_label'] = $customer->official;
                 $data['disabled'] = '';
-            } catch (midcom_error) {
+            }
+            catch (midcom_error) {
                 $data['customer_label'] = $this->_l10n->get('no customer');
                 $data['disabled'] = ' disabled="disabled"';
             }
@@ -144,7 +146,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/slideshow/style/edit.php

Braces +2 added lines, -1 removed lines

@@ -21,7 +21,8 @@
         $original = new midcom_db_attachment($image->attachment);
         $original_url = midcom_db_attachment::get_url($original);
         $name = $original->name;
-    } catch (midcom_error) {
+    }
+    catch (midcom_error) {
         $url = MIDCOM_STATIC_URL . '/stock-icons/mime/gnome-text-blank.png';
         $name = $data['l10n']->get('attachment missing');
         $original_url = '';

lib/org/openpsa/reports/viewer.php

Braces +2 added lines, -1 removed lines

@@ -98,7 +98,8 @@
             $node_guid = $siteconfig->get_node_guid($component);
             try {
                 $available_generators[$component] = midcom_db_topic::get_cached($node_guid)->get_label();
-            } catch (midcom_error) {
+            }
+            catch (midcom_error) {
                 debug_add("topic for component '{$component}' not found or accessible");
             }
         }

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

Braces +2 added lines, -1 removed lines

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

lib/org/openpsa/relatedto/handler/relatedto.php

Braces +6 added lines, -3 removed lines

@@ -107,7 +107,8 @@ 
             //TODO: check for duplicates ?
             try {
                 $result = ['other_obj' => midcom::get()->dbfactory->get_object_by_guid($link[$other . 'Guid'])];
-            } catch (midcom_error) {
+            }
+            catch (midcom_error) {
                 continue;
             }
             $result['link'] = [
@@ -378,7 +379,8 @@ 
             if (!($this->_object instanceof org_openpsa_relatedto_dba)) {
                 $response['status'] = "method requires guid of a link object as an argument";
             }
-        } catch (midcom_error $e) {
+        }
+        catch (midcom_error $e) {
             $response['status'] = "error: " . $e->getMessage();
         }
 
@@ -399,7 +401,8 @@ 
             $relation = new org_openpsa_relatedto_dba($guid);
             $result = $relation->delete();
             $status = 'Last message: ' . midcom_connection::get_error_string();
-        } catch (midcom_error $e) {
+        }
+        catch (midcom_error $e) {
             $result = false;
             $status = "Object '{$guid}' could not be loaded, error:" . $e->getMessage();
         }

Spacing +2 added lines, -2 removed lines

@@ -185,10 +185,10 @@
      *
      * @param array $link The necessary link information
      */
-    private function _render_line(array $link, midcom_core_dbaobject &$other_obj)
+    private function _render_line(array $link, midcom_core_dbaobject & $other_obj)
     {
         $this->_request_data['link'] = $link;
-        $this->_request_data['other_obj'] =& $other_obj;
+        $this->_request_data['other_obj'] = & $other_obj;
         $this->_request_data['icon'] = midcom_helper_reflector::get_object_icon($other_obj);
 
         if ($other_obj::class != $link['class']) {

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

Braces +4 added lines, -2 removed lines

@@ -23,7 +23,8 @@ 
         midcom::get()->auth->request_sudo($this->_component);
         try {
             $person = new midcom_db_person($args['guid']);
-        } catch (midcom_error) {
+        }
+        catch (midcom_error) {
             $handler->print_error('Person with guid #' . $args['guid'] . ' does not exist');
             midcom::get()->auth->drop_sudo();
             return false;
@@ -31,7 +32,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/directmarketing/handler/export.php

Braces +4 added lines, -2 removed lines

@@ -67,7 +67,8 @@ 
     {
         try {
             $person = new org_openpsa_contacts_person_dba($member->person);
-        } catch (midcom_error $e) {
+        }
+        catch (midcom_error $e) {
             $e->log();
             return;
         }
@@ -91,7 +92,8 @@ 
                 $row['organization_member'] = $membership;
                 try {
                     $row['organization'] = org_openpsa_contacts_group_dba::get_cached($membership->gid);
-                } catch (midcom_error) {
+                }
+                catch (midcom_error) {
                     debug_add("Error fetching org_openpsa_contacts_group_dba #{$membership->gid}, skipping", MIDCOM_LOG_WARN);
                     continue;
                 }

lib/org/openpsa/directmarketing/handler/subscriber.php

Braces +4 added lines, -2 removed lines

@@ -37,7 +37,8 @@ 
             // Add person to campaign
             try {
                 $campaign = new org_openpsa_directmarketing_campaign_dba($add_to);
-            } catch (midcom_error) {
+            }
+            catch (midcom_error) {
                 // FIXME: More informative error message
                 $this->notify('Failed adding person %s to campaign %s', $add_to, 'error');
                 return;
@@ -84,7 +85,8 @@ 
             try {
                 $campaigns[$membership->campaign] = new org_openpsa_directmarketing_campaign_dba($membership->campaign);
                 $campaign_membership_map[$membership->campaign] = $membership;
-            } catch (midcom_error $e) {
+            }
+            catch (midcom_error $e) {
                 debug_add('Failed to load campaign ' . $membership->campaign . ', reason: ' . $e->getMessage());
             }
         }

lib/org/openpsa/products/handler/product/view.php

Braces +2 added lines, -1 removed lines

@@ -74,7 +74,8 @@
                 $productgroup = new org_openpsa_products_product_group_dba($this->_product->productGroup);
                 $replacements['<PRODUCTGROUP_TITLE>'] = $productgroup->title;
                 $replacements['<PRODUCTGROUP_CODE>'] = $productgroup->code;
-            } catch (midcom_error) {
+            }
+            catch (midcom_error) {
                 $replacements['<PRODUCTGROUP_TITLE>'] = '';
                 $replacements['<PRODUCTGROUP_CODE>'] = '';
             }