flack / openpsa

lib/org/openpsa/contacts/handler/duplicates.php

Spacing +8 added lines, -8 removed lines

@@ -59,7 +59,7 @@ 
 
     private function load_next()
     {
-        $i =& $this->_request_data['loop_i'];
+        $i = & $this->_request_data['loop_i'];
         while ($i < 100) {
             debug_add("Loop iteration {$i}");
             $qb = new midgard_query_builder('midgard_parameter');
@@ -87,7 +87,7 @@ 
                 continue;
             }
             // Make sure we actually have enough rights to do this
-            if (   !$object1->can_do('midgard:update')
+            if (!$object1->can_do('midgard:update')
                 || !$object1->can_do('midgard:delete')
                 || !$object2->can_do('midgard:update')
                 || !$object2->can_do('midgard:delete')) {
@@ -96,7 +96,7 @@ 
                 continue;
             }
             // Extra sanity check (in case of semi-successful not-duplicate mark)
-            if (   $object1->get_parameter('org.openpsa.contacts.duplicates:not_duplicate', $object2->guid)
+            if ($object1->get_parameter('org.openpsa.contacts.duplicates:not_duplicate', $object2->guid)
                 || $object2->get_parameter('org.openpsa.contacts.duplicates:not_duplicate', $object1->guid)) {
                 debug_add("It seems these two (#{$object1->id} and #{$object2->id}) have also marked as not duplicates, some cleanup might be a good thing", MIDCOM_LOG_WARN);
                 $i++;
@@ -128,7 +128,7 @@ 
             if ($keep == 'both') {
                 $option1->require_do('midgard:update');
                 $option2->require_do('midgard:update');
-                if (   $option1->set_parameter('org.openpsa.contacts.duplicates:not_duplicate', $option2->guid, time())
+                if ($option1->set_parameter('org.openpsa.contacts.duplicates:not_duplicate', $option2->guid, time())
                     && $option2->set_parameter('org.openpsa.contacts.duplicates:not_duplicate', $option1->guid, time())) {
                     // Clear the possible duplicate parameters
                     $option1->delete_parameter('org.openpsa.contacts.duplicates:possible_duplicate', $option2->guid);
@@ -146,11 +146,11 @@ 
                 }
             } else {
                 if ($keep == $option1->guid) {
-                    $object1 =& $option1;
-                    $object2 =& $option2;
+                    $object1 = & $option1;
+                    $object2 = & $option2;
                 } elseif ($keep == $option2->guid) {
-                    $object1 =& $option2;
-                    $object2 =& $option1;
+                    $object1 = & $option2;
+                    $object2 = & $option1;
                 } else {
                     throw new midcom_error('Something weird happened (basically we got bogus data)');
                 }

Braces +4 added lines, -2 removed lines

@@ -82,7 +82,8 @@ 
             try {
                 $object1 = $this->load($param->parentguid);
                 $object2 = $this->load($param->name);
-            } catch (midcom_error $e) {
+            }
+            catch (midcom_error $e) {
                 $i++;
                 continue;
             }
@@ -160,7 +161,8 @@ 
                 try {
                     $merger = new org_openpsa_contacts_duplicates_merge($this->mode, $this->_config);
                     $merger->merge_delete($object1, $object2);
-                } catch (midcom_error $e) {
+                }
+                catch (midcom_error $e) {
                     // TODO: Localize
                     midcom::get()->uimessages->add($this->_l10n->get('org.openpsa.contacts'), 'Merge failed, errstr: ' . $e->getMessage(), 'error');
                 }