flack / openpsa

lib/org/openpsa/products/viewer.php

Spacing +3 added lines, -3 removed lines

@@ -32,11 +32,11 @@
         $object = $dm->get_storage()->get_value();
 
         $document = $indexer->new_document($dm);
-        if (   $config->get('enable_scheduling')
+        if ($config->get('enable_scheduling')
             && is_a($object, org_openpsa_products_product_dba::class)) {
             // 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/org/openpsa/reports/viewer.php

Braces +2 added lines, -1 removed lines

@@ -103,7 +103,8 @@
             try {
                 midcom_db_topic::get_cached($node_guid);
                 $available_generators[$component] = midcom::get()->i18n->get_string($component, $component);
-            } catch (midcom_error $e) {
+            }
+            catch (midcom_error $e) {
                 debug_add("topic for component '{$component}' not found or accessible");
             }
         }

lib/net/nehmer/blog/handler.php

Spacing +3 added lines, -3 removed lines

@@ -15,7 +15,7 @@ 
 {
     public function get_url(midcom_db_article $article, $allow_external = false)
     {
-        if (   $allow_external
+        if ($allow_external
             && $this->_config->get('link_to_external_url')
             && !empty($article->url)) {
             return $article->url;
@@ -47,7 +47,7 @@ 
 
         $qb->add_constraint('topic.guid', 'IN', $topic_guids);
 
-        if (   count($topic_guids) > 1
+        if (count($topic_guids) > 1
             && $list_from_folders_categories = $this->_config->get('list_from_folders_categories')) {
             $list_from_folders_categories = explode(',', $list_from_folders_categories);
 
@@ -63,7 +63,7 @@ 
 
         // Hide the articles that have the publish time in the future and if
         // the user is not administrator
-        if (   $this->_config->get('enable_scheduled_publishing')
+        if ($this->_config->get('enable_scheduled_publishing')
             && !midcom::get()->auth->admin) {
             // Show the article only if the publishing time has passed or the viewer
             // is the author

lib/net/nehmer/comments/handler.php

Spacing +1 added lines, -1 removed lines

@@ -17,7 +17,7 @@
     {
         $toolbar = new midcom_helper_toolbar();
         $buttons = [];
-        if (   midcom::get()->auth->user
+        if (midcom::get()->auth->user
             && $comment->status < net_nehmer_comments_comment::MODERATED) {
             if (!$comment->can_do('net.nehmer.comments:moderation')) {
                 // Regular users can only report abuse

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

Braces +2 added lines, -1 removed lines

@@ -76,7 +76,8 @@
             } else {
                 try {
                     $this->initialize_index_article($topic);
-                } catch (midcom_error $e) {
+                }
+                catch (midcom_error $e) {
                     // Could not create index
                     $topic->delete();
                     throw $e;

lib/org/openpsa/sales/handler.php

Braces +4 added lines, -2 removed lines

@@ -61,7 +61,8 @@ 
                     //check if related at_entry exists
                     try {
                         $notification_entry = new midcom_services_at_entry_dba($entry);
-                    } catch (midcom_error $e) {
+                    }
+                    catch (midcom_error $e) {
                         //relatedto links to a non-existing at_entry - so create a new one an link to it
                         $notification_entry = new midcom_services_at_entry_dba();
                         $notification_entry->create();
@@ -83,7 +84,8 @@ 
                     $notification_entry = new midcom_services_at_entry_dba($mc_entry->get_subkey($key, 'fromGuid'));
                     //check if related at_entry exists & delete it
                     $notification_entry->delete();
-                } catch (midcom_error $e) {
+                }
+                catch (midcom_error $e) {
                     $e->log();
                 }
             }

lib/org/openpsa/contacts/handler.php

Spacing +2 added lines, -2 removed lines

@@ -34,7 +34,7 @@ 
         $prefix = $nap->get_node($this->_topic->id)[MIDCOM_NAV_ABSOLUTEURL];
 
         $tree = new org_openpsa_widgets_tree(org_openpsa_contacts_group_dba::class, 'owner');
-        $tree->link_callback = function ($guid) use ($prefix) {
+        $tree->link_callback = function($guid) use ($prefix) {
             return $prefix . 'group/' . $guid . '/';
         };
         $tree->constraints[] = ['orgOpenpsaObtype', '<', org_openpsa_contacts_group_dba::MYCONTACTS];
@@ -52,7 +52,7 @@ 
     {
         $my_company_guid = $this->_config->get('owner_organization');
 
-        if (   empty($my_company_guid)
+        if (empty($my_company_guid)
             || !mgd_is_guid($my_company_guid)) {
             if (midcom::get()->auth->admin) {
                 midcom::get()->uimessages->add(

src/midcom/datamanager/schemadb.php

Spacing +1 added lines, -1 removed lines

@@ -64,7 +64,7 @@
 
             if ($path === '') {
                 // Infinite loop, set an UI message and stop executing
-                if (   !isset($schema['extends']['name'])
+                if (!isset($schema['extends']['name'])
                     || $schema['extends']['name'] === $schema_name) {
                     throw new midcom_error('schema ' . $path . ':' . $schema_name . ' extends itself');
                 }

lib/org/openpsa/user/accounthelper.php

Braces +2 added lines, -2 removed lines

@@ -305,7 +305,7 @@ 
         $max = $this->_config->get('min_password_length');
 
         if ($password_length < $max) {
-            if ($show_ui_message){
+            if ($show_ui_message) {
                 midcom::get()->uimessages->add($this->_l10n->get('org.openpsa.user'), $this->_l10n->get('password too short'), 'error');
             }
             return false;
@@ -320,7 +320,7 @@ 
         }
 
         if ($score < $this->_config->get('min_password_score')) {
-            if ($show_ui_message){
+            if ($show_ui_message) {
                 midcom::get()->uimessages->add($this->_l10n->get('org.openpsa.user'), $this->_l10n->get('password weak'), 'error');
             }
             return false;

Spacing +7 added lines, -7 removed lines

@@ -170,7 +170,7 @@ 
         $account = $this->get_account();
         if (!empty($new_password)) {
             //check if the new encrypted password was already used
-            if (   !$this->check_password_reuse($new_password, true)
+            if (!$this->check_password_reuse($new_password, true)
                 || !$this->check_password_strength($new_password, true)) {
                 $this->errstr = "password strength too low";
                 return false;
@@ -329,7 +329,7 @@ 
         $max = $this->_config->get('min_password_length');
 
         if ($password_length < $max) {
-            if ($show_ui_message){
+            if ($show_ui_message) {
                 midcom::get()->uimessages->add($this->_l10n->get('org.openpsa.user'), $this->_l10n->get('password too short'), 'error');
             }
             return false;
@@ -344,7 +344,7 @@ 
         }
 
         if ($score < $this->_config->get('min_password_score')) {
-            if ($show_ui_message){
+            if ($show_ui_message) {
                 midcom::get()->uimessages->add($this->_l10n->get('org.openpsa.user'), $this->_l10n->get('password weak'), 'error');
             }
             return false;
@@ -365,7 +365,7 @@ 
         for ($i = 0; $i < strlen($password); $i++) {
             $repeated = true;
             for ($j = 0; $j < $plen && ($j + $i + $plen) < strlen($password); $j++) {
-                if (   $password[$j + $i] == $password[$j + $i + $plen]
+                if ($password[$j + $i] == $password[$j + $i + $plen]
                     && $repeated) {
                         $repeated = true;
                     } else {
@@ -503,7 +503,7 @@ 
 
     public static function get_person_by_formdata($data)
     {
-        if (   empty($data['username'])
+        if (empty($data['username'])
             || empty($data['password'])) {
             return false;
         }
@@ -535,7 +535,7 @@ 
         $max_attempts = midcom_baseclasses_components_configuration::get($component, 'config')->get('max_password_attempts');
         $timeframe = midcom_baseclasses_components_configuration::get($component, 'config')->get('password_block_timeframe_min');
 
-        if (   $max_attempts == 0
+        if ($max_attempts == 0
             || $timeframe == 0) {
             return $stat;
         }
@@ -557,7 +557,7 @@ 
          * If the maximum number of attempts is reached and the oldest attempt
          * on the stack is within our defined timeframe, we block the account
          */
-        if (   count($attempts) >= $max_attempts
+        if (count($attempts) >= $max_attempts
             && $attempts[$max_attempts - 1] >= (time() - ($timeframe * 60))) {
             $this->disable_account();
             $stat = false;