flack / openpsa

lib/org/openpsa/sales/salesproject.php

Spacing +3 added lines, -3 removed lines

@@ -90,7 +90,7 @@ 
             }
         }
         $profit = $value - $cost;
-        if (   $this->value != $value
+        if ($this->value != $value
             || $this->profit != $profit) {
             $this->value = $value;
             $this->profit = $value - $cost;
@@ -220,12 +220,12 @@ 
 
     public function _on_updating()
     {
-        if (   $this->state != self::STATE_ACTIVE
+        if ($this->state != self::STATE_ACTIVE
             && !$this->end) {
             //Not active anymore and end not set, set it to now
             $this->end = time();
         }
-        if (   $this->end
+        if ($this->end
             && $this->state == self::STATE_ACTIVE) {
             //Returned to active state, clear the end marker.
             $this->end = 0;

lib/midcom/db/person.php

Spacing +1 added lines, -1 removed lines

@@ -78,7 +78,7 @@
     {
         parent::__set($property, $value);
 
-        if (   $property == 'firstname'
+        if ($property == 'firstname'
             || $property == 'lastname'
             || $property == 'homepage'
             || $property == 'email') {

lib/midcom/helper/toolbar.php

Spacing +9 added lines, -9 removed lines

@@ -364,10 +364,10 @@ 
             }
         }
 
-        if (   !empty($item[MIDCOM_TOOLBAR_ACCESSKEY])
+        if (!empty($item[MIDCOM_TOOLBAR_ACCESSKEY])
             && !array_key_exists($item[MIDCOM_TOOLBAR_ACCESSKEY], $used_access_keys)) {
             // We have valid access key, add it to help text
-            if (   isset($_SERVER['HTTP_USER_AGENT'])
+            if (isset($_SERVER['HTTP_USER_AGENT'])
                 && strstr($_SERVER['HTTP_USER_AGENT'], 'Macintosh')) {
                 // Mac users
                 $hotkey = 'Ctrl-' . strtoupper($item[MIDCOM_TOOLBAR_ACCESSKEY]);
@@ -390,7 +390,7 @@ 
     private function set_url(array &$item, $url)
     {
         $item[MIDCOM_TOOLBAR__ORIGINAL_URL] = $url;
-        if (   (   empty($item[MIDCOM_TOOLBAR_OPTIONS]["rel"])
+        if ((empty($item[MIDCOM_TOOLBAR_OPTIONS]["rel"])
                 // Some items may want to keep their links unmutilated
                 || $item[MIDCOM_TOOLBAR_OPTIONS]["rel"] != "directlink")
             && substr($url, 0, 1) != '/'
@@ -415,7 +415,7 @@ 
 
         if ($index == 0) {
             array_shift($this->items);
-        } elseif ($index == count($this->items) -1) {
+        } elseif ($index == count($this->items) - 1) {
             array_pop($this->items);
         } else {
             $this->items = array_merge(array_slice($this->items, 0, $index - 1),
@@ -545,7 +545,7 @@ 
      */
     public function render()
     {
-        $visible_items = array_filter($this->items, function ($item) {
+        $visible_items = array_filter($this->items, function($item) {
             return !$item[MIDCOM_TOOLBAR_HIDDEN];
         });
 
@@ -570,7 +570,7 @@ 
         foreach ($visible_items as $i => $item) {
             $output .= '<li class="';
             if ($i == 0) {
-                $output .= $first_class .  ' ';
+                $output .= $first_class . ' ';
             } elseif ($i == $last) {
                 $output .= 'last_item ';
             }
@@ -610,7 +610,7 @@ 
             // Try finding uppercase version of the accesskey first
             $accesskey = strtoupper($item[MIDCOM_TOOLBAR_ACCESSKEY]);
             $position = strpos($label, $accesskey);
-            if (   $position === false
+            if ($position === false
                 && midcom::get()->i18n->get_current_language() == 'en') {
                 // Try lowercase, too
                 $accesskey = strtolower($accesskey);
@@ -671,7 +671,7 @@ 
             $attributes['title'] = $item[MIDCOM_TOOLBAR_HELPTEXT];
         }
 
-        if (   $item[MIDCOM_TOOLBAR_ENABLED]
+        if ($item[MIDCOM_TOOLBAR_ENABLED]
             && !is_null($item[MIDCOM_TOOLBAR_ACCESSKEY])) {
             $attributes['class'] = 'accesskey';
             $attributes['accesskey'] = $item[MIDCOM_TOOLBAR_ACCESSKEY];
@@ -737,7 +737,7 @@ 
     public function get_index_from_url($url)
     {
         foreach ($this->items as $i => $item) {
-            if (   $item[MIDCOM_TOOLBAR_URL] == $url
+            if ($item[MIDCOM_TOOLBAR_URL] == $url
                 || $item[MIDCOM_TOOLBAR__ORIGINAL_URL] == $url) {
                 return $i;
             }

lib/midcom/helper/imagefilter.php

Spacing +5 added lines, -5 removed lines

@@ -193,7 +193,7 @@ 
             debug_add("The File {$filename} is not writeable.", MIDCOM_LOG_ERROR);
             return false;
         }
-        if (   !empty($this->_filename)
+        if (!empty($this->_filename)
             && $this->_filename !== $filename) {
             $this->_tmpfiles[] = $this->_filename;
         }
@@ -418,7 +418,7 @@ 
         $tmpfile = null;
         $imagesize = getimagesize($this->_filename);
 
-        if (   $imagesize[2] == IMAGETYPE_JPEG
+        if ($imagesize[2] == IMAGETYPE_JPEG
             && $this->_jpegtran_available()) {
             /* jpegtran */
             $operations = array(
@@ -477,7 +477,7 @@ 
         while ($rotate > 360) {
             $rotate -= 360;
         }
-        if (   $rotate == 0
+        if ($rotate == 0
             || $rotate == 360) {
             debug_add("Rotate is {$rotate}, we're happy as-is.");
             return;
@@ -487,7 +487,7 @@ 
         $imagesize = getimagesize($this->_filename);
 
         // Try lossless jpegtran rotation if possible
-        if (   $imagesize[2] == IMAGETYPE_JPEG
+        if ($imagesize[2] == IMAGETYPE_JPEG
             && ($rotate % 90 == 0)
             && $this->_jpegtran_available()) {
             $tmpfile = $this->_get_tempfile();
@@ -616,7 +616,7 @@ 
      */
     public function fill($x = null, $y = null, $color = null, $gravity = 'center')
     {
-        if (   empty($x)
+        if (empty($x)
             || empty($y)
             || empty($color)) {
             //This is a bit silly, but here for backwards compatibility...

lib/midcom/helper/xsspreventer.php

Spacing +1 added lines, -1 removed lines

@@ -37,7 +37,7 @@
     {
         return preg_replace_callback(
             "%(<\s*)+(/\s*)+{$element}%i",
-            function ($matches) {
+            function($matches) {
                 return htmlentities($matches[0]);
             },
             $input

lib/midcom/helper/nav/main.php

Spacing +3 added lines, -3 removed lines

@@ -222,7 +222,7 @@ 
         if ($uplink == $root_id) {
             return true;
         }
-        if (   $uplink == false
+        if ($uplink == false
             || $uplink == -1) {
             return false;
         }
@@ -446,7 +446,7 @@ 
                     $entry = "<span class=\"{$current_class}\">{$entry}</span>";
                 }
             } else {
-                if (   !empty($data['napobject'][MIDCOM_NAV_GUID])
+                if (!empty($data['napobject'][MIDCOM_NAV_GUID])
                     && in_array($data['napobject'][MIDCOM_NAV_GUID], $skip_guids)) {
                     continue;
                 }
@@ -569,7 +569,7 @@ 
                     $id = "custom-{$key}";
 
                     $url = "{$prefix}{$entry[MIDCOM_NAV_URL]}";
-                    if (   substr($entry[MIDCOM_NAV_URL], 0, 1) == '/'
+                    if (substr($entry[MIDCOM_NAV_URL], 0, 1) == '/'
                         || preg_match('|^https?://|', $entry[MIDCOM_NAV_URL])) {
                         $url = $entry[MIDCOM_NAV_URL];
                     }

lib/midcom/helper/formatter.php

Spacing +1 added lines, -1 removed lines

@@ -84,7 +84,7 @@
             $variable = $parts[0] . '->' . $parts[1];
         }
 
-        if (    isset($variable_parts[1])
+        if (isset($variable_parts[1])
              && array_key_exists($variable_parts[1], self::$_filters)) {
             switch ($variable_parts[1]) {
                 case 's':

lib/midcom/helper/datamanager2/controller.php

Spacing +3 added lines, -3 removed lines

@@ -120,7 +120,7 @@ 
             }
         }
 
-        $this->schemadb =& $schemadb;
+        $this->schemadb = & $schemadb;
     }
 
     /**
@@ -143,7 +143,7 @@ 
 
         if ($storage instanceof midcom_helper_datamanager2_datamanager) {
             $this->datamanager = $storage;
-        } elseif (   $storage instanceof midcom_helper_datamanager2_storage
+        } elseif ($storage instanceof midcom_helper_datamanager2_storage
                  || midcom::get()->dbclassloader->is_midcom_db_object($storage)) {
             $this->datamanager = new midcom_helper_datamanager2_datamanager($this->schemadb);
             if ($schema === null) {
@@ -203,7 +203,7 @@ 
     function display_form()
     {
         // Prevent temporary objects from failing
-        if (   $this->lock_object
+        if ($this->lock_object
             && !empty($this->datamanager->storage->object->guid)) {
             // Get the metadata object
             $metadata = $this->datamanager->storage->object->metadata;

lib/midcom/helper/datamanager2/controller/simple.php

Spacing +6 added lines, -6 removed lines

@@ -80,7 +80,7 @@ 
 
         $metadata = $this->datamanager->storage->object->metadata;
 
-        if (    $metadata
+        if ($metadata
              && array_key_exists('midcom_helper_datamanager2_unlock', $_REQUEST)
              && isset($_REQUEST['midcom_helper_datamanager2_object'])) {
             // Remove the lock, if permission is granted
@@ -102,13 +102,13 @@ 
 
         if ($metadata) {
             // Remove the lock
-            if (   $this->lock_timeout
-                && (   $result === 'save'
+            if ($this->lock_timeout
+                && ($result === 'save'
                     || $result === 'cancel')) {
                 $metadata->unlock();
             }
             // or set it, if needed
-            elseif (   $this->lock_object
+            elseif ($this->lock_object
                     && !$metadata->is_locked()
                     && $this->lock_timeout) {
                 $metadata->lock();
@@ -116,7 +116,7 @@ 
         }
 
         // Handle successful save explicitly.
-        if (   $result == 'save'
+        if ($result == 'save'
             || $result == 'next') {
             // Ok, we can save now. At this point we already have a content object.
             $result = $this->_save_form($result);
@@ -153,7 +153,7 @@ 
             return 'edit';
         }
 
-        if (   $result == 'save'
+        if ($result == 'save'
             && !$this->datamanager->save()) {
             if (count($this->datamanager->validation_errors) == 0) {
                 // It seems to be a critical error.