flack / openpsa

lib/midcom/helper/toolbar/node.php

Braces +2 added lines, -1 removed lines

@@ -88,7 +88,8 @@
                         $style = midcom_db_style::get_cached($style_id);
                         $styleeditor_url = midcom_connection::get_url('self') . "__mfa/asgard/object/view/{$style->guid}/";
                         $enabled = true;
-                    } catch (midcom_error $e) {
+                    }
+                    catch (midcom_error $e) {
                         $e->log();
                     }
                 }

Spacing +6 added lines, -6 removed lines

@@ -37,7 +37,7 @@ 
         }
         $buttons = array();
         $workflow = new midcom\workflow\datamanager2;
-        if (   $this->topic->can_do('midgard:update')
+        if ($this->topic->can_do('midgard:update')
             && $this->topic->can_do('midcom.admin.folder:topic_management')) {
             $buttons[] = $workflow->get_button("__ais/folder/edit/", array(
                 MIDCOM_TOOLBAR_LABEL => midcom::get()->i18n->get_string('edit folder', 'midcom.admin.folder'),
@@ -49,7 +49,7 @@ 
             ));
         }
 
-        if (   $urltopic->can_do('midgard:update')
+        if ($urltopic->can_do('midgard:update')
             && $urltopic->can_do('midcom.admin.folder:topic_management')) {
             // Allow to move other than root folder
             if ($urltopic->guid !== midcom::get()->config->get('midcom_root_topic_guid')) {
@@ -61,7 +61,7 @@ 
             }
         }
 
-        if (   $this->topic->can_do('midgard:update')
+        if ($this->topic->can_do('midgard:update')
             && $this->topic->can_do('midcom.admin.folder:topic_management')) {
             $buttons[] = array(
                 MIDCOM_TOOLBAR_URL => "__ais/folder/order/",
@@ -78,7 +78,7 @@ 
             );
         }
         $buttons = array_merge($buttons, $this->get_approval_controls($this->topic, false));
-        if (   $this->topic->can_do('midcom.admin.folder:template_management')
+        if ($this->topic->can_do('midcom.admin.folder:template_management')
             && midcom::get()->auth->can_user_do('midgard.admin.asgard:manage_objects', null, 'midgard_admin_asgard_plugin')) {
             $enabled = false;
             $styleeditor_url = '';
@@ -103,7 +103,7 @@ 
             );
         }
 
-        if (   $this->topic->can_do('midgard:create')
+        if ($this->topic->can_do('midgard:create')
             && $this->topic->can_do('midcom.admin.folder:topic_management')) {
             $buttons[] = $workflow->get_button("__ais/folder/create/", array(
                 MIDCOM_TOOLBAR_LABEL => midcom::get()->i18n->get_string('create subfolder', 'midcom.admin.folder'),
@@ -111,7 +111,7 @@ 
                 MIDCOM_TOOLBAR_ACCESSKEY => 'f',
             ));
         }
-        if (   $urltopic->guid !== midcom::get()->config->get('midcom_root_topic_guid')
+        if ($urltopic->guid !== midcom::get()->config->get('midcom_root_topic_guid')
             && $urltopic->can_do('midgard:delete')
             && $urltopic->can_do('midcom.admin.folder:topic_management')) {
             $workflow = new midcom\workflow\delete(array('object' => $urltopic, 'recursive' => true));

lib/midcom/helper/toolbar/view.php

Spacing +2 added lines, -2 removed lines

@@ -53,7 +53,7 @@ 
             ));
         }
 
-        if (   midcom::get()->config->get('midcom_services_rcs_enable')
+        if (midcom::get()->config->get('midcom_services_rcs_enable')
             && $object->can_do('midgard:update')
             && $object->_use_rcs) {
             $buttons[] = array(
@@ -83,7 +83,7 @@ 
             }
 
             $icon = 'stock-icons/16x16/page-' . $icontype . '.png';
-            if (   !midcom::get()->config->get('show_hidden_objects')
+            if (!midcom::get()->config->get('show_hidden_objects')
                 && !$object->metadata->is_visible()) {
                 // Take scheduling into account
                 $icon = 'stock-icons/16x16/page-' . $icontype . '-notpublished.png';

lib/midcom/helper/_dbfactory.php

Spacing +11 added lines, -11 removed lines

@@ -49,8 +49,8 @@ 
 
             throw new midcom_error_midgard($e, $guid);
         }
-        $person_class =  midcom::get()->config->get('person_class');
-        if (   get_class($tmp) == 'midgard_person'
+        $person_class = midcom::get()->config->get('person_class');
+        if (get_class($tmp) == 'midgard_person'
             && $person_class != 'midgard_person') {
             $tmp = new $person_class($guid);
         }
@@ -81,7 +81,7 @@ 
         }
         $object = new $classname($src);
         $cache[$classname][$object->guid] = $object;
-        $cache[$classname][$object->id] =& $cache[$classname][$object->guid];
+        $cache[$classname][$object->id] = & $cache[$classname][$object->guid];
         return $cache[$classname][$object->guid];
     }
 
@@ -174,7 +174,7 @@ 
             throw new midcom_error("Cannot cast the object to an MgdSchema type, it is not a MidCOM DBA object");
         }
 
-        if (   !isset($object->__object)
+        if (!isset($object->__object)
             || !is_object($object->__object)) {
             debug_print_r("Object dump:", $object);
             throw new midcom_error("Cannot cast the object to an MgdSchema type, as it doesn't contain it");
@@ -198,14 +198,14 @@ 
             return true;
         }
 
-        if (   isset($object->__object)
+        if (isset($object->__object)
             && is_object($object->__object)
             && is_a($object->__object, $class)) {
             // Decorator whose MgdSchema object matches
             return true;
         }
 
-        if (   isset($object->__mgdschema_class_name__)
+        if (isset($object->__mgdschema_class_name__)
             && $object->__mgdschema_class_name__ == $class) {
             // Decorator without object instantiated, check class match
             return true;
@@ -227,7 +227,7 @@ 
         if (is_object($object)) {
             $helper = new helper;
             // We're dealing with either MgdSchema or MidCOM DBA object
-            if (   isset($object->__object)
+            if (isset($object->__object)
                 && is_object($object->__object)) {
                 // MidCOM DBA decorator
                 return $helper->property_exists($object->__object, $property);
@@ -236,7 +236,7 @@ 
             return $helper->property_exists($object, $property);
         }
 
-        if (   !is_string($object)
+        if (!is_string($object)
             || !class_exists($object)) {
             // TODO: Raise exception?
             debug_add("\$object is not string or class_exists() returned false", MIDCOM_LOG_WARN);
@@ -261,7 +261,7 @@ 
         $parent_data = $this->_get_parent_guid_cached($object->guid, $object);
 
         $parent_guid = current($parent_data);
-        if (   empty($parent_guid)
+        if (empty($parent_guid)
             || $parent_guid === $object->guid) {
             return null;
         }
@@ -464,11 +464,11 @@ 
                 );
             }
 
-            if (   $parent_property
+            if ($parent_property
                 && $reflector->get_link_target($parent_property)) {
                 $target_class = $reflector->get_link_name($parent_property);
                 if ($target_class == 'midgard_person') {
-                    $person_class =  midcom::get()->config->get('person_class');
+                    $person_class = midcom::get()->config->get('person_class');
                     if ($person_class != 'midgard_person') {
                         $target_class = $person_class;
                     }

Braces +8 added lines, -4 removed lines

@@ -44,7 +44,8 @@ 
     {
         try {
             $tmp = midgard_object_class::get_object_by_guid($guid);
-        } catch (midgard_error_exception $e) {
+        }
+        catch (midgard_error_exception $e) {
             debug_add('Loading object by GUID ' . $guid . ' failed, reason: ' . $e->getMessage(), MIDCOM_LOG_INFO);
 
             throw new midcom_error_midgard($e, $guid);
@@ -274,7 +275,8 @@ 
                 $parent_data = array(
                     $parent->__midcom_class_name__ => $parent_guid
                 );
-            } catch (midcom_error $e) {
+            }
+            catch (midcom_error $e) {
                 $parent_data = array(
                     '' => null
                 );
@@ -288,7 +290,8 @@ 
 
         try {
             return $this->get_cached($classname, $parent_guid);
-        } catch (midcom_error $e) {
+        }
+        catch (midcom_error $e) {
             return null;
         }
     }
@@ -347,7 +350,8 @@ 
                 if ($the_object === null) {
                     try {
                         $the_object = $this->get_object_by_guid($object_guid);
-                    } catch (midcom_error $e) {
+                    }
+                    catch (midcom_error $e) {
                         return array('' => null);
                     }
                 }

lib/midcom/helper/reflector/tree.php

Spacing +7 added lines, -7 removed lines

@@ -158,7 +158,7 @@ 
     private static function _check_permissions($deleted)
     {
         // PONDER: Check for some generic user privilege instead  ??
-        if (   $deleted
+        if ($deleted
             && !midcom_connection::is_admin()
             && !midcom::get()->auth->is_component_sudo()) {
             debug_add('Non-admins are not allowed to list deleted objects', MIDCOM_LOG_ERROR);
@@ -248,7 +248,7 @@ 
                     'target' => $ref->get_link_target($field)
                 );
                 $linked_class = $ref->get_link_name($field);
-                if (   empty($linked_class)
+                if (empty($linked_class)
                     && $info['type'] === MGD_TYPE_GUID) {
                     // Guid link without class specification, valid for all classes
                     if (empty($info['target'])) {
@@ -298,7 +298,7 @@ 
             $field_type = $field_data['type'];
             $field = $field_data['name'];
 
-            if (   !$field_target
+            if (!$field_target
                 || !isset($for_object->$field_target)) {
                 // Why return false ???
                 return false;
@@ -414,7 +414,7 @@ 
             $parent_property = midgard_object_class::get_property_parent($schema_type);
             $up_property = midgard_object_class::get_property_up($schema_type);
 
-            if (   !$this->_resolve_child_classes_links_back($parent_property, $schema_type, $this->mgdschema_class)
+            if (!$this->_resolve_child_classes_links_back($parent_property, $schema_type, $this->mgdschema_class)
                 && !$this->_resolve_child_classes_links_back($up_property, $schema_type, $this->mgdschema_class)) {
                 continue;
             }
@@ -439,7 +439,7 @@ 
 
         $ref = new midgard_reflection_property($prospect_type);
         $link_class = $ref->get_link_name($property);
-        if (   empty($link_class)
+        if (empty($link_class)
             && $ref->get_midgard_type($property) === MGD_TYPE_GUID) {
             return true;
         }
@@ -534,12 +534,12 @@ 
         $ref = self::get($schema_type);
         $dummy = new $schema_type();
         $title_property = $ref->get_title_property($dummy);
-        if (   is_string($title_property)
+        if (is_string($title_property)
             && midcom::get()->dbfactory->property_exists($schema_type, $title_property)) {
             $qb->add_order($title_property);
         }
         $name_property = $ref->get_name_property($dummy);
-        if (   is_string($name_property)
+        if (is_string($name_property)
             && midcom::get()->dbfactory->property_exists($schema_type, $name_property)) {
             $qb->add_order($name_property);
         }

Braces +2 added lines, -1 removed lines

@@ -557,7 +557,8 @@
         $tree = array();
         try {
             $children = self::get_child_objects($parent);
-        } catch (midcom_error $e) {
+        }
+        catch (midcom_error $e) {
             return $tree;
         }
 

lib/midcom/helper/reflector/copy.php

Spacing +5 added lines, -5 removed lines

@@ -346,7 +346,7 @@ 
         }
 
         // Override requested root object properties
-        if (   !empty($this->target->guid)
+        if (!empty($this->target->guid)
             && $target->guid === $this->target->guid) {
             foreach ($this->root_object_values as $name => $value) {
                 $target->$name = $value;
@@ -391,7 +391,7 @@ 
         // Store for later use - if ever needed
         $this->new_objects[] = $target;
 
-        if (   !$this->_copy_data('parameters', $source, $target)
+        if (!$this->_copy_data('parameters', $source, $target)
             || !$this->_copy_data('metadata', $source, $target)
             || !$this->_copy_data('attachments', $source, $target)
             || !$this->_copy_data('privileges', $source, $target)) {
@@ -412,7 +412,7 @@ 
     private function _copy_data($type, $source, &$target)
     {
         $method = 'copy_' . $type;
-        if (   !$this->$method($source, $target)
+        if (!$this->$method($source, $target)
             && $this->halt_on_errors) {
             $this->errors[] = $this->_l10n->get('failed to copy ' . $type);
             return false;
@@ -564,8 +564,8 @@ 
     public static function copy_object_tree($source, $parent, $exclude = array(), $copy_parameters = true, $copy_metadata = true, $copy_attachments = true)
     {
         $copy = new midcom_helper_reflector_copy();
-        $copy->source =& $source;
-        $copy->target =& $parent;
+        $copy->source = & $source;
+        $copy->target = & $parent;
         $copy->copy_parameters = $copy_parameters;
         $copy->copy_metadata = $copy_metadata;
         $copy->copy_attachments = $copy_attachments;

lib/midcom/helper/search/handler/search.php

Spacing +4 added lines, -4 removed lines

@@ -53,7 +53,7 @@ 
     {
         $node = $nap->get_node($node_id);
 
-        if (   !array_key_exists($node[MIDCOM_NAV_COMPONENT], $this->_request_data['components'])
+        if (!array_key_exists($node[MIDCOM_NAV_COMPONENT], $this->_request_data['components'])
             && $node[MIDCOM_NAV_COMPONENT] != 'midcom.helper.search') {
             $l10n = $this->_i18n->get_l10n($node[MIDCOM_NAV_COMPONENT]);
             $this->_request_data['components'][$node[MIDCOM_NAV_COMPONENT]] = $l10n->get($node[MIDCOM_NAV_COMPONENT]);
@@ -112,7 +112,7 @@ 
         $this->prepare_query_data();
         $this->prepare_formdata($_REQUEST['type']);
 
-        if (   count(explode(' ', $data['query'])) == 1
+        if (count(explode(' ', $data['query'])) == 1
             && strpos($data['query'], '*') === false
             && $this->_config->get('single_term_auto_wilcard')) {
             //If there is only one search term append * to the query if auto_wildcard is enabled
@@ -192,12 +192,12 @@ 
             $this->_request_data['last_document_number'] = $last_document_id + 1;
             $this->_request_data['shown_documents'] = $last_document_id - $first_document_id + 1;
             $this->_request_data['results_per_page'] = $results_per_page;
-            $this->_request_data['all_results'] =& $result;
+            $this->_request_data['all_results'] = & $result;
             $this->_request_data['result'] = array_slice($result, $first_document_id, $results_per_page);
 
             // Register GUIDs for cache engine
             foreach ($this->_request_data['result'] as $doc) {
-                if (   !isset($doc->source)
+                if (!isset($doc->source)
                     || !mgd_is_guid($doc->source)) {
                     // Non-Midgard results don't need to go through cache registration
                     continue;

lib/midcom/baseclasses/components/configuration.php

Braces +2 added lines, -1 removed lines

@@ -143,7 +143,8 @@
 
         try {
             $data = file_get_contents($filename);
-        } catch (Exception $e) {
+        }
+        catch (Exception $e) {
             return false;
         }
 

lib/midcom/baseclasses/components/request.php

Spacing +7 added lines, -7 removed lines

@@ -433,7 +433,7 @@ 
                 $this->_request_switch[$key]['handler'] = array(&$this, $value['handler']);
             }
 
-            if (   !array_key_exists('expires', $value)
+            if (!array_key_exists('expires', $value)
                 || !is_integer($value['expires'])
                 || $value['expires'] < -1) {
                 $this->_request_switch[$key]['expires'] = -1;
@@ -465,7 +465,7 @@ 
         }
 
         // Check if we need to start up a plugin.
-        if (   $argc > 1
+        if ($argc > 1
             && array_key_exists($argv[0], self::$_plugin_namespace_config)
             && array_key_exists($argv[1], self::$_plugin_namespace_config[$argv[0]])) {
             $namespace = $argv[0];
@@ -481,7 +481,7 @@ 
                 continue;
             }
             $fixed_args_count = count($request['fixed_args']);
-            $this->_handler =& $this->_request_switch[$key];
+            $this->_handler = & $this->_request_switch[$key];
 
             $this->_handler['id'] = $key;
             $this->_handler['args'] = array_slice($argv, $fixed_args_count);
@@ -490,10 +490,10 @@ 
             $this->_prepare_handler();
 
             // If applicable, run the _can_handle check for the handler in question.
-            $handler =& $this->_handler['handler'][0];
+            $handler = & $this->_handler['handler'][0];
             $method = "_can_handle_{$this->_handler['handler'][1]}";
 
-            if (   method_exists($handler, $method)
+            if (method_exists($handler, $method)
                 && !$handler->$method($this->_handler['id'], $this->_handler['args'], $this->_request_data)) {
                 // This can_handle failed, allow next one to take over if there is one
                 continue;
@@ -510,7 +510,7 @@ 
         $variable_args_count = $request['variable_args'];
         $total_args_count = $fixed_args_count + $variable_args_count;
 
-        if (   ($argc != $total_args_count && ($variable_args_count >= 0))
+        if (($argc != $total_args_count && ($variable_args_count >= 0))
             || $fixed_args_count > $argc) {
             return false;
         }
@@ -530,7 +530,7 @@ 
                 $success = false;
                 foreach ($request['validation'][$i] as $rule) {
                     // rule is a callable function, like mgd_is_guid or is_int
-                    if (   is_callable($rule)
+                    if (is_callable($rule)
                         && $success = call_user_func($rule, $param)) {
                         break;
                     }

lib/midcom/baseclasses/components/handler/configuration.php

Spacing +1 added lines, -1 removed lines

@@ -90,7 +90,7 @@
         midcom::get()->head->set_pagetitle($this->_l10n_midcom->get('component configuration'));
 
         $workflow = $this->get_workflow('datamanager2', array('controller' => $this->_controller));
-        if (   method_exists($this, '_load_datamanagers')
+        if (method_exists($this, '_load_datamanagers')
             && method_exists($this, '_load_objects')) {
             $workflow->add_post_button('config/recreate/', $this->_l10n_midcom->get('recreate images'), array(
                 'midcom_baseclasses_components_handler_configuration_recreateok' => true,