las93 / venus3

bundles/core/Config.php

Doc Comments +3 added lines, -3 removed lines

@@ -196,8 +196,8 @@ 
 	 *
 	 * @access private
 	 * @param  string $sFileToMerge file to get
-	 * @param  object $base base
-	 * @return object
+	 * @param  stdClass $base base
+	 * @return stdClass
 	 */
 	private static function  _mergeAndGetConf(string $sFileToMerge, \StdClass $base) : \StdClass
 	{
@@ -222,7 +222,7 @@ 
 	 *
 	 * @access private
 	 * @param  $oConfFiles
-	 * @param  StdClass $base
+	 * @param  stdClass $base
 	 * @return multitype:array multitype:array
 	 */
 	private static function _recursiveGet($oConfFiles, StdClass $base) : array

Braces +6 added lines, -12 removed lines

@@ -50,8 +50,7 @@ 
 	 */
 	public static function get(string $sName, string $sPortal = null, bool $bNoDoRedirect = false)
 	{
-	    if ($bNoDoRedirect === true) { $sNameCache = $sName.'_true'; }
-	    else { $sNameCache = $sName; }
+	    if ($bNoDoRedirect === true) { $sNameCache = $sName.'_true'; } else { $sNameCache = $sName; }
 	    
 		if ($sPortal === null || !is_string($sPortal)) {
 		    
@@ -59,8 +58,7 @@ 
 
 				$sPortal = PORTAL;
 				$aDirectories = array($sPortal);
-			}
-		    else {
+			} else {
 
 				$sPortal = '';
 				$aDirectories = scandir(str_replace('core', 'src', __DIR__));
@@ -187,8 +185,7 @@ 
 	{
 	    $oConfig = self::get($sName, null, true);
 
-	    if (isset($oConfig->redirect)) { return $oConfig->redirect; }
-	    else { return PORTAL; }
+	    if (isset($oConfig->redirect)) { return $oConfig->redirect; } else { return PORTAL; }
 	}
 
 	/**
@@ -207,8 +204,7 @@ 
 
 			list($oConfFiles, $base) = self::_recursiveGet($oConfFiles, $base);
 			return $base;
-		}
-		else {
+		} else {
 
 			echo "The Json ".$sFileToMerge." has an error! Please verify!\n";
 			$oDebug = Debug::getInstance();
@@ -232,12 +228,10 @@ 
 			if (is_object($oConfFiles) && is_object($base) && !isset($base->$sKey)) {
 
 				$base->$sKey = $oConfFiles->$sKey;
-			}
-			else if (is_array($oConfFiles) && is_array($base) && !isset($base[$sKey])) {
+			} else if (is_array($oConfFiles) && is_array($base) && !isset($base[$sKey])) {
 
 				$base[$sKey] = $oConfFiles[$sKey];
-			}
-			else if (!isset($base->$sKey) && is_array($mOne)) {
+			} else if (!isset($base->$sKey) && is_array($mOne)) {
 
 				$base->$sKey = new \StdClass;
 				list($oConfFiles, $base) = self::_recursiveGet($mOne, $base->$sKey);

bundles/core/Mother.php

Doc Comments +2 added lines, -2 removed lines

@@ -118,7 +118,7 @@ 
 	 *
 	 * @access public
 	 * @param  mixed $mKey
-	 * @return void
+	 * @return boolean
 	 */
 	public function __isset($mKey)
 	{
@@ -130,7 +130,7 @@ 
 	 *
 	 * @access public
 	 * @param  mixed $mOffset
-	 * @return mixed
+	 * @return boolean
 	 */
 	function offsetExists($mOffset)
 	{

Braces +3 added lines, -6 removed lines

@@ -62,15 +62,13 @@ 
 			if (!is_callable($data = $this->_aDataType[$mKey][$mKey]) || (is_string($data) && function_exists($data))) {
 
 				return $data;
-			}
-			else {
+			} else {
 
 				$dataStore = &$this->_aDataType[$mKey];
 				$dataStore[$mKey] = call_user_func($data, null);
 				return $dataStore[$mKey];
 			}
-		}
-		else {
+		} else {
 
 			return null;
 		}
@@ -89,8 +87,7 @@ 
 
 			$this->_aClosures[$mKey] = $mValue;
 			$this->_aDataType[$mKey] = &$this->_aClosures;
-		}
-		else {
+		} else {
 
 			$this->_aData[$mKey] = $mValue;
 			$this->_aDataType[$mKey] = &$this->_aData;

bundles/core/Router.php

Doc Comments +2 added lines, -2 removed lines

@@ -90,7 +90,7 @@ 
 	 * run the routeur
 	 *
 	 * @access public
-	 * @return void
+	 * @return null|boolean
 	 */
 	public function run() 
 	{
@@ -378,7 +378,7 @@ 
 	 * load a route
 	 *
 	 * @access private
-	 * @param  stdClass $oRoute one route
+	 * @param  \stdClass $oRoute one route
 	 * @param  string $RequestUri URI
 	 * @return void
 	 */

Braces +26 added lines, -52 removed lines

@@ -131,14 +131,12 @@ 
     		            
     		            echo file_get_contents(__DIR__.DIRECTORY_SEPARATOR.'..'.DIRECTORY_SEPARATOR.'src'.DIRECTORY_SEPARATOR.'plugins'.DIRECTORY_SEPARATOR.$sPlugin.DIRECTORY_SEPARATOR.'public'.$_SERVER['REQUEST_URI']);
                         exit;
-    		        }
-                    else if (strstr($_SERVER['REQUEST_URI'], '.css')
+    		        } else if (strstr($_SERVER['REQUEST_URI'], '.css')
 		                && file_exists(__DIR__.DIRECTORY_SEPARATOR.'..'.DIRECTORY_SEPARATOR.'src'.DIRECTORY_SEPARATOR.'plugins'.DIRECTORY_SEPARATOR.$sPlugin.DIRECTORY_SEPARATOR.'public'.preg_replace('/\.css/', '.less', $_SERVER['REQUEST_URI']))) {
 
 		                Less::toCss(__DIR__.DIRECTORY_SEPARATOR.'..'.DIRECTORY_SEPARATOR.'src'.DIRECTORY_SEPARATOR.'plugins'.DIRECTORY_SEPARATOR.$sPlugin.DIRECTORY_SEPARATOR.'public'.preg_replace('/\.css/', '.less', $_SERVER['REQUEST_URI']));
 		                exit;
-                    }
-		            else if (strstr($_SERVER['REQUEST_URI'], '.js')
+                    } else if (strstr($_SERVER['REQUEST_URI'], '.js')
 		                && file_exists(__DIR__.DIRECTORY_SEPARATOR.'..'.DIRECTORY_SEPARATOR.'src'.DIRECTORY_SEPARATOR.'plugins'.DIRECTORY_SEPARATOR.$sPlugin.DIRECTORY_SEPARATOR.'public'.preg_replace('/\.js/', '.ts', $_SERVER['REQUEST_URI']))) {
 
 		                Typescript::toJs(__DIR__.DIRECTORY_SEPARATOR.'..'.DIRECTORY_SEPARATOR.'src'.DIRECTORY_SEPARATOR.'plugins'.DIRECTORY_SEPARATOR.$sPlugin.DIRECTORY_SEPARATOR.'public'.preg_replace('/\.js/', '.ts', $_SERVER['REQUEST_URI']));
@@ -167,8 +165,7 @@ 
     						header('Status: 301 Moved Permanently', false, 301);
     	  					header('Location: '.$oHost->location);
     	  					exit;
-    					}
-    					else if (preg_match('#getCss\?#', $_SERVER['REQUEST_URI'])) {
+    					} else if (preg_match('#getCss\?#', $_SERVER['REQUEST_URI'])) {
     					    
     					    foreach ($_GET as $sKey => $sValue) {
 
@@ -179,8 +176,7 @@ 
     					    }
     					    
     					    exit;
-    					}
-    					else if (preg_match('#getJs\?#', $_SERVER['REQUEST_URI'])) {
+    					} else if (preg_match('#getJs\?#', $_SERVER['REQUEST_URI'])) {
     					    
     					    foreach ($_GET as $sKey => $sValue) {
 
@@ -191,8 +187,7 @@ 
     					    }
     					    
     					    exit;
-    					}
-    					else if (isset($oHost->routes)) {
+    					} else if (isset($oHost->routes)) {
     
     						foreach($oHost->routes as $sKey => $oRoute) {
 
@@ -201,8 +196,7 @@ 
     							if ($mReturn === 403) {
     
     								$this->_getPage403();
-    							}
-    							else if ($mReturn == true) {
+    							} else if ($mReturn == true) {
     
     								if (isset($oRoute->cache)) { $this->_checkCache($oRoute->cache); }
     
@@ -226,11 +220,9 @@ 
 //					trigger_error("Votre route n'existe pas : ".$_SERVER['REQUEST_URI'], E_USER_NOTICE);
 //				}
 			}
-		}
-        else if (Request::isCliRequest()) {
+		} else if (Request::isCliRequest()) {
 
-			if (isset($_SERVER['argv'])) { $aArguments = $_SERVER['argv']; }
-			else { $aArguments = $argv; }
+			if (isset($_SERVER['argv'])) { $aArguments = $_SERVER['argv']; } else { $aArguments = $argv; }
 
 			define('PORTAL', 'Batch');
 			set_include_path(get_include_path().PATH_SEPARATOR.'src'.PATH_SEPARATOR.PORTAL.PATH_SEPARATOR.'public');
@@ -240,8 +232,7 @@ 
                 $sBatchName = "phpunit";
                 $aArguments[0] = "bin/console";
                 $aArguments[1] = "phpunit";
-            }
-            else {
+            } else {
                 $sBatchName = $aArguments[1];
             }
 
@@ -290,8 +281,7 @@ 
 			if (isset($oBatch->controller) && isset($oBatch->action)) {
 
 				echo $this->_loadController($oBatch->controller, $oBatch->action, array($aOptions));
-			}
-			else {
+			} else {
 
 				if (Request::isCliRequest()) {
 
@@ -334,8 +324,7 @@ 
 					}
 				}
 			}
-		}
-		else if (defined('STDIN')) {
+		} else if (defined('STDIN')) {
 
 			$oBatch = Config::get('Route')->batch->script->{$sRoute};
 			echo $this->_loadController($oBatch->controller, $oBatch->action, $aParams);
@@ -397,8 +386,7 @@ 
 				},
 				$sRoute
 			);
-		}
-		else {
+		} else {
 
 			$sFinalRoute = '.*';
 		}
@@ -448,8 +436,7 @@ 
 
 				$oMobileDetect = new \Mobile_Detect;
 
-				if ($oMobileDetect->isMobile()) { $sCacheExt = '.mobi'; }
-				else { $sCacheExt = ''; }
+				if ($oMobileDetect->isMobile()) { $sCacheExt = '.mobi'; } else { $sCacheExt = ''; }
 
 				$mCacheReturn = Cache::get($RequestUri.$sCacheExt, $oRoute->cache->max_age);
 
@@ -472,17 +459,14 @@ 
 					if ($oRoute->content_type == 'json') {
 
 						header('Content-type: application/json; charset='.$sCharset.'');
-					}
-					else if ($oRoute->content_type == 'html') {
+					} else if ($oRoute->content_type == 'html') {
 
 						header('Content-type: text/html; charset='.$sCharset.'');
-					}
-					else if ($oRoute->content_type == 'jpeg') {
+					} else if ($oRoute->content_type == 'jpeg') {
 
 						header('Content-type: image/jpeg');
 					}
-				}
-				else {
+				} else {
 
 					header('Content-type: text/html; charset='.$sCharset.'');
 				}
@@ -503,21 +487,17 @@ 
 						if (isset($_GET[$sName]) && $_GET[$sName] != '') {
 
 							$aEntries[] = $_GET[$sName];
-						}
-						else if (isset($oRoute->defaults_constraints) && is_object($oRoute->defaults_constraints)
+						} else if (isset($oRoute->defaults_constraints) && is_object($oRoute->defaults_constraints)
 							&& isset($oRoute->defaults_constraints->{$sName})) {
 
 							$aEntries[] = $oRoute->defaults_constraints->{$sName};
-						}
-						else if (isset($_GET[$sName])) {
+						} else if (isset($_GET[$sName])) {
 
 							$aEntries[] = $_GET[$sName];
-						}
-						else if (preg_match('/'.$sType.'/', '')) {
+						} else if (preg_match('/'.$sType.'/', '')) {
 
 							$aEntries[] = '';
-						}
-						else {
+						} else {
 
 						    $this->_oLogger->warning('Error: Parameter '.$sName.' not exists!');
 							break;
@@ -529,8 +509,7 @@ 
 						$mReturn = $this->_loadController($oController, $sActionName, $aEntries);
 
 					}
-				}
-				else {
+				} else {
 
 					$mReturn = $this->_loadController($oController, $sActionName, $aEntries);
 				}
@@ -542,8 +521,7 @@ 
 						$mReturn = json_encode($mReturn, JSON_PRETTY_PRINT);
 					}
 				}
-			}
-			else if (isset($oRoute->template) && isset($oRoute->layout) && $oRoute->layout === true) {
+			} else if (isset($oRoute->template) && isset($oRoute->layout) && $oRoute->layout === true) {
 
 			    define('PORTAL', preg_replace('/^\\\\Venus\\\\src\\\\([a-zA-Z0-9_]+)\\\\.+$/', '$1', $oRoute->template));
 			    set_include_path(get_include_path().PATH_SEPARATOR.'src'.PATH_SEPARATOR.PORTAL.PATH_SEPARATOR.'public');
@@ -560,8 +538,7 @@ 
 
 				$mReturn = $oLayout->assign('model', DIRECTORY_SEPARATOR.'src'.DIRECTORY_SEPARATOR.PORTAL.DIRECTORY_SEPARATOR.'View'.DIRECTORY_SEPARATOR.$oRoute->template.'.tpl')
 								   ->fetch();
-			}
-			else if (isset($oRoute->template)) {
+			} else if (isset($oRoute->template)) {
 
 			    define('PORTAL', preg_replace('/^\\\\Venus\\\\src\\\\([a-zA-Z0-9_]+)\\\\.+$/', '$1', $oRoute->template));
 			    set_include_path(get_include_path().PATH_SEPARATOR.'src'.PATH_SEPARATOR.PORTAL.PATH_SEPARATOR.'public');
@@ -585,8 +562,7 @@ 
 
 				$oMobileDetect = new \Mobile_Detect;
 
-				if ($oMobileDetect->isMobile()) { $sCacheExt = '.mobi'; }
-				else { $sCacheExt = ''; }
+				if ($oMobileDetect->isMobile()) { $sCacheExt = '.mobi'; } else { $sCacheExt = ''; }
 
 				if (defined('COMPRESS_HTML') && COMPRESS_HTML) {
 
@@ -640,8 +616,7 @@ 
 
 			$oMobileDetect = new \Mobile_Detect;
 
-			if ($oMobileDetect->isMobile()) { $sCacheExt = '.mobi'; }
-			else { $sCacheExt = ''; }
+			if ($oMobileDetect->isMobile()) { $sCacheExt = '.mobi'; } else { $sCacheExt = ''; }
 
 			$mCacheReturn = Cache::get($sActionName.$sCacheExt, $aPhpDoc['Cache']['maxage']);
 
@@ -682,8 +657,7 @@ 
 
 			$oMobileDetect = new \Mobile_Detect;
 
-			if ($oMobileDetect->isMobile()) { $sCacheExt = '.mobi'; }
-			else { $sCacheExt = ''; }
+			if ($oMobileDetect->isMobile()) { $sCacheExt = '.mobi'; } else { $sCacheExt = ''; }
 
 			if (defined('COMPRESS_HTML') && COMPRESS_HTML) {
 

bundles/core/Security.php

Doc Comments +1 added lines, -1 removed lines

@@ -63,7 +63,7 @@
 	 * check security of access
 	 *
 	 * @access public
-	 * @return void
+	 * @return null|boolean
 	 */
 
 	public function checkSecurity() {

Braces +9 added lines, -19 removed lines

@@ -95,8 +95,7 @@ 
 					header('HTTP/1.0 401 Unauthorized');
 					echo $oSecurity->cancelled;
 					exit;
-				}
-				else {
+				} else {
 
 					self::$_sLogin = $_SERVER['PHP_AUTH_USER'];
 					self::$_sPassword = $_SERVER['PHP_AUTH_PW'];
@@ -105,8 +104,7 @@ 
 					if (!$this->_checkAccess()) { return false; }
 					if (!$this->_checkBlackListIps()) { return false; }
 				}
-			}
-			else if (isset($oSecurity->authentification) && $oSecurity->authentification === 'http_basic_validate_by_controller') {
+			} else if (isset($oSecurity->authentification) && $oSecurity->authentification === 'http_basic_validate_by_controller') {
 
 				if (!isset($_SERVER['PHP_AUTH_USER'])) {
 
@@ -117,8 +115,7 @@ 
 					header('HTTP/1.0 401 Unauthorized');
 					echo $oSecurity->cancelled;
 					exit;
-				}
-				else {
+				} else {
 
 					self::$_sLogin = $_SERVER['PHP_AUTH_USER'];
 					self::$_sPassword = $_SERVER['PHP_AUTH_PW'];
@@ -132,8 +129,7 @@ 
 					if (!$this->_checkAccess()) { return false; }
 					if (!$this->_checkBlackListIps()) { return false; }
 				}
-			}
-			else if (isset($oSecurity->authentification) && $oSecurity->authentification === 'controller') {
+			} else if (isset($oSecurity->authentification) && $oSecurity->authentification === 'controller') {
 
 				// it's an action of one controller that it return true or false for the authentification
 
@@ -152,8 +148,7 @@ 
 			if (isset($oSecurity->requires_channel) && $oSecurity->requires_channel == 'https' && !Request::isHttpsRequest()) {
 
 				return false;
-			}
-			else if (isset($oSecurity->requires_channel) && $oSecurity->requires_channel == 'http' && ((Request::isHttpRequest()
+			} else if (isset($oSecurity->requires_channel) && $oSecurity->requires_channel == 'http' && ((Request::isHttpRequest()
 				&& Request::isHttpsRequest()) || !Request::isHttpRequest())) {
 
 				return false;
@@ -180,8 +175,7 @@ 
 
 				if (preg_match('#'.$sPathAccess.'#', str_replace($this->_sBaseUri, '', $_SERVER['REQUEST_URI']))) {
 
-					if (in_array($this->getUserRole(), $aParams->roles)) { return true; }
-					else { return false; }
+					if (in_array($this->getUserRole(), $aParams->roles)) { return true; } else { return false; }
 				}
 			}
 		}
@@ -223,9 +217,7 @@ 
 		$sLogin = self::$_sLogin;
 		$sPassword = Config::get('Security')->users->$sLogin->password;
 
-		if ($sPassword == self::$_sPassword) { return true; }
-		else if ($sPassword == md5(self::$_sPassword)) { return true; }
-		else { return false; }
+		if ($sPassword == self::$_sPassword) { return true; } else if ($sPassword == md5(self::$_sPassword)) { return true; } else { return false; }
 	}
 
 	/**
@@ -241,8 +233,7 @@ 
 
 			$sLogin = self::$_sLogin;
 			return Config::get('Security')->users->$sLogin->roles;
-		}
-		else {
+		} else {
 
 			return '';
 		}
@@ -259,8 +250,7 @@ 
 
 	public function isGranted(string $sRole) : bool {
 
-		if ($sRole == $this->getUserRole() || $this->getUserRole() == '') { return true; }
-		else { return false; }
+		if ($sRole == $this->getUserRole() || $this->getUserRole() == '') { return true; } else { return false; }
 	}
 
 	/**

bundles/core/UrlManager.php

Doc Comments +1 added lines, -1 removed lines

@@ -42,7 +42,7 @@
 	 * @access public
 	 * @param  string $sCode code of the url between "routes" and "route" in Route.conf
 	 * @param  array $aParams parameters to create the url
-	 * @return void
+	 * @return string
 	 *
 	 * @tutorial	If I have this route I could make my URL:
 	 *

Braces +2 added lines, -4 removed lines

@@ -96,12 +96,10 @@
 
 										if (preg_match('#'.$sType.'#', $aParams[$sName])) {
 
-											if ($aParams[$sName]) { $sRoute = str_replace('[/:'.$sName.']', '/'.$aParams[$sName], $sRoute); }
-											else { $sRoute = str_replace('[/:'.$sName.']', '', $sRoute); }
+											if ($aParams[$sName]) { $sRoute = str_replace('[/:'.$sName.']', '/'.$aParams[$sName], $sRoute); } else { $sRoute = str_replace('[/:'.$sName.']', '', $sRoute); }
 											$sRoute = str_replace('[:'.$sName.']', $aParams[$sName], $sRoute);
 											continue;
-										}
-										else if (isset($oRoute->defaults_constraints) 
+										} else if (isset($oRoute->defaults_constraints) 
 											&& isset($oRoute->defaults_constraints->{$sName}) 
 											&& preg_match('#'.$sType.'#', $oRoute->defaults_constraints->{$sName})) {
 

bundles/ext/facebook/base_facebook.php

Doc Comments +6 added lines, -2 removed lines

@@ -503,7 +503,7 @@ 
    * Get the UID of the connected user, or 0
    * if the Facebook user is not connected.
    *
-   * @return string the UID if available.
+   * @return integer the UID if available.
    */
   public function getUser() {
     if ($this->user !== null) {
@@ -757,6 +757,7 @@ 
    * either logged in to Facebook or has granted an offline access permission.
    *
    * @param string $code An authorization code.
+   * @param string $redirect_uri
    * @return mixed An access token exchanged for the authorization code, or
    *               false if an access token could not be generated.
    */
@@ -1135,7 +1136,7 @@ 
   /**
    * Build the URL for given domain alias, path and parameters.
    *
-   * @param $name string The name of the domain
+   * @param string $name string The name of the domain
    * @param $path string Optional path (without a leading slash)
    * @param $params array Optional query parameters
    *
@@ -1402,6 +1403,9 @@ 
     return self::endsWith($big, '.'.$small);
   }
 
+  /**
+   * @param string $small
+   */
   protected static function endsWith($big, $small) {
     $len = strlen($small);
     if ($len === 0) {

Braces +1 added lines, -2 removed lines

@@ -361,8 +361,7 @@
           'fb_exchange_token' => $this->getAccessToken(),
         )
       );
-    }
-    catch (FacebookApiException $e) {
+    } catch (FacebookApiException $e) {
       // most likely that user very recently revoked authorization.
       // In any event, we don't have an access token, so say so.
       return false;

bundles/ext/facebook/facebook.php

Doc Comments +3 added lines

@@ -150,6 +150,9 @@
     return self::FBSS_COOKIE_NAME . '_' . $this->getAppId();
   }
 
+  /**
+   * @param string $key
+   */
   protected function constructSessionVariableName($key) {
     $parts = array('fb', $this->getAppId(), $key);
     if ($this->sharedSessionID) {

bundles/lib/Cache/File.php

Doc Comments +2 added lines, -3 removed lines

@@ -97,7 +97,7 @@ 
 	 *
 	 * @access public
 	 * @param  string $sName name of the session
-	 * @return mixed
+	 * @return boolean
 	 */
 	public function delete(string $sName)
 	{
@@ -108,7 +108,6 @@ 
 	 * flush the cache
 	 *
 	 * @access public
-	 * @param  string $sName name of the session
 	 * @return mixed
 	 */
 	public function flush()
@@ -121,7 +120,7 @@ 
 	 *
 	 * @access public
 	 * @param  string $sName name of the session
-	 * @return mixed
+	 * @return string
 	 */
 	private function _getSubDirectory($sName)
 	{

Braces +2 added lines, -4 removed lines

@@ -85,8 +85,7 @@ 
 		if (file_exists($this->_sFolder.$this->_getSubDirectory($sName).md5($sName).'.fil.cac')) {
 
 			return unserialize(file_get_contents($this->_sFolder.$this->_getSubDirectory($sName).md5($sName).'.fil.cac'));
-		}
-		else {
+		} else {
 
 			return false;
 		}
@@ -146,8 +145,7 @@ 
 
 			while (($sFile = readdir($rDirectory)) !== false) {
 
-				if ($sFile > '0' and filetype($sName.$sFile) == "file") { unlink($sName.$sFile); }
-				elseif ($sFile > '0' and filetype($sName.$sFile) == "dir") { remove_dir($sName.$sFile."\\"); }
+				if ($sFile > '0' and filetype($sName.$sFile) == "file") { unlink($sName.$sFile); } elseif ($sFile > '0' and filetype($sName.$sFile) == "dir") { remove_dir($sName.$sFile."\\"); }
 			}
 
 			closedir($rDirectory);

bundles/lib/Cache/Mock.php

Doc Comments +5 added lines, -6 removed lines

@@ -37,7 +37,7 @@ 
 	 * @param  string $sName name of the session
 	 * @param  int $iFlags flags
 	 * @param  int $iTimeout expiration of cache
-	 * @return mixed
+	 * @return boolean
 	 */
 	public function get(string $sName, int &$iFlags = null, int $iTimeout = 0)
 	{ 
@@ -52,7 +52,7 @@ 
 	 * @param  mixed $mValue value of this sesion var
 	 * @param  int $iFlag unused
 	 * @param  int $iExpire expiration of cache
-	 * @return \Venus\lib\Cache\Apc
+	 * @return boolean
 	 */
 	public function set(string $sName, $mValue, int $iFlag = 0, int $iExpire = false)
 	{ 
@@ -63,7 +63,7 @@ 
 	 * flush the cache
 	 *
 	 * @access public
-	 * @return mixed
+	 * @return boolean
 	 */
 	public function flush()
 	{
@@ -75,7 +75,7 @@ 
 	 *
 	 * @access public
 	 * @param  string $sName name of the session
-	 * @return mixed
+	 * @return boolean
 	 */
 	public function delete(string $sName)
 	{
@@ -88,9 +88,8 @@ 
 	 * @access public
 	 * @param  string $sName name of the session
 	 * @param  mixed $mValue value of this sesion var
-	 * @param  int $iFlag unused
 	 * @param  int $iExpire expiration of cache
-	 * @return mixed
+	 * @return boolean
 	 */
 	public function add($sName, $mValue, $iExpire = false)
 	{