flack / openpsa

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

Spacing +1 added lines, -1 removed lines

@@ -75,7 +75,7 @@
             $data['query'] = new org_openpsa_reports_query_dba($guid);
             $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;
         }
 

lib/net/nemein/redirector/handler/redirect.php

Spacing +3 added lines, -3 removed lines

@@ -34,7 +34,7 @@ 
 
         // Catch first the configuration option for showing editing interface instead
         // of redirecting administrators
-        if (   $this->_topic->can_do('net.nemein.redirector:noredirect')
+        if ($this->_topic->can_do('net.nemein.redirector:noredirect')
             && !$this->_config->get('admin_redirection')) {
             return new midcom_response_relocate($this->router->generate('edit', ['tinyurl' => $tinyurl]));
         }
@@ -68,8 +68,8 @@ 
     {
         $data['url'] = net_nemein_redirector_viewer::topic_links_to($this->_config);
 
-        if (   $this->_config->get('redirection_type') !== null
-            && (   !$this->_topic->can_do('net.nemein.redirector:noredirect')
+        if ($this->_config->get('redirection_type') !== null
+            && (!$this->_topic->can_do('net.nemein.redirector:noredirect')
                 || $this->_config->get('admin_redirection'))) {
             return $this->redirect($data['url']);
         }

lib/net/nehmer/static/viewer.php

Spacing +1 added lines, -1 removed lines

@@ -54,7 +54,7 @@
             }
         }
 
-        if (   $this->_topic->can_do('midgard:update')
+        if ($this->_topic->can_do('midgard:update')
             && $this->_topic->can_do('midcom:component_config')) {
                 $buttons[] = $workflow->get_button($this->router->generate('config'), [
                 MIDCOM_TOOLBAR_LABEL => $this->_l10n_midcom->get('component configuration'),

lib/net/nehmer/static/handler/view.php

Braces +2 added lines, -1 removed lines

@@ -112,7 +112,8 @@
         foreach ($qb->execute() as $article) {
             try {
                 $datamanager->set_storage($article);
-            } catch (midcom_error $e) {
+            }
+            catch (midcom_error $e) {
                 $e->log();
                 continue;
             }

Spacing +4 added lines, -4 removed lines

@@ -40,7 +40,7 @@ 
             ]);
         }
 
-        if (   $this->_article->topic === $this->_topic->id
+        if ($this->_article->topic === $this->_topic->id
             && $this->_article->can_do('midgard:delete')) {
             $delete = $this->get_workflow('delete', ['object' => $this->_article]);
             $buttons[] = $delete->get_button($this->router->generate('delete', ['guid' => $this->_article->guid]));
@@ -81,7 +81,7 @@ 
         $this->_datamanager->set_storage($this->_article);
 
         $arg = $this->_article->name ?: $this->_article->guid;
-        if (   $arg != 'index'
+        if ($arg != 'index'
             && $this->_config->get('hide_navigation')) {
             $this->add_breadcrumb($this->router->generate('view', ['name' =>$arg]), $this->_article->title);
         }
@@ -91,13 +91,13 @@ 
         midcom::get()->metadata->set_request_metadata($this->_article->metadata->revised, $this->_article->guid);
         $this->bind_view_to_object($this->_article, $this->_datamanager->get_schema()->get_name());
 
-        if (   $this->_config->get('indexinnav')
+        if ($this->_config->get('indexinnav')
             || $this->_config->get('autoindex')
             || $this->_article->name != 'index') {
             $this->set_active_leaf($this->_article->id);
         }
 
-        if (   $this->_config->get('folder_in_title')
+        if ($this->_config->get('folder_in_title')
             && $this->_topic->extra != $this->_article->title) {
             midcom::get()->head->set_pagetitle("{$this->_topic->extra}: {$this->_article->title}");
         } else {

lib/org/openpsa/projects/style/show-project.php

Braces +2 added lines, -1 removed lines

@@ -46,7 +46,8 @@
             $customer_html = '<a href="' . $contacts_url . '/group/' . $customer->guid . '/">' . $customer_html . "</a>\n";
         }
         echo $customer_html;
-    } catch (midcom_error) {
+    }
+    catch (midcom_error) {
     }
 
     if ($project->manager) {

lib/midcom/core/query.php

Spacing +1 added lines, -1 removed lines

@@ -102,7 +102,7 @@
             $this->_user_id = midcom::get()->auth->acl->get_user_id();
         }
 
-        if (   $this->hide_invisible
+        if ($this->hide_invisible
             && !$this->_visibility_checks_added) {
             if (!midcom::get()->config->get('show_hidden_objects')) {
                 $this->add_constraint('metadata.hidden', '=', false);

src/midcom/datamanager/extension/transformer/multipleTransformer.php

Spacing +2 added lines, -2 removed lines

@@ -54,7 +54,7 @@ 
                     return [];
                 }
                 $results = explode($this->multiple_separator, substr($input, 1, -1));
-                if (   !empty($this->config['widget_config']['id_field'])
+                if (!empty($this->config['widget_config']['id_field'])
                     && $this->config['widget_config']['id_field'] == 'id') {
                     $results = array_map(intval(...), $results);
                 }
@@ -67,7 +67,7 @@ 
 
     public function reverseTransform(mixed $array) : mixed
     {
-        if (!is_array($array) ) {
+        if (!is_array($array)) {
             throw new TransformationFailedException('Expected an array.');
         }
 

src/midcom/dba/privileges.php

Spacing +3 added lines, -3 removed lines

@@ -66,7 +66,7 @@ 
      */
     public function set_privilege($privilege, $assignee = null, int $value = MIDCOM_PRIVILEGE_ALLOW, string $classname = '') : bool
     {
-        if (   !$this->can_do('midgard:update')
+        if (!$this->can_do('midgard:update')
             || !$this->can_do('midgard:privileges')) {
             debug_add("Failed to set a privilege, midgard:update or midgard:privileges on " . static::class . " {$this->guid} not granted for the current user.",
             MIDCOM_LOG_ERROR);
@@ -96,7 +96,7 @@ 
      */
     public function unset_privilege($privilege, $assignee = null, string $classname = '') : bool
     {
-        if (   !$this->can_do('midgard:update')
+        if (!$this->can_do('midgard:update')
             || !$this->can_do('midgard:privileges')) {
             debug_add("Failed to unset a privilege, midgard:update or midgard:privileges on " . static::class . " {$this->guid} not granted for the current user.",
             MIDCOM_LOG_ERROR);
@@ -125,7 +125,7 @@ 
      *     midcom_core_group object or subtype thereof.
      * @param string $classname An optional class name to which a SELF privilege is restricted to.
      */
-    public function get_privilege(string $privilege, midcom_core_user|midcom_core_group|string $assignee, string $classname = '')
+    public function get_privilege(string $privilege, midcom_core_user | midcom_core_group | string $assignee, string $classname = '')
     {
         if (!$this->can_do('midgard:privileges')) {
             debug_add("Failed to get a privilege, midgard:update or midgard:privileges on " . static::class . " {$this->guid} not granted for the current user.",

lib/midcom/helper/imagefilter.php

Spacing +3 added lines, -3 removed lines

@@ -323,7 +323,7 @@ 
         $tmpfile = null;
         $imagesize = getimagesize($this->_filename);
 
-        if (   $imagesize[2] == IMAGETYPE_JPEG
+        if ($imagesize[2] == IMAGETYPE_JPEG
             && $this->_jpegtran_available()) {
             /* jpegtran */
             $operations = [
@@ -390,7 +390,7 @@ 
         $imagesize = getimagesize($this->_filename);
 
         // Try lossless jpegtran rotation if possible
-        if (   $imagesize[2] == IMAGETYPE_JPEG
+        if ($imagesize[2] == IMAGETYPE_JPEG
             && ($rotate % 90 == 0)
             && $this->_jpegtran_available()) {
             $tmpfile = $this->_get_tempfile();
@@ -502,7 +502,7 @@ 
      */
     public function fill(?int $x = null, ?int $y = null, ?string $color = null, string $gravity = 'center')
     {
-        if (   empty($x)
+        if (empty($x)
             || empty($y)
             || empty($color)) {
             //This is a bit silly, but here for backwards compatibility...