| Index: trunk/phase3/includes/filerepo/RepoGroup.php |
| — | — | @@ -177,9 +177,11 @@ |
| 178 | 178 | } |
| 179 | 179 | |
| 180 | 180 | /** |
| 181 | | - * Run a function across all foreign repos. |
| 182 | | - * @param $callback string Name of method to call |
| 183 | | - * @param $params array Optional params to pass to the function |
| | 181 | + * Call a function for each foreign repo, with the repo object as the |
| | 182 | + * first parameter. |
| | 183 | + * |
| | 184 | + * @param $callback callback The function to call |
| | 185 | + * @param $params array Optional additional parameters to pass to the function |
| 184 | 186 | */ |
| 185 | 187 | function forEachForeignRepo( $callback, $params = array() ) { |
| 186 | 188 | foreach( $this->foreignRepos as $repo ) { |
| — | — | @@ -192,11 +194,11 @@ |
| 193 | 195 | } |
| 194 | 196 | |
| 195 | 197 | /** |
| 196 | | - * Does the installation have foreign repos set up? |
| | 198 | + * Does the installation have any foreign repos set up? |
| 197 | 199 | * @return bool |
| 198 | 200 | */ |
| 199 | 201 | function hasForeignRepos() { |
| 200 | | - return !empty( $this->foreignRepos ); |
| | 202 | + return (bool)$this->foreignRepos; |
| 201 | 203 | } |
| 202 | 204 | |
| 203 | 205 | /** |