flack / openpsa

lib/org/openpsa/user/handler/person/privileges.php

Spacing +2 added lines, -2 removed lines

@@ -23,9 +23,9 @@
         $schemadb = schemadb::from_path($this->_config->get('schemadb_acl'));
 
         // Get the calendar root event
-        if (   class_exists('org_openpsa_calendar_interface')
+        if (class_exists('org_openpsa_calendar_interface')
             && $root_event = org_openpsa_calendar_interface::find_root_event()) {
-            $field =& $schemadb->get('default')->get_field('calendar');
+            $field = & $schemadb->get('default')->get_field('calendar');
             $field['type_config']['privilege_object'] = $root_event;
             $field['type_config']['assignee'] = 'user:' . $person->guid;
         }

lib/midgard/admin/asgard/style/midgard_admin_asgard_rcs_history.php

Spacing +1 added lines, -1 removed lines

@@ -33,7 +33,7 @@
                 echo "                    <td>" . $formatter->datetime($revision['date'], IntlDateFormatter::MEDIUM, IntlDateFormatter::LONG) . "</td>\n";
                 echo "                    <td>";
 
-                if (   $revision['user']
+                if ($revision['user']
                     && $user = midcom::get()->auth->get_user($revision['user'])) {
                     echo $user->get_storage()->name;
                 } elseif ($revision['ip']) {

lib/org/openpsa/directmarketing/handler/message/compose.php

Spacing +2 added lines, -2 removed lines

@@ -64,7 +64,7 @@ 
         }
 
         //Substyle handling
-        if (   !empty($data['message_array']['substyle'])
+        if (!empty($data['message_array']['substyle'])
             && !str_starts_with($data['message_array']['substyle'], 'builtin:')) {
             debug_add("Appending substyle {$data['message_array']['substyle']}");
             midcom::get()->style->append_substyle($data['message_array']['substyle']);
@@ -113,7 +113,7 @@ 
     private function _real_show_compose(array $data)
     {
         $prefix = '';
-        if (   !empty($data['message_array']['substyle'])
+        if (!empty($data['message_array']['substyle'])
             && preg_match('/^builtin:(.*)/', $data['message_array']['substyle'], $matches_style)) {
             $prefix = $matches_style[1] . '-';
         }

lib/org/openpsa/slideshow/handler/edit.php

Spacing +2 added lines, -2 removed lines

@@ -61,7 +61,7 @@ 
         $images = $qb->execute();
         $failed = 0;
         foreach ($images as $image) {
-            if (   !$image->generate_image('thumbnail', $this->_config->get('thumbnail_filter'))
+            if (!$image->generate_image('thumbnail', $this->_config->get('thumbnail_filter'))
                 || !$image->generate_image('image', $this->_config->get('image_filter'))) {
                 $failed++;
             }
@@ -241,7 +241,7 @@ 
         $attachment->title = $title;
         $attachment->mimetype = $file->getMimeType();
         $attachment->parentguid = $image->guid;
-        if (   !$attachment->create()
+        if (!$attachment->create()
             || !$attachment->copy_from_file($file->getRealPath())) {
             throw new midcom_error('Failed to create attachment: ' . midcom_connection::get_error_string());
         }

lib/org/openpsa/user/handler/person/account.php

Spacing +3 added lines, -3 removed lines

@@ -50,7 +50,7 @@ 
         $workflow = $this->get_workflow('datamanager', ['controller' => $data['controller']]);
         $response = $workflow->run($request);
 
-        if (   $workflow->get_state() == 'save'
+        if ($workflow->get_state() == 'save'
             && $this->create_account($this->person, $data["controller"]->get_form_values())) {
             midcom::get()->uimessages->add($this->_l10n->get('org.openpsa.user'), sprintf($this->_l10n->get('person %s created'), $this->person->name));
         }
@@ -83,7 +83,7 @@ 
         $person = new midcom_db_person($guid);
         $accounthelper = new org_openpsa_user_accounthelper($person);
         if ($accounthelper->welcome_email()) {
-            midcom::get()->uimessages->add($this->_l10n->get($this->_component), sprintf($this->_l10n->get('password reset and mail to %s sent'), $person->email ));
+            midcom::get()->uimessages->add($this->_l10n->get($this->_component), sprintf($this->_l10n->get('password reset and mail to %s sent'), $person->email));
         } else {
             midcom::get()->uimessages->add($this->_l10n->get($this->_component), $accounthelper->errstr, 'error'); 
         }
@@ -139,7 +139,7 @@ 
         $accounthelper = new org_openpsa_user_accounthelper($this->person);
 
         // Update account
-        if (   !$accounthelper->set_account($controller->get_form_values()["username"], $password)
+        if (!$accounthelper->set_account($controller->get_form_values()["username"], $password)
             && midcom_connection::get_error() != MGD_ERR_OK) {
             // Failure, give a message
             midcom::get()->uimessages->add($this->_l10n->get('org.openpsa.user'), $this->_l10n->get("failed to update the user account, reason") . ': ' . midcom_connection::get_error_string(), 'error');

lib/midcom/helper/nav/main.php

Spacing +3 added lines, -3 removed lines

@@ -340,7 +340,7 @@ 
         while ($parent = $object->get_parent()) {
             // Verify that this topic is within the current sites tree, if it is not,
             // we ignore it.
-            if (   $parent instanceof midcom_db_topic
+            if ($parent instanceof midcom_db_topic
                 && $this->is_node_in_tree($parent->id, $this->get_root_node())) {
                 return $parent;
             }
@@ -394,7 +394,7 @@ 
                     $entry = "<span class=\"{$current_class}\">{$entry}</span>";
                 }
             } else {
-                if (   !empty($data['napobject'][MIDCOM_NAV_GUID])
+                if (!empty($data['napobject'][MIDCOM_NAV_GUID])
                     && in_array($data['napobject'][MIDCOM_NAV_GUID], $skip_guids)) {
                     continue;
                 }
@@ -508,7 +508,7 @@ 
                     $id = "custom-{$key}";
 
                     $url = "{$prefix}{$entry[MIDCOM_NAV_URL]}";
-                    if (   str_starts_with($entry[MIDCOM_NAV_URL], '/')
+                    if (str_starts_with($entry[MIDCOM_NAV_URL], '/')
                         || preg_match('|^https?://|', $entry[MIDCOM_NAV_URL])) {
                         $url = $entry[MIDCOM_NAV_URL];
                     }

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

Spacing +3 added lines, -3 removed lines

@@ -33,7 +33,7 @@ 
     {
         $this->_object = midcom::get()->dbfactory->get_object_by_guid($guid);
 
-        if (   !($this->_object instanceof midcom_db_topic)
+        if (!($this->_object instanceof midcom_db_topic)
             && !($this->_object instanceof midcom_db_article)) {
             throw new midcom_error_notfound("Moving only topics and articles is supported.");
         }
@@ -74,7 +74,7 @@ 
             $folder = midcom_core_context::get()->get_key(MIDCOM_CONTEXT_ROOTTOPIC);
         }
 
-        if (   $this->_object instanceof midcom_db_topic
+        if ($this->_object instanceof midcom_db_topic
             && $folder->up == $this->_object->id) {
             $tree_disabled = true;
         }
@@ -87,7 +87,7 @@ 
             $selected = ' checked="checked"';
         }
 
-        if (   !($this->_object instanceof midcom_db_topic)
+        if (!($this->_object instanceof midcom_db_topic)
             && $folder->component !== $this->current_folder->component) {
             // Non-topic objects may only be moved under folders of same component
             $class = 'wrong_component';

lib/org/openpsa/products/viewer.php

Spacing +3 added lines, -3 removed lines

@@ -30,11 +30,11 @@
         $object = $dm->get_storage()->get_value();
 
         $document = $indexer->new_document($dm);
-        if (   $config->get('enable_scheduling')
+        if ($config->get('enable_scheduling')
             && $object instanceof org_openpsa_products_product_dba) {
             // Check start/end for products
-            if (   $object->start > time()
-                || (   $object->end != 0
+            if ($object->start > time()
+                || ($object->end != 0
                     && $object->end < time())) {
                 // Not in market, remove from index
                 $indexer->delete($document->RI);

lib/midgard/admin/asgard/plugin.php

Spacing +2 added lines, -2 removed lines

@@ -161,7 +161,7 @@ 
 
         $parent = $object->get_parent();
 
-        if (   $object instanceof midcom_db_parameter
+        if ($object instanceof midcom_db_parameter
             && !empty($parent->guid)) {
             // Add "parameters" list to breadcrumb if we're in a param
             $breadcrumb[] = [
@@ -171,7 +171,7 @@ 
         }
 
         $i = 0;
-        while (   !empty($parent->guid)
+        while (!empty($parent->guid)
                && $i < 10) {
             $i++;
             $parent_reflector = midcom_helper_reflector::get($parent);