@@ -149,7 +149,7 @@ |
||
149 | 149 | } |
150 | 150 | |
151 | 151 | $reflector = midcom_helper_reflector::get($linked_object); |
152 | - $link_html = "<a href='" . midcom::get()->permalinks->create_permalink($linked_object->guid) . "'>" . $reflector->get_object_label($linked_object) ."</a>"; |
|
152 | + $link_html = "<a href='" . midcom::get()->permalinks->create_permalink($linked_object->guid) . "'>" . $reflector->get_object_label($linked_object) . "</a>"; |
|
153 | 153 | $data['linked_objects'][$entry->linkGuid] = $link_html; |
154 | 154 | $data['linked_raw_objects'][$entry->linkGuid] = $reflector->get_object_label($linked_object); |
155 | 155 | } |
@@ -103,7 +103,7 @@ |
||
103 | 103 | foreach ($parameters as $key => $value) { |
104 | 104 | $this->_patterns[] = "/__({$key})__/"; |
105 | 105 | |
106 | - if ( is_array($value) |
|
106 | + if (is_array($value) |
|
107 | 107 | || is_object($value)) { |
108 | 108 | $this->_patterns[] = "/__({$key})_([^ \.>\"-]*?)__/"; |
109 | 109 | } |
@@ -75,7 +75,7 @@ discard block |
||
75 | 75 | $this->from = $from; |
76 | 76 | $this->subject = $subject; |
77 | 77 | |
78 | - if ( $message->orgOpenpsaObtype != org_openpsa_directmarketing_campaign_message_dba::EMAIL_TEXT |
|
78 | + if ($message->orgOpenpsaObtype != org_openpsa_directmarketing_campaign_message_dba::EMAIL_TEXT |
|
79 | 79 | && $message->orgOpenpsaObtype != org_openpsa_directmarketing_campaign_message_dba::EMAIL_HTML) { |
80 | 80 | throw new midcom_error('unsupported message type'); |
81 | 81 | } |
@@ -219,7 +219,7 @@ discard block |
||
219 | 219 | $token .= midcom_helper_misc::random_string($this->token_size - 1, 'abcdefghijklmnopqrstuvwxyz0123456789'); |
220 | 220 | |
221 | 221 | //If token is not free or (very, very unlikely) matches our dummy token, recurse. |
222 | - if ( $token === 'dummy' |
|
222 | + if ($token === 'dummy' |
|
223 | 223 | || !org_openpsa_directmarketing_campaign_messagereceipt_dba::token_is_free($token)) { |
224 | 224 | return $this->_create_token(); |
225 | 225 | } |
@@ -240,7 +240,7 @@ discard block |
||
240 | 240 | return null; |
241 | 241 | } |
242 | 242 | $type = $this->_backend->get_type(); |
243 | - if ( $person->get_parameter('org.openpsa.directmarketing', "send_all_denied") |
|
243 | + if ($person->get_parameter('org.openpsa.directmarketing', "send_all_denied") |
|
244 | 244 | || $person->get_parameter('org.openpsa.directmarketing', "send_{$type}_denied")) { |
245 | 245 | debug_add("Sending {$type} messages to person {$person->rname} is denied, unsubscribing member (member #{$member->id})"); |
246 | 246 | $member->orgOpenpsaObtype = org_openpsa_directmarketing_campaign_member_dba::UNSUBSCRIBED; |
@@ -100,7 +100,7 @@ discard block |
||
100 | 100 | return null; |
101 | 101 | } |
102 | 102 | |
103 | - if ( midcom::get()->config->get('auth_check_client_ip') |
|
103 | + if (midcom::get()->config->get('auth_check_client_ip') |
|
104 | 104 | && $data['clientip'] != $request->getClientIp()) { |
105 | 105 | debug_add("The session had mismatching client IP.", MIDCOM_LOG_INFO); |
106 | 106 | debug_add("Expected {$data['clientip']}, got {$request->getClientIp()}."); |
@@ -114,7 +114,7 @@ discard block |
||
114 | 114 | return null; |
115 | 115 | } |
116 | 116 | |
117 | - if ( !$this->check_timestamp($data['timestamp'], $user) |
|
117 | + if (!$this->check_timestamp($data['timestamp'], $user) |
|
118 | 118 | || !$this->authenticate($user->username, '', true)) { |
119 | 119 | $this->logout($user); |
120 | 120 | return null; |
@@ -149,7 +149,7 @@ |
||
149 | 149 | $child_indent = $indent + 20; |
150 | 150 | echo "{$prefix}<tbody class=\"children\">\n"; |
151 | 151 | foreach ($child_types as $type => $children) { |
152 | - if ( count($children) < 10 |
|
152 | + if (count($children) < 10 |
|
153 | 153 | || isset($_GET['show_children'][$object->guid][$type])) { |
154 | 154 | foreach ($children as $child) { |
155 | 155 | $this->show_type($child, $child_indent, "{$prefix} ", false); |
@@ -28,7 +28,7 @@ |
||
28 | 28 | &(published:h); |
29 | 29 | <?php |
30 | 30 | if ($data['linked']) { |
31 | - echo $data['l10n']->get('to') ." <a href=\"{$data['node'][MIDCOM_NAV_ABSOLUTEURL]}\">{$data['node'][MIDCOM_NAV_NAME]}</a>\n"; |
|
31 | + echo $data['l10n']->get('to') . " <a href=\"{$data['node'][MIDCOM_NAV_ABSOLUTEURL]}\">{$data['node'][MIDCOM_NAV_NAME]}</a>\n"; |
|
32 | 32 | } |
33 | 33 | ?> |
34 | 34 | </p> |
@@ -52,7 +52,7 @@ |
||
52 | 52 | */ |
53 | 53 | public function _on_reindex($topic, midcom_helper_configuration $config, midcom_services_indexer $indexer) |
54 | 54 | { |
55 | - if ( !$config->get('index_products') |
|
55 | + if (!$config->get('index_products') |
|
56 | 56 | && !$config->get('index_groups')) { |
57 | 57 | debug_add("No indexing to groups and products, skipping", MIDCOM_LOG_WARN); |
58 | 58 | return true; |
@@ -141,8 +141,8 @@ |
||
141 | 141 | $schema->get_field('paid')['hidden'] = empty($this->invoice->paid); |
142 | 142 | |
143 | 143 | $contact = $this->invoice->customerContact ?: $this->contact_id; |
144 | - $customer_field =& $schema->get_field('customer'); |
|
145 | - $contact_field =& $schema->get_field('customerContact'); |
|
144 | + $customer_field = & $schema->get_field('customer'); |
|
145 | + $contact_field = & $schema->get_field('customerContact'); |
|
146 | 146 | |
147 | 147 | if (!empty($contact)) { |
148 | 148 | $customer_field['type_config']['options'] = $this->get_customers_for_contact($contact); |
@@ -47,12 +47,12 @@ discard block |
||
47 | 47 | $qb_hr = org_openpsa_expenses_hour_report_dba::new_query_builder(); |
48 | 48 | $qb_hr->add_constraint('date', '<=', (int) $query_data['end']); |
49 | 49 | $qb_hr->add_constraint('date', '>=', (int) $query_data['start']); |
50 | - if ( array_key_exists('invoiceable_filter', $query_data) |
|
50 | + if (array_key_exists('invoiceable_filter', $query_data) |
|
51 | 51 | && $query_data['invoiceable_filter'] != -1) { |
52 | 52 | $qb_hr->add_constraint('invoiceable', '=', (bool) $query_data['invoiceable_filter']); |
53 | 53 | } |
54 | 54 | |
55 | - if ( array_key_exists('invoiced_filter', $query_data) |
|
55 | + if (array_key_exists('invoiced_filter', $query_data) |
|
56 | 56 | && $query_data['invoiced_filter'] != -1) { |
57 | 57 | if ((int) $query_data['invoiced_filter']) { |
58 | 58 | debug_add('invoiced_filter parsed as ONLY, adding constraint'); |
@@ -71,7 +71,7 @@ discard block |
||
71 | 71 | $tasks = $this->_expand_task($query_data['task']); |
72 | 72 | $qb_hr->add_constraint('task', 'IN', $tasks); |
73 | 73 | } |
74 | - if ( array_key_exists('hour_type_filter', $query_data) |
|
74 | + if (array_key_exists('hour_type_filter', $query_data) |
|
75 | 75 | && $query_data['hour_type_filter'] != 'builtin:all') { |
76 | 76 | $qb_hr->add_constraint('reportType', '=', $query_data['hour_type_filter']); |
77 | 77 | } |
@@ -140,7 +140,7 @@ discard block |
||
140 | 140 | |
141 | 141 | private function add_to_group(array $new_row, string $matching, string $sort, string $title) |
142 | 142 | { |
143 | - $rows =& $this->_request_data['report']['rows']; |
|
143 | + $rows = & $this->_request_data['report']['rows']; |
|
144 | 144 | if (array_key_exists($matching, $rows)) { |
145 | 145 | $rows[$matching]['rows'][] = $new_row; |
146 | 146 | $rows[$matching]['total_hours'] += $new_row['hour']->hours; |