@@ -67,7 +67,8 @@ discard block |
||
67 | 67 | { |
68 | 68 | try { |
69 | 69 | $person = new org_openpsa_contacts_person_dba($member->person); |
70 | - } catch (midcom_error $e) { |
|
70 | + } |
|
71 | + catch (midcom_error $e) { |
|
71 | 72 | $e->log(); |
72 | 73 | return; |
73 | 74 | } |
@@ -91,7 +92,8 @@ discard block |
||
91 | 92 | $row['organization_member'] = $membership; |
92 | 93 | try { |
93 | 94 | $row['organization'] = org_openpsa_contacts_group_dba::get_cached($membership->gid); |
94 | - } catch (midcom_error) { |
|
95 | + } |
|
96 | + catch (midcom_error) { |
|
95 | 97 | debug_add("Error fetching org_openpsa_contacts_group_dba #{$membership->gid}, skipping", MIDCOM_LOG_WARN); |
96 | 98 | continue; |
97 | 99 | } |
@@ -37,7 +37,8 @@ discard block |
||
37 | 37 | // Add person to campaign |
38 | 38 | try { |
39 | 39 | $campaign = new org_openpsa_directmarketing_campaign_dba($add_to); |
40 | - } catch (midcom_error) { |
|
40 | + } |
|
41 | + catch (midcom_error) { |
|
41 | 42 | // FIXME: More informative error message |
42 | 43 | $this->notify('Failed adding person %s to campaign %s', $add_to, 'error'); |
43 | 44 | return; |
@@ -84,7 +85,8 @@ discard block |
||
84 | 85 | try { |
85 | 86 | $campaigns[$membership->campaign] = new org_openpsa_directmarketing_campaign_dba($membership->campaign); |
86 | 87 | $campaign_membership_map[$membership->campaign] = $membership; |
87 | - } catch (midcom_error $e) { |
|
88 | + } |
|
89 | + catch (midcom_error $e) { |
|
88 | 90 | debug_add('Failed to load campaign ' . $membership->campaign . ', reason: ' . $e->getMessage()); |
89 | 91 | } |
90 | 92 | } |
@@ -74,7 +74,8 @@ |
||
74 | 74 | $productgroup = new org_openpsa_products_product_group_dba($this->_product->productGroup); |
75 | 75 | $replacements['<PRODUCTGROUP_TITLE>'] = $productgroup->title; |
76 | 76 | $replacements['<PRODUCTGROUP_CODE>'] = $productgroup->code; |
77 | - } catch (midcom_error) { |
|
77 | + } |
|
78 | + catch (midcom_error) { |
|
78 | 79 | $replacements['<PRODUCTGROUP_TITLE>'] = ''; |
79 | 80 | $replacements['<PRODUCTGROUP_CODE>'] = ''; |
80 | 81 | } |
@@ -6,7 +6,8 @@ |
||
6 | 6 | foreach ($data['prospects'] as $prospect) { |
7 | 7 | try { |
8 | 8 | $person = org_openpsa_contacts_person_dba::get_cached($prospect->person); |
9 | - } catch (midcom_error) { |
|
9 | + } |
|
10 | + catch (midcom_error) { |
|
10 | 11 | continue; |
11 | 12 | } ?> |
12 | 13 | <person> |
@@ -47,7 +47,8 @@ |
||
47 | 47 | if ($node_id) { |
48 | 48 | try { |
49 | 49 | $comments_topic = new midcom_db_topic($node_id); |
50 | - } catch (midcom_error) { |
|
50 | + } |
|
51 | + catch (midcom_error) { |
|
51 | 52 | return null; |
52 | 53 | } |
53 | 54 |
@@ -29,7 +29,8 @@ |
||
29 | 29 | foreach ($qb->execute() as $feed) { |
30 | 30 | try { |
31 | 31 | midcom_db_topic::get_cached($feed->node); |
32 | - } catch (midcom_error) { |
|
32 | + } |
|
33 | + catch (midcom_error) { |
|
33 | 34 | debug_add("Node #{$feed->node} does not exist, skipping feed #{$feed->id}", MIDCOM_LOG_ERROR); |
34 | 35 | continue; |
35 | 36 | } |
@@ -17,7 +17,8 @@ |
||
17 | 17 | foreach ($data['objects'] as $guid => $privs) { |
18 | 18 | try { |
19 | 19 | $object = new midcom_db_topic($guid); |
20 | - } catch (midcom_error) { |
|
20 | + } |
|
21 | + catch (midcom_error) { |
|
21 | 22 | continue; |
22 | 23 | } |
23 | 24 | echo "<tr>\n"; |
@@ -185,7 +185,8 @@ |
||
185 | 185 | |
186 | 186 | try { |
187 | 187 | $account = new midcom_core_account($person); |
188 | - } catch (midcom_error) { |
|
188 | + } |
|
189 | + catch (midcom_error) { |
|
189 | 190 | midcom::get()->uimessages->add($this->_l10n->get('midgard.admin.user'), sprintf($this->_l10n->get('failed to get the user with id %s'), $person->id), 'error'); |
190 | 191 | return; |
191 | 192 | } |
@@ -26,7 +26,8 @@ discard block |
||
26 | 26 | try { |
27 | 27 | $person = new midcom_db_person($this->uid); |
28 | 28 | $grp = new midcom_db_group($this->gid); |
29 | - } catch (midcom_error $e) { |
|
29 | + } |
|
30 | + catch (midcom_error $e) { |
|
30 | 31 | $e->log(); |
31 | 32 | return 'Invalid membership record'; |
32 | 33 | } |
@@ -43,7 +44,8 @@ discard block |
||
43 | 44 | } |
44 | 45 | try { |
45 | 46 | $person = new midcom_db_person($this->uid); |
46 | - } catch (midcom_error) { |
|
47 | + } |
|
48 | + catch (midcom_error) { |
|
47 | 49 | return; |
48 | 50 | } |
49 | 51 | midcom::get()->cache->invalidate($person->guid); |