shogodev / argilla

protected/extensions/eauth/EOAuthService.php

Doc Comments +2 added lines, -3 removed lines

@@ -141,8 +141,7 @@ 
 	 *
 	 * @param string $url url to request.
 	 * @param array $options HTTP request options. Keys: query, data, referer.
-	 * @param boolean $parseJson Whether to parse response in json format.
-	 * @return cURL handle.
+	 * @return resource handle.
 	 */
 	protected function initRequest($url, $options = array()) {
 		$ch = parent::initRequest($url, $options);
@@ -156,7 +155,7 @@ 
 	 * @param string $url url to request.
 	 * @param array $options HTTP request options. Keys: query, data, referer.
 	 * @param boolean $parseJson Whether to parse response in json format.
-	 * @return string the response.
+	 * @return stdClass the response.
 	 * @see makeRequest
 	 */
 	public function makeSignedRequest($url, $options = array(), $parseJson = true) {

protected/extensions/elFinder/php/elFinderConnector.class.php

Doc Comments +1 added lines

@@ -31,6 +31,7 @@
 	/**
 	 * Constructor
 	 *
+	 * @param elFinder $elFinder
 	 * @return void
 	 * @author Dmitry (dio) Levashov
 	 **/

protected/extensions/elFinder/php/elFinderVolumeDriver.class.php

Doc Comments +15 added lines, -20 removed lines

@@ -1000,7 +1000,6 @@ 
 	 * Return file info or false on error
 	 *
 	 * @param  string   $hash      file hash
-	 * @param  bool     $realpath  add realpath field to file info
 	 * @return array|false
 	 * @author Dmitry (dio) Levashov
 	 **/
@@ -1022,7 +1021,6 @@ 
 	 * Return folder info
 	 *
 	 * @param  string   $hash  folder hash
-	 * @param  bool     $hidden  return hidden file info
 	 * @return array|false
 	 * @author Dmitry (dio) Levashov
 	 **/
@@ -1349,7 +1347,6 @@ 
 	 *
 	 * @param  Resource $fp      file pointer
 	 * @param  string   $dst     destination folder hash
-	 * @param  string   $src     file name
 	 * @param  string   $tmpname file tmp name - required to detect mime type
 	 * @return array|false
 	 * @author Dmitry (dio) Levashov
@@ -1443,7 +1440,6 @@ 
 	 * Paste files
 	 *
 	 * @param  Object  $volume  source volume
-	 * @param  string  $source  file hash
 	 * @param  string  $dst     destination dir hash
 	 * @param  bool    $rmSrc   remove source after copy?
 	 * @return array|false
@@ -1543,7 +1539,7 @@ 
 	 * Return file contents
 	 *
 	 * @param  string  $hash  file hash
-	 * @return string|false
+	 * @return boolean|string
 	 * @author Dmitry (dio) Levashov
 	 **/
 	public function getContents($hash) {
@@ -1765,7 +1761,7 @@ 
 	 * Return image dimensions
 	 *
 	 * @param  string  $hash  file hash
-	 * @return array
+	 * @return false|string
 	 * @author Dmitry (dio) Levashov
 	 **/
 	public function dimensions($hash) {
@@ -1861,7 +1857,8 @@ 
 	 * Not implemented
 	 *
 	 * @param  string  path
-	 * @return mixed
+	 * @param string $path
+	 * @return string
 	 * @author Dmitry (dio) Levashov
 	 **/
 	protected function crypt($path) {
@@ -1873,7 +1870,8 @@ 
 	 * Not implemented
 	 *
 	 * @param  mixed  hash
-	 * @return mixed
+	 * @param string $hash
+	 * @return string
 	 * @author Dmitry (dio) Levashov
 	 **/
 	protected function uncrypt($hash) {
@@ -1902,7 +1900,6 @@ 
 	/**
 	 * Return new unique name based on file name and suffix
 	 *
-	 * @param  string  $path    file path
 	 * @param  string  $suffix  suffix append to name
 	 * @return string
 	 * @author Dmitry (dio) Levashov
@@ -2418,7 +2415,7 @@ 
 	 * @param  string  $src   source path
 	 * @param  string  $dst   destination dir path
 	 * @param  string  $name  new file name (optionaly)
-	 * @return string|false
+	 * @return string|boolean
 	 * @author Dmitry (dio) Levashov
 	 **/
 	protected function copy($src, $dst, $name) {
@@ -2468,7 +2465,7 @@ 
 	 * @param  string  $src   source path
 	 * @param  string  $dst   destination dir path
 	 * @param  string  $name  new file name
-	 * @return string|false
+	 * @return string|boolean
 	 * @author Dmitry (dio) Levashov
 	 **/
 	protected function move($src, $dst, $name) {
@@ -2494,7 +2491,7 @@ 
 	 * @param  string  $src          source file hash
 	 * @param  string  $destination  destination dir path
 	 * @param  string  $name         file name
-	 * @return string|false
+	 * @return boolean|string
 	 * @author Dmitry (dio) Levashov
 	 **/
 	protected function copyFrom($volume, $src, $destination, $name) {
@@ -2634,7 +2631,7 @@ 
 	 *
 	 * @param  string  $path  thumnbnail path
 	 * @param  array   $stat  file stat
-	 * @return string|bool
+	 * @return boolean
 	 * @author Dmitry (dio) Levashov
 	 **/
 	protected function canCreateTmb($path, $stat) {
@@ -2651,7 +2648,7 @@ 
 	 *
 	 * @param  string  $path  thumnbnail path
 	 * @param  array   $stat  file stat
-	 * @return string|bool
+	 * @return boolean
 	 * @author Dmitry (dio) Levashov
 	 **/
 	protected function canResize($path, $stat) {
@@ -2662,7 +2659,6 @@ 
 	 * Create thumnbnail and return it's URL on success
 	 *
 	 * @param  string  $path  file path
-	 * @param  string  $mime  file mime type
 	 * @return string|false
 	 * @author Dmitry (dio) Levashov
 	 **/
@@ -3024,7 +3020,7 @@ 
 	 *
 	 * @param  string  $command       command line
 	 * @param  array   $output        stdout strings
-	 * @param  array   $return_var    process exit code
+	 * @param  integer   $return_var    process exit code
 	 * @param  array   $error_output  stderr strings
 	 * @return int     exit code
 	 * @author Alexey Sukhotin
@@ -3174,7 +3170,7 @@ 
 	* @param  string  $haystack  The string being checked.
 	* @param  string  $needle    The string to find in haystack.
 	* @param  int     $offset    The search offset. If it is not specified, 0 is used.
-	* @return int|bool
+	* @return integer
 	* @author Alexey Sukhotin
 	**/
 	protected function stripos($haystack , $needle , $offset = 0) {
@@ -3462,7 +3458,6 @@ 
 	 * Open file and return file pointer
 	 *
 	 * @param  string  $path  file path
-	 * @param  bool    $write open file for writing
 	 * @return resource|false
 	 * @author Dmitry (dio) Levashov
 	 **/
@@ -3515,7 +3510,7 @@ 
 	 * Copy file into another file (only inside one volume)
 	 *
 	 * @param  string  $source  source file path
-	 * @param  string  $target  target dir path
+	 * @param  string  $targetDir  target dir path
 	 * @param  string  $name    file name
 	 * @return bool
 	 * @author Dmitry (dio) Levashov
@@ -3527,7 +3522,7 @@ 
 	 * Return new file path or false.
 	 *
 	 * @param  string  $source  source file path
-	 * @param  string  $target  target dir path
+	 * @param  string  $targetDir  target dir path
 	 * @param  string  $name    file name
 	 * @return string|bool
 	 * @author Dmitry (dio) Levashov

protected/extensions/elFinder/php/elFinderVolumeFTP.class.php

Doc Comments +16 added lines, -16 removed lines

@@ -355,7 +355,7 @@ 
 	 * Return ftp transfer mode for file
 	 *
 	 * @param  string  $path  file path
-	 * @return string
+	 * @return integer
 	 * @author Dmitry (dio) Levashov
 	 **/
 	protected function ftpMode($path) {
@@ -513,7 +513,7 @@ 
 	 * If file does not exists - returns empty array or false.
 	 *
 	 * @param  string  $path    file path 
-	 * @return array|false
+	 * @return string
 	 * @author Dmitry (dio) Levashov
 	 **/
 	protected function _stat($path) {
@@ -651,7 +651,7 @@ 
 	 *
 	 * @param  string  $path  file path
 	 * @param  string  $mime  file mime type
-	 * @return string
+	 * @return boolean
 	 * @author Dmitry (dio) Levashov
 	 **/
 	protected function _dimensions($path, $mime) {
@@ -684,7 +684,6 @@ 
 	 * Open file and return file pointer
 	 *
 	 * @param  string  $path  file path
-	 * @param  bool    $write open file for writing
 	 * @return resource|false
 	 * @author Dmitry (dio) Levashov
 	 **/
@@ -705,7 +704,7 @@ 
 	 * Close opened file
 	 *
 	 * @param  resource  $fp  file pointer
-	 * @return bool
+	 * @return boolean|null
 	 * @author Dmitry (dio) Levashov
 	 **/
 	protected function _fclose($fp, $path='') {
@@ -722,7 +721,7 @@ 
 	 *
 	 * @param  string  $path  parent dir path
 	 * @param string  $name  new directory name
-	 * @return string|bool
+	 * @return false|string
 	 * @author Dmitry (dio) Levashov
 	 **/
 	protected function _mkdir($path, $name) {
@@ -740,7 +739,7 @@ 
 	 *
 	 * @param  string  $path  parent dir path
 	 * @param string  $name  new file name
-	 * @return string|bool
+	 * @return string|false
 	 * @author Dmitry (dio) Levashov
 	 **/
 	protected function _mkfile($path, $name) {
@@ -772,7 +771,7 @@ 
 	 * @param  string  $source     source file path
 	 * @param  string  $targetDir  target directory path
 	 * @param  string  $name       new file name
-	 * @return bool
+	 * @return string|false
 	 * @author Dmitry (dio) Levashov
 	 **/
 	protected function _copy($source, $targetDir, $name) {
@@ -797,9 +796,9 @@ 
 	 * Return new file path or false.
 	 *
 	 * @param  string  $source  source file path
-	 * @param  string  $target  target dir path
+	 * @param  string  $targetDir  target dir path
 	 * @param  string  $name    file name
-	 * @return string|bool
+	 * @return string|false
 	 * @author Dmitry (dio) Levashov
 	 **/
 	protected function _move($source, $targetDir, $name) {
@@ -837,7 +836,7 @@ 
 	 * @param  string    $dir  target dir path
 	 * @param  string    $name file name
 	 * @param  array     $stat file stat (required by some virtual fs)
-	 * @return bool|string
+	 * @return string|false
 	 * @author Dmitry (dio) Levashov
 	 **/
 	protected function _save($fp, $dir, $name, $stat) {
@@ -955,7 +954,7 @@ 
 	 *
 	 * @param  string  $path  archive path
 	 * @param  array   $arc   archiver command and arguments (same as in $this->archivers)
-	 * @return true
+	 * @return false|string
 	 * @author Dmitry (dio) Levashov,
 	 * @author Alexey Sukhotin
 	 **/
@@ -1062,7 +1061,7 @@ 
 	 * @param  array   $files  files names list
 	 * @param  string  $name   archive name
 	 * @param  array   $arc    archiver options
-	 * @return string|bool
+	 * @return false|string
 	 * @author Dmitry (dio) Levashov,
 	 * @author Alexey Sukhotin
 	 **/
@@ -1209,9 +1208,9 @@ 
 	/**
 	 * Downloads specified files from remote directory
 	 * if there is a directory among files it is downloaded recursively (omitting symbolic links).
-	 * @param $remote_directory string remote FTP path to a source directory to download from.
+	 * @param string $remote_directory string remote FTP path to a source directory to download from.
 	 * @param array $files list of files to download from remote directory.
-	 * @param $dest_local_directory string destination folder to store downloaded files.
+	 * @param string $dest_local_directory string destination folder to store downloaded files.
 	 * @return bool true on success and false on failure.
 	 */
 	private function ftp_download_files($remote_directory, array $files, $dest_local_directory)
@@ -1252,7 +1251,7 @@ 
 
 	/**
 	 * Delete local directory recursively.
-	 * @param $dirPath string to directory to be erased.
+	 * @param string $dirPath string to directory to be erased.
 	 * @return bool true on success and false on failure.
 	 */
 	private function deleteDir($dirPath)
@@ -1289,6 +1288,7 @@ 
 	 * Returns array of strings containing all files and folders in the specified local directory.
 	 * @param $dir
 	 * @param string $prefix
+	 * @param boolean $omitSymlinks
 	 * @internal param string $path path to directory to scan.
 	 * @return array array of files and folders names relative to the $path
 	 * or an empty array if the directory $path is empty,

protected/extensions/elFinder/php/elFinderVolumeLocalFileSystem.class.php

Doc Comments +7 added lines, -8 removed lines

@@ -402,7 +402,6 @@ 
 	 * Open file and return file pointer
 	 *
 	 * @param  string  $path  file path
-	 * @param  bool    $write open file for writing
 	 * @return resource|false
 	 * @author Dmitry (dio) Levashov
 	 **/
@@ -428,7 +427,7 @@ 
 	 *
 	 * @param  string  $path  parent dir path
 	 * @param string  $name  new directory name
-	 * @return string|bool
+	 * @return string|false
 	 * @author Dmitry (dio) Levashov
 	 **/
 	protected function _mkdir($path, $name) {
@@ -447,7 +446,7 @@ 
 	 *
 	 * @param  string  $path  parent dir path
 	 * @param string  $name  new file name
-	 * @return string|bool
+	 * @return string|false
 	 * @author Dmitry (dio) Levashov
 	 **/
 	protected function _mkfile($path, $name) {
@@ -492,9 +491,9 @@ 
 	 * Return new file path or false.
 	 *
 	 * @param  string  $source  source file path
-	 * @param  string  $target  target dir path
+	 * @param  string  $targetDir  target dir path
 	 * @param  string  $name    file name
-	 * @return string|bool
+	 * @return string|false
 	 * @author Dmitry (dio) Levashov
 	 **/
 	protected function _move($source, $targetDir, $name) {
@@ -532,7 +531,7 @@ 
 	 * @param  string    $dir  target dir path
 	 * @param  string    $name file name
 	 * @param  array     $stat file stat (required by some virtual fs)
-	 * @return bool|string
+	 * @return false|string
 	 * @author Dmitry (dio) Levashov
 	 **/
 	protected function _save($fp, $dir, $name, $stat) {
@@ -555,7 +554,7 @@ 
 	 * Get file contents
 	 *
 	 * @param  string  $path  file path
-	 * @return string|false
+	 * @return string
 	 * @author Dmitry (dio) Levashov
 	 **/
 	protected function _getContents($path) {
@@ -741,7 +740,7 @@ 
 	 * @param  array   $files  files names list
 	 * @param  string  $name   archive name
 	 * @param  array   $arc    archiver options
-	 * @return string|bool
+	 * @return string|false
 	 * @author Dmitry (dio) Levashov, 
 	 * @author Alexey Sukhotin
 	 **/

protected/extensions/elFinder/php/elFinderVolumeMySQL.class.php

Doc Comments +7 added lines, -8 removed lines

@@ -273,7 +273,6 @@ 
 	 * @param  string   $hash    image file
 	 * @param  int      $width   new width
 	 * @param  int      $height  new height
-	 * @param  bool     $crop    crop image
 	 * @return array|false
 	 * @author Dmitry (dio) Levashov
 	 * @author Alexey Sukhotin
@@ -686,7 +685,7 @@ 
 	 * Close opened file
 	 *
 	 * @param  resource  $fp  file pointer
-	 * @return bool
+	 * @return boolean|null
 	 * @author Dmitry (dio) Levashov
 	 **/
 	protected function _fclose($fp, $path='') {
@@ -703,7 +702,7 @@ 
 	 *
 	 * @param  string  $path  parent dir path
 	 * @param string  $name  new directory name
-	 * @return string|bool
+	 * @return string|false
 	 * @author Dmitry (dio) Levashov
 	 **/
 	protected function _mkdir($path, $name) {
@@ -715,7 +714,7 @@ 
 	 *
 	 * @param  string  $path  parent dir path
 	 * @param string  $name  new file name
-	 * @return string|bool
+	 * @return string|false
 	 * @author Dmitry (dio) Levashov
 	 **/
 	protected function _mkfile($path, $name) {
@@ -759,9 +758,9 @@ 
 	 * Return new file path or false.
 	 *
 	 * @param  string  $source  source file path
-	 * @param  string  $target  target dir path
+	 * @param  string  $targetDir  target dir path
 	 * @param  string  $name    file name
-	 * @return string|bool
+	 * @return string|false
 	 * @author Dmitry (dio) Levashov
 	 **/
 	protected function _move($source, $targetDir, $name) {
@@ -934,7 +933,7 @@ 
 	 *
 	 * @param  string  $path  archive path
 	 * @param  array   $arc   archiver command and arguments (same as in $this->archivers)
-	 * @return true
+	 * @return boolean
 	 * @author Dmitry (dio) Levashov, 
 	 * @author Alexey Sukhotin
 	 **/
@@ -949,7 +948,7 @@ 
 	 * @param  array   $files  files names list
 	 * @param  string  $name   archive name
 	 * @param  array   $arc    archiver options
-	 * @return string|bool
+	 * @return boolean
 	 * @author Dmitry (dio) Levashov, 
 	 * @author Alexey Sukhotin
 	 **/

protected/extensions/eoauth/lib/OAuthConsumer.php

Doc Comments +4 added lines

@@ -12,6 +12,10 @@
   public $key;
   public $secret;
 
+  /**
+   * @param string $key
+   * @param string $secret
+   */
   function __construct($key, $secret, $callback_url=NULL) {
     $this->key = $key;
     $this->secret = $secret;

protected/extensions/eoauth/lib/OAuthRequest.php

Doc Comments +9 added lines

@@ -77,6 +77,8 @@ 
 
   /**
    * pretty much a helper function to set up the request
+   * @param string $http_method
+   * @param string $http_url
    */
   public static function from_consumer_and_token($consumer, $token, $http_method, $http_url, $parameters=NULL) {
     @$parameters or $parameters = array();
@@ -92,6 +94,9 @@ 
     return new OAuthRequest($http_method, $http_url, $parameters);
   }
 
+  /**
+   * @param string $name
+   */
   public function set_parameter($name, $value, $allow_duplicates = true) {
     if ($allow_duplicates && isset($this->parameters[$name])) {
       // We have already added parameter(s) with this name, so add to the list
@@ -185,6 +190,7 @@ 
 
   /**
    * builds a url usable for a GET request
+   * @return string
    */
   public function to_url() {
     $post_data = $this->to_postdata();
@@ -227,6 +233,9 @@ 
   }
 
 
+  /**
+   * @param OAuthSignatureMethod_HMAC_SHA1 $signature_method
+   */
   public function sign_request($signature_method, $consumer, $token) {
     $this->set_parameter(
       "oauth_signature_method",

protected/extensions/nested-set-behavior/NestedSetBehavior.php

Doc Comments +6 added lines

@@ -592,6 +592,8 @@ 
 	 * @param int $levelUp.
 	 * @param boolean $runValidation.
 	 * @param array $attributes.
+	 * @param integer $levelUp
+	 * @param boolean $runValidation
 	 * @return boolean.
 	 */
 	private function addNode($target,$key,$levelUp,$runValidation,$attributes)
@@ -660,6 +662,7 @@ 
 
 	/**
 	 * @param array $attributes.
+	 * @param boolean|null $attributes
 	 * @return boolean.
 	 */
 	private function makeRoot($attributes)
@@ -724,6 +727,7 @@ 
 	 * @param CActiveRecord $target.
 	 * @param int $key.
 	 * @param int $levelUp.
+	 * @param integer $levelUp
 	 * @return boolean.
 	 */
 	private function moveNode($target,$key,$levelUp)
@@ -879,6 +883,7 @@ 
 	/**
 	 * Correct cache for {@link NestedSetBehavior::addNode()}.
 	 * @param int $key.
+	 * @param integer $key
 	 */
 	private function correctCachedOnAddNode($key)
 	{
@@ -907,6 +912,7 @@ 
 	 * Correct cache for {@link NestedSetBehavior::moveNode()}.
 	 * @param int $key.
 	 * @param int $levelDelta.
+	 * @param double $levelDelta
 	 */
 	private function correctCachedOnMoveNode($key,$levelDelta)
 	{