flack / openpsa

lib/midcom/helper/datamanager2/type/privilege.php

Spacing +1 added lines, -1 removed lines

@@ -66,7 +66,7 @@
      */
     public function _on_initialize()
     {
-        if (   !$this->name
+        if (!$this->name
             || !$this->assignee) {
             throw new midcom_error("The field {$this->name} had no name or assignee specified with it, cannot start up.");
         }

lib/midcom/helper/configuration.php

Spacing +5 added lines, -5 removed lines

@@ -144,7 +144,7 @@ 
     private function _update_cache()
     {
         $this->_merged = $this->_global;
-        if (   !empty($this->_local)
+        if (!empty($this->_local)
             && is_array($this->_local)) {
             $this->_merged = array_merge($this->_merged, $this->_local);
         }
@@ -186,7 +186,7 @@ 
      */
     public function store(array $params, $reset = true)
     {
-        if (   !$this->_object_stored
+        if (!$this->_object_stored
             && $this->_object) {
             $this->_store_from_object();
         }
@@ -246,7 +246,7 @@ 
      */
     public function get($key)
     {
-        if (   !$this->_object_stored
+        if (!$this->_object_stored
             && $this->_object) {
             $this->_store_from_object();
         }
@@ -278,7 +278,7 @@ 
      */
     public function get_all()
     {
-        if (   !$this->_object_stored
+        if (!$this->_object_stored
             && $this->_object) {
             $this->_store_from_object();
         }
@@ -294,7 +294,7 @@ 
      */
     public function exists($key)
     {
-        if (   !$this->_object_stored
+        if (!$this->_object_stored
             && $this->_object) {
             $this->_store_from_object();
         }

lib/midcom/admin/folder/handler/order.php

Braces +2 added lines, -1 removed lines

@@ -69,7 +69,8 @@
         // Use the DB Factory to resolve the class and to get the object
         try {
             $object = midcom::get()->dbfactory->get_object_by_guid($identifier);
-        } catch (midcom_error $e) {
+        }
+        catch (midcom_error $e) {
             // This is probably a pseudo leaf, store the score to the current node
             $this->_topic->set_parameter('midcom.helper.nav.score', $identifier, $score);
             return true;

Spacing +3 added lines, -3 removed lines

@@ -89,7 +89,7 @@ 
         }
 
         // Approve if possible
-        if (   $approval_status
+        if ($approval_status
             && $object->can_do('midcom:approve')) {
             $object->metadata->approve();
         }
@@ -135,10 +135,10 @@ 
             // jQuery sorting
             midcom::get()->head->enable_jquery_ui(array('mouse', 'sortable'));
 
-            midcom::get()->head->add_jsfile(MIDCOM_STATIC_URL.'/midcom.admin.folder/jquery-postfix.js');
+            midcom::get()->head->add_jsfile(MIDCOM_STATIC_URL . '/midcom.admin.folder/jquery-postfix.js');
 
             // Custom styles
-            $this->add_stylesheet(MIDCOM_STATIC_URL.'/midcom.admin.folder/midcom-admin-order.css');
+            $this->add_stylesheet(MIDCOM_STATIC_URL . '/midcom.admin.folder/midcom-admin-order.css');
         }
     }
 

lib/midcom/admin/folder/handler/approvals.php

Spacing +1 added lines, -1 removed lines

@@ -23,7 +23,7 @@
      */
     public function _handler_approval($handler_id, array $args, array &$data)
     {
-        if (   !array_key_exists('guid', $_REQUEST)
+        if (!array_key_exists('guid', $_REQUEST)
             || !array_key_exists('return_to', $_REQUEST)) {
             throw new midcom_error('Cannot process approval request, request is incomplete.');
         }

lib/midcom/admin/folder/selectcomponent.php

Indentation +1 added lines, -1 removed lines

@@ -45,7 +45,7 @@
         $select_attributes['class'] .= ' selectcomponent';
 
         $select_element = $this->_form->createElement('select', $this->name, $this->_translate($this->_field['title']),
-           array(), $select_attributes);
+            array(), $select_attributes);
 
         // Translate and add
         foreach ($this->_all_elements as $key => $value) {

lib/midcom/admin/help/help.php

Spacing +8 added lines, -8 removed lines

@@ -33,7 +33,7 @@ 
         parent::__construct();
         $this->add_stylesheet(MIDCOM_STATIC_URL . '/midcom.admin.help/style-editor.css');
 
-        midcom::get()->head->add_jsfile(MIDCOM_STATIC_URL.'/midcom.admin.help/twisty.js');
+        midcom::get()->head->add_jsfile(MIDCOM_STATIC_URL . '/midcom.admin.help/twisty.js');
         if (defined('MGD_TYPE_NONE')) {
             $this->mgdtypes[MGD_TYPE_NONE] = 'none';
         }
@@ -48,7 +48,7 @@ 
 
     static function check_component($component)
     {
-        if (   !midcom::get()->componentloader->is_installed($component)
+        if (!midcom::get()->componentloader->is_installed($component)
             && $component != 'midcom') {
             throw new midcom_error("Failed to generate documentation path for component {$component} as it is not installed.");
         }
@@ -234,9 +234,9 @@ 
         $files = array();
         $pattern = $component_dir . '*.{' . midcom::get()->i18n->get_current_language() . ',' . midcom::get()->config->get('i18n_fallback_language') . '}.txt';
 
-        foreach (glob($pattern, GLOB_NOSORT|GLOB_BRACE) as $path) {
+        foreach (glob($pattern, GLOB_NOSORT | GLOB_BRACE) as $path) {
             $entry = basename($path);
-            if (    substr($entry, 0, 5) == 'index'
+            if (substr($entry, 0, 5) == 'index'
                  || substr($entry, 0, 7) == 'handler'
                  || substr($entry, 0, 9) == 'urlmethod') {
                 // Ignore dotfiles, handlers & index.lang.txt
@@ -261,7 +261,7 @@ 
 
         // TODO: We're using "private" members here, better expose them through a method
         $handler = midcom::get()->componentloader->get_interface_class($component);
-        $request =& $handler->_context_data[midcom_core_context::get()->id]['handler'];
+        $request = & $handler->_context_data[midcom_core_context::get()->id]['handler'];
         if (!isset($request->_request_switch)) {
             // No request switch available, skip loading it
             return $data;
@@ -316,7 +316,7 @@ 
             $exec_path = MIDCOM_ROOT . '/' . $component_path . '/exec';
         }
 
-        if (   !is_dir($exec_path)
+        if (!is_dir($exec_path)
             || !is_readable($exec_path)) {
             // Directory not accessible, skip loading it
             return $data;
@@ -427,7 +427,7 @@ 
     {
         $this->add_breadcrumb("__ais/help/", $this->_l10n->get('midcom.admin.help'));
 
-        if (   $handler_id == '____ais-help-help'
+        if ($handler_id == '____ais-help-help'
             || $handler_id == '____ais-help-component') {
             $this->add_breadcrumb(
                 "__ais/help/{$this->_request_data['component']}/",
@@ -436,7 +436,7 @@ 
         }
 
         if ($handler_id == '____ais-help-help') {
-            if (   $this->_request_data['help_id'] == 'handlers'
+            if ($this->_request_data['help_id'] == 'handlers'
                 || $this->_request_data['help_id'] == 'urlmethods'
                 || $this->_request_data['help_id'] == 'mgdschemas') {
                 $this->add_breadcrumb(

lib/midcom/db/topic.php

Spacing +1 added lines, -1 removed lines

@@ -30,7 +30,7 @@
     public function _on_loaded()
     {
         // if we are missing the component, use the nullcomponent.
-        if (   !$this->component
+        if (!$this->component
             || !array_key_exists($this->component, midcom::get()->componentloader->manifests)) {
             debug_add("Topic {$this->id} has no component assigned to it, using 'midcom.core.nullcomponent'.",
             MIDCOM_LOG_INFO);

lib/midcom/response/styled.php

Spacing +1 added lines, -1 removed lines

@@ -49,7 +49,7 @@
             $this->context->set_key(MIDCOM_CONTEXT_PAGETITLE, $meta[MIDCOM_NAV_NAME]);
         }
 
-        if (   $this->context->id == 0
+        if ($this->context->id == 0
             && !midcom::get()->skip_page_style) {
             // Let metadata service add its meta tags
             midcom::get()->metadata->populate_meta_head();

lib/midcom/response/relocate.php

Spacing +3 added lines, -3 removed lines

@@ -28,14 +28,14 @@
 
     public function send()
     {
-        if (   $this->url == ''
-            || (   substr($this->url, 0, 1) != "/")
+        if ($this->url == ''
+            || (substr($this->url, 0, 1) != "/")
                 && !preg_match('|^https?://|', $this->url)) {
             $prefix = midcom_core_context::get()->get_key(MIDCOM_CONTEXT_ANCHORPREFIX);
             if ($prefix == '') {
                 $prefix = '/';
             }
-            $this->url =  "{$prefix}{$this->url}";
+            $this->url = "{$prefix}{$this->url}";
             debug_add("This is a relative URL from the local site, prepending anchor prefix: {$this->url}");
         }
         $location = "Location: {$this->url}";