There is currently only one permission that allows for replacing any files. In some cases, we want to only allow a user to use this feature if they are replacing their own file.

Thoughts? Mods?

CommentFileSizeAuthor
#5 add_permission_for_own_files-r2-240426.patch786 byteskyilmaz80
#3 add_permission_for_own_files-r1-240426.patch4.51 KBkyilmaz80
add_permission_for_own_files.patch1.72 KBjohnsicili

Comments

johnsicili created an issue. See original summary.

casey’s picture

casey commented
Status: Needs review » Needs work

Interesting, we would need a update path however.

kyilmaz80’s picture

kyilmaz80 commented
StatusFileSize
new add_permission_for_own_files-r1-240426.patch4.51 KB

Hello and thanks for adding this feature,

I tested the new patch on Drupal 10.2.5 and Php 8.2.7. The access logic did not worked as expected. The code below have to be changed:

$is_owner = (bool) $account->id() == $file->getOwnerId();

to

$is_owner = $account->id() == $file->getOwnerId();

bool cast have a problem i think. Overall the patch worked after that. I am also adding the revised patch (add_permission_for_own_files-r1-240429.patch).

Best regards,

-Koray

kyilmaz80’s picture

kyilmaz80 commented
kyilmaz80’s picture

kyilmaz80 commented
StatusFileSize
new add_permission_for_own_files-r2-240426.patch786 bytes
kyilmaz80’s picture

kyilmaz80 commented