@@ -72,7 +72,7 @@ |
||
| 72 | 72 | |
| 73 | 73 | $this->_group = new org_openpsa_contacts_group_dba(); |
| 74 | 74 | |
| 75 | - if ( $this->_type == 'organization' |
|
| 75 | + if ($this->_type == 'organization' |
|
| 76 | 76 | && $this->_parent_group) { |
| 77 | 77 | $this->_group->owner = $this->_parent_group->id; |
| 78 | 78 | } else { |
@@ -50,6 +50,6 @@ |
||
| 50 | 50 | $indexer = new org_openpsa_documents_midcom_indexer($this->_topic); |
| 51 | 51 | $indexer->index($controller->get_datamanager()); |
| 52 | 52 | |
| 53 | - return $controller->get_datamanager()->get_storage()->get_value()->name. "/"; |
|
| 53 | + return $controller->get_datamanager()->get_storage()->get_value()->name . "/"; |
|
| 54 | 54 | } |
| 55 | 55 | } |
@@ -9,11 +9,11 @@ |
||
| 9 | 9 | use midcom\datamanager\indexer\client; |
| 10 | 10 | use midcom\datamanager\datamanager; |
| 11 | 11 | |
| 12 | - /** |
|
| 13 | - * Indexer client class |
|
| 14 | - * |
|
| 15 | - * @package org.openpsa.invoices |
|
| 16 | - */ |
|
| 12 | + /** |
|
| 13 | + * Indexer client class |
|
| 14 | + * |
|
| 15 | + * @package org.openpsa.invoices |
|
| 16 | + */ |
|
| 17 | 17 | class org_openpsa_invoices_midcom_indexer extends client |
| 18 | 18 | { |
| 19 | 19 | public function prepare_document(midcom_services_indexer_document &$document, datamanager $dm) |
@@ -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 | } |
@@ -22,7 +22,7 @@ |
||
| 22 | 22 | |
| 23 | 23 | // Get the calendar root event |
| 24 | 24 | if ($root_event = org_openpsa_calendar_interface::find_root_event()) { |
| 25 | - $field =& $schemadb->get('default')->get_field('calendar'); |
|
| 25 | + $field = & $schemadb->get('default')->get_field('calendar'); |
|
| 26 | 26 | $field['type_config']['privilege_object'] = $root_event; |
| 27 | 27 | $field['type_config']['assignee'] = 'group:' . $group->guid; |
| 28 | 28 | } |
@@ -22,7 +22,7 @@ |
||
| 22 | 22 | |
| 23 | 23 | // Get the calendar root event |
| 24 | 24 | if ($root_event = org_openpsa_calendar_interface::find_root_event()) { |
| 25 | - $field =& $schemadb->get('default')->get_field('calendar'); |
|
| 25 | + $field = & $schemadb->get('default')->get_field('calendar'); |
|
| 26 | 26 | $field['type_config']['privilege_object'] = $root_event; |
| 27 | 27 | $field['type_config']['assignee'] = 'user:' . $person->guid; |
| 28 | 28 | } |
@@ -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 | } |
@@ -134,8 +134,8 @@ discard block |
||
| 134 | 134 | $year_data = []; |
| 135 | 135 | $first_post = $this->_compute_welcome_first_post(); |
| 136 | 136 | $this->_request_data['first_post'] = $first_post; |
| 137 | - $this->_request_data['total_count'] =& $total_count; |
|
| 138 | - $this->_request_data['year_data'] =& $year_data; |
|
| 137 | + $this->_request_data['total_count'] = & $total_count; |
|
| 138 | + $this->_request_data['year_data'] = & $year_data; |
|
| 139 | 139 | if (!$first_post) { |
| 140 | 140 | return; |
| 141 | 141 | } |
@@ -271,7 +271,7 @@ discard block |
||
| 271 | 271 | switch ($handler_id) { |
| 272 | 272 | case 'archive-year-category': |
| 273 | 273 | $data['category'] = trim(strip_tags($args[1])); |
| 274 | - if ( $data['datamanager']->get_schema('default')->has_field('categories') |
|
| 274 | + if ($data['datamanager']->get_schema('default')->has_field('categories') |
|
| 275 | 275 | && !$data['datamanager']->get_schema('default')->get_field('categories')['type_config']['allow_multiple']) { |
| 276 | 276 | $qb->add_constraint('extra1', '=', (string) $data['category']); |
| 277 | 277 | } else { |
@@ -360,7 +360,7 @@ discard block |
||
| 360 | 360 | throw new midcom_error_notfound("The year '{$year}' is not a valid year identifier."); |
| 361 | 361 | } |
| 362 | 362 | |
| 363 | - if ( $month < 1 |
|
| 363 | + if ($month < 1 |
|
| 364 | 364 | || $month > 12) { |
| 365 | 365 | throw new midcom_error_notfound("The year {$month} is not a valid year identifier."); |
| 366 | 366 | } |
@@ -141,7 +141,8 @@ discard block |
||
| 141 | 141 | foreach ($this->_articles as $article_counter => $article) { |
| 142 | 142 | try { |
| 143 | 143 | $data['datamanager']->set_storage($article); |
| 144 | - } catch (midcom_error $e) { |
|
| 144 | + } |
|
| 145 | + catch (midcom_error $e) { |
|
| 145 | 146 | $e->log(); |
| 146 | 147 | continue; |
| 147 | 148 | } |
@@ -178,7 +179,8 @@ discard block |
||
| 178 | 179 | if ($this->_config->get('comments_topic')) { |
| 179 | 180 | try { |
| 180 | 181 | $comments_topic = new midcom_db_topic($this->_config->get('comments_topic')); |
| 181 | - } catch (midcom_error $e) { |
|
| 182 | + } |
|
| 183 | + catch (midcom_error $e) { |
|
| 182 | 184 | return false; |
| 183 | 185 | } |
| 184 | 186 | |
@@ -93,7 +93,7 @@ |
||
| 93 | 93 | midcom::get()->head->set_pagetitle($this->_request_data['page_title']); |
| 94 | 94 | |
| 95 | 95 | // Activate correct leaf |
| 96 | - if ( $this->_config->get('show_navigation_pseudo_leaves') |
|
| 96 | + if ($this->_config->get('show_navigation_pseudo_leaves') |
|
| 97 | 97 | && in_array($this->_request_data['category'], $this->_request_data['categories'])) { |
| 98 | 98 | $this->set_active_leaf($this->_topic->id . '_CAT_' . $this->_request_data['category']); |
| 99 | 99 | } |
@@ -17,7 +17,7 @@ |
||
| 17 | 17 | |
| 18 | 18 | public function form_widget_simple(FormView $view, array $data) |
| 19 | 19 | { |
| 20 | - if ( !empty($data['value']) |
|
| 20 | + if (!empty($data['value']) |
|
| 21 | 21 | || is_numeric($data['value'])) { |
| 22 | 22 | return $data['value']; |
| 23 | 23 | } |