flack / openpsa

lib/org/openpsa/directmarketing/handler/message/report.php

Spacing +15 added lines, -15 removed lines

@@ -54,7 +54,7 @@ 
         $qb_receipts->add_constraint('message', '=', $this->_message->id);
         $qb_receipts->add_constraint('orgOpenpsaObtype', '=', org_openpsa_directmarketing_campaign_messagereceipt_dba::SENT);
         $receipts = $qb_receipts->execute_unchecked();
-        $receipt_data =& $this->_request_data['report']['receipt_data'];
+        $receipt_data = & $this->_request_data['report']['receipt_data'];
         $receipt_data['first_send'] = time();
         $receipt_data['last_send'] = 0;
         $receipt_data['sent'] = count($receipts);
@@ -87,7 +87,7 @@ 
 
     private function _get_campaign_data($first_send)
     {
-        $campaign_data =& $this->_request_data['report']['campaign_data'];
+        $campaign_data = & $this->_request_data['report']['campaign_data'];
         $qb_unsub = org_openpsa_directmarketing_campaign_member_dba::new_query_builder();
         $qb_unsub->add_constraint('campaign', '=', $this->_message->campaign);
         $qb_unsub->add_constraint('orgOpenpsaObtype', '=', org_openpsa_directmarketing_campaign_member_dba::UNSUBSCRIBED);
@@ -95,8 +95,8 @@ 
         $campaign_data['next_message'] = false;
         // Find "next message" and if present use its sendStarted as constraint for this query
         $qb_messages = org_openpsa_directmarketing_campaign_message_dba::new_query_builder();
-        $qb_messages->add_constraint('campaign', '=',  $this->_message->campaign);
-        $qb_messages->add_constraint('id', '<>',  $this->_message->id);
+        $qb_messages->add_constraint('campaign', '=', $this->_message->campaign);
+        $qb_messages->add_constraint('id', '<>', $this->_message->id);
         $qb_messages->add_constraint('sendStarted', '>', $first_send);
         $qb_messages->add_order('sendStarted', 'DESC');
         $qb_messages->set_limit(1);
@@ -111,7 +111,7 @@ 
     private function _get_link_data($segmentation_param)
     {
         $this->_request_data['report']['link_data'] = array();
-        $link_data =& $this->_request_data['report']['link_data'];
+        $link_data = & $this->_request_data['report']['link_data'];
 
         $link_data['counts'] = array();
         $link_data['percentages'] = array('of_links' => array(), 'of_recipients' => array());
@@ -137,7 +137,7 @@ 
         foreach ($links as $link) {
             $segment = '';
             $segment_notfound = false;
-            if (   $segmentation_param
+            if ($segmentation_param
                 && !empty($link->person)) {
                 try {
                     $person = org_openpsa_contacts_person_dba::get_cached($link->person);
@@ -151,7 +151,7 @@ 
                 if (!isset($link_data['segments'][$segment])) {
                     $link_data['segments'][$segment] = $segment_prototype;
                 }
-                $segment_data =& $link_data['segments'][$segment];
+                $segment_data = & $link_data['segments'][$segment];
             } else {
                 $segment_data = $segment_prototype;
             }
@@ -261,15 +261,15 @@ 
         $this->_initialize_field($array['percentages']['of_links'], $link);
         $this->_initialize_field($array['percentages']['of_recipients'], $link);
 
-        $link_data =& $this->_request_data['report']['link_data'];
-        $array['percentages']['of_links'][$link->target]['total'] = ($array['counts'][$link->target]['total']/$link_data['total'])*100;
-        $array['percentages']['of_links'][$link->target][$link->token] = ($array['counts'][$link->target][$link->token]/$link_data['total'])*100;
+        $link_data = & $this->_request_data['report']['link_data'];
+        $array['percentages']['of_links'][$link->target]['total'] = ($array['counts'][$link->target]['total'] / $link_data['total']) * 100;
+        $array['percentages']['of_links'][$link->target][$link->token] = ($array['counts'][$link->target][$link->token] / $link_data['total']) * 100;
 
-        $receipt_data =& $this->_request_data['report']['receipt_data'];
-        $array['percentages']['of_recipients'][$link->target]['total'] = ((count($array['counts'][$link->target])-1)/($receipt_data['sent']-$receipt_data['bounced']))*100;
-        $array['percentages']['of_recipients'][$link->target][$link->token] = ($array['counts'][$link->target][$link->token]/($receipt_data['sent']-$receipt_data['bounced']))*100;
+        $receipt_data = & $this->_request_data['report']['receipt_data'];
+        $array['percentages']['of_recipients'][$link->target]['total'] = ((count($array['counts'][$link->target]) - 1) / ($receipt_data['sent'] - $receipt_data['bounced'])) * 100;
+        $array['percentages']['of_recipients'][$link->target][$link->token] = ($array['counts'][$link->target][$link->token] / ($receipt_data['sent'] - $receipt_data['bounced'])) * 100;
 
-        if (   !isset($array['percentages']['of_recipients']['total'])
+        if (!isset($array['percentages']['of_recipients']['total'])
             || $array['percentages']['of_recipients'][$link->target]['total'] > $array['percentages']['of_recipients']['total']) {
             $array['percentages']['of_recipients']['total'] = $array['percentages']['of_recipients'][$link->target]['total'];
         }
@@ -335,7 +335,7 @@ 
         $this->_campaign = $this->_master->load_campaign($this->_message->campaign);
         $data['campaign'] = $this->_campaign;
 
-        if (   isset($_POST['oo_dirmar_userule'])
+        if (isset($_POST['oo_dirmar_userule'])
             && !empty($_POST['oo_dirmar_rule_' . $_POST['oo_dirmar_userule']])) {
             return $this->_create_campaign_from_link($_POST['oo_dirmar_userule']);
         }

lib/org/openpsa/documents/midcom/interfaces.php

Spacing +1 added lines, -1 removed lines

@@ -39,7 +39,7 @@
 
     public function resolve_object_link(midcom_db_topic $topic, midcom_core_dbaobject $object)
     {
-        if (   $object instanceof org_openpsa_documents_document_dba
+        if ($object instanceof org_openpsa_documents_document_dba
             && $object->topic == $topic->id) {
             return "document/{$object->guid}/";
         }

lib/org/openpsa/documents/handler/document/admin.php

Spacing +5 added lines, -5 removed lines

@@ -114,7 +114,7 @@ 
 
         $this->_controller = $this->get_controller('simple', $this->_document);
 
-        if (   $data['enable_versioning']
+        if ($data['enable_versioning']
             && !empty($_POST)) {
             $this->_backup_attachment();
         }
@@ -144,7 +144,7 @@ 
             $prefix = $node[MIDCOM_NAV_ABSOLUTEURL];
         }
 
-        return $prefix  . "document/" . $this->_document->guid . "/";
+        return $prefix . "document/" . $this->_document->guid . "/";
     }
 
     /**
@@ -157,8 +157,8 @@ 
         // First, look at post data (from in-form replace/delete buttons)
         if (!empty($_POST['document'])) {
             foreach (array_keys($_POST['document']) as $key) {
-                if (    strpos($key, '_delete')
-                    || (    strpos($key, '_upload')
+                if (strpos($key, '_delete')
+                    || (strpos($key, '_upload')
                         && !strpos($key, 'new_upload'))) {
                     $this->_document->backup_version();
                     return;
@@ -169,7 +169,7 @@ 
         // If nothing is found, try looking in quickform (regular form submission)
         $group = $this->_controller->formmanager->form->getElement('document');
         foreach ($group->getElements() as $element) {
-            if (   preg_match('/e_exist_.+?_file$/', $element->getName())
+            if (preg_match('/e_exist_.+?_file$/', $element->getName())
                 && $element->isUploadedFile()) {
                 $this->_document->backup_version();
                 return;

lib/org/openpsa/expenses/viewer.php

Spacing +3 added lines, -3 removed lines

@@ -39,10 +39,10 @@ 
      */
     public function _on_handle($handler, array $args)
     {
-        if (   strpos($handler, 'index') !== false
+        if (strpos($handler, 'index') !== false
             || strpos($handler, 'list') !== false) {
             $task = false;
-            if (   $handler == 'list_hours_task'
+            if ($handler == 'list_hours_task'
                 || $handler == 'list_hours_task_all') {
                 $task = $args[0];
             }
@@ -64,7 +64,7 @@ 
         $person_filter->set_label($this->_l10n->get("choose user"));
         $qf->add_filter($person_filter);
 
-        if (   $this->_request_data['handler_id'] != 'index_timestamp'
+        if ($this->_request_data['handler_id'] != 'index_timestamp'
             && $this->_request_data['handler_id'] != 'index') {
             $date_filter = new org_openpsa_core_filter_timeframe('date');
             $date_filter->set_label($this->_l10n->get("timeframe"));

lib/org/openpsa/products/product.php

Spacing +1 added lines, -1 removed lines

@@ -107,7 +107,7 @@
             $qb->add_constraint('id', '<>', $this->id);
         }
         // Make sure the product is in the same product group
-        $qb->add_constraint('productGroup', '=', (int)$this->productGroup);
+        $qb->add_constraint('productGroup', '=', (int) $this->productGroup);
 
         return ($qb->count() == 0);
     }

lib/org/openpsa/projects/workflow.php

Spacing +2 added lines, -2 removed lines

@@ -170,7 +170,7 @@ 
     {
         debug_add("task->start() called with user #" . midcom_connection::get_user());
         //PONDER: Check actual status objects for more accurate logic ?
-        if (   $task->status >= org_openpsa_projects_task_status_dba::STARTED
+        if ($task->status >= org_openpsa_projects_task_status_dba::STARTED
             && $task->status <= org_openpsa_projects_task_status_dba::APPROVED) {
             //We already have started status
             debug_add('Task has already been started');
@@ -382,7 +382,7 @@ 
 
     private static function is_manager($task)
     {
-        return (   $task->manager == 0
+        return ($task->manager == 0
                 || midcom_connection::get_user() == $task->manager);
     }
 }

lib/org/openpsa/projects/task.php

Spacing +2 added lines, -2 removed lines

@@ -93,7 +93,7 @@ 
     public function _on_updated()
     {
         // Sync the object's ACL properties into MidCOM ACL system
-        if (   !$this->_skip_acl_refresh) {
+        if (!$this->_skip_acl_refresh) {
             if ($this->orgOpenpsaAccesstype && $this->orgOpenpsaOwnerWg) {
                 debug_add("Synchronizing task ACLs to MidCOM");
                 $sync = new org_openpsa_core_acl_synchronizer();
@@ -326,7 +326,7 @@ 
 
             if ($report_data['invoice']) {
                 $hours['invoiced'] += $report_hours;
-            } elseif (   $report_data['invoiceable']
+            } elseif ($report_data['invoiceable']
                       && ($report_data['isapproved'] || !$invoice_approved_only)) {
                 $hours['invoiceable'] += $report_hours;
             }

lib/org/openpsa/reports/handler/projects/report.php

Spacing +3 added lines, -3 removed lines

@@ -44,7 +44,7 @@ 
         $qb_hr = org_openpsa_projects_hour_report_dba::new_query_builder();
         $qb_hr->add_constraint('date', '<=', (int) $this->_request_data['query_data']['end']);
         $qb_hr->add_constraint('date', '>=', (int) $this->_request_data['query_data']['start']);
-        if (   array_key_exists('invoiceable_filter', $this->_request_data['query_data'])
+        if (array_key_exists('invoiceable_filter', $this->_request_data['query_data'])
             && $this->_request_data['query_data']['invoiceable_filter'] != -1) {
             $qb_hr->add_constraint('invoiceable', '=', (bool) $this->_request_data['query_data']['invoiceable_filter']);
         }
@@ -60,7 +60,7 @@ 
             $tasks = $this->_expand_task($this->_request_data['query_data']['task']);
             $qb_hr->add_constraint('task', 'IN', $tasks);
         }
-        if (   array_key_exists('hour_type_filter', $this->_request_data['query_data'])
+        if (array_key_exists('hour_type_filter', $this->_request_data['query_data'])
             && $this->_request_data['query_data']['hour_type_filter'] != 'builtin:all') {
             $qb_hr->add_constraint('reportType', '=', $this->_request_data['query_data']['hour_type_filter']);
         }
@@ -151,7 +151,7 @@ 
 
     private function add_to_group($new_row, $matching, $sort, $title)
     {
-        $rows =& $this->_request_data['report']['rows'];
+        $rows = & $this->_request_data['report']['rows'];
         if (array_key_exists($matching, $rows)) {
             $rows[$matching]['rows'][] = $new_row;
             $rows[$matching]['total_hours'] += $new_row['hour']->hours;

lib/org/openpsa/relatedto/handler/relatedto.php

Spacing +6 added lines, -6 removed lines

@@ -228,7 +228,7 @@ 
     private function _render_line(array $link, &$other_obj)
     {
         $this->_request_data['link'] = $link;
-        $this->_request_data['other_obj'] =& $other_obj;
+        $this->_request_data['other_obj'] = & $other_obj;
 
         $ref = midcom_helper_reflector::get($link['class']);
 
@@ -316,7 +316,7 @@ 
             //Calendar node found, render a better view
             $this->_request_data['raw_url'] = $url . 'event/raw/' . $other_obj->guid . '/';
             $workflow = $this->get_workflow('viewer');
-            $title = '<a href="' . $url . 'event/' . $other_obj->guid .  '/" ' . $workflow->render_attributes() . '>' . $title . "</a>\n";
+            $title = '<a href="' . $url . 'event/' . $other_obj->guid . '/" ' . $workflow->render_attributes() . '>' . $title . "</a>\n";
         }
 
         $this->_request_data['title'] = $title;
@@ -340,7 +340,7 @@ 
         $title = $other_obj->title;
 
         if ($url = $this->get_node_url('org.openpsa.projects')) {
-            $title = '<a href="' . $url . $type . '/' . $other_obj->guid  . '/" target="task_' . $other_obj->guid . '">' . $title . "</a>\n";
+            $title = '<a href="' . $url . $type . '/' . $other_obj->guid . '/" target="task_' . $other_obj->guid . '">' . $title . "</a>\n";
         }
         $this->_request_data['title'] = $title;
         $this->_request_data['type'] = $type;
@@ -358,7 +358,7 @@ 
         $title = $other_obj->title;
 
         if ($url = $this->get_node_url('org.openpsa.sales')) {
-            $title = '<a href="' . $url . 'salesproject/' . $other_obj->guid  . '/" target="salesproject_' . $other_obj->guid . '">' . $title . "</a>\n";
+            $title = '<a href="' . $url . 'salesproject/' . $other_obj->guid . '/" target="salesproject_' . $other_obj->guid . '">' . $title . "</a>\n";
         }
 
         $this->_request_data['title'] = $title;
@@ -376,7 +376,7 @@ 
         $title = $this->_i18n->get_string('invoice', 'org.openpsa.invoices') . ' ' . $other_obj->get_label();
 
         if ($url = $this->get_node_url('org.openpsa.invoices')) {
-            $title = '<a href="' . $url . 'invoice/' . $other_obj->guid  . '/" target="invoice_' . $other_obj->guid . '">' . $title . "</a>\n";
+            $title = '<a href="' . $url . 'invoice/' . $other_obj->guid . '/" target="invoice_' . $other_obj->guid . '">' . $title . "</a>\n";
         }
         $this->_request_data['title'] = $title;
 
@@ -422,7 +422,7 @@ 
     {
         echo "<ul class=\"relatedto_toolbar\" data-link-guid=\"{$link['guid']}\" data-other-guid=\"{$other_obj->guid}\">\n";
 
-        if (   $link['component'] == 'net.nemein.wiki'
+        if ($link['component'] == 'net.nemein.wiki'
             || $link['component'] == 'org.openpsa.calendar') {
             echo "<li><input type=\"button\" class=\"button info\" value=\"" . midcom::get()->i18n->get_string('details', 'org.openpsa.relatedto') . "\" /></li>\n";
         }