Test Failed
Push — 1.0.0-dev ( 90e1c2...b109e3 )
by nguereza
02:32
created
core/classes/Controller.php 1 patch
Indentation   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -93,7 +93,7 @@  discard block
 block discarded – undo
93 93
          *
94 94
          * @codeCoverageIgnore
95 95
          */
96
-         private function startAppSession() {
96
+            private function startAppSession() {
97 97
             //$_SESSION is not available on cli mode 
98 98
             if (!IS_CLI) {
99 99
                 //set session params
@@ -143,7 +143,7 @@  discard block
 block discarded – undo
143 143
          * @codeCoverageIgnore
144 144
          */
145 145
         private function setAppSessionConfig() {
146
-             //the default is to store in the files
146
+                //the default is to store in the files
147 147
             $sessionHandler = $this->config->get('session_handler', 'files');
148 148
             $this->logger->info('Session handler: ' . $sessionHandler);
149 149
             if ($sessionHandler == 'files') {
Please login to merge, or discard this patch.
core/classes/model/Model.php 1 patch
Indentation   +31 added lines, -31 removed lines patch added patch discarded remove patch
@@ -30,7 +30,7 @@  discard block
 block discarded – undo
30 30
 
31 31
     class Model {
32 32
 
33
-       /**
33
+        /**
34 34
          * This model's default database table. 
35 35
          * @var string the name of table
36 36
          */
@@ -173,12 +173,12 @@  discard block
 block discarded – undo
173 173
             if ($db !== null) {
174 174
                 $this->setDb($db);
175 175
             } else {
176
-                 /**
177
-                 * NOTE: Need use "clone" because some Model need have the personal instance of the database library
178
-                 * to prevent duplication
179
-                 */
180
-                 $obj = & get_instance();
181
-                 $this->setDb(clone $obj->database);
176
+                    /**
177
+                     * NOTE: Need use "clone" because some Model need have the personal instance of the database library
178
+                     * to prevent duplication
179
+                     */
180
+                    $obj = & get_instance();
181
+                    $this->setDb(clone $obj->database);
182 182
             }
183 183
             array_unshift($this->beforeCreateCallbacks, 'removeProtectedTableColumns');
184 184
             array_unshift($this->beforeUpdateCallbacks, 'removeProtectedTableColumns');
@@ -623,7 +623,7 @@  discard block
 block discarded – undo
623 623
          * Only get deleted rows on the next call
624 624
          * 
625 625
          * @return object the current instance
626
-        */
626
+         */
627 627
         public function onlyRecordDeleted() {
628 628
             $this->returnOnlyRecordDeleted = true;
629 629
             return $this;
@@ -649,8 +649,8 @@  discard block
 block discarded – undo
649 649
          * @return array the data after add field for updated time
650 650
          */
651 651
         public function updatedAt($row) {
652
-           $row['updated_at'] = date('Y-m-d H:i:s');
653
-           return $row;
652
+            $row['updated_at'] = date('Y-m-d H:i:s');
653
+            return $row;
654 654
         }
655 655
 
656 656
         /**
@@ -695,7 +695,7 @@  discard block
 block discarded – undo
695 695
          */
696 696
         public function removeProtectedTableColumns($row) {
697 697
             foreach ($this->protectedTableColumns as $attr) {
698
-               if (isset($row[$attr])) {
698
+                if (isset($row[$attr])) {
699 699
                     unset($row[$attr]);
700 700
                 }
701 701
             }
@@ -818,23 +818,23 @@  discard block
 block discarded – undo
818 818
             return false;
819 819
         }
820 820
 
821
-         /**
822
-         * Get the record return type array or object
823
-         * 
824
-         * @return string|boolean
825
-         */
821
+            /**
822
+             * Get the record return type array or object
823
+             * 
824
+             * @return string|boolean
825
+             */
826 826
         protected function getReturnType(){
827 827
             $type = false;
828 828
             if ($this->temporaryReturnRecordType == 'array') {
829
-               $type = 'array';
829
+                $type = 'array';
830 830
             }
831 831
             return $type;
832 832
         }
833 833
 
834
-         /**
835
-         * Check if soft delete is enable setting the condition
836
-         * @return object the current instance 
837
-         */
834
+            /**
835
+             * Check if soft delete is enable setting the condition
836
+             * @return object the current instance 
837
+             */
838 838
         protected function checkForSoftDelete(){
839 839
             if ($this->softDeleteStatus && $this->returnRecordWithDeleted !== true) {
840 840
                 $this->getQueryBuilder()->where(
@@ -845,16 +845,16 @@  discard block
 block discarded – undo
845 845
             return $this;
846 846
         }
847 847
 
848
-         /**
849
-         * Relate for "manyToOne" and "oneToMany"
850
-         * 
851
-         * @param  string $relationship the name of relation
852
-         * @param  string|array $options      the model and primary key values
853
-         * @param  object|array $row          the row to update
854
-         * @param  string $type the type can be "manyToOne", "oneToMany"
855
-         * 
856
-         * @return array|object the final row values
857
-         */
848
+            /**
849
+             * Relate for "manyToOne" and "oneToMany"
850
+             * 
851
+             * @param  string $relationship the name of relation
852
+             * @param  string|array $options      the model and primary key values
853
+             * @param  object|array $row          the row to update
854
+             * @param  string $type the type can be "manyToOne", "oneToMany"
855
+             * 
856
+             * @return array|object the final row values
857
+             */
858 858
         protected function relateOneToManyAndManyToOne($relationship, $options, $row, $type){
859 859
             if (in_array($relationship, $this->withs)) {
860 860
                 get_instance()->loader->model($options['model'], $relationship . '_model');
Please login to merge, or discard this patch.
core/bootstrap.php 1 patch
Indentation   +28 added lines, -28 removed lines patch added patch discarded remove patch
@@ -1,32 +1,32 @@  discard block
 block discarded – undo
1 1
 <?php
2 2
     defined('ROOT_PATH') || exit('Access denied');
3
-     /**
4
-     * TNH Framework
5
-     *
6
-     * A simple PHP framework using HMVC architecture
7
-     *
8
-     * This content is released under the MIT License (MIT)
9
-     *
10
-     * Copyright (c) 2017 TNH Framework
11
-     *
12
-     * Permission is hereby granted, free of charge, to any person obtaining a copy
13
-     * of this software and associated documentation files (the "Software"), to deal
14
-     * in the Software without restriction, including without limitation the rights
15
-     * to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
16
-     * copies of the Software, and to permit persons to whom the Software is
17
-     * furnished to do so, subject to the following conditions:
18
-     *
19
-     * The above copyright notice and this permission notice shall be included in all
20
-     * copies or substantial portions of the Software.
21
-     *
22
-     * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
23
-     * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
24
-     * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
25
-     * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
26
-     * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
27
-     * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
28
-     * SOFTWARE.
29
-     */
3
+        /**
4
+         * TNH Framework
5
+         *
6
+         * A simple PHP framework using HMVC architecture
7
+         *
8
+         * This content is released under the MIT License (MIT)
9
+         *
10
+         * Copyright (c) 2017 TNH Framework
11
+         *
12
+         * Permission is hereby granted, free of charge, to any person obtaining a copy
13
+         * of this software and associated documentation files (the "Software"), to deal
14
+         * in the Software without restriction, including without limitation the rights
15
+         * to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
16
+         * copies of the Software, and to permit persons to whom the Software is
17
+         * furnished to do so, subject to the following conditions:
18
+         *
19
+         * The above copyright notice and this permission notice shall be included in all
20
+         * copies or substantial portions of the Software.
21
+         *
22
+         * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
23
+         * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
24
+         * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
25
+         * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
26
+         * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
27
+         * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
28
+         * SOFTWARE.
29
+         */
30 30
 
31 31
     /**
32 32
      *  @file bootstrap.php
@@ -143,7 +143,7 @@  discard block
 block discarded – undo
143 143
     /**
144 144
      * Register controllers autoload function
145 145
      */
146
-     spl_autoload_register('autoload_controller');
146
+        spl_autoload_register('autoload_controller');
147 147
 
148 148
     /**
149 149
      * Loading Security class
Please login to merge, or discard this patch.
core/classes/Module.php 1 patch
Indentation   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -58,8 +58,8 @@  discard block
 block discarded – undo
58 58
          */
59 59
         public function add($name) {
60 60
             if (in_array($name, $this->list)) {
61
-               $this->logger->info('The module [' .$name. '] already added skipping.');
62
-               return $this;
61
+                $this->logger->info('The module [' .$name. '] already added skipping.');
62
+                return $this;
63 63
             }
64 64
             $this->list[] = $name;
65 65
             return $this;
@@ -90,10 +90,10 @@  discard block
 block discarded – undo
90 90
             $this->list = array();
91 91
         }
92 92
 
93
-         /**
94
-         * Get the list of module loaded
95
-         * @return array the module list
96
-         */
93
+            /**
94
+             * Get the list of module loaded
95
+             * @return array the module list
96
+             */
97 97
         public function getModuleList() {
98 98
             return $this->list;
99 99
         }
@@ -240,7 +240,7 @@  discard block
 block discarded – undo
240 240
             $this->logger->debug('Check if the application contains the modules ...');
241 241
             $dirList = glob(MODULE_PATH . '*', GLOB_ONLYDIR);
242 242
             if ($dirList !== false) {
243
-               $this->list = array_map('basename', $dirList);
243
+                $this->list = array_map('basename', $dirList);
244 244
             }
245 245
             if (!empty($this->list)) {
246 246
                 $this->logger->info('The application contains the module below [' . implode(', ', $this->list) . ']');
@@ -263,7 +263,7 @@  discard block
 block discarded – undo
263 263
             $filePath = MODULE_PATH . $module . DS . $type . DS . $classFile;
264 264
             if (file_exists($filePath)) {
265 265
                 $this->logger->info('Found class [' . $class . '] in module [' . $module . '] '
266
-                                     . 'for [' . $type . '] the file path is [' . $filePath . ']');
266
+                                        . 'for [' . $type . '] the file path is [' . $filePath . ']');
267 267
                 return $filePath;
268 268
             }
269 269
             $this->logger->info('Class [' . $class . '] does not exist in the module [' . $module . '] for [' . $type . ']');
Please login to merge, or discard this patch.
core/classes/database/DatabaseConnection.php 1 patch
Indentation   +39 added lines, -40 removed lines patch added patch discarded remove patch
@@ -42,28 +42,28 @@  discard block
 block discarded – undo
42 42
          */
43 43
         private $driver = null;
44 44
 
45
-         /**
46
-         * The database hostname
47
-         * @var string
48
-         */
45
+            /**
46
+             * The database hostname
47
+             * @var string
48
+             */
49 49
         private $hostname = null;
50 50
 
51
-          /**
52
-         * The database port
53
-         * @var integer
54
-         */
51
+            /**
52
+             * The database port
53
+             * @var integer
54
+             */
55 55
         private $port = null;
56 56
         
57
-         /**
58
-         * The database username
59
-         * @var string
60
-         */
57
+            /**
58
+             * The database username
59
+             * @var string
60
+             */
61 61
         private $username = null;
62 62
 
63
-         /**
64
-         * The database password
65
-         * @var string
66
-         */
63
+            /**
64
+             * The database password
65
+             * @var string
66
+             */
67 67
         private $password = null;
68 68
 
69 69
         /**
@@ -72,10 +72,10 @@  discard block
 block discarded – undo
72 72
          */
73 73
         private $databaseName = null;
74 74
 
75
-         /**
76
-         * The database charset
77
-         * @var string
78
-         */
75
+            /**
76
+             * The database charset
77
+             * @var string
78
+             */
79 79
         private $charset = null;
80 80
 
81 81
         /**
@@ -84,10 +84,10 @@  discard block
 block discarded – undo
84 84
          */
85 85
         private $collation = null;
86 86
 
87
-         /**
88
-         * The database tables prefix
89
-         * @var string
90
-         */
87
+            /**
88
+             * The database tables prefix
89
+             * @var string
90
+             */
91 91
         private $prefix = null;
92 92
 
93 93
         /**
@@ -114,11 +114,11 @@  discard block
 block discarded – undo
114 114
             }
115 115
         }
116 116
 
117
-         /**
118
-         * This is method is used to connect to database
119
-         * 
120
-         * @return boolean true in case of successfully connection false if error
121
-         */
117
+            /**
118
+             * This is method is used to connect to database
119
+             * 
120
+             * @return boolean true in case of successfully connection false if error
121
+             */
122 122
         public function connect() {
123 123
             try {
124 124
                 $options = array(
@@ -414,7 +414,7 @@  discard block
 block discarded – undo
414 414
             $this->populatePropertiesFromConfig();
415 415
 
416 416
             if (!empty($this->config)) {
417
-               //For logging
417
+                //For logging
418 418
                 $configInfo = $this->config;
419 419
                 //Hide password from log
420 420
                 $configInfo['password'] = string_hidden($this->getPassword());
@@ -423,11 +423,10 @@  discard block
 block discarded – undo
423 423
             return $this;
424 424
         }
425 425
 
426
-         /**
427
-         * Get the database configuration using the configuration file
428
-         
429
-         * @return array the database configuration from file
430
-         */
426
+            /**
427
+             * Get the database configuration using the configuration file
428
+             * @return array the database configuration from file
429
+             */
431 430
         public function getDatabaseConfigFromFile() {
432 431
             $db = array();
433 432
             if (file_exists(CONFIG_PATH . 'database.php')) {
@@ -437,11 +436,11 @@  discard block
 block discarded – undo
437 436
             return $db;
438 437
         }
439 438
 
440
-         /**
441
-         * Update the properties using the current database configuration
442
-         * 
443
-         * @return object the current instance
444
-         */
439
+            /**
440
+             * Update the properties using the current database configuration
441
+             * 
442
+             * @return object the current instance
443
+             */
445 444
         protected function populatePropertiesFromConfig() {
446 445
             foreach ($this->config as $key => $value) {
447 446
                 $setter = 'set' . ucfirst($key);
Please login to merge, or discard this patch.
core/classes/database/Database.php 1 patch
Indentation   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -111,7 +111,7 @@  discard block
 block discarded – undo
111 111
          */
112 112
         public function __construct(DatabaseConnection $connection = null) {
113 113
             parent::__construct();
114
-    		if ($connection !== null) {
114
+            if ($connection !== null) {
115 115
                 $this->connection = $connection;
116 116
             } 
117 117
         }
@@ -385,9 +385,9 @@  discard block
 block discarded – undo
385 385
                 $this->queryCount++;
386 386
                 
387 387
                 $queryResult = $this->queryRunner->setQuery($query)
388
-                                                 ->setReturnType($returnAsList)
389
-                                                 ->setReturnAsArray($returnAsArray)
390
-                                                 ->execute();
388
+                                                    ->setReturnType($returnAsList)
389
+                                                    ->setReturnAsArray($returnAsArray)
390
+                                                    ->execute();
391 391
 
392 392
                 if (is_object($queryResult)) {
393 393
                     $this->result  = $queryResult->getResult();
Please login to merge, or discard this patch.
core/classes/Response.php 1 patch
Indentation   +26 added lines, -26 removed lines patch added patch discarded remove patch
@@ -190,7 +190,7 @@  discard block
 block discarded – undo
190 190
                 $this->logger->info('Cannot find view [' . $view . '] in module [' . $module . '] '
191 191
                                     . 'using the default location');
192 192
             }
193
-			if (!$path) {
193
+            if (!$path) {
194 194
                 $path = $this->getDefaultFilePathForView($viewFile);
195 195
             }
196 196
             $this->logger->info('The view file path to be loaded is [' . $path . ']');
@@ -266,12 +266,12 @@  discard block
 block discarded – undo
266 266
             return $this->finalPageContent;
267 267
         }
268 268
 
269
-         /**
270
-         * Set the final page to be rendered
271
-         * @param string $finalPage the content of the final page
272
-         * 
273
-         * @return object
274
-         */
269
+            /**
270
+             * Set the final page to be rendered
271
+             * @param string $finalPage the content of the final page
272
+             * 
273
+             * @return object
274
+             */
275 275
         public function setFinalPageContent($finalPage) {
276 276
             $this->finalPageContent = $finalPage;
277 277
             return $this;
@@ -345,11 +345,11 @@  discard block
 block discarded – undo
345 345
         }
346 346
 
347 347
                 /**
348
-         * Dispatch the FINAL_VIEW_READY event
349
-         *             
350
-         * @return string|null the final view content after processing by each listener
351
-         * if they exists otherwise the same content will be returned
352
-         */
348
+                 * Dispatch the FINAL_VIEW_READY event
349
+                 *             
350
+                 * @return string|null the final view content after processing by each listener
351
+                 * if they exists otherwise the same content will be returned
352
+                 */
353 353
         protected function dispatchFinalViewEvent() {
354 354
             //dispatch
355 355
             $event = get_instance()->eventdispatcher->dispatch(
@@ -383,12 +383,12 @@  discard block
 block discarded – undo
383 383
         }
384 384
 
385 385
 
386
-         /**
387
-         * Return the default full file path for view
388
-         * @param  string $file    the filename
389
-         * 
390
-         * @return string|null          the full file path
391
-         */
386
+            /**
387
+             * Return the default full file path for view
388
+             * @param  string $file    the filename
389
+             * 
390
+             * @return string|null          the full file path
391
+             */
392 392
         protected function getDefaultFilePathForView($file){
393 393
             $searchDir = array(APPS_VIEWS_PATH, CORE_VIEWS_PATH);
394 394
             $fullFilePath = null;
@@ -421,7 +421,7 @@  discard block
 block discarded – undo
421 421
                 $headerModifiedSince = $globals->server('HTTP_IF_MODIFIED_SINCE');
422 422
                 if (!empty($headerModifiedSince) && $lastModified <= strtotime($headerModifiedSince)) {
423 423
                     $this->logger->info('The cache page content is not yet expire for the '
424
-                                         . 'URL [' . $this->currentUrl . '] send 304 header to browser');
424
+                                            . 'URL [' . $this->currentUrl . '] send 304 header to browser');
425 425
                     $this->sendHeaders(304);
426 426
                     return true;
427 427
                 }
@@ -436,8 +436,8 @@  discard block
 block discarded – undo
436 436
          */
437 437
         protected function sendCachePageContentToBrowser(&$cache) {
438 438
             $this->logger->info('The cache page content is expired or the browser does '
439
-                 . 'not send the HTTP_IF_MODIFIED_SINCE header for the URL [' . $this->currentUrl . '] '
440
-                 . 'send cache headers to tell the browser');
439
+                    . 'not send the HTTP_IF_MODIFIED_SINCE header for the URL [' . $this->currentUrl . '] '
440
+                    . 'send cache headers to tell the browser');
441 441
             $this->sendHeaders(200);
442 442
             //current page cache key
443 443
             $pageCacheKey = $this->currentUrlCacheKey;
@@ -576,18 +576,18 @@  discard block
 block discarded – undo
576 576
             }
577 577
         }
578 578
 
579
-         /**
580
-         * Set the mandory headers, like security, etc.
581
-         */
579
+            /**
580
+             * Set the mandory headers, like security, etc.
581
+             */
582 582
         protected function setRequiredHeaders() {
583 583
             $requiredHeaders = array(
584 584
                                 'X-XSS-Protection' => '1; mode=block',
585 585
                                 'X-Frame-Options'  => 'SAMEORIGIN'
586 586
                             );
587 587
             foreach ($requiredHeaders as $key => $value) {
588
-               if (!isset($this->headers[$key])) {
588
+                if (!isset($this->headers[$key])) {
589 589
                     $this->headers[$key] = $value;
590
-               } 
590
+                } 
591 591
             }
592 592
         }
593 593
     }
Please login to merge, or discard this patch.
core/classes/Loader.php 1 patch
Indentation   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -220,7 +220,7 @@  discard block
 block discarded – undo
220 220
             $moduleConfigPath = get_instance()->module->findConfigFullPath($filename, $module);
221 221
             if ($moduleConfigPath) {
222 222
                 $this->logger->info('Found config [' . $filename . '] from module [' . $module . '], '
223
-                                  . 'the file path is [' . $moduleConfigPath . '] we will used it');
223
+                                    . 'the file path is [' . $moduleConfigPath . '] we will used it');
224 224
                 $configFilePath = $moduleConfigPath;
225 225
             } else {
226 226
                 $this->logger->info('Cannot find config [' . $filename . '] from modules using the default location');
@@ -339,7 +339,7 @@  discard block
 block discarded – undo
339 339
         protected function getDefaultFilePathForFunctionLanguage($file, $type, $appLang = null){
340 340
             $searchDir = null;
341 341
             if ($type == 'function') {
342
-               $searchDir = array(FUNCTIONS_PATH, CORE_FUNCTIONS_PATH);
342
+                $searchDir = array(FUNCTIONS_PATH, CORE_FUNCTIONS_PATH);
343 343
             }
344 344
             else if ($type == 'language') {
345 345
                 $searchDir = array(APP_LANG_PATH, CORE_LANG_PATH);
@@ -507,7 +507,7 @@  discard block
 block discarded – undo
507 507
             $moduleLibraryPath = get_instance()->module->findLibraryFullPath($class, $module);
508 508
             if ($moduleLibraryPath) {
509 509
                 $this->logger->info('Found library [' . $class . '] from module [' . $module . '], the '
510
-                                   . 'file path is [' . $moduleLibraryPath . '] we will used it');
510
+                                    . 'file path is [' . $moduleLibraryPath . '] we will used it');
511 511
                 $libraryFilePath = $moduleLibraryPath;
512 512
             } else {
513 513
                 $this->logger->info('Cannot find library [' . $class . '] from modules using the default location');
Please login to merge, or discard this patch.
core/classes/Url.php 1 patch
Indentation   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -128,7 +128,7 @@
 block discarded – undo
128 128
                     break;
129 129
                 }
130 130
             }
131
-			$port = get_instance()->request->server('SERVER_PORT');
131
+            $port = get_instance()->request->server('SERVER_PORT');
132 132
             
133 133
             if ($port && !in_array($port, array(80, 443))) {
134 134
                 $domain .= ':' . $port;
Please login to merge, or discard this patch.