flack / openpsa

lib/net/nehmer/blog/style/feeds-item.php

Spacing +2 added lines, -2 removed lines

@@ -18,7 +18,7 @@ 
 $item->title = $data['article']->title;
 $arg = $data['article']->name ?: $data['article']->guid;
 
-if (   $data['config']->get('link_to_external_url')
+if ($data['config']->get('link_to_external_url')
     && !empty($data['article']->url)) {
     $item->link = $data['article']->url;
 } else {
@@ -37,7 +37,7 @@ 
     $item->description .= '<div class="abstract">' . $view['abstract'] . '</div>';
 }
 
-if (   array_key_exists('image', $view)
+if (array_key_exists('image', $view)
     && $data['config']->get('rss_use_image')) {
     $item->description .= "\n<div class=\"image\">" . $view['image'] . '</div>';
 }

lib/net/nemein/wiki/handler/view.php

Spacing +4 added lines, -4 removed lines

@@ -34,7 +34,7 @@ 
 
     public function _on_initialize()
     {
-        $this->_request_data['page'] =& $this->_page;
+        $this->_request_data['page'] = & $this->_page;
     }
 
     /**
@@ -216,9 +216,9 @@ 
         $toc .= "\n<ol class=\"midcom_helper_toc_formatter level_{$current_list_level}\">\n";
         foreach ($headings[4] as $key => $heading) {
             $anchor = 'heading-' . md5($heading);
-            $tag_level =& $headings[3][$key];
-            $heading_code =& $headings[0][$key];
-            $heading_tag =& $headings[2][$key];
+            $tag_level = & $headings[3][$key];
+            $heading_code = & $headings[0][$key];
+            $heading_tag = & $headings[2][$key];
             $heading_new_code = "<a id='{$anchor}'></a>{$heading_code}";
             $content = str_replace($heading_code, $heading_new_code, $content);
             if ($current_tag_level === false) {

lib/org/openpsa/core/acl/synchronizer.php

Spacing +3 added lines, -3 removed lines

@@ -15,7 +15,7 @@ 
 {
     public function write_acls(midcom_core_dbaobject $object, $owner_id, $accesstype)
     {
-        if (   empty($owner_id)
+        if (empty($owner_id)
             || empty($accesstype)) {
             return false;
         }
@@ -63,14 +63,14 @@ 
 
         if ($privileges) {
             foreach ($privileges as $privilege) {
-                if (   !empty($needed_privileges[$privilege->privilegename])
+                if (!empty($needed_privileges[$privilege->privilegename])
                     && $needed_privileges[$privilege->privilegename]['assignee'] == $privilege->assignee
                     && $needed_privileges[$privilege->privilegename]['value'] == $privilege->value) {
                     unset($needed_privileges[$privilege->privilegename]);
                     continue;
                 }
                 // Clear old ACLs applying to others than current user or selected owner group
-                if (   $privilege->assignee != midcom::get()->auth->user->id
+                if ($privilege->assignee != midcom::get()->auth->user->id
                     && $privilege->assignee != $owner_id) {
                     debug_add("Removing privilege {$privilege->privilegename} from {$privilege->assignee}");
                     $object->unset_privilege($privilege->privilegename, $privilege->assignee);

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

Spacing +2 added lines, -2 removed lines

@@ -215,7 +215,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']);
 
@@ -408,7 +408,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";
         }

Braces +6 added lines, -3 removed lines

@@ -134,7 +134,8 @@ 
             ];
             try {
                 $to_arr['other_obj'] = midcom::get()->dbfactory->get_object_by_guid($mc->get_subkey($guid, $object_prefix . 'Guid'));
-            } catch (midcom_error $e) {
+            }
+            catch (midcom_error $e) {
                 continue;
             }
 
@@ -438,7 +439,8 @@ 
             if (!($this->_object instanceof org_openpsa_relatedto_dba)) {
                 $response->status = "method requires guid of a link object as an argument";
             }
-        } catch (midcom_error $e) {
+        }
+        catch (midcom_error $e) {
             $response->status = "error: " . $e->getMessage();
         }
 
@@ -464,7 +466,8 @@ 
             $relation = new org_openpsa_relatedto_dba($guid);
             $response->result = $relation->delete();
             $response->status = 'Last message: ' . midcom_connection::get_error_string();
-        } catch (midcom_error $e) {
+        }
+        catch (midcom_error $e) {
             $response->result = false;
             $response->status = "Object '{$guid}' could not be loaded, error:" . $e->getMessage();
         }

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

Spacing +1 added lines, -1 removed lines

@@ -101,7 +101,7 @@
 
         $this->_controller = $this->load_controller();
 
-        if (   $data['enable_versioning']
+        if ($data['enable_versioning']
             && $request->request->count() > 0) {
             $this->_backup_attachment();
         }

lib/org/openpsa/qbpager/main.php

Spacing +6 added lines, -6 removed lines

@@ -63,13 +63,13 @@ 
         $results_var = $this->_prefix . 'results';
         if (!empty($_REQUEST[$page_var])) {
             debug_add("{$page_var} has value: {$_REQUEST[$page_var]}");
-            $this->_current_page = (int)$_REQUEST[$page_var];
+            $this->_current_page = (int) $_REQUEST[$page_var];
         }
         if (!empty($_REQUEST[$results_var])) {
             debug_add("{$results_var} has value: {$_REQUEST[$results_var]}");
-            $this->results_per_page = (int)$_REQUEST[$results_var];
+            $this->results_per_page = (int) $_REQUEST[$results_var];
         }
-        $this->_offset = ($this->_current_page-1)*$this->results_per_page;
+        $this->_offset = ($this->_current_page - 1) * $this->results_per_page;
         if ($this->_offset < 0) {
             $this->_offset = 0;
         }
@@ -291,7 +291,7 @@ 
         return $this->_midcom_qb->add_constraint($param, $op, $val);
     }
 
-    public function add_order($param, $sort='ASC')
+    public function add_order($param, $sort = 'ASC')
     {
         if (!$this->_sanity_check()) {
             return false;
@@ -328,7 +328,7 @@ 
         if (!$this->_sanity_check()) {
             return false;
         }
-        if (   !$this->count
+        if (!$this->count
             || $this->_count_mode != 'count') {
             $this->count = $this->_midcom_qb_count->count();
         }
@@ -341,7 +341,7 @@ 
         if (!$this->_sanity_check()) {
             return false;
         }
-        if (   !$this->count
+        if (!$this->count
             || $this->_count_mode != 'count_unchecked') {
             $this->count = $this->_midcom_qb_count->count_unchecked();
         }

lib/org/openpsa/reports/handler/base.php

Spacing +2 added lines, -2 removed lines

@@ -88,7 +88,7 @@ 
             $data['query'] = new org_openpsa_reports_query_dba($args[0]);
             $data['query']->require_do('midgard:update');
         } else {
-            $data['query']= new org_openpsa_reports_query_dba();
+            $data['query'] = new org_openpsa_reports_query_dba();
             $data['query']->component = $this->_component;
         }
 
@@ -116,7 +116,7 @@ 
 
     protected function _generator_load_redirect(&$args)
     {
-        $this->_request_data['query'] = new org_openpsa_reports_query_dba($args[0]);;
+        $this->_request_data['query'] = new org_openpsa_reports_query_dba($args[0]); ;
 
         if (empty($args[1])) {
             debug_add('Filename part not specified in URL, generating');

lib/midgard/admin/asgard/handler/type.php

Spacing +1 added lines, -1 removed lines

@@ -104,7 +104,7 @@
         }
 
         $help = new midcom_admin_help_help();
-        $this->_request_data['help'] = $help->get_help_contents('asgard_'.$this->type, $help_component);
+        $this->_request_data['help'] = $help->get_help_contents('asgard_' . $this->type, $help_component);
         $this->_request_data['component'] = $component;
     }
 

lib/midcom/helper/misc.php

Spacing +1 added lines, -1 removed lines

@@ -281,7 +281,7 @@
             return $data;
         }
 
-        $preg='/s:([0-9]+):"(.*?)";/ms';
+        $preg = '/s:([0-9]+):"(.*?)";/ms';
         preg_match_all($preg, $data, $matches);
         $cache = [];