[5.3] Adding Filesystem::isReadable() #15289

themsaid · 2016-09-05T22:49:31Z

Since isWritable() already exists.

GrahamCampbell · 2016-09-06T07:48:59Z

src/Illuminate/Filesystem/Filesystem.php

+     */
+    public function isReadable($path)
+    {
+        return is_writable($path);


GrahamCampbell reviewed Sep 6, 2016
View reviewed changes

src/Illuminate/Filesystem/Filesystem.php

*/

public function isReadable($path)

{

return is_writable($path);

Copy link

Member

GrahamCampbell Sep 6, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

writable?

This closes #14832 by adding isReadable

4c0c08c

taylorotwell merged commit 4c0c08c into laravel:5.3 Sep 6, 2016

vlakoff mentioned this pull request Oct 30, 2016

[5.3] Skip tests using chmod on Windows #16182

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[5.3] Adding Filesystem::isReadable() #15289

[5.3] Adding Filesystem::isReadable() #15289

themsaid commented Sep 5, 2016

GrahamCampbell Sep 6, 2016

[5.3] Adding Filesystem::isReadable() #15289

[5.3] Adding Filesystem::isReadable() #15289

Conversation

themsaid commented Sep 5, 2016

GrahamCampbell Sep 6, 2016

Choose a reason for hiding this comment