flack / openpsa

lib/org/openpsa/expenses/handler/index.php

Spacing +1 added lines, -1 removed lines

@@ -125,7 +125,7 @@
             }
 
             $date_identifier = date('Y-m-d', $row['date']);
-            $row_identifier = $task->id . '-' .  $row['person'];
+            $row_identifier = $task->id . '-' . $row['person'];
 
             if (!isset($reports[$row_identifier])) {
                 try {

Braces +6 added lines, -3 removed lines

@@ -32,7 +32,8 @@ 
         }
         try {
             $date = new DateTime($requested_time);
-        } catch (Exception $e) {
+        }
+        catch (Exception $e) {
             return false;
         }
         $offset = $date->format('N') - 1;
@@ -119,7 +120,8 @@ 
         foreach ($hours as $guid => $row) {
             try {
                 $task = org_openpsa_projects_task_dba::get_cached($row['task']);
-            } catch (midcom_error $e) {
+            }
+            catch (midcom_error $e) {
                 // Task couldn't be loaded, probably because of ACL
                 continue;
             }
@@ -132,7 +134,8 @@ 
                     $person_object = org_openpsa_contacts_person_dba::get_cached($row['person']);
                     $person_label = $this->_get_list_link($person_object->name, null, null, $row['person']);
                     $person_name = $person_object->name;
-                } catch (midcom_error $e) {
+                }
+                catch (midcom_error $e) {
                     $person_label = $this->_l10n->get('no person');
                     $person_name = '';
                 }

lib/org/openpsa/mail/backend/bouncer.php

Braces +2 added lines, -1 removed lines

@@ -23,7 +23,8 @@
         foreach ($this->_try_backends as $backend) {
             try {
                 $this->_backend = org_openpsa_mail_backend::get($backend, $params);
-            } catch (midcom_error $e) {
+            }
+            catch (midcom_error $e) {
                 debug_add('Failed to load backend ' . $backend . ', message:' . $e->getMessage());
             }
         }

lib/org/openpsa/mail/backend.php

Braces +4 added lines, -2 removed lines

@@ -42,7 +42,8 @@ 
                     $object = self::_load_backend($backend, $params);
                     debug_add('Using backend ' . $backend);
                     return $object;
-                } catch (midcom_error $e) {
+                }
+                catch (midcom_error $e) {
                     debug_add('Failed to load backend ' . $backend . ', message:' . $e->getMessage());
                 }
             }
@@ -67,7 +68,8 @@ 
             $ret = $this->mail($message);
             $this->error = false;
             return $ret;
-        } catch (Exception $e) {
+        }
+        catch (Exception $e) {
             $this->error = $e->getMessage();
             return false;
         }

Spacing +1 added lines, -1 removed lines

@@ -79,7 +79,7 @@
             return false;
         }
 
-        if (   is_string($this->error)
+        if (is_string($this->error)
             && !empty($this->error)) {
             return $this->error;
         }

lib/org/openpsa/mail/main.php

Spacing +1 added lines, -1 removed lines

@@ -191,7 +191,7 @@
         $this->html_body = preg_replace("/\n\r|\r\n|\r/", "\n", $this->html_body);
 
         //Try to translate HTML-only body to plaintext as well
-        if (   strlen($this->body) == 0
+        if (strlen($this->body) == 0
             && strlen($this->html_body) > 0
             && !$this->allow_only_html) {
             $this->body = $this->html2text($this->html_body);

lib/org/openpsa/mail/message.php

Spacing +1 added lines, -1 removed lines

@@ -102,7 +102,7 @@
             if (is_string($value)) {
                 $this->_headers[$header] = trim($value);
             }
-            if (   strtolower($header) == 'from'
+            if (strtolower($header) == 'from'
                 || strtolower($header) == 'reply-to'
                 || strtolower($header) == 'to') {
                 $this->_headers[$header] = $this->_encode_address_field($value);

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

Indentation +3 added lines, -3 removed lines

@@ -252,9 +252,9 @@
                 break;
             case 'delete':
                  if (empty($_POST['guids'])) {
-                     throw new midcom_error('Invalid request');
-                 }
-                 break;
+                        throw new midcom_error('Invalid request');
+                    }
+                    break;
             default:
                 throw new midcom_error('Invalid request');
         }

Spacing +3 added lines, -3 removed lines

@@ -63,7 +63,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++;
             }
@@ -244,7 +244,7 @@ 
                 }
                 //Fall-through
             case 'create':
-                if (   !isset($_POST['title'])
+                if (!isset($_POST['title'])
                     || !isset($_POST['description'])
                     || !isset($_POST['position'])) {
                     throw new midcom_error('Invalid request');
@@ -267,7 +267,7 @@ 
         $attachment->title = $_POST['title'];
         $attachment->mimetype = $file['type'];
         $attachment->parentguid = $image->guid;
-        if (   !$attachment->create()
+        if (!$attachment->create()
             || !$attachment->copy_from_file($file['tmp_name'])) {
             throw new midcom_error('Failed to create attachment: ' . midcom_connection::get_error_string());
         }

Braces +2 added lines, -1 removed lines

@@ -148,7 +148,8 @@
         try {
             $this->$function();
             $this->_response->success = true;
-        } catch (midcom_error $e) {
+        }
+        catch (midcom_error $e) {
             $this->_response->success = false;
             $this->_response->error = $e->getMessage();
         }

lib/org/openpsa/slideshow/image.php

Spacing +1 added lines, -1 removed lines

@@ -150,7 +150,7 @@
         $rows = $mc->get_rows(array('id', 'name', 'guid'), 'id');
 
         foreach ($images as $image) {
-            if (   !isset($rows[$image->attachment])
+            if (!isset($rows[$image->attachment])
                 || !isset($rows[$image->image])
                 || !isset($rows[$image->thumbnail])) {
                 continue;

Braces +4 added lines, -2 removed lines

@@ -41,7 +41,8 @@ 
 
         try {
             $folder = midcom_db_topic::get_cached($this->topic);
-        } catch (midcom_error $e) {
+        }
+        catch (midcom_error $e) {
             $e->log();
         }
         $folder->delete_attachment(self::FOLDER_THUMBNAIL);
@@ -51,7 +52,8 @@ 
     {
         try {
             return new midcom_db_attachment($this->$type);
-        } catch (midcom_error $e) {
+        }
+        catch (midcom_error $e) {
             $e->log();
             return false;
         }

lib/org/openpsa/core/acl/synchronizer.php

Spacing +3 added lines, -3 removed lines

@@ -15,7 +15,7 @@ 
 {
     public function write_acls(midcom_core_dbaobject $object, $owner_id, $accesstype)
     {
-        if (   empty($owner_id)
+        if (empty($owner_id)
             || empty($accesstype)) {
             return false;
         }
@@ -63,14 +63,14 @@ 
 
         if ($privileges) {
             foreach ($privileges as $privilege) {
-                if (   !empty($needed_privileges[$privilege->privilegename])
+                if (!empty($needed_privileges[$privilege->privilegename])
                     && $needed_privileges[$privilege->privilegename]['assignee'] == $privilege->assignee
                     && $needed_privileges[$privilege->privilegename]['value'] == $privilege->value) {
                     unset($needed_privileges[$privilege->privilegename]);
                     continue;
                 }
                 // Clear old ACLs applying to others than current user or selected owner group
-                if (   $privilege->assignee != midcom::get()->auth->user->id
+                if ($privilege->assignee != midcom::get()->auth->user->id
                     && $privilege->assignee != $owner_id) {
                     if (is_array($privilege->assignee)) {
                         $assignee_key = $privilege->assignee['identifier'];

lib/org/openpsa/core/filter/select.php

Spacing +1 added lines, -1 removed lines

@@ -77,7 +77,7 @@
             echo '<select class="filter_input" onchange="document.forms[\'' . $this->name . '_filter\'].submit();" name="' . $this->name . '">';
 
             foreach ($options as $option) {
-                echo '<option value="' .  $option['id'] . '"';
+                echo '<option value="' . $option['id'] . '"';
                 if ($option['selected'] == true) {
                     echo " selected=\"selected\"";
                 }