@@ -100,7 +100,7 @@ |
||
| 100 | 100 | foreach ($arguments as $i => $argument) { |
| 101 | 101 | if ($argument === '__request_data__') { |
| 102 | 102 | $context = $event->getRequest()->attributes->get('context'); |
| 103 | - $arguments[$i] =& $context->get_custom_key('request_data'); |
|
| 103 | + $arguments[$i] = & $context->get_custom_key('request_data'); |
|
| 104 | 104 | } |
| 105 | 105 | } |
| 106 | 106 | $event->setArguments($arguments); |
@@ -261,7 +261,7 @@ |
||
| 261 | 261 | if ($part === '') { |
| 262 | 262 | continue; |
| 263 | 263 | } |
| 264 | - if ( midcom::get()->config->get('theme') |
|
| 264 | + if (midcom::get()->config->get('theme') |
|
| 265 | 265 | && !$args_started |
| 266 | 266 | && self::check_page_exists($part)) { |
| 267 | 267 | $page_style .= '/' . $part; |
@@ -93,7 +93,8 @@ |
||
| 93 | 93 | |
| 94 | 94 | try { |
| 95 | 95 | $user = new midgard_user($login_tokens); |
| 96 | - } catch (mgd_exception $e) { |
|
| 96 | + } |
|
| 97 | + catch (mgd_exception $e) { |
|
| 97 | 98 | return null; |
| 98 | 99 | } |
| 99 | 100 | if (!$trusted && !self::verify_password($password, $user->password)) { |
@@ -15,7 +15,7 @@ |
||
| 15 | 15 | |
| 16 | 16 | foreach ($properties as $propname => $val) { |
| 17 | 17 | $description = preg_replace('/ *\n */', "\n", $val['value']); |
| 18 | - if ( $val['link'] |
|
| 18 | + if ($val['link'] |
|
| 19 | 19 | && $linked_component = midcom::get()->dbclassloader->get_component_for_class($val['link_name'])) { |
| 20 | 20 | $proplink = $data['router']->generate('help', ['component' => $linked_component, 'help_id' => 'mgdschemas']); |
| 21 | 21 | $proplink = "<a href='{$proplink}#{$val['link_name']}' title='{$linked_component}/{$val['link_name']}::{$val['link_target']}'>{$val['link_name']}:{$val['link_target']}</a>"; |
@@ -110,7 +110,7 @@ |
||
| 110 | 110 | public function _on_updated() |
| 111 | 111 | { |
| 112 | 112 | // Sync the object's ACL properties into MidCOM ACL system |
| 113 | - if ( !$this->_skip_acl_refresh) { |
|
| 113 | + if (!$this->_skip_acl_refresh) { |
|
| 114 | 114 | if ($this->orgOpenpsaAccesstype && $this->orgOpenpsaOwnerWg) { |
| 115 | 115 | debug_add("Synchronizing task ACLs to MidCOM"); |
| 116 | 116 | $sync = new org_openpsa_core_acl_synchronizer(); |
@@ -91,7 +91,7 @@ discard block |
||
| 91 | 91 | } |
| 92 | 92 | |
| 93 | 93 | if ($this->_config->get('sales_pdfbuilder_class')) { |
| 94 | - if ( $this->_salesproject->can_do('midgard:update') |
|
| 94 | + if ($this->_salesproject->can_do('midgard:update') |
|
| 95 | 95 | && $this->is_pdf_creatable()) { |
| 96 | 96 | $workflow = $this->get_workflow('datamanager'); |
| 97 | 97 | $buttons[] = $workflow->get_button($this->router->generate('create_offer', ['guid' => $this->_salesproject->guid]), [ |
@@ -218,7 +218,7 @@ discard block |
||
| 218 | 218 | case 'ordered': |
| 219 | 219 | if ($deliverable->orgOpenpsaObtype == org_openpsa_products_product_dba::DELIVERY_SUBSCRIPTION) { |
| 220 | 220 | $entries = $deliverable->get_at_entries(); |
| 221 | - if ( $entries |
|
| 221 | + if ($entries |
|
| 222 | 222 | && $entries[0]->status == midcom_services_at_entry_dba::SCHEDULED |
| 223 | 223 | && midcom::get()->auth->can_user_do('midgard:create', null, org_openpsa_invoices_invoice_dba::class)) { |
| 224 | 224 | return '<button class="run_cycle"><i class="fa fa-refresh"></i>' . $this->_l10n->get('generate now') . '</button>'; |
@@ -228,7 +228,7 @@ discard block |
||
| 228 | 228 | } |
| 229 | 229 | break; |
| 230 | 230 | case 'delivered': |
| 231 | - if ( $deliverable->orgOpenpsaObtype != org_openpsa_products_product_dba::DELIVERY_SUBSCRIPTION |
|
| 231 | + if ($deliverable->orgOpenpsaObtype != org_openpsa_products_product_dba::DELIVERY_SUBSCRIPTION |
|
| 232 | 232 | && midcom::get()->auth->can_user_do('midgard:create', null, org_openpsa_invoices_invoice_dba::class)) { |
| 233 | 233 | $client_class = $this->_config->get('calculator'); |
| 234 | 234 | $client = new $client_class(); |
@@ -3,7 +3,7 @@ discard block |
||
| 3 | 3 | $state = $data['deliverable']->get_state(); |
| 4 | 4 | $formatter = $data['l10n']->get_formatter(); |
| 5 | 5 | $per_unit = $data['l10n']->get('per unit'); |
| 6 | -if ( $data['product'] |
|
| 6 | +if ($data['product'] |
|
| 7 | 7 | && $unit_option = org_openpsa_sales_viewer::get_unit_option($data['product']->unit)) { |
| 8 | 8 | $per_unit = sprintf($data['l10n']->get('per %s'), $unit_option); |
| 9 | 9 | } |
@@ -124,7 +124,7 @@ discard block |
||
| 124 | 124 | </div> |
| 125 | 125 | <?php |
| 126 | 126 | $tabs = []; |
| 127 | - if ( $data['invoices_url'] |
|
| 127 | + if ($data['invoices_url'] |
|
| 128 | 128 | && $data['deliverable']->invoiced > 0) { |
| 129 | 129 | $tabs[] = [ |
| 130 | 130 | 'url' => $data['invoices_url'] . "list/deliverable/{$data['deliverable']->guid}/", |
@@ -132,7 +132,7 @@ discard block |
||
| 132 | 132 | ]; |
| 133 | 133 | } |
| 134 | 134 | |
| 135 | - if ( $data['projects_url'] |
|
| 135 | + if ($data['projects_url'] |
|
| 136 | 136 | && $data['deliverable']->state >= org_openpsa_sales_salesproject_deliverable_dba::STATE_ORDERED |
| 137 | 137 | && $data['product'] |
| 138 | 138 | && $data['product']->orgOpenpsaObtype == org_openpsa_products_product_dba::TYPE_SERVICE) { |
@@ -191,7 +191,7 @@ discard block |
||
| 191 | 191 | } |
| 192 | 192 | echo " </tbody>\n"; |
| 193 | 193 | echo "</table>\n"; |
| 194 | - if ( $at_entries[0]->status == midcom_services_at_entry_dba::SCHEDULED |
|
| 194 | + if ($at_entries[0]->status == midcom_services_at_entry_dba::SCHEDULED |
|
| 195 | 195 | && midcom::get()->auth->can_user_do('midgard:create', null, org_openpsa_invoices_invoice_dba::class)) { |
| 196 | 196 | $label = $data['l10n']->get('generate now'); |
| 197 | 197 | $process_link = $data['router']->generate('deliverable_run_cycle', ['guid' => $data['deliverable']->guid]); |
@@ -74,12 +74,12 @@ |
||
| 74 | 74 | <tr> |
| 75 | 75 | <td class="title"><?php echo $data['l10n']->get('cost'); ?></td> |
| 76 | 76 | <?php |
| 77 | - if ($data['deliverable']->costType == '%') { ?> |
|
| 77 | + if ($data['deliverable']->costType == '%') { ?> |
|
| 78 | 78 | <td class="numeric"><?php echo $view['costPerUnit']; ?> %</td> |
| 79 | 79 | <td class="numeric"> </td> |
| 80 | 80 | <td class="numeric"> </td> |
| 81 | 81 | <?php |
| 82 | - } else { ?> |
|
| 82 | + } else { ?> |
|
| 83 | 83 | <td class="numeric"><?php echo $formatter->number($data['deliverable']->costPerUnit); ?></td> |
| 84 | 84 | <td class="numeric"><?php echo $view['plannedUnits']; ?></td> |
| 85 | 85 | <td class="numeric"><?php echo $view['units']; ?></td> |
@@ -34,7 +34,7 @@ discard block |
||
| 34 | 34 | public function configureOptions(OptionsResolver $resolver) |
| 35 | 35 | { |
| 36 | 36 | $resolver->setDefault('error_bubbling', false); |
| 37 | - $resolver->setNormalizer('widget_config', function (Options $options, $value) { |
|
| 37 | + $resolver->setNormalizer('widget_config', function(Options $options, $value) { |
|
| 38 | 38 | $widget_defaults = [ |
| 39 | 39 | 'creation_mode_enabled' => false, |
| 40 | 40 | 'class' => null, |
@@ -66,7 +66,7 @@ discard block |
||
| 66 | 66 | |
| 67 | 67 | return helper::normalize($widget_defaults, $value); |
| 68 | 68 | }); |
| 69 | - $resolver->setNormalizer('type_config', function (Options $options, $value) { |
|
| 69 | + $resolver->setNormalizer('type_config', function(Options $options, $value) { |
|
| 70 | 70 | $type_defaults = [ |
| 71 | 71 | 'options' => [], |
| 72 | 72 | 'method' => 'GET', |
@@ -116,7 +116,8 @@ |
||
| 116 | 116 | try { |
| 117 | 117 | $object = $options['widget_config']['class']::get_cached($identifier); |
| 118 | 118 | $preset[$identifier] = autocomplete_helper::create_item_label($object, $options['widget_config']['result_headers'], $options['widget_config']['titlefield']); |
| 119 | - } catch (midcom_error $e) { |
|
| 119 | + } |
|
| 120 | + catch (midcom_error $e) { |
|
| 120 | 121 | $e->log(); |
| 121 | 122 | } |
| 122 | 123 | } |
@@ -68,7 +68,8 @@ discard block |
||
| 68 | 68 | $field['type_config']['options'] = [0 => '', $customer->id => $customer->official]; |
| 69 | 69 | |
| 70 | 70 | $defaults['customer'] = $customer->id; |
| 71 | - } catch (midcom_error $e) { |
|
| 71 | + } |
|
| 72 | + catch (midcom_error $e) { |
|
| 72 | 73 | $customer = new org_openpsa_contacts_person_dba($guid); |
| 73 | 74 | $defaults['customerContact'] = $customer->id; |
| 74 | 75 | $field['type_config']['options'] = $this->list_groups(new org_openpsa_sales_salesproject_dba, [$customer->id => true]); |
@@ -142,7 +143,8 @@ discard block |
||
| 142 | 143 | try { |
| 143 | 144 | $company = org_openpsa_contacts_group_dba::get_cached($company_id); |
| 144 | 145 | $ret[$company->id] = $company->get_label(); |
| 145 | - } catch (midcom_error $e) { |
|
| 146 | + } |
|
| 147 | + catch (midcom_error $e) { |
|
| 146 | 148 | $e->log(); |
| 147 | 149 | } |
| 148 | 150 | } |
@@ -61,7 +61,7 @@ |
||
| 61 | 61 | $schemadb = schemadb::from_path($this->_config->get('schemadb_salesproject')); |
| 62 | 62 | |
| 63 | 63 | if (!empty($guid)) { |
| 64 | - $field =& $schemadb->get('default')->get_field('customer'); |
|
| 64 | + $field = & $schemadb->get('default')->get_field('customer'); |
|
| 65 | 65 | try { |
| 66 | 66 | $customer = new org_openpsa_contacts_group_dba($guid); |
| 67 | 67 | $field['type_config']['options'] = [0 => '', $customer->id => $customer->official]; |
@@ -153,7 +153,7 @@ discard block |
||
| 153 | 153 | return false; |
| 154 | 154 | } |
| 155 | 155 | |
| 156 | - if ( $key === MIDCOM_CONTEXT_ROOTTOPIC |
|
| 156 | + if ($key === MIDCOM_CONTEXT_ROOTTOPIC |
|
| 157 | 157 | && $this->_data[$key] === null) { |
| 158 | 158 | $this->_data[$key] = self::initialize_root_topic(); |
| 159 | 159 | } |
@@ -277,7 +277,7 @@ discard block |
||
| 277 | 277 | */ |
| 278 | 278 | public function set_custom_key($key, &$value) |
| 279 | 279 | { |
| 280 | - $this->_data[MIDCOM_CONTEXT_CUSTOMDATA][$key] =& $value; |
|
| 280 | + $this->_data[MIDCOM_CONTEXT_CUSTOMDATA][$key] = & $value; |
|
| 281 | 281 | } |
| 282 | 282 | |
| 283 | 283 | public function show() |
@@ -165,7 +165,8 @@ |
||
| 165 | 165 | try { |
| 166 | 166 | self::$root_topic = midcom_db_topic::get_cached($guid); |
| 167 | 167 | return self::$root_topic; |
| 168 | - } catch (midcom_error $e) { |
|
| 168 | + } |
|
| 169 | + catch (midcom_error $e) { |
|
| 169 | 170 | if ($e instanceof midcom_error_forbidden) { |
| 170 | 171 | throw $e; |
| 171 | 172 | } |