flack / openpsa

lib/org/openpsa/invoices/invoice/pdf.php

Spacing +2 added lines, -2 removed lines

@@ -87,7 +87,7 @@ 
         $filename = $generator->from_string($this->invoice->get_label()) . '.pdf';
 
         // tmp filename
-        $tmp_file = midcom::get()->config->get('midcom_tempdir') . "/". $filename;
+        $tmp_file = midcom::get()->config->get('midcom_tempdir') . "/" . $filename;
 
         // render pdf to tmp filename
         $pdf_builder->render($tmp_file);
@@ -100,7 +100,7 @@ 
             $attachment->update();
         } else {
             $attachment = $this->invoice->create_attachment($filename, $this->invoice->get_label(), "application/pdf");
-            if (   !$attachment
+            if (!$attachment
                 || !$this->invoice->set_parameter("midcom.helper.datamanager2.type.blobs", "guids_pdf_file", $attachment->guid . ":" . $attachment->guid)) {
                 throw new midcom_error("Failed to create invoice attachment for pdf");
             }

lib/org/openpsa/products/handler/product/view.php

Spacing +1 added lines, -1 removed lines

@@ -58,7 +58,7 @@
 
         if (midcom::get()->config->get('enable_ajax_editing')) {
             $data['controller'] = midcom_helper_datamanager2_controller::create('ajax');
-            $data['controller']->schemadb =& $data['schemadb_product'];
+            $data['controller']->schemadb = & $data['schemadb_product'];
             $data['controller']->set_storage($this->_product);
             $data['controller']->process_ajax();
             $data['datamanager'] = $data['controller']->datamanager;

Braces +2 added lines, -1 removed lines

@@ -85,7 +85,8 @@
                 $productgroup = new org_openpsa_products_product_group_dba($this->_product->productGroup);
                 $title = str_replace('<PRODUCTGROUP_TITLE>', $productgroup->title, $title);
                 $title = str_replace('<PRODUCTGROUP_CODE>', $productgroup->code, $title);
-            } catch (midcom_error $e) {
+            }
+            catch (midcom_error $e) {
                 $title = str_replace(array('<PRODUCTGROUP_TITLE>', '<PRODUCTGROUP_CODE>'), '', $title);
             }
         }

lib/org/openpsa/user/accounthelper.php

Spacing +5 added lines, -5 removed lines

@@ -146,7 +146,7 @@ 
             $new_password_encrypted = midcom_connection::prepare_password($new_password);
 
             //check if the new encrypted password was already used
-            if (    $this->check_password_reuse($new_password_encrypted)
+            if ($this->check_password_reuse($new_password_encrypted)
                  && $this->check_password_strength($new_password)) {
                 $this->_save_old_password();
                 $account->set_password($new_password);
@@ -336,7 +336,7 @@ 
         for ($i = 0; $i < strlen($password); $i++) {
             $repeated = true;
             for ($j = 0; $j < $plen && ($j + $i + $plen) < strlen($password); $j++) {
-                if (   (substr($password, $j + $i, 1) == substr($password, $j + $i + $plen, 1))
+                if ((substr($password, $j + $i, 1) == substr($password, $j + $i + $plen, 1))
                     && $repeated) {
                     $repeated = true;
                 } else {
@@ -478,7 +478,7 @@ 
 
     public static function get_person_by_formdata($data)
     {
-        if (   empty($data['username'])
+        if (empty($data['username'])
             || empty($data['password'])) {
             return false;
         }
@@ -512,7 +512,7 @@ 
         $max_attempts = midcom_baseclasses_components_configuration::get($component, 'config')->get('max_password_attempts');
         $timeframe = midcom_baseclasses_components_configuration::get($component, 'config')->get('password_block_timeframe_min');
 
-        if (   $max_attempts == 0
+        if ($max_attempts == 0
             || $timeframe == 0) {
             return $stat;
         }
@@ -534,7 +534,7 @@ 
          * If the maximum number of attemps is reached and the oldest attempt
          * on the stack is within our defined timeframe, we block the account
          */
-        if (   count($attempts) >= $max_attempts
+        if (count($attempts) >= $max_attempts
             && $attempts[$max_attempts - 1] >= (time() - ($timeframe * 60))) {
             $this->disable_account();
             $stat = false;

lib/org/openpsa/relatedto/handler/journal/list.php

Spacing +3 added lines, -3 removed lines

@@ -121,7 +121,7 @@ 
                 }
 
                 $reflector = midcom_helper_reflector::get($linked_object);
-                $link_html = "<a href='" . midcom::get()->permalinks->create_permalink($linked_object->guid) . "'>" . $reflector->get_object_label($linked_object) ."</a>";
+                $link_html = "<a href='" . midcom::get()->permalinks->create_permalink($linked_object->guid) . "'>" . $reflector->get_object_label($linked_object) . "</a>";
                 $data['linked_objects'][$entry->linkGuid] = $link_html;
                 $data['linked_raw_objects'][$entry->linkGuid] = $reflector->get_object_label($linked_object);
             }
@@ -151,8 +151,8 @@ 
         //check if there is a page & rows - parameter passed - if add them to qb
         if (array_key_exists('page', $_POST) && array_key_exists('rows', $_POST)) {
             $this->_request_data['page'] = $_POST['page'];
-            $this->qb->set_limit((int)$_POST['rows']);
-            $offset = ((int)$_POST['page'] - 1) * (int)$_POST['rows'];
+            $this->qb->set_limit((int) $_POST['rows']);
+            $offset = ((int) $_POST['page'] - 1) * (int) $_POST['rows'];
             $this->qb->set_offset($offset);
         }
     }

Braces +2 added lines, -1 removed lines

@@ -171,7 +171,8 @@
                 //create reflector with linked object to get the right label
                 try {
                     $linked_object = midcom::get()->dbfactory->get_object_by_guid($entry->linkGuid);
-                } catch (midcom_error $e) {
+                }
+                catch (midcom_error $e) {
                     unset($data['entries'][$i]);
                     $e->log();
                     continue;

lib/org/openpsa/projects/handler/task/list.php

Spacing +4 added lines, -4 removed lines

@@ -122,7 +122,7 @@ 
             case 'proposed':
                 $html = $this->render_status($task->manager, "from %s");
                 $task->get_members();
-                if (   $task->can_do('midgard:update')
+                if ($task->can_do('midgard:update')
                     && isset($task->resources[midcom_connection::get_user()])) {
                     $html .= '<form method="post" action="' . $prefix . 'workflow/' . $task->guid . '/">';
                     //TODO: If we need all resources to accept task hide tools when we have accepted and replace with "pending acceptance from..."
@@ -244,7 +244,7 @@ 
         $entry['id'] = $task->id;
         $entry['index_task'] = $task->title;
         $entry['task'] = '<a href="' . $task_url . '"><img class="status-icon" src="' . MIDCOM_STATIC_URL . '/stock-icons/16x16/' . $task->get_icon() . '" /> ' . $task->title . '</a>';
-        if (   $this->_request_data['view_identifier'] == 'my_tasks'
+        if ($this->_request_data['view_identifier'] == 'my_tasks'
             || $this->_request_data['view_identifier'] == 'project_tasks') {
             $entry['status_control'] = org_openpsa_projects_workflow::render_status_control($task);
             $status_type = $this->_get_status_type($task);
@@ -393,7 +393,7 @@ 
         if ($data['view'] == 'json') {
             midcom_show_style('show-json-tasks');
         } else {
-            if (   $data['view_identifier'] != 'my_tasks'
+            if ($data['view_identifier'] != 'my_tasks'
                 && $data['view_identifier'] != 'agreement') {
                 midcom_show_style('show-priority-filter');
             }
@@ -430,7 +430,7 @@ 
             $e->log();
         }
 
-        if (   $this->_request_data['view_identifier'] != 'agreement'
+        if ($this->_request_data['view_identifier'] != 'agreement'
             && $this->_request_data['view_identifier'] != 'project_tasks') {
             try {
                 $customer = org_openpsa_contacts_group_dba::get_cached($task->customer);

Braces +4 added lines, -2 removed lines

@@ -426,7 +426,8 @@ 
             $prefix = midcom_core_context::get()->get_key(MIDCOM_CONTEXT_ANCHORPREFIX);
             $ret['project'] = '<a href="' . $prefix . 'project/' . $project->guid . '/">' . $project->title . '</a>';
             $ret['index_project'] = $project->title;
-        } catch (midcom_error $e) {
+        }
+        catch (midcom_error $e) {
             $e->log();
         }
 
@@ -437,7 +438,8 @@ 
                 $customer_url = "{$this->_request_data['contacts_url']}group/{$customer->guid}/";
                 $ret['customer'] = "<a href='{$customer_url}' title='{$customer->official}'>{$customer->get_label()}</a>";
                 $ret['index_customer'] = $customer->name;
-            } catch (midcom_error $e) {
+            }
+            catch (midcom_error $e) {
                 $ret['customer'] = '';
                 $ret['index_customer'] = '';
             }

lib/org/openpsa/notifications/main.php

Spacing +1 added lines, -1 removed lines

@@ -93,7 +93,7 @@
 
         // If user has preference for this message, we use that
         $personal_preferences = $recipient->list_parameters('org.openpsa.notifications');
-        if (   count($personal_preferences) > 0
+        if (count($personal_preferences) > 0
             && array_key_exists("{$component}:{$action}", $personal_preferences)) {
             return $personal_preferences[$action];
         }

Braces +2 added lines, -1 removed lines

@@ -52,7 +52,8 @@
         // TODO: Should we sudo here to ensure getting correct prefs regardless of ACLs?
         try {
             $recipient = midcom_db_person::get_cached($recipient);
-        } catch (midcom_error $e) {
+        }
+        catch (midcom_error $e) {
             return false;
         }
 

lib/midgard/admin/asgard/handler/object/manage.php

Braces +3 added lines, -3 removed lines

@@ -64,7 +64,8 @@ 
     {
         try {
             $this->_object = midcom::get()->dbfactory->get_object_by_guid($guid);
-        } catch (midcom_error $e) {
+        }
+        catch (midcom_error $e) {
             if (midcom_connection::get_error() == MGD_ERR_OBJECT_DELETED) {
                 $relocate = midcom_core_context::get()->get_key(MIDCOM_CONTEXT_ANCHORPREFIX) . '__mfa/asgard/object/deleted/' . $guid . '/';
                 midcom::get()->relocate($relocate);
@@ -232,8 +233,7 @@ 
         $this->_new_object = new $create_type();
         $mgd_type = midcom::get()->dbclassloader->get_mgdschema_class_name_for_midcom_class($create_type);
 
-        if ($parent_property = midgard_object_class::get_property_parent($mgd_type))
-        {
+        if ($parent_property = midgard_object_class::get_property_parent($mgd_type)) {
             $this->_new_object->$parent_property = $controller->formmanager->get_value($parent_property);
         }
 

Spacing +14 added lines, -14 removed lines

@@ -167,7 +167,7 @@ 
 
         $this->_load_schemadb();
         $this->_controller = midcom_helper_datamanager2_controller::create('simple');
-        $this->_controller->schemadb =& $this->_schemadb;
+        $this->_controller->schemadb = & $this->_schemadb;
         $this->_controller->set_storage($this->_object, 'object');
         if (!$this->_controller->initialize()) {
             throw new midcom_error("Failed to initialize a DM2 controller instance for object {$this->_object->guid}.");
@@ -184,11 +184,11 @@ 
                 return $this->_prepare_relocate($this->_object);
 
             case 'edit':
-                $qf =& $this->_controller->formmanager->form;
-                if (   $qf->isSubmitted()
+                $qf = & $this->_controller->formmanager->form;
+                if ($qf->isSubmitted()
                     && !$qf->validate()) {
                     foreach ($qf->_errors as $field => $error) {
-                        $element =& $qf->getElement($field);
+                        $element = & $qf->getElement($field);
                         $message = sprintf($this->_l10n->get('validation error in field %s: %s'), $element->getLabel(), $error);
                         midcom::get()->uimessages->add(
                             $this->_l10n->get('midgard.admin.asgard'),
@@ -231,7 +231,7 @@ 
 
         if (!$this->_new_object->create()) {
             debug_print_r('We operated on this object:', $this->_new_object);
-            throw new midcom_error('Failed to create a new object. Last Midgard error was: '. midcom_connection::get_error_string());
+            throw new midcom_error('Failed to create a new object. Last Midgard error was: ' . midcom_connection::get_error_string());
         }
 
         return $this->_new_object;
@@ -260,7 +260,7 @@ 
         midcom::get()->auth->require_user_do('midgard.admin.asgard:manage_objects', null, 'midgard_admin_asgard_plugin');
 
         $data['defaults'] = array();
-        if (   $handler_id == '____mfa-asgard-object_create_toplevel'
+        if ($handler_id == '____mfa-asgard-object_create_toplevel'
             || $handler_id == '____mfa-asgard-object_create_chooser') {
             midcom::get()->auth->require_user_do('midgard:create', null, $this->_new_type);
 
@@ -278,9 +278,9 @@ 
         }
 
         $this->_controller = midcom_helper_datamanager2_controller::create('create');
-        $this->_controller->schemadb =& $this->_schemadb;
+        $this->_controller->schemadb = & $this->_schemadb;
         $this->_controller->schema = 'object';
-        $this->_controller->callback_object =& $this;
+        $this->_controller->callback_object = & $this;
         $this->_controller->defaults = $this->_get_defaults();
         if (!$this->_controller->initialize()) {
             throw new midcom_error("Failed to initialize a DM2 create controller.");
@@ -321,11 +321,11 @@ 
             $parent_property = null;
             $new_type_reflector = midcom_helper_reflector::get($this->_new_type);
             $link_properties = $new_type_reflector->get_link_properties();
-            $type_to_link_to =  midcom_helper_reflector::class_rewrite(get_class($this->_object));
+            $type_to_link_to = midcom_helper_reflector::class_rewrite(get_class($this->_object));
             foreach ($link_properties as $child_property => $link) {
                 $linked_type = midcom_helper_reflector::class_rewrite($link['class']);
                 if (midcom_helper_reflector::is_same_class($linked_type, $type_to_link_to)
-                    || (   $link['type'] == MGD_TYPE_GUID
+                    || ($link['type'] == MGD_TYPE_GUID
                         && is_null($link['class']))) {
                     $parent_property = $link['target'];
                     break;
@@ -338,7 +338,7 @@ 
         }
 
         // Allow setting defaults from query string, useful for things like "create event for today" and chooser
-        if (   isset($_GET['defaults'])
+        if (isset($_GET['defaults'])
             && is_array($_GET['defaults'])) {
             $get_defaults = array_intersect_key($_GET['defaults'], $this->_schemadb['object']->fields);
             $defaults = array_merge($defaults, array_map('trim', $get_defaults));
@@ -356,7 +356,7 @@ 
     {
         if ($handler_id == '____mfa-asgard-object_create_chooser') {
             midcom_show_style('midgard_admin_asgard_popup_header');
-            if (   $this->_new_object
+            if ($this->_new_object
                 || isset($data['cancelled'])) {
                 $data['jsdata'] = $this->_object_to_jsdata($this->_new_object);
                 midcom_show_style('midgard_admin_asgard_object_create_after');
@@ -407,7 +407,7 @@ 
             $url = $type;
 
             $class_extends = $this->_config->get('class_extends');
-            if (   is_array($class_extends)
+            if (is_array($class_extends)
                 && array_key_exists($type, $class_extends)) {
                 $url = $class_extends[$type];
             }
@@ -506,7 +506,7 @@ 
 
         // Load the nullstorage controller
         $this->_controller = midcom_helper_datamanager2_controller::create('nullstorage');
-        $this->_controller->schemadb =& $this->_schemadb;
+        $this->_controller->schemadb = & $this->_schemadb;
 
         if (!$this->_controller->initialize()) {
             throw new midcom_error('Failed to initialize the controller');

lib/midcom/admin/folder/management.php

Spacing +5 added lines, -5 removed lines

@@ -82,7 +82,7 @@ 
             }
 
             // Skip components beginning with midcom or midgard
-            if (   preg_match('/^(midcom|midgard)\./', $manifest->name)
+            if (preg_match('/^(midcom|midgard)\./', $manifest->name)
                 && $manifest->name != 'midcom.helper.search') {
                 continue;
             }
@@ -99,7 +99,7 @@ 
         asort($components);
 
         // Set the parent component to be the first if applicable
-        if (   $parent_component !== ''
+        if ($parent_component !== ''
             && array_key_exists($parent_component, $components)) {
             $temp = array();
             $temp[$parent_component] = $components[$parent_component];
@@ -128,14 +128,14 @@ 
         }
 
         foreach (self::get_component_list() as $component => $details) {
-            if (   $component !== $parent_component
+            if ($component !== $parent_component
                 && !$all) {
-                if (   is_array(midcom::get()->config->get('component_listing_allowed'))
+                if (is_array(midcom::get()->config->get('component_listing_allowed'))
                     && !in_array($component, midcom::get()->config->get('component_listing_allowed'))) {
                     continue;
                 }
 
-                if (   is_array(midcom::get()->config->get('component_listing_excluded'))
+                if (is_array(midcom::get()->config->get('component_listing_excluded'))
                     && in_array($component, midcom::get()->config->get('component_listing_excluded'))) {
                     continue;
                 }

lib/midcom/config/test.php

Spacing +1 added lines, -1 removed lines

@@ -14,7 +14,7 @@
 class midcom_config_test
 {
     const OK = 0;
-    const WARNING =  1;
+    const WARNING = 1;
     const ERROR = 2;
 
     private $messages = array(

Indentation +1 added lines, -1 removed lines

@@ -104,7 +104,7 @@
             $this->add('Setting: upload_max_filesize', self::OK, ini_get('upload_max_filesize'));
         } else {
             $this->add('Setting: upload_max_filesize',
-                             self::WARNING, "To make bulk uploads (for exampe in the Image Gallery) useful, you should increase the Upload limit to something above 50 MB. (Current setting: {$upload_limit})");
+                                self::WARNING, "To make bulk uploads (for exampe in the Image Gallery) useful, you should increase the Upload limit to something above 50 MB. (Current setting: {$upload_limit})");
         }
 
         $post_limit = $this->ini_get_filesize('post_max_size');

Braces +2 added lines, -1 removed lines

@@ -74,7 +74,8 @@
                 $config = new midcom_services_rcs_config($config);
                 $config->test_rcs_config();
                 $this->add("MidCOM RCS", self::OK);
-            } catch (midcom_error $e) {
+            }
+            catch (midcom_error $e) {
                 $this->add("MidCOM RCS", self::ERROR, $e->getMessage());
             }
         } else {