Passed
Push — 1.0.0-dev ( a1ce08...4f1d46 )
by nguereza
10:50
created
core/classes/Response.php 1 patch
Indentation   +18 added lines, -18 removed lines patch added patch discarded remove patch
@@ -185,7 +185,7 @@  discard block
 block discarded – undo
185 185
             } else {
186 186
                 $logger->info('Cannot find view [' . $view . '] in module [' . $module . '] using the default location');
187 187
             }
188
-			if (!$path) {
188
+            if (!$path) {
189 189
                 $path = $this->getDefaultFilePathForView($viewFile);
190 190
             }
191 191
             $logger->info('The view file path to be loaded is [' . $path . ']');
@@ -273,12 +273,12 @@  discard block
 block discarded – undo
273 273
             return $this->_pageRender;
274 274
         }
275 275
 
276
-         /**
277
-         * Set the final page to be rendered
278
-         * @param string $finalPage the content of the final page
279
-         * 
280
-         * @return object
281
-         */
276
+            /**
277
+             * Set the final page to be rendered
278
+             * @param string $finalPage the content of the final page
279
+             * 
280
+             * @return object
281
+             */
282 282
         public function setFinalPageContent($finalPage) {
283 283
             $this->_pageRender = $finalPage;
284 284
             return $this;
@@ -351,12 +351,12 @@  discard block
 block discarded – undo
351 351
             ob_end_flush();
352 352
         }
353 353
 
354
-         /**
355
-         * Return the default full file path for view
356
-         * @param  string $file    the filename
357
-         * 
358
-         * @return string|null          the full file path
359
-         */
354
+            /**
355
+             * Return the default full file path for view
356
+             * @param  string $file    the filename
357
+             * 
358
+             * @return string|null          the full file path
359
+             */
360 360
         protected static function getDefaultFilePathForView($file){
361 361
             $searchDir = array(APPS_VIEWS_PATH, CORE_VIEWS_PATH);
362 362
             $fullFilePath = null;
@@ -548,18 +548,18 @@  discard block
 block discarded – undo
548 548
             }
549 549
         }
550 550
 
551
-         /**
552
-         * Set the mandory headers, like security, etc.
553
-         */
551
+            /**
552
+             * Set the mandory headers, like security, etc.
553
+             */
554 554
         protected static function setRequiredHeaders() {
555 555
             $requiredHeaders = array(
556 556
                                 'X-XSS-Protection' => '1; mode=block',
557 557
                                 'X-Frame-Options'  => 'SAMEORIGIN'
558 558
                             );
559 559
             foreach ($requiredHeaders as $key => $value) {
560
-               if (!isset(self::$headers[$key])) {
560
+                if (!isset(self::$headers[$key])) {
561 561
                     self::$headers[$key] = $value;
562
-               } 
562
+                } 
563 563
             }
564 564
         }
565 565
     }
Please login to merge, or discard this patch.
core/classes/Router.php 2 patches
Indentation   +12 added lines, -12 removed lines patch added patch discarded remove patch
@@ -241,12 +241,12 @@  discard block
 block discarded – undo
241 241
         }
242 242
 
243 243
         /**
244
-        * Setting the route configuration using the configuration file and additional configuration from param
245
-        * @param array $overwriteConfig the additional configuration to overwrite with the existing one
246
-        * @param boolean $useConfigFile whether to use route configuration file
247
-        * 
248
-        * @return object
249
-        */
244
+         * Setting the route configuration using the configuration file and additional configuration from param
245
+         * @param array $overwriteConfig the additional configuration to overwrite with the existing one
246
+         * @param boolean $useConfigFile whether to use route configuration file
247
+         * 
248
+         * @return object
249
+         */
250 250
         public function setRouteConfiguration(array $overwriteConfig = array(), $useConfigFile = true) {
251 251
             $route = array();
252 252
             if ($useConfigFile && file_exists(CONFIG_PATH . 'routes.php')) {
@@ -423,11 +423,11 @@  discard block
 block discarded – undo
423 423
             }
424 424
         }
425 425
 
426
-         /**
427
-         * Remove the URL suffix and query string values if exists
428
-         * @param  string $uri the route URI to process
429
-         * @return string      the final route uri after processed
430
-         */
426
+            /**
427
+             * Remove the URL suffix and query string values if exists
428
+             * @param  string $uri the route URI to process
429
+             * @return string      the final route uri after processed
430
+             */
431 431
         protected function removeSuffixAndQueryStringFromUri($uri) {
432 432
             $this->logger->debug('Check if URL suffix is enabled in the configuration');
433 433
             //remove url suffix from the request URI
@@ -540,7 +540,7 @@  discard block
 block discarded – undo
540 540
                 $this->logger->info(
541 541
                                     'After loop in predefined routes configuration,'
542 542
                                     . 'the module name is set but the controller is not set,' 
543
-									. 'so we will use module as the controller'
543
+                                    . 'so we will use module as the controller'
544 544
                                 );
545 545
                 $this->controller = $this->module;
546 546
             }
Please login to merge, or discard this patch.
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -407,7 +407,7 @@  discard block
 block discarded – undo
407 407
          * Remove the DOCUMENT_ROOT and front controller from segments if exists
408 408
          * @return void
409 409
          */
410
-        protected function removeDocumentRootFrontControllerFromSegments(){
410
+        protected function removeDocumentRootFrontControllerFromSegments() {
411 411
             $segment = $this->segments;
412 412
             $baseUrl = get_config('base_url');
413 413
             //check if the app is not in DOCUMENT_ROOT
@@ -525,7 +525,7 @@  discard block
 block discarded – undo
525 525
                     break;
526 526
                 }
527 527
             }
528
-            if($findIndex !== -1){
528
+            if ($findIndex !== -1) {
529 529
                 //$args[0] => full string captured by preg_match
530 530
                 //$args[1], $args[2], $args[n] => contains the value of 
531 531
                 //(:num), (:alpha), (:alnum), (:any)
@@ -550,7 +550,7 @@  discard block
 block discarded – undo
550 550
          * Find file path of the current controller using the current module
551 551
          * @return boolean true if the file path is found otherwise false.
552 552
          */
553
-        protected function findControllerFullPathUsingCurrentModule(){
553
+        protected function findControllerFullPathUsingCurrentModule() {
554 554
             $path = Module::findControllerFullPath(ucfirst($this->controller), $this->module);
555 555
             if (!$path) {
556 556
                 $this->logger->info('The controller [' . $this->controller . '] not found in the module, may be will use the module [' . $this->module . '] as controller');
@@ -566,7 +566,7 @@  discard block
 block discarded – undo
566 566
          * or the current request does not use module
567 567
          * @return void
568 568
          */
569
-        protected function setRouteParamsIfNoModuleOrNotFound(){
569
+        protected function setRouteParamsIfNoModuleOrNotFound() {
570 570
             $segment = $this->segments;
571 571
             //controller
572 572
             if (isset($segment[0])) {
@@ -587,7 +587,7 @@  discard block
 block discarded – undo
587 587
          * or the current request use module
588 588
          * @return void
589 589
          */
590
-        protected function setRouteParamsIfAppHasModuleOrFound(){
590
+        protected function setRouteParamsIfAppHasModuleOrFound() {
591 591
             //get the module list
592 592
             $modules = Module::getModuleList();
593 593
             $segment = $this->segments;
@@ -600,7 +600,7 @@  discard block
 block discarded – undo
600 600
                     $this->controller = $segment[0];
601 601
 
602 602
                     //check if the request use the same module name and controller
603
-                    if($this->findControllerFullPathUsingCurrentModule()){
603
+                    if ($this->findControllerFullPathUsingCurrentModule()) {
604 604
                         array_shift($segment);
605 605
                     }
606 606
                 }
Please login to merge, or discard this patch.