HexMakina / kadro

Models/Abilities/Event.php

Spacing +1 added lines, -1 removed lines

@@ -22,7 +22,7 @@
 
     public function event_label()
     {
-        return '' . $this;
+        return ''.$this;
     }
 
     public static function today($format = Dato::FORMAT)

Controllers/Operator.php

Spacing +2 added lines, -2 removed lines

@@ -13,7 +13,7 @@ 
 namespace HexMakina\kadro\Controllers;
 
 use HexMakina\Crudites\Crudites;
-use HexMakina\kadro\Auth\{ACL,AccessRefusedException};
+use HexMakina\kadro\Auth\{ACL, AccessRefusedException};
 
 class Operator extends \HexMakina\kadro\Controllers\ORM
 {
@@ -56,7 +56,7 @@ 
     public function dashboard()
     {
         $real_operator_class = get_class($this->operator());
-        $this->viewport('users', $real_operator_class::filter([], ['order_by' => [null,'username', 'ASC']]));
+        $this->viewport('users', $real_operator_class::filter([], ['order_by' => [null, 'username', 'ASC']]));
     }
 
     public function destroy()

kadro.php

Spacing +4 added lines, -4 removed lines

@@ -51,7 +51,7 @@ 
         self::$box->get('\Smarty')->assign('lezer', $lezer);
         self::$box->get('\Smarty')->assign('language', $language);
 
-        setcookie('lang', $language, time() + (365 * 24 * 60 * 60), "/", "");
+        setcookie('lang', $language, time()+(365 * 24 * 60 * 60), "/", "");
 
         return self::$box;
     }
@@ -62,7 +62,7 @@ 
         $setting = 'settings.default.charset';
         if (is_string(self::$box->get($setting))) {
             ini_set('default_charset', self::$box->get($setting));
-            header('Content-type: text/html; charset=' . strtolower(self::$box->get($setting)));
+            header('Content-type: text/html; charset='.strtolower(self::$box->get($setting)));
         } else {
             throw new \UnexpectedValueException($setting);
         }
@@ -70,7 +70,7 @@ 
         // ----     parametroj:linguo
         $setting = 'settings.default.language';
         if (is_string(self::$box->get($setting))) {
-            putenv('LANG=' . self::$box->get($setting));
+            putenv('LANG='.self::$box->get($setting));
             setlocale(LC_ALL, self::$box->get($setting));
         } else {
             throw new \UnexpectedValueException($setting);
@@ -94,7 +94,7 @@ 
         foreach (self::$box->get('settings.smarty.template_extra_directories') as $i => $template_dir) {
             $smarty->addTemplateDir($template_dir);
         }
-        $smarty->addTemplateDir(__DIR__ . '/Views/'); //kadro templates
+        $smarty->addTemplateDir(__DIR__.'/Views/'); //kadro templates
 
         $setting = 'settings.smarty.compiled_path';
         if (is_string(self::$box->get($setting))) {

TableToForm.php

Spacing +2 added lines, -2 removed lines

@@ -3,7 +3,7 @@ 
 namespace HexMakina\kadro;
 
 use HexMakina\BlackBox\ORM\ModelInterface;
-use HexMakina\Marker\{Form,Element};
+use HexMakina\Marker\{Form, Element};
 
 class TableToForm
 {
@@ -82,7 +82,7 @@ 
                     $form_field = new Element('div', $form_field, ['class' => $group_class]);
                 }
             }
-            $ret .= PHP_EOL . $form_field;
+            $ret .= PHP_EOL.$form_field;
         }
 
         return $ret;

Auth/Operatorability.php

Spacing +3 added lines, -3 removed lines

@@ -50,12 +50,12 @@
 
     public static function enhance_query_retrieve($Query, $filters, $options)
     {
-        $Query->autoJoin([ACL::table(),'ACL'], null, 'LEFT OUTER');
+        $Query->autoJoin([ACL::table(), 'ACL'], null, 'LEFT OUTER');
         $permission_alias = $Query->autoJoin([Permission::table(), 'permission'], null, 'LEFT OUTER');
 
         $permission_ids_and_names = [];
-        $permission_ids_and_names [] = sprintf('GROUP_CONCAT(DISTINCT %s.%s) as %s', $permission_alias, 'id', $permission_alias . '_ids');
-        $permission_ids_and_names [] = sprintf('GROUP_CONCAT(DISTINCT %s.%s) as %s', $permission_alias, 'name', $permission_alias . '_names');
+        $permission_ids_and_names [] = sprintf('GROUP_CONCAT(DISTINCT %s.%s) as %s', $permission_alias, 'id', $permission_alias.'_ids');
+        $permission_ids_and_names [] = sprintf('GROUP_CONCAT(DISTINCT %s.%s) as %s', $permission_alias, 'name', $permission_alias.'_names');
         $Query->selectAlso($permission_ids_and_names);
 
         $Query->selectAlso(['operator.name as operator_name', 'operator.active as operator_active']);

Auth/Operator.php

Spacing +3 added lines, -3 removed lines

@@ -4,7 +4,7 @@ 
 
 use HexMakina\BlackBox\Database\SelectInterface;
 use HexMakina\BlackBox\Auth\OperatorInterface;
-use HexMakina\TightORM\{TightModel,RelationManyToMany};
+use HexMakina\TightORM\{TightModel, RelationManyToMany};
 
 class Operator extends TightModel implements OperatorInterface
 {
@@ -91,7 +91,7 @@ 
         }
 
         if (isset($filters['model']) && !empty($filters['model'])) {
-            $Query->join([static::otm('t'), static::otm('a')], [[static::otm('a'),static::otm('k'), 't_from','id']], 'INNER');
+            $Query->join([static::otm('t'), static::otm('a')], [[static::otm('a'), static::otm('k'), 't_from', 'id']], 'INNER');
             $Query->whereFieldsEQ(['model_id' => $filters['model']->getId(), 'model_type' => get_class($filters['model'])::model_type()], static::otm('a'));
         }
 
@@ -146,7 +146,7 @@ 
     public function hasPermission($p): bool
     {
       // new instances or inactive operators, none shall pass
-        if ($this->isNew() === true || $this->isActive()  === false) {
+        if ($this->isNew() === true || $this->isActive() === false) {
             return false;
         }
 

Controllers/Base.php

Spacing +1 added lines, -1 removed lines

@@ -2,7 +2,7 @@
 
 namespace HexMakina\kadro\Controllers;
 
-use Psr\Container\{ContainerInterface,ContainerExceptionInterface,NotFoundExceptionInterface};
+use Psr\Container\{ContainerInterface, ContainerExceptionInterface, NotFoundExceptionInterface};
 use HexMakina\BlackBox\RouterInterface;
 use HexMakina\BlackBox\Controllers\BaseControllerInterface;
 use Psr\Log\LoggerInterface;

Controllers/Display.php

Spacing +5 added lines, -5 removed lines

@@ -51,9 +51,9 @@ 
         $this->viewport('user_messages', $this->get('HexMakina\BlackBox\StateAgentInterface')->messages());
 
         $this->viewport('web_root', $this->router()->webRoot());
-        $this->viewport('view_path', $this->router()->filePath() . $this->get('settings.smarty.template_path') . 'app/');
-        $this->viewport('view_url', $this->router()->webRoot() . $this->get('settings.smarty.template_path'));
-        $this->viewport('images_url', $this->router()->webRoot() . $this->get('settings.smarty.template_path') . 'images/');
+        $this->viewport('view_path', $this->router()->filePath().$this->get('settings.smarty.template_path').'app/');
+        $this->viewport('view_url', $this->router()->webRoot().$this->get('settings.smarty.template_path'));
+        $this->viewport('images_url', $this->router()->webRoot().$this->get('settings.smarty.template_path').'images/');
 
         foreach ($this->viewport() as $template_var_name => $value) {
             $smarty->assign($template_var_name, $value);
@@ -83,8 +83,8 @@ 
           // 1. check for custom template in the current controller directory
             $templates ['custom_3'] = sprintf('%s/%s.html', $controller_template_path, $custom_template);
           // 2. check for custom template formatted as controller/view
-            $templates ['custom_2'] = $custom_template . '.html';
-            $templates ['custom_1'] = '_layouts/' . $custom_template . '.html';
+            $templates ['custom_2'] = $custom_template.'.html';
+            $templates ['custom_1'] = '_layouts/'.$custom_template.'.html';
         }
 
         if (!empty($this->router()->targetMethod())) {

Controllers/ORM.php

Spacing +13 added lines, -13 removed lines

@@ -58,7 +58,7 @@ 
         $ret = $this->modelType();
 
         if (!is_null($suffix)) {
-            $ret .= '_' . $suffix;
+            $ret .= '_'.$suffix;
         }
 
         return $ret;
@@ -167,8 +167,8 @@ 
         $this->viewport('listing_fields', $listing_fields);
         $this->viewport('listing_template', $listing_template);
 
-        $this->viewport('route_new', $this->router()->hyp($class_name::model_type() . '_new'));
-        $this->viewport('route_export', $this->router()->hyp($class_name::model_type() . '_export'));
+        $this->viewport('route_new', $this->router()->hyp($class_name::model_type().'_new'));
+        $this->viewport('route_export', $this->router()->hyp($class_name::model_type().'_export'));
     }
 
     public function copy()
@@ -199,7 +199,7 @@ 
     {
         $this->errors = $model->save($this->operator()->operatorId()); // returns [errors]
         if (empty($this->errors())) {
-            $this->logger()->notice($this->l('CRUDITES_INSTANCE_ALTERED', [$this->l('MODEL_' . get_class($model)::model_type() . '_INSTANCE')]));
+            $this->logger()->notice($this->l('CRUDITES_INSTANCE_ALTERED', [$this->l('MODEL_'.get_class($model)::model_type().'_INSTANCE')]));
             return $model;
         }
         foreach ($this->errors() as $field => $error_msg) {
@@ -212,7 +212,7 @@ 
     public function before_edit()
     {
         if (!is_null($this->router()->params('id')) && is_null($this->load_model)) {
-            $this->logger()->warning($this->l('CRUDITES_ERR_INSTANCE_NOT_FOUND', [$this->l('MODEL_' . $this->modelClassName()::model_type() . '_INSTANCE')]));
+            $this->logger()->warning($this->l('CRUDITES_ERR_INSTANCE_NOT_FOUND', [$this->l('MODEL_'.$this->modelClassName()::model_type().'_INSTANCE')]));
             $this->router()->hop($this->modelClassName()::model_type());
         }
     }
@@ -231,7 +231,7 @@ 
     public function destroy_confirm()
     {
         if (is_null($this->load_model)) {
-            $this->logger()->warning($this->l('CRUDITES_ERR_INSTANCE_NOT_FOUND', [$this->l('MODEL_' . $this->model_type . '_INSTANCE')]));
+            $this->logger()->warning($this->l('CRUDITES_ERR_INSTANCE_NOT_FOUND', [$this->l('MODEL_'.$this->model_type.'_INSTANCE')]));
             $this->router()->hop($this->model_type);
         }
 
@@ -243,10 +243,10 @@ 
     public function before_destroy() // default: checks for load_model and immortality, hops back to object on failure
     {
         if (is_null($this->load_model)) {
-            $this->logger()->warning($this->l('CRUDITES_ERR_INSTANCE_NOT_FOUND', [$this->l('MODEL_' . $this->model_type . '_INSTANCE')]));
+            $this->logger()->warning($this->l('CRUDITES_ERR_INSTANCE_NOT_FOUND', [$this->l('MODEL_'.$this->model_type.'_INSTANCE')]));
             $this->router()->hop($this->model_type);
         } elseif ($this->load_model->immortal()) {
-            $this->logger()->warning($this->l('CRUDITES_ERR_INSTANCE_IS_IMMORTAL', [$this->l('MODEL_' . $this->model_type . '_INSTANCE')]));
+            $this->logger()->warning($this->l('CRUDITES_ERR_INSTANCE_IS_IMMORTAL', [$this->l('MODEL_'.$this->model_type.'_INSTANCE')]));
             $this->router()->hop($this->route_model($this->load_model));
         }
     }
@@ -258,10 +258,10 @@ 
         }
 
         if ($this->load_model->destroy($this->operator()->operatorId()) === false) {
-            $this->logger()->info($this->l('CRUDITES_ERR_INSTANCE_IS_UNDELETABLE', ['' . $this->load_model]));
+            $this->logger()->info($this->l('CRUDITES_ERR_INSTANCE_IS_UNDELETABLE', [''.$this->load_model]));
             $this->routeBack($this->load_model);
         } else {
-            $this->logger()->notice($this->l('CRUDITES_INSTANCE_DESTROYED', [$this->l('MODEL_' . $this->model_type . '_INSTANCE')]));
+            $this->logger()->notice($this->l('CRUDITES_INSTANCE_DESTROYED', [$this->l('MODEL_'.$this->model_type.'_INSTANCE')]));
             $this->routeBack($this->model_type);
         }
     }
@@ -285,7 +285,7 @@ 
     public function collection_to_csv($collection, $filename)
     {
       // TODO use Format/File/CSV class to generate file
-        $file_path = $this->get('settings.export.directory') . $filename . '.csv';
+        $file_path = $this->get('settings.export.directory').$filename.'.csv';
         $fp = fopen($file_path, 'w');
 
         $header = false;
@@ -322,7 +322,7 @@ 
 
     public function route_new(ModelInterface $model): string
     {
-        return $this->router()->hyp(get_class($model)::model_type() . '_new');
+        return $this->router()->hyp(get_class($model)::model_type().'_new');
     }
 
     public function route_list(ModelInterface $model): string
@@ -334,7 +334,7 @@ 
     {
         $route_params = [];
 
-        $route_name = get_class($model)::model_type() . '_';
+        $route_name = get_class($model)::model_type().'_';
         if ($model->isNew()) {
             $route_name .= 'new';
         } else {