flack / openpsa

src/midcom/datamanager/storage/parameter.php

Indentation +1 added lines, -1 removed lines

@@ -87,7 +87,7 @@
      * Converts the selected options according to the multiple_storagemode setting.
      *
      * @return mixed The data converted to the final data storage.
-    */
+     */
     function convert_multiple_to_storage()
     {
         switch ($this->config['type_config']['multiple_storagemode']) {

Spacing +2 added lines, -2 removed lines

@@ -44,7 +44,7 @@ 
     {
         //workaround for weird mgd API behavior where setting falsy (i.e. deleting) a nonexistent parameter
         //returns an error
-        if (   !$this->value
+        if (!$this->value
             && $this->load() === null) {
             return true;
         }
@@ -63,7 +63,7 @@ 
         switch ($this->config['type_config']['multiple_storagemode']) {
             case 'serialized':
             case 'array':
-                if (   !is_array($source)
+                if (!is_array($source)
                     && empty($source)) {
                     $source = array();
                 }

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

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,

src/midcom/datamanager/schema.php

Spacing +5 added lines, -5 removed lines

@@ -144,15 +144,15 @@ 
             'validation' => array()
         ));
 
-        $normalize_widget = function (Options $options, $value) {
-            if (   $value == 'images'
+        $normalize_widget = function(Options $options, $value) {
+            if ($value == 'images'
                 || $value == 'downloads') {
                 return 'subform';
             }
             return $value;
         };
 
-        $normalize_storage = function (Options $options, $value) use ($name) {
+        $normalize_storage = function(Options $options, $value) use ($name) {
             $default = array(
                 'location' => 'parameter',
                 'domain' => 'midcom.helper.datamanager2',
@@ -191,7 +191,7 @@ 
             return $value;
         };
 
-        $normalize_validation = function (Options $options, $config) {
+        $normalize_validation = function(Options $options, $config) {
             $validation = array();
             if (array_key_exists('validation', (array) $config)) {
                 $validation = (array) $config['validation'];
@@ -202,7 +202,7 @@ 
                     $rule = array('type' => $rule);
                 } elseif (!array_key_exists('type', $rule)) {
                     throw new midcom_error("Missing validation rule type for rule {$key} on field {$config['name']}, this is a required option.");
-                } elseif (   $rule['type'] == 'compare'
+                } elseif ($rule['type'] == 'compare'
                           && !array_key_exists('compare_with', $rule)) {
                     throw new midcom_error("Missing compare_with option for compare type rule {$key} on field {$config['name']}, this is a required option.");
                 }

src/midcom/datamanager/template/form.php

Spacing +13 added lines, -13 removed lines

@@ -36,7 +36,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 .= '<fieldset class="fieldset">';
                 if (!empty($child->vars['start_fieldset']['title'])) {
@@ -44,7 +44,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++) {
@@ -58,7 +58,7 @@ 
     public function form_end(FormView $view, array $data)
     {
         $string = '';
-        if (   !isset($data['render_rest'])
+        if (!isset($data['render_rest'])
             || $data['render_rest']) {
             $string .= $this->renderer->rest($view);
         }
@@ -143,7 +143,7 @@ 
     public function form_widget_simple(FormView $view, array $data)
     {
         $type = isset($data['type']) ? $data['type'] : 'text';
-        if (   $type == 'text'
+        if ($type == 'text'
             || $type == 'password'
             || $type == 'email') {
             $view->vars['attr']['class'] = 'shorttext';
@@ -151,7 +151,7 @@ 
 
         $string = '<input type="' . $type . '"';
         $string .= $this->renderer->block($view, 'widget_attributes');
-        if (   !empty($data['value'])
+        if (!empty($data['value'])
             || is_numeric($data['value'])) {
             $string .= ' value="' . $this->escape($data['value']) . '"';
         }
@@ -184,7 +184,7 @@ 
         $jsinit .= "midcom_helper_datamanager2_autocomplete.create_dm2_widget('{$element_id}_search_input', {$data['min_chars']});\n";
 
         $string = '<fieldset ' . $this->renderer->block($view, 'widget_container_attributes') . '>';
-        $string .=  $this->renderer->widget($view['selection']);
+        $string .= $this->renderer->widget($view['selection']);
         $string .= ' ' . $this->renderer->widget($view['search_input']);
         $string .= '</fieldset>';
         return $string . $this->jsinit($jsinit);
@@ -219,7 +219,7 @@ 
     public function choice_widget_collapsed(FormView $view, array $data)
     {
         $string = '<select';
-        if (   $data['required']
+        if ($data['required']
             && null === $data['empty_value']
             && $data['empty_value_in_choices'] === false
             && $data['multiple'] === false) {
@@ -233,7 +233,7 @@ 
         $string .= '>';
         if (null !== $data['empty_value']) {
             $string .= '<option value=""';
-            if (   $data['required']
+            if ($data['required']
                 && empty($data['value'])
                 && "0" !== $data['value']) {
                 $string .= ' selected="selected"';
@@ -255,7 +255,7 @@ 
         if (!empty($data['form']->vars['type'])) {
             $obj = $data['form'];
             $obj = $data['form']->vars['type']->search_for_object($obj);
-            $new_label = $data['form']->vars['type']->render_choices($data['form']->vars['type_conf'],$obj);
+            $new_label = $data['form']->vars['type']->render_choices($data['form']->vars['type_conf'], $obj);
             $view->children[2]->vars['label'] = $new_label;
         }
         return $this->choice_widget_expanded($view, $data);
@@ -309,7 +309,7 @@ 
         $string .= $this->renderer->widget($view['date']);
 
         if (isset($view['time'])) {
-            $string .= ' '. $this->renderer->widget($view['time']);
+            $string .= ' ' . $this->renderer->widget($view['time']);
         }
         $string .= $data['jsinit'];
         return $string . '</fieldset>';
@@ -323,7 +323,7 @@ 
         if (!empty($data['value'])) {
             $string .= '<a href="' . $view->vars['value']['url'] . '" target="_new"><img src="' . $view->vars['value']['url'] . '" class="preview-image">';
 
-            if (   $data['value']['size_x']
+            if ($data['value']['size_x']
                 && $data['value']['size_y']) {
                 $size = "{$data['value']['size_x']}&times;{$data['value']['size_y']}";
             } else {
@@ -362,7 +362,7 @@ 
         if (!empty($values)) {
             $string .= '<ul>';
             foreach ($values as $identifier => $info) {
-                if (   $info['size_x']
+                if ($info['size_x']
                     && $info['size_y']) {
                     $size = "{$info['size_x']}x{$info['size_y']}";
                 } else {
@@ -380,7 +380,7 @@ 
         $string .= '<label class="midcom_datamanager_photo_lable">' . $this->renderer->humanize('delete photo') . ' ' . $this->renderer->widget($data['form']['delete']) . '</label>';
         $string .= '</td></tr></table></div>';
 
-        return $string . $this->jsinit('init_photo_widget("' . $view->vars['id'] .'");');
+        return $string . $this->jsinit('init_photo_widget("' . $view->vars['id'] . '");');
     }
 
     public function subform_widget(FormView $view, array $data)

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

Spacing +2 added lines, -2 removed lines

@@ -42,7 +42,7 @@ 
      */
     public function configureOptions(OptionsResolver $resolver)
     {
-        $map_privilege = function (Options $options) {
+        $map_privilege = function(Options $options) {
             $return_options = $this->defaultChoices;
             return $return_options;
         };
@@ -88,7 +88,7 @@ 
     public function render_choices(array $options, $object = null)
     {
         $l10n = midcom::get()->i18n->get_l10n('midcom.datamanager');
-        $inherit = $this->get_effective_value($options,$object);
+        $inherit = $this->get_effective_value($options, $object);
         $allow = 'widget privilege: allow';
         $deny = 'widget privilege: deny';
         $base = $l10n->get('widget privilege: inherit %s');

Braces +2 added lines, -4 removed lines

@@ -65,8 +65,7 @@ 
 
     public function get_effective_value(array $options, $object = null)
     {
-        if (!$object)
-        {
+        if (!$object) {
             $defaults = midcom::get()->auth->acl->get_default_privileges();
             return $defaults[$options['privilege_name']] === MIDCOM_PRIVILEGE_ALLOW;
         }
@@ -107,8 +106,7 @@ 
     {
         $help_obj = $object;
 
-        while (true)
-        {
+        while (true) {
             if ($help_obj instanceof dbacontainer) {
                 return $help_obj->get_value();
             } else {