@@ -222,7 +222,8 @@ discard block |
||
| 222 | 222 | $this->_backend->send($person, $member, $token, $subject, $content, $from); |
| 223 | 223 | self::$_messages_sent++; |
| 224 | 224 | $status = org_openpsa_directmarketing_campaign_messagereceipt_dba::SENT; |
| 225 | - } catch (midcom_error $e) { |
|
| 225 | + } |
|
| 226 | + catch (midcom_error $e) { |
|
| 226 | 227 | $status = org_openpsa_directmarketing_campaign_messagereceipt_dba::FAILURE; |
| 227 | 228 | if (!$this->test_mode) { |
| 228 | 229 | $params[] = [ |
@@ -279,7 +280,8 @@ discard block |
||
| 279 | 280 | { |
| 280 | 281 | try { |
| 281 | 282 | $person = org_openpsa_contacts_person_dba::get_cached($member->person); |
| 282 | - } catch (midcom_error $e) { |
|
| 283 | + } |
|
| 284 | + catch (midcom_error $e) { |
|
| 283 | 285 | debug_add("Person #{$member->person} deleted or missing, removing member (member #{$member->id})"); |
| 284 | 286 | $member->orgOpenpsaObtype = org_openpsa_directmarketing_campaign_member_dba::UNSUBSCRIBED; |
| 285 | 287 | $member->delete(); |
@@ -210,8 +210,7 @@ |
||
| 210 | 210 | { |
| 211 | 211 | $ret = []; |
| 212 | 212 | |
| 213 | - foreach ($this->storage as $field => $value) |
|
| 214 | - { |
|
| 213 | + foreach ($this->storage as $field => $value) { |
|
| 215 | 214 | $ret[$field] = $value->get_value(); |
| 216 | 215 | $config = $this->schema->get_field($field); |
| 217 | 216 | if (!empty($config['type_config']['allow_multiple'])) { |
@@ -30,7 +30,8 @@ |
||
| 30 | 30 | $this->config = $config; |
| 31 | 31 | } |
| 32 | 32 | |
| 33 | - protected function cast($value) { |
|
| 33 | + protected function cast($value) |
|
| 34 | + { |
|
| 34 | 35 | if ($this->config['type'] == 'number' && !is_numeric($value)) { |
| 35 | 36 | $value = (float) $value; |
| 36 | 37 | } |
@@ -30,7 +30,8 @@ |
||
| 30 | 30 | foreach ($result as $wikipage) { |
| 31 | 31 | try { |
| 32 | 32 | $datamanager->set_storage($wikipage); |
| 33 | - } catch (midcom_error $e) { |
|
| 33 | + } |
|
| 34 | + catch (midcom_error $e) { |
|
| 34 | 35 | $e->log(MIDCOM_LOG_WARN); |
| 35 | 36 | continue; |
| 36 | 37 | } |
@@ -409,7 +409,8 @@ |
||
| 409 | 409 | foreach ($this->_articles as $article_counter => $article) { |
| 410 | 410 | try { |
| 411 | 411 | $data['datamanager']->set_storage($article); |
| 412 | - } catch (midcom_error $e) { |
|
| 412 | + } |
|
| 413 | + catch (midcom_error $e) { |
|
| 413 | 414 | $e->log(); |
| 414 | 415 | continue; |
| 415 | 416 | } |
@@ -7,7 +7,8 @@ |
||
| 7 | 7 | try { |
| 8 | 8 | $person = new midcom_db_person($metadata->locker); |
| 9 | 9 | $name = $person->name; |
| 10 | -} catch (midcom_error $e) { |
|
| 10 | +} |
|
| 11 | +catch (midcom_error $e) { |
|
| 11 | 12 | $name = $l10n_midcom->get('unknown user'); |
| 12 | 13 | $e->log(); |
| 13 | 14 | } |
@@ -12,7 +12,8 @@ |
||
| 12 | 12 | foreach ($data['wikilinks'] as $link) { |
| 13 | 13 | try { |
| 14 | 14 | $page = new net_nemein_wiki_wikipage($link->frompage); |
| 15 | - } catch (midcom_error $e) { |
|
| 15 | + } |
|
| 16 | + catch (midcom_error $e) { |
|
| 16 | 17 | $e->log(); |
| 17 | 18 | continue; |
| 18 | 19 | } |
@@ -35,7 +35,8 @@ |
||
| 35 | 35 | try { |
| 36 | 36 | $article = new midcom_db_article($related); |
| 37 | 37 | echo "<li><a href=\"" . midcom::get()->permalinks->create_permalink($article->guid) . "\">{$article->title}</a></li>\n"; |
| 38 | - } catch (midcom_error $e) { |
|
| 38 | + } |
|
| 39 | + catch (midcom_error $e) { |
|
| 39 | 40 | $e->log(); |
| 40 | 41 | } |
| 41 | 42 | } |
@@ -33,7 +33,8 @@ |
||
| 33 | 33 | $reporter_card = new org_openpsa_widgets_contact($reporter); |
| 34 | 34 | $entry['index_reporter'] = $reporter->rname; |
| 35 | 35 | $entry['reporter'] = $reporter_card->show_inline(); |
| 36 | - } catch (midcom_error $e) { |
|
| 36 | + } |
|
| 37 | + catch (midcom_error $e) { |
|
| 37 | 38 | $e->log(); |
| 38 | 39 | $entry['index_reporter'] = ''; |
| 39 | 40 | $entry['reporter'] = ''; |