flack / openpsa

lib/org/openpsa/user/handler/person/create.php

Spacing +1 added lines, -1 removed lines

@@ -103,7 +103,7 @@
         $this->_person = new midcom_db_person;
         if (!$this->_person->create()) {
             debug_print_r('We operated on this object:', $this->_person);
-            throw new midcom_error('Failed to create a new person. Last Midgard error was: '. midcom_connection::get_error_string());
+            throw new midcom_error('Failed to create a new person. Last Midgard error was: ' . midcom_connection::get_error_string());
         }
 
         return $this->_person;

src/midcom/events/watcher.php

Braces +2 added lines, -1 removed lines

@@ -48,7 +48,8 @@
 
         try {
             $interface = \midcom::get()->componentloader->get_interface_class($this->component);
-        } catch (\midcom_error $e) {
+        }
+        catch (\midcom_error $e) {
             debug_add("Failed to load the component {$this->component}: " . $e->getMessage(), MIDCOM_LOG_INFO);
             return;
         }

src/midcom/workflow/dialog.php

Spacing +1 added lines, -1 removed lines

@@ -73,7 +73,7 @@
         $button_config[MIDCOM_TOOLBAR_URL] = $url;
         //The constants are numeric, so array_merge won't work...
         foreach ($options as $key => $value) {
-            if (   is_array($value)
+            if (is_array($value)
                 && !empty($button_config[$key])) {
                 $value = array_merge($button_config[$key], $value);
             }

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->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') {

src/midcom/datamanager/template/view.php

Spacing +4 added lines, -4 removed lines

@@ -24,7 +24,7 @@ 
     {
         $string = '';
         foreach ($view as $child) {
-            if (    array_key_exists('start_fieldset', $child->vars)
+            if (array_key_exists('start_fieldset', $child->vars)
                 && $child->vars['start_fieldset'] !== null) {
                 $string .= '<div class="fieldset">';
                 if (!empty($child->vars['start_fieldset']['title'])) {
@@ -32,7 +32,7 @@ 
                 }
             }
             $string .= $this->renderer->row($child);
-            if (    array_key_exists('end_fieldset', $child->vars)
+            if (array_key_exists('end_fieldset', $child->vars)
                 && $child->vars['end_fieldset'] !== null) {
                 $end_fieldsets = max(1, (int) $child->vars['end_fieldset']);
                 for ($i = 0; $i < $end_fieldsets; $i++) {
@@ -84,7 +84,7 @@ 
 
     public function form_widget_simple(FormView $view, array $data)
     {
-        if (   !empty($data['value'])
+        if (!empty($data['value'])
             || is_numeric($data['value'])) {
             return $this->escape($data['value']);
         }
@@ -131,7 +131,7 @@ 
         $string = $this->renderer->widget($view['date']);
 
         if (isset($view['time'])) {
-            $string .= ' '. $this->renderer->widget($view['time']);
+            $string .= ' ' . $this->renderer->widget($view['time']);
         }
         return $string;
     }

src/midcom/datamanager/extension/transformer/blobs.php

Spacing +1 added lines, -1 removed lines

@@ -98,7 +98,7 @@
 
     public function reverseTransform($array)
     {
-        if (!is_array($array) ) {
+        if (!is_array($array)) {
             throw new TransformationFailedException('Expected an array.');
         }
 

src/midcom/datamanager/extension/transformer/jsdate.php

Spacing +4 added lines, -4 removed lines

@@ -33,8 +33,8 @@ 
             }
         }
 
-        if (   $input === null
-            || (   $input instanceof DateTime
+        if ($input === null
+            || ($input instanceof DateTime
                 && $input->format('Y-m-d H:i:s') == '0001-01-01 00:00:00')) {
             return $result;
         }
@@ -55,12 +55,12 @@ 
 
     public function reverseTransform($array)
     {
-        if (!is_array($array) ) {
+        if (!is_array($array)) {
             throw new TransformationFailedException('Expected an array.');
         }
 
         if (empty($array['date'])
-            || (   $array['date'] instanceof DateTime
+            || ($array['date'] instanceof DateTime
                 && $array['date']->format('Y-m-d H:i:s') == '0001-01-01 00:00:00')) {
             return;
         }

src/midcom/datamanager/extension/transformer/autocomplete.php

Spacing +3 added lines, -3 removed lines

@@ -22,7 +22,7 @@ 
 
     public function transform($input)
     {
-        if (   $this->config['dm2_type'] == 'select'
+        if ($this->config['dm2_type'] == 'select'
             && $this->config['type_config']['allow_multiple']) {
             switch ($this->config['type_config']['multiple_storagemode']) {
                 case 'serialized':
@@ -42,7 +42,7 @@ 
 
     public function reverseTransform($array)
     {
-        if (!is_array($array) ) {
+        if (!is_array($array)) {
             throw new TransformationFailedException('Expected an array.');
         }
 
@@ -54,7 +54,7 @@ 
             return reset($array['selection']);
         }
 
-        if (   $this->config['dm2_type'] == 'select'
+        if ($this->config['dm2_type'] == 'select'
             && $this->config['type_config']['allow_multiple']) {
             switch ($this->config['type_config']['multiple_storagemode']) {
                 case 'serialized':

src/midcom/datamanager/extension/type/autocomplete.php

Braces +2 added lines, -1 removed lines

@@ -106,7 +106,8 @@
                 try {
                     $object = new $options['widget_config']['class']($identifier);
                     $preset[$identifier] = \midcom_helper_datamanager2_widget_autocomplete::create_item_label($object, $options['widget_config']['result_headers'], $options['widget_config']['get_label_for']);
-                } catch (midcom_error $e) {
+                }
+                catch (midcom_error $e) {
                     $e->log();
                 }
             }

Spacing +2 added lines, -2 removed lines

@@ -29,7 +29,7 @@ 
      */
     public function configureOptions(OptionsResolver $resolver)
     {
-        $resolver->setNormalizer('widget_config', function (Options $options, $value) {
+        $resolver->setNormalizer('widget_config', function(Options $options, $value) {
             $widget_defaults = array(
                 'creation_mode_enabled' => false,
                 'class' => null,
@@ -61,7 +61,7 @@ 
 
             return helper::resolve_options($widget_defaults, $value);
         });
-        $resolver->setNormalizer('type_config', function (Options $options, $value) {
+        $resolver->setNormalizer('type_config', function(Options $options, $value) {
             $type_defaults = array(
                 'options' => array(),
                 'allow_other' => false,