flack / openpsa

lib/org/openpsa/directmarketing/importer.php

Braces +2 added lines, -1 removed lines

@@ -300,7 +300,8 @@
                     $organization = $this->_import_subscribers_organization($subscriber);
                     $this->_import_subscribers_organization_member($subscriber, $person, $organization);
                 }
-            } catch (midcom_error $e) {
+            }
+            catch (midcom_error $e) {
                 $e->log();
                 // Clean up possibly created data
                 $this->_clean_new_objects();

Spacing +3 added lines, -3 removed lines

@@ -133,7 +133,7 @@ 
             // User is or has been subscriber earlier, update status
             $member = $members[0];
 
-            if (   $member->orgOpenpsaObtype == org_openpsa_directmarketing_campaign_member_dba::UNSUBSCRIBED
+            if ($member->orgOpenpsaObtype == org_openpsa_directmarketing_campaign_member_dba::UNSUBSCRIBED
                 || $member->orgOpenpsaObtype == org_openpsa_directmarketing_campaign_member_dba::NORMAL) {
                 $this->_import_status['already_subscribed']++;
                 return;
@@ -173,7 +173,7 @@ 
 
             $qb = org_openpsa_contacts_group_dba::new_query_builder();
             $schema = $this->_schemadbs['organization']->get('default');
-            if (   $schema->has_field('company_id')
+            if ($schema->has_field('company_id')
                 && !empty($subscriber['organization']['company_id'])) {
                 // Imported data has a company id, we use that instead of name
                 $qb->add_constraint($schema->get_field('company_id')['storage']['location'], '=', $subscriber['organization']['company_id']);
@@ -181,7 +181,7 @@ 
                 // Seek by official name
                 $qb->add_constraint('official', '=', $subscriber['organization']['official']);
 
-                if (   $schema->has_field('city')
+                if ($schema->has_field('city')
                     && !empty($subscriber['organization']['city'])) {
                     // Imported data has a city, we use also that for matching
                     $qb->add_constraint($schema->get_field('city')['storage']['location'], '=', $subscriber['organization']['city']);

lib/org/openpsa/directmarketing/exec/cleanup.php

Spacing +1 added lines, -1 removed lines

@@ -34,7 +34,7 @@
     echo "            <th>{$type}</th>\n";
     echo "            <td style=\"text-align: right;\">" . number_format($count) . "</td>\n";
     if (isset($cleanups_kept[$type])) {
-        echo "            <td style=\"text-align: right;\">" . number_format($cleanups_kept[$type]) ."</td>\n";
+        echo "            <td style=\"text-align: right;\">" . number_format($cleanups_kept[$type]) . "</td>\n";
     }
     echo "        </tr>\n";
 }

src/midcom/workflow/dialog.php

Spacing +1 added lines, -1 removed lines

@@ -73,7 +73,7 @@
         $button_config[MIDCOM_TOOLBAR_URL] = $url;
         //The constants are numeric, so array_merge won't work...
         foreach ($options as $key => $value) {
-            if (   is_array($value)
+            if (is_array($value)
                 && !empty($button_config[$key])) {
                 $value = array_merge($button_config[$key], $value);
             }

lib/org/openpsa/mail/exec/test-html.php

Spacing +1 added lines, -1 removed lines

@@ -1,7 +1,7 @@
 <?php
 midcom::get()->auth->require_admin_user();
 
-if (   empty($_POST['to'])
+if (empty($_POST['to'])
     || empty($_POST['from'])) {
     ?>
     <h2>Send test email</h2>

web/rootfile.php

Spacing +1 added lines, -1 removed lines

@@ -23,7 +23,7 @@
     include dirname(__DIR__) . '/config-default.inc.php';
 }
 
-if (! defined('MIDCOM_STATIC_URL')) {
+if (!defined('MIDCOM_STATIC_URL')) {
     define('MIDCOM_STATIC_URL', '/midcom-static');
 }
 

lib/org/openpsa/invoices/handler/invoice/action.php

Braces +2 added lines, -1 removed lines

@@ -48,7 +48,8 @@
         try {
             $pdf_helper->render_and_attach();
             return $this->reply(true, $this->_l10n->get('pdf created'));
-        } catch (midcom_error $e) {
+        }
+        catch (midcom_error $e) {
             return $this->reply(false, $this->_l10n->get('pdf creation failed') . ': ' . $e->getMessage());
         }
     }

lib/midcom/core/querybuilder.php

Spacing +2 added lines, -2 removed lines

@@ -105,7 +105,7 @@ 
 
         $this->_add_visibility_checks();
 
-        if (   empty($this->_limit)
+        if (empty($this->_limit)
             && empty($this->_offset)) {
             // No point to do windowing
             $newresult = $this->_execute_and_check_privileges();
@@ -150,7 +150,7 @@ 
         }
         $this->_query->set_limit($window_size);
 
-        while (    ($resultset = $this->_execute_and_check_privileges())
+        while (($resultset = $this->_execute_and_check_privileges())
                 || $this->denied > $denied) {
             $size = count($resultset);
             $total_size = $size + ($this->denied - $denied);

lib/net/nemein/rss/manage.php

Braces +2 added lines, -1 removed lines

@@ -25,7 +25,8 @@
     {
         try {
             $viewer->register_plugin_namespace('__feeds', ['rss' => ['class' => __CLASS__]]);
-        } catch (midcom_error $e) {
+        }
+        catch (midcom_error $e) {
             $e->log();
         }
     }

lib/net/nehmer/static/viewer.php

Spacing +1 added lines, -1 removed lines

@@ -73,7 +73,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('config/', [
                 MIDCOM_TOOLBAR_LABEL => $this->_l10n_midcom->get('component configuration'),