flack / openpsa

lib/org/openpsa/projects/handler/task/resourcing.php

Spacing +2 added lines, -2 removed lines

@@ -64,7 +64,7 @@ 
             return new midcom_response_relocate($this->router->generate('task_view', ['guid' => $guid]));
         }
 
-        if (   $request->request->has('save')
+        if ($request->request->has('save')
             && $prospects = $request->request->get('org_openpsa_projects_prospects')) {
             $qb = org_openpsa_projects_task_resource_dba::new_query_builder();
             $qb->add_constraint('guid', 'IN', array_keys($prospects));
@@ -95,7 +95,7 @@ 
                     // create relatedto
                     org_openpsa_relatedto_plugin::create($event, 'org.openpsa.calendar', $this->_task, 'org.openpsa.projects');
                 }
-                if (   $update_prospect
+                if ($update_prospect
                     && !$prospect->update()) {
                     debug_add('Failed to update prospect: ' . midcom_connection::get_error_string(), MIDCOM_LOG_ERROR);
                 }

lib/org/openpsa/directmarketing/handler/subscriber.php

Braces +4 added lines, -2 removed lines

@@ -36,7 +36,8 @@ 
             // Add person to campaign
             try {
                 $campaign = new org_openpsa_directmarketing_campaign_dba($add_to);
-            } catch (midcom_error $e) {
+            }
+            catch (midcom_error $e) {
                 // FIXME: More informative error message
                 $this->notify('Failed adding person %s to campaign %s', $add_to, 'error');
                 return;
@@ -86,7 +87,8 @@ 
             try {
                 $campaigns[$membership->campaign] = new org_openpsa_directmarketing_campaign_dba($membership->campaign);
                 $campaign_membership_map[$membership->campaign] = $membership;
-            } catch (midcom_error $e) {
+            }
+            catch (midcom_error $e) {
                 debug_add('Failed to load campaign ' . $membership->campaign . ', reason: ' . $e->getMessage());
             }
         }

lib/org/openpsa/directmarketing/handler/campaign/rules.php

Braces +2 added lines, -1 removed lines

@@ -105,7 +105,8 @@
         if ($request->request->has('midcom_helper_datamanager2_save')) {
             try {
                 $rules = $this->_load_rules();
-            } catch (midcom_error $e) {
+            }
+            catch (midcom_error $e) {
                 midcom::get()->uimessages->add('org.openpsa.directmarketing', $this->_l10n->get($e->getMessage()), 'error');
                 return;
             }

lib/org/openpsa/products/handler/product/create.php

Braces +2 added lines, -1 removed lines

@@ -89,7 +89,8 @@
         } elseif ((int) $group > 0) {
             try {
                 $this->parent = new org_openpsa_products_product_group_dba((int) $group);
-            } catch (midcom_error $e) {
+            }
+            catch (midcom_error $e) {
                 $e->log();
             }
         }

lib/org/openpsa/contacts/handler/group/action.php

Braces +2 added lines, -1 removed lines

@@ -29,7 +29,8 @@
                 $member->require_do('midgard:update');
                 $member->extra = $request->request->get('title');
                 $response->status = $member->update();
-            } catch (midcom_error $e) {
+            }
+            catch (midcom_error $e) {
                 $e->log();
             }
             $response->message = midcom_connection::get_error_string();

lib/org/openpsa/calendar/handler/event/create.php

Spacing +1 added lines, -1 removed lines

@@ -44,7 +44,7 @@
         if ($query->has('start')) {
             $defaults['start'] = strtotime($query->get('start'));
             if ($query->has('end')) {
-                $defaults['end']= strtotime($query->get('end'));
+                $defaults['end'] = strtotime($query->get('end'));
             } else {
                 $defaults['end'] = $defaults['start'] + 3600;
             }

lib/midgard/admin/asgard/handler/object/attachments.php

Spacing +3 added lines, -3 removed lines

@@ -96,13 +96,13 @@ 
         }
 
         $mimetype = $request->request->get('midgard_admin_asgard_mimetype');
-        if (   !empty($mimetype)
+        if (!empty($mimetype)
             && $local_file->mimetype != $mimetype) {
             $local_file->mimetype = $mimetype;
             $needs_update = true;
         }
 
-        if (   $needs_update
+        if ($needs_update
             && !$local_file->update()) {
             return false;
         }
@@ -243,7 +243,7 @@ 
         $this->bind_view_to_object($this->_file);
 
         $filename = $this->_process_form($request);
-        if (   $filename
+        if ($filename
             && $filename != $data['filename']) {
             return $this->relocate_to_file($filename);
         }

lib/midgard/admin/user/handler/group/list.php

Spacing +1 added lines, -1 removed lines

@@ -58,7 +58,7 @@
                 midcom::get()->uimessages->add($this->_l10n->get('midgard.admin.user'), $this->_l10n_midcom->get('updated'));
                 return new midcom_response_relocate($this->router->generate('group_edit', ['guid' => $guid]));
             }
-            debug_add('Failed to update the group, last error was '. midcom_connection::get_error_string(), MIDCOM_LOG_ERROR);
+            debug_add('Failed to update the group, last error was ' . midcom_connection::get_error_string(), MIDCOM_LOG_ERROR);
             debug_print_r('We operated on this object', $data['group'], MIDCOM_LOG_ERROR);
 
             throw new midcom_error('Failed to update the group, see error level log for details');

lib/midcom/baseclasses/components/viewer.php

Braces +2 added lines, -1 removed lines

@@ -348,7 +348,8 @@
             $result = $this->router->match($url);
             $this->_prepare_handler($result);
             return $this->_handler;
-        } catch (ResourceNotFoundException $e) {
+        }
+        catch (ResourceNotFoundException $e) {
             // No match
             return false;
         }

Spacing +2 added lines, -2 removed lines

@@ -328,7 +328,7 @@ 
         $prefix = midcom_core_context::get()->get_key(MIDCOM_CONTEXT_ANCHORPREFIX);
 
         // Check if we need to start up a plugin.
-        if (   count($argv) > 1
+        if (count($argv) > 1
             && array_key_exists($argv[0], self::$_plugin_namespace_config)
             && array_key_exists($argv[1], self::$_plugin_namespace_config[$argv[0]])) {
             $namespace = array_shift($argv);
@@ -375,7 +375,7 @@ 
      */
     private function _prepare_handler(array $request)
     {
-        $this->_handler =& $request;
+        $this->_handler = & $request;
         $args = [];
         foreach ($request as $name => $value) {
             if (substr($name, 0, 1) !== '_') {