| @@ -35,7 +35,7 @@ | ||
| 35 | 35 | </div> | 
| 36 | 36 | |
| 37 | 37 | <?php | 
| 38 | - //TODO: necessary JS stuff to load details (which should in turn include the invoice's own relatedtos) via AHAH | |
| 39 | - org_openpsa_relatedto_handler_relatedto::render_line_controls($link, $data['other_obj']); | |
| 40 | - ?> | |
| 38 | + //TODO: necessary JS stuff to load details (which should in turn include the invoice's own relatedtos) via AHAH | |
| 39 | + org_openpsa_relatedto_handler_relatedto::render_line_controls($link, $data['other_obj']); | |
| 40 | + ?> | |
| 41 | 41 | </li> | 
| @@ -27,7 +27,7 @@ | ||
| 27 | 27 | <div id="org_openpsa_relatedto_details_&(salesproject.guid);" class="details hidden" style="display: none;"> | 
| 28 | 28 | </div> | 
| 29 | 29 | <?php | 
| 30 | - //TODO: necessary JS stuff to load details (which should in turn include the invoice's own relatedtos) via AHAH | |
| 31 | - org_openpsa_relatedto_handler_relatedto::render_line_controls($link, $data['other_obj']); | |
| 32 | - ?> | |
| 30 | + //TODO: necessary JS stuff to load details (which should in turn include the invoice's own relatedtos) via AHAH | |
| 31 | + org_openpsa_relatedto_handler_relatedto::render_line_controls($link, $data['other_obj']); | |
| 32 | + ?> | |
| 33 | 33 | </li> | 
| @@ -31,7 +31,7 @@ | ||
| 31 | 31 | <div id="org_openpsa_relatedto_details_&(document.guid);" class="details hidden" style="display: none;"> | 
| 32 | 32 | </div> | 
| 33 | 33 | <?php | 
| 34 | - //TODO: get correct node and via it then handle details trough AHAH (and when we have node we can use proper link in document_url as well | |
| 35 | - org_openpsa_relatedto_handler_relatedto::render_line_controls($link, $data['other_obj']); | |
| 36 | - ?> | |
| 34 | + //TODO: get correct node and via it then handle details trough AHAH (and when we have node we can use proper link in document_url as well | |
| 35 | + org_openpsa_relatedto_handler_relatedto::render_line_controls($link, $data['other_obj']); | |
| 36 | + ?> | |
| 37 | 37 | </li> | 
| @@ -25,6 +25,6 @@ | ||
| 25 | 25 | </div> | 
| 26 | 26 | <?php | 
| 27 | 27 | //TODO: necessary JS stuff to load details (which should in turn include the events own relatedtos) via AHAH | 
| 28 | - org_openpsa_relatedto_handler_relatedto::render_line_controls($link, $data['other_obj']); | |
| 29 | - ?> | |
| 28 | + org_openpsa_relatedto_handler_relatedto::render_line_controls($link, $data['other_obj']); | |
| 29 | + ?> | |
| 30 | 30 | </li> | 
| @@ -407,7 +407,7 @@ | ||
| 407 | 407 | |
| 408 | 408 | $parent = $parent->get_parent_group(); | 
| 409 | 409 | } | 
| 410 | - */ | |
| 410 | + */ | |
| 411 | 411 | $this->_inheritance_chains[$id] = $inheritance_chain; | 
| 412 | 412 | } | 
| 413 | 413 | } | 
| @@ -182,7 +182,8 @@ discard block | ||
| 182 | 182 | } | 
| 183 | 183 |              try { | 
| 184 | 184 | $this->_storage = new $person_class($id); | 
| 185 | -            } catch (mgd_exception $e) { | |
| 185 | + } | |
| 186 | +            catch (mgd_exception $e) { | |
| 186 | 187 |                  debug_add("Failed to retrieve the person ID {$id}: " . $e->getMessage(), MIDCOM_LOG_INFO); | 
| 187 | 188 | throw new midcom_error_midgard($e, $id); | 
| 188 | 189 | } | 
| @@ -230,7 +231,8 @@ discard block | ||
| 230 | 231 |          if (mgd_is_guid($id)) { | 
| 231 | 232 |              try { | 
| 232 | 233 | return new $person_class($id); | 
| 233 | -            } catch (mgd_exception $e) { | |
| 234 | + } | |
| 235 | +            catch (mgd_exception $e) { | |
| 234 | 236 |                  debug_add("Failed to retrieve the person GUID {$id}: " . $e->getMessage(), MIDCOM_LOG_INFO); | 
| 235 | 237 | throw new midcom_error_midgard($e, $id); | 
| 236 | 238 | } | 
| @@ -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 | } | 
| @@ -25,7 +25,8 @@ discard block | ||
| 25 | 25 | midcom::get()->auth->request_sudo($this->_component); | 
| 26 | 26 |          try { | 
| 27 | 27 | $person = new midcom_db_person($args['guid']); | 
| 28 | -        } catch (midcom_error $e) { | |
| 28 | + } | |
| 29 | +        catch (midcom_error $e) { | |
| 29 | 30 |              $handler->print_error('Person with guid #' . $args['guid'] . ' does not exist'); | 
| 30 | 31 | midcom::get()->auth->drop_sudo(); | 
| 31 | 32 | return false; | 
| @@ -33,7 +34,8 @@ discard block | ||
| 33 | 34 | $accounthelper = new org_openpsa_user_accounthelper($person); | 
| 34 | 35 |          try { | 
| 35 | 36 | $accounthelper->reopen_account(); | 
| 36 | -        } catch (midcom_error $e) { | |
| 37 | + } | |
| 38 | +        catch (midcom_error $e) { | |
| 37 | 39 | $handler->print_error($e->getMessage()); | 
| 38 | 40 | midcom::get()->auth->drop_sudo(); | 
| 39 | 41 | return false; | 
| @@ -23,7 +23,8 @@ discard block | ||
| 23 | 23 |          try { | 
| 24 | 24 | $person = new midcom_db_person($this->uid); | 
| 25 | 25 | $grp = new midcom_db_group($this->gid); | 
| 26 | -        } catch (midcom_error $e) { | |
| 26 | + } | |
| 27 | +        catch (midcom_error $e) { | |
| 27 | 28 | $e->log(); | 
| 28 | 29 | return 'Invalid membership record'; | 
| 29 | 30 | } | 
| @@ -40,7 +41,8 @@ discard block | ||
| 40 | 41 | } | 
| 41 | 42 |          try { | 
| 42 | 43 | $person = new midcom_db_person($this->uid); | 
| 43 | -        } catch (midcom_error $e) { | |
| 44 | + } | |
| 45 | +        catch (midcom_error $e) { | |
| 44 | 46 | return; | 
| 45 | 47 | } | 
| 46 | 48 | midcom::get()->cache->invalidate($person->guid); |