| @@ -19,14 +19,14 @@ discard block | ||
| 19 | 19 | abstract protected function test(); | 
| 20 | 20 | |
| 21 | 21 | /** | 
| 22 | - * @return mixed | |
| 22 | + * @return string | |
| 23 | 23 | */ | 
| 24 | 24 |  	public function name() { | 
| 25 | 25 | return $this->name; | 
| 26 | 26 | } | 
| 27 | 27 | |
| 28 | 28 | /** | 
| 29 | - * @return mixed|string | |
| 29 | + * @return string | |
| 30 | 30 | */ | 
| 31 | 31 |  	public function result() { | 
| 32 | 32 | |
| @@ -524,7 +524,7 @@ discard block | ||
| 524 | 524 | var $name = 'WP Cron Test Failed'; | 
| 525 | 525 | |
| 526 | 526 | /** | 
| 527 | - * @return mixed | |
| 527 | + * @return boolean | |
| 528 | 528 | */ | 
| 529 | 529 |  	protected function test() { | 
| 530 | 530 | return (bool) get_option( 'hmbkp_wp_cron_test_failed' ); | 
| @@ -177,7 +177,7 @@ | ||
| 177 | 177 | /** | 
| 178 | 178 | * Get the connection args for the mysqldump command | 
| 179 | 179 | * | 
| 180 | - * @return array The array of connection args | |
| 180 | + * @return string[] The array of connection args | |
| 181 | 181 | */ | 
| 182 | 182 |  	public function get_mysql_connection_args() { | 
| 183 | 183 | |
| @@ -141,7 +141,6 @@ | ||
| 141 | 141 | * issues with the backup process. | 
| 142 | 142 | * | 
| 143 | 143 | * @param string $context The context for the warning. | 
| 144 | - * @param string $error The warning that was encountered. | |
| 145 | 144 | */ | 
| 146 | 145 |  	private function warning( $context, $warning ) { | 
| 147 | 146 | |
| @@ -91,7 +91,7 @@ | ||
| 91 | 91 | /** | 
| 92 | 92 | * Attempt to work out path to a cli executable. | 
| 93 | 93 | * | 
| 94 | - * @param array $paths An array of paths to check against. | |
| 94 | + * @param string[] $paths An array of paths to check against. | |
| 95 | 95 | * | 
| 96 | 96 | * @return string|false The path to the executable. | 
| 97 | 97 | */ | 
| @@ -282,7 +282,6 @@ | ||
| 282 | 282 | * Move backup files from an existing directory and the new | 
| 283 | 283 | * location | 
| 284 | 284 | * | 
| 285 | - * @param string $path The path to move the backups from | |
| 286 | 285 | * @return void | 
| 287 | 286 | */ | 
| 288 | 287 |  	public function move_old_backups( $from ) { | 
| @@ -147,7 +147,7 @@ | ||
| 147 | 147 | * Returns a formatted string containing the calculated total site size or a message | 
| 148 | 148 | * to indicate it is being calculated. | 
| 149 | 149 | * | 
| 150 | - * @param HM\BackUpWordPress\Scheduled_Backup $schedule | |
| 150 | + * @param Scheduled_Backup $schedule | |
| 151 | 151 | * | 
| 152 | 152 | * @return string | 
| 153 | 153 | */ | 
| @@ -38,7 +38,7 @@ | ||
| 38 | 38 | /** | 
| 39 | 39 | * Set the excludes rules for the backup. | 
| 40 | 40 | * | 
| 41 | - * @param array $excludes The exclude rules. | |
| 41 | + * @param Excludes $excludes The exclude rules. | |
| 42 | 42 | */ | 
| 43 | 43 |  	public function set_excludes( Excludes $excludes ) { | 
| 44 | 44 | $this->excludes = $excludes; | 
| @@ -158,7 +158,7 @@ discard block | ||
| 158 | 158 | /** | 
| 159 | 159 | * Get the exclude rules | 
| 160 | 160 | * | 
| 161 | - * @return array | |
| 161 | + * @return Excludes | |
| 162 | 162 | */ | 
| 163 | 163 |  	public function get_excludes() { | 
| 164 | 164 | return new Excludes( $this->options['excludes'] ); | 
| @@ -167,7 +167,7 @@ discard block | ||
| 167 | 167 | /** | 
| 168 | 168 | * Set the exclude rules | 
| 169 | 169 | * | 
| 170 | - * @param mixed $excludes A comma separated list or array of exclude rules | |
| 170 | + * @param mixed $exclude_rules A comma separated list or array of exclude rules | |
| 171 | 171 | * @param bool $append Whether to replace or append to existing rules | 
| 172 | 172 | * | 
| 173 | 173 | * @return string | 
| @@ -213,7 +213,7 @@ discard block | ||
| 213 | 213 | * Get the array of services options for this schedule | 
| 214 | 214 | * | 
| 215 | 215 | * @param $service | 
| 216 | - * @param null $option | |
| 216 | + * @param string $option | |
| 217 | 217 | * | 
| 218 | 218 | * @return array | 
| 219 | 219 | */ | 
| @@ -290,6 +290,7 @@ discard block | ||
| 290 | 290 | /** | 
| 291 | 291 | * Get the schedule reoccurrence | 
| 292 | 292 | * | 
| 293 | + * @return string | |
| 293 | 294 | */ | 
| 294 | 295 |  	public function get_reoccurrence() { | 
| 295 | 296 | return $this->options['reoccurrence']; | 
| @@ -460,7 +461,7 @@ discard block | ||
| 460 | 461 | /** | 
| 461 | 462 | * Hook into the actions fired in the Backup class and set the status | 
| 462 | 463 | * | 
| 463 | - * @param $action | |
| 464 | + * @param string $action | |
| 464 | 465 | */ | 
| 465 | 466 |  	public function do_action( $action, Site_Backup $backup ) { | 
| 466 | 467 | |
| @@ -479,6 +480,7 @@ discard block | ||
| 479 | 480 | * Calculate schedule run time. | 
| 480 | 481 | * | 
| 481 | 482 | * @param int Timestamp $end | 
| 483 | + * @param integer $end | |
| 482 | 484 | */ | 
| 483 | 485 |  	public function update_average_schedule_run_time( $start, $end ) { | 
| 484 | 486 | |
| @@ -22,7 +22,7 @@ | ||
| 22 | 22 | * | 
| 23 | 23 | * @param string $type Whether to calculate the size of the database, files | 
| 24 | 24 | * or both. Should be one of 'file', 'database' or 'complete' | 
| 25 | - * @param array $excludes An array of exclude rules | |
| 25 | + * @param Excludes $excludes An array of exclude rules | |
| 26 | 26 | */ | 
| 27 | 27 |  	public function __construct( $type = 'complete', Excludes $excludes = null ) { | 
| 28 | 28 | $this->type = $type; |