tnhfw / tnh-fw

core/classes/database/Database.php

Indentation +4 added lines, -4 removed lines

@@ -111,7 +111,7 @@ 
          */
         public function __construct(DatabaseConnection $connection = null) {
             parent::__construct();
-    		if ($connection !== null) {
+            if ($connection !== null) {
                 $this->connection = $connection;
             } 
         }
@@ -385,9 +385,9 @@ 
                 $this->queryCount++;
                 
                 $queryResult = $this->queryRunner->setQuery($query)
-                                                 ->setReturnType($returnAsList)
-                                                 ->setReturnAsArray($returnAsArray)
-                                                 ->execute();
+                                                    ->setReturnType($returnAsList)
+                                                    ->setReturnAsArray($returnAsArray)
+                                                    ->execute();
 
                 if (is_object($queryResult)) {
                     $this->result  = $queryResult->getResult();

core/classes/Response.php

Indentation +26 added lines, -26 removed lines

@@ -190,7 +190,7 @@ 
                 $this->logger->info('Cannot find view [' . $view . '] in module [' . $module . '] '
                                     . 'using the default location');
             }
-			if (!$path) {
+            if (!$path) {
                 $path = $this->getDefaultFilePathForView($viewFile);
             }
             $this->logger->info('The view file path to be loaded is [' . $path . ']');
@@ -266,12 +266,12 @@ 
             return $this->finalPageContent;
         }
 
-         /**
-         * Set the final page to be rendered
-         * @param string $finalPage the content of the final page
-         * 
-         * @return object
-         */
+            /**
+             * Set the final page to be rendered
+             * @param string $finalPage the content of the final page
+             * 
+             * @return object
+             */
         public function setFinalPageContent($finalPage) {
             $this->finalPageContent = $finalPage;
             return $this;
@@ -345,11 +345,11 @@ 
         }
 
                 /**
-         * Dispatch the FINAL_VIEW_READY event
-         *             
-         * @return string|null the final view content after processing by each listener
-         * if they exists otherwise the same content will be returned
-         */
+                 * Dispatch the FINAL_VIEW_READY event
+                 *             
+                 * @return string|null the final view content after processing by each listener
+                 * if they exists otherwise the same content will be returned
+                 */
         protected function dispatchFinalViewEvent() {
             //dispatch
             $event = get_instance()->eventdispatcher->dispatch(
@@ -383,12 +383,12 @@ 
         }
 
 
-         /**
-         * Return the default full file path for view
-         * @param  string $file    the filename
-         * 
-         * @return string|null          the full file path
-         */
+            /**
+             * Return the default full file path for view
+             * @param  string $file    the filename
+             * 
+             * @return string|null          the full file path
+             */
         protected function getDefaultFilePathForView($file){
             $searchDir = array(APPS_VIEWS_PATH, CORE_VIEWS_PATH);
             $fullFilePath = null;
@@ -421,7 +421,7 @@ 
                 $headerModifiedSince = $globals->server('HTTP_IF_MODIFIED_SINCE');
                 if (!empty($headerModifiedSince) && $lastModified <= strtotime($headerModifiedSince)) {
                     $this->logger->info('The cache page content is not yet expire for the '
-                                         . 'URL [' . $this->currentUrl . '] send 304 header to browser');
+                                            . 'URL [' . $this->currentUrl . '] send 304 header to browser');
                     $this->sendHeaders(304);
                     return true;
                 }
@@ -436,8 +436,8 @@ 
          */
         protected function sendCachePageContentToBrowser(&$cache) {
             $this->logger->info('The cache page content is expired or the browser does '
-                 . 'not send the HTTP_IF_MODIFIED_SINCE header for the URL [' . $this->currentUrl . '] '
-                 . 'send cache headers to tell the browser');
+                    . 'not send the HTTP_IF_MODIFIED_SINCE header for the URL [' . $this->currentUrl . '] '
+                    . 'send cache headers to tell the browser');
             $this->sendHeaders(200);
             //current page cache key
             $pageCacheKey = $this->currentUrlCacheKey;
@@ -576,18 +576,18 @@ 
             }
         }
 
-         /**
-         * Set the mandory headers, like security, etc.
-         */
+            /**
+             * Set the mandory headers, like security, etc.
+             */
         protected function setRequiredHeaders() {
             $requiredHeaders = array(
                                 'X-XSS-Protection' => '1; mode=block',
                                 'X-Frame-Options'  => 'SAMEORIGIN'
                             );
             foreach ($requiredHeaders as $key => $value) {
-               if (!isset($this->headers[$key])) {
+                if (!isset($this->headers[$key])) {
                     $this->headers[$key] = $value;
-               } 
+                } 
             }
         }
     }

Spacing +4 added lines, -4 removed lines

@@ -249,7 +249,7 @@ 
             $cacheInfo = $cache->getInfo($pageCacheKey);
             if ($cacheInfo) {
                 $status = $this->sendCacheNotYetExpireInfoToBrowser($cacheInfo);
-                if($status === false) {
+                if ($status === false) {
                     return $this->sendCachePageContentToBrowser($cache);
                 }
                 return true;
@@ -323,7 +323,7 @@ 
          */
         public function sendError(array $data = array()) {
             $path = CORE_VIEWS_PATH . 'errors.php';
-            if(file_exists($path)){
+            if (file_exists($path)) {
                 //compress the output if is available
                 $compressOutputHandler = $this->getCompressOutputHandler();
                 ob_start($compressOutputHandler);
@@ -335,7 +335,7 @@ 
                 echo $content;
             }
             //@codeCoverageIgnoreStart
-            else{
+            else {
                 //can't use show_error() at this time because 
                 //some dependencies not yet loaded
                 set_http_status_header(503);
@@ -389,7 +389,7 @@ 
          * 
          * @return string|null          the full file path
          */
-        protected function getDefaultFilePathForView($file){
+        protected function getDefaultFilePathForView($file) {
             $searchDir = array(APPS_VIEWS_PATH, CORE_VIEWS_PATH);
             $fullFilePath = null;
             foreach ($searchDir as $dir) {

core/classes/EventDispatcher.php

Indentation +1 added lines, -1 removed lines

@@ -156,7 +156,7 @@
                 return;
             } 
             $this->logger->info('Found the registered event listener for the '
-                                 . 'event [' . $event->name . '] the list are: ' . stringfy_vars($list));
+                                    . 'event [' . $event->name . '] the list are: ' . stringfy_vars($list));
             foreach ($list as $listener) {
                 $result = call_user_func_array($listener, array($event));
                 if ($eBackup->returnBack === true) {

core/classes/Log.php

Spacing +1 added lines, -1 removed lines

@@ -214,7 +214,7 @@
         protected function getLogDebugBacktraceInfo() {
             $dtrace = debug_backtrace(DEBUG_BACKTRACE_IGNORE_ARGS);
             $i = 0;
-            while ($dtrace[$i]['file'] == __FILE__ ) {
+            while ($dtrace[$i]['file'] == __FILE__) {
                 $i++;
             } 
             $fileInfo = $dtrace[$i];

Indentation +8 added lines, -8 removed lines

@@ -169,7 +169,7 @@ 
             $this->log(self::DEBUG, $message);
         } 
 		
-		/**
+        /**
          * Logs with an arbitrary level.
          *
          * @param  integer|string $level   the log level in integer or string format,
@@ -187,11 +187,11 @@ 
                 //NOTE: here need put the show_error() "logging" to false 
                 //to prevent self function loop call
                 show_error('Invalid config log level [' . $configLogLevel . '], '
-                           . 'the value must be one of the following: ' 
-                           . implode(', ', array_map('strtoupper', self::$validConfigLevel))
-                           , 'Log Config Error', 
-                           $logging = false
-                       );
+                            . 'the value must be one of the following: ' 
+                            . implode(', ', array_map('strtoupper', self::$validConfigLevel))
+                            , 'Log Config Error', 
+                            $logging = false
+                        );
                 return;	
             }
 			
@@ -261,7 +261,7 @@ 
         protected function levelCanSaveLog($level) {
             $result = true;
             $configLogLevel = get_config('log_level');
-             //check if can save log regarding the log level configuration
+                //check if can save log regarding the log level configuration
             $configLevel = self::getLevelValue($configLogLevel);
             if ($configLevel > $level) {
                 //can't log
@@ -341,7 +341,7 @@ 
                 //NOTE: here need put the show_error() "logging" to false 
                 //to prevent self function loop call
                 show_error('Error : the log dir does not exist or is not writable',
-                           'Log directory error', $logging = false);
+                            'Log directory error', $logging = false);
             }
 			
             $path = $logSavePath . 'logs-' . date('Y-m-d') . '.log';

core/classes/Router.php

Indentation +15 added lines, -15 removed lines

@@ -277,14 +277,14 @@ 
         }
 
         /**
-        * Setting the route configuration using the configuration file 
-        * and additional configuration from param
-        * @param array $overwriteConfig the additional configuration 
-        * to overwrite with the existing one
-        * @param boolean $useConfigFile whether to use route configuration file
-        * 
-        * @return object
-        */
+         * Setting the route configuration using the configuration file 
+         * and additional configuration from param
+         * @param array $overwriteConfig the additional configuration 
+         * to overwrite with the existing one
+         * @param boolean $useConfigFile whether to use route configuration file
+         * 
+         * @return object
+         */
         public function setRouteConfiguration(array $overwriteConfig = array(), $useConfigFile = true) {
             $route = array();
             if ($useConfigFile && file_exists(CONFIG_PATH . 'routes.php')) {
@@ -365,7 +365,7 @@ 
             //the URL like http://domain.com/module/controller/method/arg1/arg2
             if (!$this->controller) {
                 $this->logger->info('Cannot determine the routing information ' 
-                       . 'using the predefined routes configuration, will use the request URI parameters');
+                        . 'using the predefined routes configuration, will use the request URI parameters');
                 //determine route parameters using the route URI param
                 $this->determineRouteParamsFromRequestUri();
             }
@@ -478,11 +478,11 @@ 
             }
         }
 
-         /**
-         * Remove the URL suffix and query string values if exists
-         * @param  string $uri the route URI to process
-         * @return string      the final route uri after processed
-         */
+            /**
+             * Remove the URL suffix and query string values if exists
+             * @param  string $uri the route URI to process
+             * @return string      the final route uri after processed
+             */
         protected function removeSuffixAndQueryStringFromUri($uri) {
             $this->logger->debug('Check if URL suffix is enabled in the configuration');
             //remove url suffix from the request URI
@@ -596,7 +596,7 @@ 
                 $this->logger->info(
                                     'After loop in predefined routes configuration,'
                                     . 'the module name is set but the controller is not set,' 
-									. 'so we will use module as the controller'
+                                    . 'so we will use module as the controller'
                                 );
                 $this->controller = $this->module;
             }

Spacing +6 added lines, -6 removed lines

@@ -450,7 +450,7 @@ 
          * Remove the DOCUMENT_ROOT and front controller from segments if exists
          * @return void
          */
-        protected function removeDocumentRootFrontControllerFromSegments(){
+        protected function removeDocumentRootFrontControllerFromSegments() {
             $segment = $this->segments;
             $globals = & class_loader('GlobalVar', 'classes');
             $rootFolder = substr($globals->server('SCRIPT_NAME'), 0, strpos(
@@ -581,7 +581,7 @@ 
                     break;
                 }
             }
-            if($findIndex !== -1){
+            if ($findIndex !== -1) {
                 //$args[0] => full string captured by preg_match
                 //$args[1], $args[2], $args[n] => contains the value of 
                 //(:num), (:alpha), (:alnum), (:any)
@@ -606,7 +606,7 @@ 
          * Find file path of the current controller using the current module
          * @return boolean true if the file path is found otherwise false.
          */
-        protected function findControllerFullPathUsingCurrentModule(){
+        protected function findControllerFullPathUsingCurrentModule() {
             $path = $this->moduleInstance->findControllerFullPath(ucfirst($this->controller), $this->module);
             if (!$path) {
                 $this->logger->info('The controller [' . $this->controller . '] not ' 
@@ -623,7 +623,7 @@ 
          * or the current request does not use module
          * @return void
          */
-        protected function setRouteParamsIfNoModuleOrNotFound(){
+        protected function setRouteParamsIfNoModuleOrNotFound() {
             $segment = $this->segments;
             //controller
             if (isset($segment[0])) {
@@ -644,7 +644,7 @@ 
          * or the current request use module
          * @return void
          */
-        protected function setRouteParamsIfAppHasModuleOrFound(){
+        protected function setRouteParamsIfAppHasModuleOrFound() {
             //get the module list
             $modules = $this->moduleInstance->getModuleList();
             $segment = $this->segments;
@@ -657,7 +657,7 @@ 
                     $this->controller = $segment[0];
 
                     //check if the request use the same module name and controller
-                    if($this->findControllerFullPathUsingCurrentModule()){
+                    if ($this->findControllerFullPathUsingCurrentModule()) {
                         array_shift($segment);
                     }
                 }

app/config/config.php

Indentation +34 added lines, -34 removed lines

@@ -134,26 +134,26 @@ 
 	*/
 	
     /** 
-    * The log level
-    *
-    * The valid log level are: OFF, NONE, DEBUG, INFO, NOTICE, WARNING, ERROR, CRITICAL, ALERT, EMERGENCY
-    *
-    * 'OFF' or 'NONE' = do not save log
-    * 'EMERGENCY' = enable log for emergency level and above (EMERGENCY)
-    * 'ALERT' = enable log for alert level and above (ALERT, EMERGENCY)
-    * 'CRITICAL' = enable log for critical level and above (CRITICAL, ALERT, EMERGENCY)
-    * 'ERROR' = enable log for error level and above (ERROR, CRITICAL, ALERT, EMERGENCY)
-    * 'WARNING' = enable log for warning level and above (WARNING, ERROR, CRITICAL, ALERT, EMERGENCY)
-    * 'NOTICE' = enable log for notice level and above (NOTICE, WARNING, ERROR, CRITICAL, ALERT, EMERGENCY)
-    * 'INFO' = enable log for info level and above (INFO, NOTICE, WARNING, ERROR, CRITICAL, ALERT, EMERGENCY)
-    * 'DEBUG' = enable log for debug level and above (DEBUG, INFO, NOTICE, WARNING, ERROR, CRITICAL, ALERT, EMERGENCY)
-    *
-    * The default value is NONE if the config value is: null, '', 0, false
-    * 
-    * Note: in production environment it's recommand to set the log level to 'WARNING' or 'ERROR' if not, in small
-    * time the log file size will increase very fast and will cost the application performance
-    * and also the filesystem usage of your server.
-    */
+     * The log level
+     *
+     * The valid log level are: OFF, NONE, DEBUG, INFO, NOTICE, WARNING, ERROR, CRITICAL, ALERT, EMERGENCY
+     *
+     * 'OFF' or 'NONE' = do not save log
+     * 'EMERGENCY' = enable log for emergency level and above (EMERGENCY)
+     * 'ALERT' = enable log for alert level and above (ALERT, EMERGENCY)
+     * 'CRITICAL' = enable log for critical level and above (CRITICAL, ALERT, EMERGENCY)
+     * 'ERROR' = enable log for error level and above (ERROR, CRITICAL, ALERT, EMERGENCY)
+     * 'WARNING' = enable log for warning level and above (WARNING, ERROR, CRITICAL, ALERT, EMERGENCY)
+     * 'NOTICE' = enable log for notice level and above (NOTICE, WARNING, ERROR, CRITICAL, ALERT, EMERGENCY)
+     * 'INFO' = enable log for info level and above (INFO, NOTICE, WARNING, ERROR, CRITICAL, ALERT, EMERGENCY)
+     * 'DEBUG' = enable log for debug level and above (DEBUG, INFO, NOTICE, WARNING, ERROR, CRITICAL, ALERT, EMERGENCY)
+     *
+     * The default value is NONE if the config value is: null, '', 0, false
+     * 
+     * Note: in production environment it's recommand to set the log level to 'WARNING' or 'ERROR' if not, in small
+     * time the log file size will increase very fast and will cost the application performance
+     * and also the filesystem usage of your server.
+     */
     $config['log_level'] = 'NONE';
 
 
@@ -180,20 +180,20 @@ 
     $config['log_logger_name'] = array();
 
     /**
-    * The logger name custom level to use for the log
-    * 
-    * If this config is set so means the logger level will be used to overwrite 
-    * the default log level configuration above. 
-    *
-    * Example:
-    * $config['log_logger_name_level'] = array('MY_LOGGER1' => 'WARNING'); 
-    * So if $config['log_level'] = 'ERROR' but all log messages with "MY_LOGGER1" as logger name
-    *  will be saved for WARNING message and above
-    *  Note: You can also use an regular expression for the logger name.
-    *  Example:
-    *  $config['log_logger_name_level'] = array('^Class::Con(.*)' => 'info');
-    *  So all logger name like "Class::Config", "Class::Cookie", etc. will be match
-    */  
+     * The logger name custom level to use for the log
+     * 
+     * If this config is set so means the logger level will be used to overwrite 
+     * the default log level configuration above. 
+     *
+     * Example:
+     * $config['log_logger_name_level'] = array('MY_LOGGER1' => 'WARNING'); 
+     * So if $config['log_level'] = 'ERROR' but all log messages with "MY_LOGGER1" as logger name
+     *  will be saved for WARNING message and above
+     *  Note: You can also use an regular expression for the logger name.
+     *  Example:
+     *  $config['log_logger_name_level'] = array('^Class::Con(.*)' => 'info');
+     *  So all logger name like "Class::Config", "Class::Cookie", etc. will be match
+     */  
     $config['log_logger_name_level'] = array();
 	
 	

core/classes/Config.php

Indentation +9 added lines, -9 removed lines

@@ -46,9 +46,9 @@ 
             if ($init) {
                 $this->init();
                 //@codeCoverageIgnoreStart
-                 if (ENVIRONMENT == 'production' && in_array(strtolower($this->config['log_level']), array('debug', 'info', 'all'))) {
+                    if (ENVIRONMENT == 'production' && in_array(strtolower($this->config['log_level']), array('debug', 'info', 'all'))) {
                     $this->logger->warning('You are in production environment, please set '
-                                           . 'log level to WARNING, ERROR, CRITICAL, ALERT, EMERGENCY to increase the application performance');
+                                            . 'log level to WARNING, ERROR, CRITICAL, ALERT, EMERGENCY to increase the application performance');
                 }
                 //@codeCoverageIgnoreEnd
             }
@@ -146,7 +146,7 @@ 
             if (empty($this->config['base_url'])) {
                 if (ENVIRONMENT == 'production') {
                     $this->logger->warning('Application base URL is not set or invalid, please'
-                                           . ' set application base URL to increase the application loading time');
+                                            . ' set application base URL to increase the application loading time');
                 }
                 $baseUrl = null;
                 $protocol = 'http';
@@ -173,7 +173,7 @@ 
                                                                     );
                 } else {
                     $this->logger->warning('Can not determine the application '
-                                           . 'base URL automatically, use http://localhost as default');
+                                            . 'base URL automatically, use http://localhost as default');
                     $baseUrl = 'http://localhost/';
                 }
                 $this->config['base_url'] = $baseUrl;
@@ -182,11 +182,11 @@ 
         }
          
         /**
-        * Return the server port using variable
-        *
-        * @codeCoverageIgnore
-        * @return string
-        */
+         * Return the server port using variable
+         *
+         * @codeCoverageIgnore
+         * @return string
+         */
         protected function getServerPort() {
             $globals = & class_loader('GlobalVar', 'classes');
             $serverPort = $globals->server('SERVER_PORT');

core/common.php

Spacing +2 added lines, -2 removed lines

@@ -147,7 +147,7 @@ 
         static $cfg;
         if (empty($cfg)) {
             $cfg[0] = & load_configurations();
-            if(! is_array($cfg[0])){
+            if (!is_array($cfg[0])) {
                 $cfg[0] = array();
             }
         }
@@ -197,7 +197,7 @@ 
             set_http_status_header(500);
         }
         $errorType = 'Error';
-        $errorsType = array (
+        $errorsType = array(
                 E_ERROR              => 'Error',
                 E_WARNING            => 'Warning',
                 E_PARSE              => 'Parsing Error',

Indentation +13 added lines, -13 removed lines

@@ -91,7 +91,7 @@ 
 		
         //record the class instance
         $classes[$class] = isset($params) ? new $class($params) : new $class();
-		return $classes[$class];
+        return $classes[$class];
     }
 
     /**
@@ -179,17 +179,17 @@ 
         die();
     }
 
-     /**
-     *  Function defined for PHP error message handling
-     *              
-     *  @param int $errno the type of error for example: E_USER_ERROR, E_USER_WARNING, etc.
-     *  @param string $errstr the error message
-     *  @param string $errfile the file where the error occurred
-     *  @param int $errline the line number where the error occurred
-     *  @codeCoverageIgnore
-     *  
-     *  @return boolean 
-     */
+        /**
+         *  Function defined for PHP error message handling
+         *              
+         *  @param int $errno the type of error for example: E_USER_ERROR, E_USER_WARNING, etc.
+         *  @param string $errstr the error message
+         *  @param string $errfile the file where the error occurred
+         *  @param int $errline the line number where the error occurred
+         *  @codeCoverageIgnore
+         *  
+         *  @return boolean 
+         */
     function fw_error_handler($errno, $errstr, $errfile, $errline) {
         $isError = (((E_ERROR | E_COMPILE_ERROR | E_CORE_ERROR | E_USER_ERROR) & $errno) === $errno);
         if ($isError) {
@@ -212,7 +212,7 @@ 
                 E_RECOVERABLE_ERROR  => 'Catchable Fatal Error'
         );
         if (isset($errorsType[$errno])) {
-           $errorType = $errorsType[$errno];
+            $errorType = $errorsType[$errno];
         }
         $errorText = 'An error is occurred in the file ' . $errfile . ' at line ' . $errline . ' raison : ' . $errstr;
         if ((error_reporting() & $errno) !== $errno) {

core/classes/cache/ApcCache.php

Indentation +3 added lines, -3 removed lines

@@ -33,7 +33,7 @@ 
         /**
          * Construct new ApcCache instance
          */
-	public function __construct() {
+    public function __construct() {
             parent::__construct();
             if (!$this->isSupported()) {
                 show_error('The cache for APC[u] driver is not available. Check if APC[u] extension is loaded and enabled.');
@@ -79,7 +79,7 @@ 
         public function set($key, $data, $ttl = 0) {
             $expire = time() + $ttl;
             $this->logger->debug('Setting cache data for key [' . $key . '], time to live [' . $ttl . '], '
-                                 . 'expire at [' . date('Y-m-d H:i:s', $expire) . ']');
+                                    . 'expire at [' . date('Y-m-d H:i:s', $expire) . ']');
             $result = apc_store($key, $data, $ttl);
             if ($result === false) {
                 $this->logger->error('Can not save cache data for the key [' . $key . '], return false');
@@ -127,7 +127,7 @@ 
                             'mtime'  => $cacheInfos['creation_time'],
                             'expire' => $cacheInfos['creation_time'] + $cacheInfos['ttl'],
                             'ttl'    => $cacheInfos['ttl']
-                           );
+                            );
             } 
             $this->logger->info('This cache does not exists skipping');
             return false;