flack / openpsa

lib/org/openpsa/directmarketing/support/bounce_detector.php

Spacing +2 added lines, -2 removed lines

@@ -45,7 +45,7 @@ 
 }
 $recipient = $_ENV["RECIPIENT"];
 $token_length = strlen($recipient) - strlen($prefix) - strlen($suffix);
-if (   $token_length <= 0
+if ($token_length <= 0
     || substr($recipient, 0, strlen($prefix)) != $prefix
     || substr($recipient, strlen($prefix) + $token_length) != $suffix) {
     error_log("Recipient address does not match address template: $recipient");
@@ -61,7 +61,7 @@ 
         fclose($fh);
         exit(0);
     }
-} elseif (   substr($BOUNCE_LOGGER, 0, strlen("http://")) == "http://"
+} elseif (substr($BOUNCE_LOGGER, 0, strlen("http://")) == "http://"
           || substr($BOUNCE_LOGGER, 0, strlen("https://")) == "https://") {
     $client = new org_openpsa_httplib();
     $client->post($BOUNCE_LOGGER, ["token" => $token]);

lib/org/openpsa/documents/document.php

Spacing +4 added lines, -4 removed lines

@@ -74,7 +74,7 @@ 
             return;
         }
         $parent = $this->get_parent();
-        if (   $parent
+        if ($parent
             && $parent->component == 'org.openpsa.documents') {
             midcom::get()->auth->request_sudo('org.openpsa.documents');
 
@@ -170,7 +170,7 @@ 
          * if nothing matched so far and the subtype is alphanumeric, uppercase it on the theory
          * that it's probably a file extension
          */
-        if (   $st_orig == $subtype
+        if ($st_orig == $subtype
             && preg_match('/^[a-z0-9]+$/', $subtype)) {
             $subtype = strtoupper($subtype);
         }
@@ -185,7 +185,7 @@ 
         $properties = $this->get_properties();
         // Copy current properties
         foreach ($properties as $key) {
-            if (   $key != 'guid'
+            if ($key != 'guid'
                 && $key != 'id'
                 && $key != 'metadata') {
                 $backup->$key = $this->{$key};
@@ -217,7 +217,7 @@ 
             $backup_attachment = $backup->create_attachment($original_attachment->name, $original_attachment->title, $original_attachment->mimetype);
 
             $original_handle = $original_attachment->open('r');
-            if (   !$backup_attachment
+            if (!$backup_attachment
                 || !$original_handle) {
                 // Failed to copy the attachment, abort
                 return $backup->delete();

src/midcom/workflow/datamanager.php

Spacing +2 added lines, -2 removed lines

@@ -93,8 +93,8 @@
 
     private function prepare_url($url)
     {
-        if (   substr($url, 0, 1) != '/'
-            && ! preg_match('|^https?://|', $url)) {
+        if (substr($url, 0, 1) != '/'
+            && !preg_match('|^https?://|', $url)) {
             $url = midcom_core_context::get()->get_key(MIDCOM_CONTEXT_ANCHORPREFIX) . $url;
         }
         return '"' . $url . '"';

src/midcom/datamanager/storage/container/dbacontainer.php

Spacing +2 added lines, -2 removed lines

@@ -32,7 +32,7 @@ 
             }
             $config['name'] = $name;
             $field = $this->prepare_field($config);
-            if (   isset($config['default'])
+            if (isset($config['default'])
                 && (!$this->object->id || $field instanceof transientnode)) {
                 $field->set_value($config['default']);
             }
@@ -64,7 +64,7 @@ 
      */
     private function prepare_field(array $config)
     {
-        if (   empty($config['storage']['location'])
+        if (empty($config['storage']['location'])
                // This line is needed because a parameter default is set by the schema parser and then ignored
                // by the type. The things we do for backwards compatibility...
             || $config['storage']['location'] === 'parameter') {

lib/org/openpsa/invoices/handler/billingdata.php

Spacing +3 added lines, -3 removed lines

@@ -30,8 +30,8 @@ 
     private function load_controller(org_openpsa_invoices_billing_data_dba $bd)
     {
         $schemadb = schemadb::from_path($this->_config->get('schemadb_billing_data'));
-        $vat =& $schemadb->get('default')->get_field('vat');
-        $due =& $schemadb->get('default')->get_field('due');
+        $vat = & $schemadb->get('default')->get_field('vat');
+        $due = & $schemadb->get('default')->get_field('due');
         // Fill VAT select
         $vat_array = explode(',', $this->_config->get('vat_percentages'));
         if (!empty($vat_array)) {
@@ -74,7 +74,7 @@ 
         $data['controller'] = $this->load_controller($bd);
 
         $workflow = $this->get_workflow('datamanager', ['controller' => $data['controller']]);
-        if (   $mode == 'edit'
+        if ($mode == 'edit'
             && $bd->can_do('midgard:delete')) {
             $delete = $this->get_workflow('delete', [
                 'object' => $bd,

lib/net/nemein/rss/handler/admin.php

Spacing +1 added lines, -1 removed lines

@@ -82,7 +82,7 @@
         }
 
         // OPML subscription list import support
-        if (   array_key_exists('net_nemein_rss_manage_opml', $_FILES)
+        if (array_key_exists('net_nemein_rss_manage_opml', $_FILES)
             && is_uploaded_file($_FILES['net_nemein_rss_manage_opml']['tmp_name'])) {
             $opml_file = $_FILES['net_nemein_rss_manage_opml']['tmp_name'];
 

src/midcom/datamanager/storage/parameter.php

Spacing +1 added lines, -1 removed lines

@@ -25,7 +25,7 @@
     {
         // workaround for weird mgd API behavior where setting empty (i.e. deleting) a
         // nonexistent parameter returns false
-        if (   ($this->value === false || $this->value === null || $this->value === "")
+        if (($this->value === false || $this->value === null || $this->value === "")
             && $this->load() === null) {
             return true;
         }

lib/org/openpsa/contacts/handler/group/create.php

Spacing +1 added lines, -1 removed lines

@@ -72,7 +72,7 @@
 
         $this->_group = new org_openpsa_contacts_group_dba();
 
-        if (   $this->_type == 'organization'
+        if ($this->_type == 'organization'
             && $this->_parent_group) {
             $this->_group->owner = $this->_parent_group->id;
         } else {

lib/org/openpsa/documents/handler/directory/create.php

Spacing +1 added lines, -1 removed lines

@@ -50,6 +50,6 @@
         $indexer = new org_openpsa_documents_midcom_indexer($this->_topic);
         $indexer->index($controller->get_datamanager());
 
-        return $controller->get_datamanager()->get_storage()->get_value()->name. "/";
+        return $controller->get_datamanager()->get_storage()->get_value()->name . "/";
     }
 }