Remove the delete function about simple-file-manager HOT 3 CLOSED

I think this would do it. However, note that you can still overwrite a file with an empty (effectively blanking it).

28a29,30
> $allow_delete = false;
> 
57,58c59,60
<               'is_deleteable' => (!is_dir($i) && is_writable($directory)) || 
<                                  (is_dir($i) && is_writable($directory) && is_recursively_deleteable($i)),
---
>               'is_deleteable' => $allow_delete && ((!is_dir($i) && is_writable($directory)) || 
>                                  (is_dir($i) && is_writable($directory) && is_recursively_deleteable($i))),
70c72,74
<   rmrf($file);
---
>   if($allow_delete) {
>       rmrf($file);
>   }

from simple-file-manager.

send a pull request with allow_delete at the top, and default to true.
I'll merge it.

On Wed, Jun 22, 2016 at 10:31 AM, mesler1 [email protected] wrote:

I think this would do it. However, note that you can still overwrite a
file with an empty (effectively blanking it).

28a29,30

$allow_delete = false;

57,58c59,60
< 'is_deleteable' => (!is_dir($i) && is_writable($directory)) ||
< (is_dir($i) && is_writable($directory) && is_recursively_deleteable($i)),

      'is_deleteable' => $allow_delete && ((!is_dir($i) && is_writable($directory)) ||
                         (is_dir($i) && is_writable($directory) && is_recursively_deleteable($i))),

70c72,74
< rmrf($file);

if($allow_delete) {
rmrf($file);
}

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#20 (comment),
or mute the thread
https://github.com/notifications/unsubscribe/AABjG6cjD3s4pcYyVTJYqxbJdertRWfVks5qOUdfgaJpZM4IuKlZ
.

from simple-file-manager.

Looks like this has been merged. Closing.

from simple-file-manager.