flack / openpsa

src/midcom/datamanager/storage/blobs.php

Braces +2 added lines, -1 removed lines

@@ -37,7 +37,8 @@
         foreach ($items as $identifier => $guid) {
             try {
                 $results[$identifier] = new midcom_db_attachment($guid);
-            } catch (midcom_error $e) {
+            }
+            catch (midcom_error $e) {
                 $e->log();
             }
         }

Spacing +1 added lines, -1 removed lines

@@ -143,7 +143,7 @@
     protected function save_attachment_list()
     {
         if (!empty($this->config['widget_config']['sortable'])) {
-            uasort($this->map, function ($a, $b) {
+            uasort($this->map, function($a, $b) {
                 if ($a->metadata->score == $b->metadata->score) {
                     return strnatcasecmp($a->name, $b->name);
                 }

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

Indentation +2 added lines, -2 removed lines

@@ -65,8 +65,8 @@
     private function prepare_field(array $config)
     {
         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...
+                // 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') {
             if (class_exists('midcom\datamanager\storage\\' . $config['type'])) {
                 $classname = 'midcom\datamanager\storage\\' . $config['type'];

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/net/nehmer/blog/handler/view.php

Braces +2 added lines, -1 removed lines

@@ -218,7 +218,8 @@
         if ($this->_config->get('comments_topic')) {
             try {
                 $comments_topic = new midcom_db_topic($this->_config->get('comments_topic'));
-            } catch (midcom_error $e) {
+            }
+            catch (midcom_error $e) {
                 return false;
             }
 

Spacing +2 added lines, -2 removed lines

@@ -44,7 +44,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("delete/{$this->_article->guid}/");
@@ -96,7 +96,7 @@ 
         if ($this->_config->get('comments_enable')) {
             if ($comments_node = $this->_seek_comments()) {
                 $this->_request_data['comments_url'] = $comments_node[MIDCOM_NAV_RELATIVEURL] . "comment/{$this->_article->guid}";
-                if (   $this->_topic->can_do('midgard:update')
+                if ($this->_topic->can_do('midgard:update')
                     && $this->_topic->can_do('net.nehmer.comments:moderation')) {
                     net_nehmer_comments_viewer::add_head_elements();
                 }

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/billing/data.php

Braces +6 added lines, -3 removed lines

@@ -77,7 +77,8 @@ 
             // if the property useContactAddress is set
             $billing_data[0]->set_address();
             return $billing_data[0];
-        } catch (midcom_error $e) {
+        }
+        catch (midcom_error $e) {
             $e->log();
             return false;
         }
@@ -122,10 +123,12 @@ 
     {
         try {
             return new org_openpsa_contacts_person_dba($this->linkGuid);
-        } catch (midcom_error $e) {
+        }
+        catch (midcom_error $e) {
             try {
                 return new org_openpsa_contacts_group_dba($this->linkGuid);
-            } catch (midcom_error $e) {
+            }
+            catch (midcom_error $e) {
                 debug_add("Failed to load contact with GUID: " . $this->linkGuid . " - last error:" . $e->getMessage(), MIDCOM_LOG_ERROR);
                 return false;
             }

Indentation +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
     public static function get_by_object(org_openpsa_invoices_interfaces_customer $object)
     {
         if (   !($bd = self::get_billing_data(org_openpsa_contacts_group_dba::class, $object->customer))
-               // check if the customerContact is set and has invoice_data
+                // check if the customerContact is set and has invoice_data
             && !($bd = self::get_billing_data(org_openpsa_contacts_person_dba::class, $object->customerContact))) {
             $bd = new org_openpsa_invoices_billing_data_dba();
             $due = midcom_baseclasses_components_configuration::get('org.openpsa.invoices', 'config')->get('default_due_days');

Spacing +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
      */
     public static function get_by_object(org_openpsa_invoices_interfaces_customer $object)
     {
-        if (   !($bd = self::get_billing_data(org_openpsa_contacts_group_dba::class, $object->customer))
+        if (!($bd = self::get_billing_data(org_openpsa_contacts_group_dba::class, $object->customer))
                // check if the customerContact is set and has invoice_data
             && !($bd = self::get_billing_data(org_openpsa_contacts_person_dba::class, $object->customerContact))) {
             $bd = new org_openpsa_invoices_billing_data_dba();

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/org/openpsa/calendar/event/member.php

Braces +2 added lines, -1 removed lines

@@ -152,7 +152,8 @@
                 debug_add('person #' . $person->id . 'has no email address, aborting');
                 return false;
             }
-        } catch (midcom_error $e) {
+        }
+        catch (midcom_error $e) {
             return false;
         }
 

Spacing +6 added lines, -6 removed lines

@@ -69,7 +69,7 @@ 
             $event = new org_openpsa_calendar_event_dba($this->eid);
         }
 
-        if (    $recipient->id == midcom_connection::get_user()
+        if ($recipient->id == midcom_connection::get_user()
              && !$event->send_notify_me) {
             //Do not send notification to current user
             debug_add('event->send_notify_me is false and recipient is current user, aborting notify');
@@ -114,7 +114,7 @@ 
                 return false;
         }
 
-        if (   $type == 'cancel'
+        if ($type == 'cancel'
             || $type == 'remove') {
             // TODO: Create iCal export with correct delete commands
         } else {
@@ -206,12 +206,12 @@ 
             $workday_starts = 8;
             $workday_ends = 16;
 
-            $workday_starts_ts = mktime($workday_starts, 0, 0, (int)$ymd_matches[2], (int)$ymd_matches[3], (int)$ymd_matches[1]);
-            $workday_ends_ts = mktime($workday_ends, 0, 0, (int)$ymd_matches[2], (int)$ymd_matches[3], (int)$ymd_matches[1]);
+            $workday_starts_ts = mktime($workday_starts, 0, 0, (int) $ymd_matches[2], (int) $ymd_matches[3], (int) $ymd_matches[1]);
+            $workday_ends_ts = mktime($workday_ends, 0, 0, (int) $ymd_matches[2], (int) $ymd_matches[3], (int) $ymd_matches[1]);
             $last_end_time = false;
             $last_event = false;
             foreach ($events as $event) {
-                if (   $event->end <= $workday_starts_ts
+                if ($event->end <= $workday_starts_ts
                     || $event->start >= $workday_ends_ts) {
                     // We need not to consider this event, it is outside the defined workday
                     continue;
@@ -241,7 +241,7 @@ 
             if ($last_end_time === false) {
                 $last_end_time = $workday_starts_ts;
             }
-            if (   $last_end_time < $workday_ends_ts
+            if ($last_end_time < $workday_ends_ts
                 && (($workday_ends_ts - $last_end_time) >= $amount)) {
                 $slots[] = self::_create_slot($last_end_time, $workday_ends_ts, $last_event);
             }

lib/net/nehmer/blog/handler/index.php

Spacing +2 added lines, -2 removed lines

@@ -62,7 +62,7 @@ 
         $data['page_title'] = $this->_topic->extra;
 
         // Filter by categories
-        if (   $handler_id == 'index-category'
+        if ($handler_id == 'index-category'
             || $handler_id == 'latest-category') {
             $data['category'] = trim(strip_tags($args[0]));
 
@@ -111,7 +111,7 @@ 
         midcom::get()->head->set_pagetitle($this->_request_data['page_title']);
 
         // Activate correct leaf
-        if (   $this->_config->get('show_navigation_pseudo_leaves')
+        if ($this->_config->get('show_navigation_pseudo_leaves')
             && in_array($this->_request_data['category'], $this->_request_data['categories'])) {
             $this->set_active_leaf($this->_topic->id . '_CAT_' . $this->_request_data['category']);
         }

Braces +4 added lines, -2 removed lines

@@ -141,7 +141,8 @@ 
             foreach ($this->_articles as $article_counter => $article) {
                 try {
                     $data['datamanager']->set_storage($article);
-                } catch (midcom_error $e) {
+                }
+                catch (midcom_error $e) {
                     $e->log();
                     continue;
                 }
@@ -178,7 +179,8 @@ 
         if ($this->_config->get('comments_topic')) {
             try {
                 $comments_topic = new midcom_db_topic($this->_config->get('comments_topic'));
-            } catch (midcom_error $e) {
+            }
+            catch (midcom_error $e) {
                 return false;
             }