flack / openpsa

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

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/widgets/contact.php

Braces +4 added lines, -2 removed lines

@@ -103,7 +103,8 @@ 
 
         try {
             $person = midcom_db_person::get_cached($src);
-        } catch (midcom_error) {
+        }
+        catch (midcom_error) {
             return $cache[$src] = new self(new midcom_db_person);
         }
 
@@ -303,7 +304,8 @@ 
         foreach ($mc->get_rows(['gid', 'extra']) as $data) {
             try {
                 $group = org_openpsa_contacts_group_dba::get_cached($data['gid']);
-            } catch (midcom_error $e) {
+            }
+            catch (midcom_error $e) {
                 $e->log();
                 continue;
             }

lib/org/openpsa/directmarketing/sender.php

Braces +4 added lines, -2 removed lines

@@ -160,7 +160,8 @@ 
             $this->_backend->send($person, $member, $token, $subject, $content, $from);
             self::$_messages_sent++;
             $status = org_openpsa_directmarketing_campaign_messagereceipt_dba::SENT;
-        } catch (midcom_error $e) {
+        }
+        catch (midcom_error $e) {
             $status = org_openpsa_directmarketing_campaign_messagereceipt_dba::FAILURE;
             if (!$this->test_mode) {
                 $params[] = [
@@ -209,7 +210,8 @@ 
     {
         try {
             $person = org_openpsa_contacts_person_dba::get_cached($member->person);
-        } catch (midcom_error) {
+        }
+        catch (midcom_error) {
             debug_add("Person #{$member->person} deleted or missing, removing member (member #{$member->id})");
             $member->orgOpenpsaObtype = org_openpsa_directmarketing_campaign_member_dba::UNSUBSCRIBED;
             $member->delete();

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

Braces +4 added lines, -2 removed lines

@@ -51,7 +51,8 @@ 
         try {
             midcom::get()->dynamic_load($batch_url);
             $ret = true;
-        } catch (midcom_error $e) {
+        }
+        catch (midcom_error $e) {
             $ret = $e->getMessage();
         }
         ob_end_clean();
@@ -74,7 +75,8 @@ 
         $stat = true;
         try {
             $campaign = new org_openpsa_directmarketing_campaign_dba($args['campaign_guid']);
-        } catch (midcom_error) {
+        }
+        catch (midcom_error) {
             $handler->print_error("{$args['campaign_guid']} is not a valid campaign GUID");
             $stat = false;
         }

lib/org/openpsa/directmarketing/handler/message/report.php

Braces +2 added lines, -1 removed lines

@@ -114,7 +114,8 @@
                 try {
                     $person = org_openpsa_contacts_person_dba::get_cached($link->person);
                     $segment = $person->get_parameter('org.openpsa.directmarketing.segments', $segmentation_param);
-                } catch (midcom_error) {
+                }
+                catch (midcom_error) {
                 }
                 if (empty($segment)) {
                     $segment = $this->_l10n->get('no segment');

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