also looking at this
Apply fixes from StyleCI
#53
1 file
+32
-51
Review threads live on the whole diff, not on one commit, so none are shown here - a thread's line means something in the branch's final form, and painting it into an intermediate step would put it on code it is not about.
| @@ -21,10 +21,9 @@ class MeemaAdapter extends AbstractAdapter | ||
| 21 | 21 | /** |
| 22 | 22 | * Write a new file. |
| 23 | 23 | * |
| 24 | * @param string $path | |
| 25 | * @param string $contents | |
| 26 | * @param Config $config Config object | |
| 27 | * | |
| 24 | * @param string $path | |
| 25 | * @param string $contents | |
| 26 | * @param Config $config Config object | |
| 28 | 27 | * @return false|array false on failure file meta data on success |
| 29 | 28 | */ |
| 30 | 29 | public function write($path, $contents, Config $config) |
| @@ -35,10 +34,9 @@ class MeemaAdapter extends AbstractAdapter | ||
| 35 | 34 | /** |
| 36 | 35 | * Write a new file using a stream. |
| 37 | 36 | * |
| 38 | * @param string $path | |
| 39 | * @param resource $resource | |
| 40 | * @param Config $config Config object | |
| 41 | * | |
| 37 | * @param string $path | |
| 38 | * @param resource $resource | |
| 39 | * @param Config $config Config object | |
| 42 | 40 | * @return array|false false on failure file meta data on success |
| 43 | 41 | */ |
| 44 | 42 | public function writeStream($path, $resource, Config $config) |
| @@ -49,10 +47,9 @@ class MeemaAdapter extends AbstractAdapter | ||
| 49 | 47 | /** |
| 50 | 48 | * Update a file. |
| 51 | 49 | * |
| 52 | * @param string $path | |
| 53 | * @param string $contents | |
| 54 | * @param Config $config Config object | |
| 55 | * | |
| 50 | * @param string $path | |
| 51 | * @param string $contents | |
| 52 | * @param Config $config Config object | |
| 56 | 53 | * @return array|false false on failure file meta data on success |
| 57 | 54 | */ |
| 58 | 55 | public function update($path, $contents, Config $config) |
| @@ -63,10 +60,9 @@ class MeemaAdapter extends AbstractAdapter | ||
| 63 | 60 | /** |
| 64 | 61 | * Update a file using a stream. |
| 65 | 62 | * |
| 66 | * @param string $path | |
| 67 | * @param resource $resource | |
| 68 | * @param Config $config Config object | |
| 69 | * | |
| 63 | * @param string $path | |
| 64 | * @param resource $resource | |
| 65 | * @param Config $config Config object | |
| 70 | 66 | * @return array|false false on failure file meta data on success |
| 71 | 67 | */ |
| 72 | 68 | public function updateStream($path, $resource, Config $config) |
| @@ -77,9 +73,8 @@ class MeemaAdapter extends AbstractAdapter | ||
| 77 | 73 | /** |
| 78 | 74 | * Rename a file. |
| 79 | 75 | * |
| 80 | * @param string $path | |
| 81 | * @param string $newpath | |
| 82 | * | |
| 76 | * @param string $path | |
| 77 | * @param string $newpath | |
| 83 | 78 | * @return bool |
| 84 | 79 | */ |
| 85 | 80 | public function rename($path, $newpath) |
| @@ -90,9 +85,8 @@ class MeemaAdapter extends AbstractAdapter | ||
| 90 | 85 | /** |
| 91 | 86 | * Copy a file. |
| 92 | 87 | * |
| 93 | * @param string $path | |
| 94 | * @param string $newpath | |
| 95 | * | |
| 88 | * @param string $path | |
| 89 | * @param string $newpath | |
| 96 | 90 | * @return bool |
| 97 | 91 | */ |
| 98 | 92 | public function copy($path, $newpath) |
| @@ -103,8 +97,7 @@ class MeemaAdapter extends AbstractAdapter | ||
| 103 | 97 | /** |
| 104 | 98 | * Delete a file. |
| 105 | 99 | * |
| 106 | * @param string $path | |
| 107 | * | |
| 100 | * @param string $path | |
| 108 | 101 | * @return bool |
| 109 | 102 | */ |
| 110 | 103 | public function delete($path) |
| @@ -115,8 +108,7 @@ class MeemaAdapter extends AbstractAdapter | ||
| 115 | 108 | /** |
| 116 | 109 | * Delete a directory. |
| 117 | 110 | * |
| 118 | * @param string $dirname | |
| 119 | * | |
| 111 | * @param string $dirname | |
| 120 | 112 | * @return bool |
| 121 | 113 | */ |
| 122 | 114 | public function deleteDir($dirname) |
| @@ -127,9 +119,8 @@ class MeemaAdapter extends AbstractAdapter | ||
| 127 | 119 | /** |
| 128 | 120 | * Create a directory. |
| 129 | 121 | * |
| 130 | * @param string $dirname directory name | |
| 131 | * @param Config $config | |
| 132 | * | |
| 122 | * @param string $dirname directory name | |
| 123 | * @param Config $config | |
| 133 | 124 | * @return array|false |
| 134 | 125 | */ |
| 135 | 126 | public function createDir($dirname, Config $config) |
| @@ -140,9 +131,8 @@ class MeemaAdapter extends AbstractAdapter | ||
| 140 | 131 | /** |
| 141 | 132 | * Set the visibility for a file. |
| 142 | 133 | * |
| 143 | * @param string $path | |
| 144 | * @param string $visibility | |
| 145 | * | |
| 134 | * @param string $path | |
| 135 | * @param string $visibility | |
| 146 | 136 | * @return array|false file meta data |
| 147 | 137 | */ |
| 148 | 138 | public function setVisibility($path, $visibility) |
| @@ -153,8 +143,7 @@ class MeemaAdapter extends AbstractAdapter | ||
| 153 | 143 | /** |
| 154 | 144 | * Check whether a file exists. |
| 155 | 145 | * |
| 156 | * @param string $path | |
| 157 | * | |
| 146 | * @param string $path | |
| 158 | 147 | * @return bool |
| 159 | 148 | */ |
| 160 | 149 | public function has($path) |
| @@ -165,8 +154,7 @@ class MeemaAdapter extends AbstractAdapter | ||
| 165 | 154 | /** |
| 166 | 155 | * Read a file. |
| 167 | 156 | * |
| 168 | * @param string $path | |
| 169 | * | |
| 157 | * @param string $path | |
| 170 | 158 | * @return false|array |
| 171 | 159 | */ |
| 172 | 160 | public function read($path) |
| @@ -177,9 +165,8 @@ class MeemaAdapter extends AbstractAdapter | ||
| 177 | 165 | /** |
| 178 | 166 | * List contents of a directory. |
| 179 | 167 | * |
| 180 | * @param string $directory | |
| 181 | * @param bool $recursive | |
| 182 | * | |
| 168 | * @param string $directory | |
| 169 | * @param bool $recursive | |
| 183 | 170 | * @return array |
| 184 | 171 | */ |
| 185 | 172 | public function listContents($directory = '', $recursive = false) |
| @@ -190,8 +177,7 @@ class MeemaAdapter extends AbstractAdapter | ||
| 190 | 177 | /** |
| 191 | 178 | * Read a file as a stream. |
| 192 | 179 | * |
| 193 | * @param string $path | |
| 194 | * | |
| 180 | * @param string $path | |
| 195 | 181 | * @return array|false |
| 196 | 182 | */ |
| 197 | 183 | public function readStream($path) |
| @@ -202,8 +188,7 @@ class MeemaAdapter extends AbstractAdapter | ||
| 202 | 188 | /** |
| 203 | 189 | * Get all the meta data of a file or directory. |
| 204 | 190 | * |
| 205 | * @param string $path | |
| 206 | * | |
| 191 | * @param string $path | |
| 207 | 192 | * @return false|array |
| 208 | 193 | */ |
| 209 | 194 | public function getMetadata($path) |
| @@ -214,8 +199,7 @@ class MeemaAdapter extends AbstractAdapter | ||
| 214 | 199 | /** |
| 215 | 200 | * Get all the meta data of a file or directory. |
| 216 | 201 | * |
| 217 | * @param string $path | |
| 218 | * | |
| 202 | * @param string $path | |
| 219 | 203 | * @return false|array |
| 220 | 204 | */ |
| 221 | 205 | public function getSize($path) |
| @@ -226,8 +210,7 @@ class MeemaAdapter extends AbstractAdapter | ||
| 226 | 210 | /** |
| 227 | 211 | * Get the mimetype of a file. |
| 228 | 212 | * |
| 229 | * @param string $path | |
| 230 | * | |
| 213 | * @param string $path | |
| 231 | 214 | * @return false|array |
| 232 | 215 | */ |
| 233 | 216 | public function getMimetype($path) |
| @@ -238,8 +221,7 @@ class MeemaAdapter extends AbstractAdapter | ||
| 238 | 221 | /** |
| 239 | 222 | * Get the timestamp of a file. |
| 240 | 223 | * |
| 241 | * @param string $path | |
| 242 | * | |
| 224 | * @param string $path | |
| 243 | 225 | * @return false|array |
| 244 | 226 | */ |
| 245 | 227 | public function getTimestamp($path) |
| @@ -250,8 +232,7 @@ class MeemaAdapter extends AbstractAdapter | ||
| 250 | 232 | /** |
| 251 | 233 | * Get the visibility of a file. |
| 252 | 234 | * |
| 253 | * @param string $path | |
| 254 | * | |
| 235 | * @param string $path | |
| 255 | 236 | * @return array|false |
| 256 | 237 | */ |
| 257 | 238 | public function getVisibility($path) |