@@ -136,7 +136,8 @@ |
||
| 136 | 136 | { |
| 137 | 137 | $ret = array('org_openpsa_relatedto' => array()); |
| 138 | 138 | foreach ($array as $rel) { |
| 139 | - if (!midcom::get()->dbfactory->is_a($rel, 'org_openpsa_relatedto_dba')) { //Matches also 'org_openpsa_relatedto' |
|
| 139 | + if (!midcom::get()->dbfactory->is_a($rel, 'org_openpsa_relatedto_dba')) { |
|
| 140 | +//Matches also 'org_openpsa_relatedto' |
|
| 140 | 141 | //Wrong type of object found in array, cruelly abort the whole procedure |
| 141 | 142 | return false; |
| 142 | 143 | } |
@@ -30,7 +30,8 @@ |
||
| 30 | 30 | try { |
| 31 | 31 | midcom::get()->dbfactory->get_object_by_guid($result->fromGuid); |
| 32 | 32 | midcom::get()->dbfactory->get_object_by_guid($result->toGuid); |
| 33 | - } catch (midcom_error $e) { |
|
| 33 | + } |
|
| 34 | + catch (midcom_error $e) { |
|
| 34 | 35 | echo $i . "/" . $total . ": Deleting relatedto #" . $result->id . "\n"; |
| 35 | 36 | flush(); |
| 36 | 37 | $result->delete(); |
@@ -42,7 +42,8 @@ |
||
| 42 | 42 | } |
| 43 | 43 | try { |
| 44 | 44 | $resource = org_openpsa_calendar_resource_dba::get_cached($this->resource); |
| 45 | - } catch (midcom_error $e) { |
|
| 45 | + } |
|
| 46 | + catch (midcom_error $e) { |
|
| 46 | 47 | debug_add("Cannot fetch resource #{$this->resource} returning false", MIDCOM_LOG_INFO); |
| 47 | 48 | return false; |
| 48 | 49 | } |
@@ -66,7 +66,8 @@ discard block |
||
| 66 | 66 | try { |
| 67 | 67 | $task = org_openpsa_projects_task_dba::get_cached($this->task); |
| 68 | 68 | $url = $projects_url . 'task/' . $task->guid . '/'; |
| 69 | - } catch (midcom_error $e) { |
|
| 69 | + } |
|
| 70 | + catch (midcom_error $e) { |
|
| 70 | 71 | } |
| 71 | 72 | } |
| 72 | 73 | if ( $url == '' |
@@ -74,7 +75,8 @@ discard block |
||
| 74 | 75 | try { |
| 75 | 76 | $deliverable = org_openpsa_sales_salesproject_deliverable_dba::get_cached($this->deliverable); |
| 76 | 77 | $url = $sales_url . 'deliverable/' . $deliverable->guid . '/'; |
| 77 | - } catch (midcom_error $e) { |
|
| 78 | + } |
|
| 79 | + catch (midcom_error $e) { |
|
| 78 | 80 | } |
| 79 | 81 | } |
| 80 | 82 | if ($url != '') { |
@@ -96,7 +98,8 @@ discard block |
||
| 96 | 98 | $deliverable = new org_openpsa_sales_salesproject_deliverable_dba($this->deliverable); |
| 97 | 99 | self::update_deliverable($deliverable); |
| 98 | 100 | } |
| 99 | - } catch (midcom_error $e) { |
|
| 101 | + } |
|
| 102 | + catch (midcom_error $e) { |
|
| 100 | 103 | $e->log(); |
| 101 | 104 | } |
| 102 | 105 | } |
@@ -174,7 +174,8 @@ |
||
| 174 | 174 | try { |
| 175 | 175 | midcom_db_topic::get_cached($node_guid); |
| 176 | 176 | $this->_available_generators[$component] = $loc; |
| 177 | - } catch (midcom_error $e) { |
|
| 177 | + } |
|
| 178 | + catch (midcom_error $e) { |
|
| 178 | 179 | debug_add("topic for component '{$component}' not found or accessible"); |
| 179 | 180 | } |
| 180 | 181 | } |
@@ -89,7 +89,8 @@ |
||
| 89 | 89 | // ->source will always contain the document GUID |
| 90 | 90 | $data['document'] = new org_openpsa_documents_document_dba($document->source); |
| 91 | 91 | $this->_datamanagers['document']->autoset_storage($data['document']); |
| 92 | - } catch (Exception $e) { |
|
| 92 | + } |
|
| 93 | + catch (Exception $e) { |
|
| 93 | 94 | $e->log(); |
| 94 | 95 | continue; |
| 95 | 96 | } |
@@ -107,7 +107,8 @@ |
||
| 107 | 107 | if (!empty($_POST['midcom_helper_datamanager2_save'])) { |
| 108 | 108 | try { |
| 109 | 109 | $rules = $this->_load_rules(); |
| 110 | - } catch (midcom_error $e) { |
|
| 110 | + } |
|
| 111 | + catch (midcom_error $e) { |
|
| 111 | 112 | midcom::get()->uimessages->add('org.openpsa.directmarketing', $this->_l10n->get($e->getMessage()), 'error'); |
| 112 | 113 | return; |
| 113 | 114 | } |
@@ -23,7 +23,8 @@ |
||
| 23 | 23 | foreach ($this->_try_backends as $backend) { |
| 24 | 24 | try { |
| 25 | 25 | $this->_backend = org_openpsa_mail_backend::get($backend, $params); |
| 26 | - } catch (midcom_error $e) { |
|
| 26 | + } |
|
| 27 | + catch (midcom_error $e) { |
|
| 27 | 28 | debug_add('Failed to load backend ' . $backend . ', message:' . $e->getMessage()); |
| 28 | 29 | } |
| 29 | 30 | } |
@@ -42,7 +42,8 @@ discard block |
||
| 42 | 42 | $object = self::_load_backend($backend, $params); |
| 43 | 43 | debug_add('Using backend ' . $backend); |
| 44 | 44 | return $object; |
| 45 | - } catch (midcom_error $e) { |
|
| 45 | + } |
|
| 46 | + catch (midcom_error $e) { |
|
| 46 | 47 | debug_add('Failed to load backend ' . $backend . ', message:' . $e->getMessage()); |
| 47 | 48 | } |
| 48 | 49 | } |
@@ -67,7 +68,8 @@ discard block |
||
| 67 | 68 | $ret = $this->mail($message); |
| 68 | 69 | $this->error = false; |
| 69 | 70 | return $ret; |
| 70 | - } catch (Exception $e) { |
|
| 71 | + } |
|
| 72 | + catch (Exception $e) { |
|
| 71 | 73 | $this->error = $e->getMessage(); |
| 72 | 74 | return false; |
| 73 | 75 | } |