I found that users were not given access to editable fields in a node view unless they had the 'Bypass content access control' permission. I think users should have access to editable fields if they have access to edit the entity to which the fields are attached.

CommentFileSizeAuthor
#2 editablefields-3247569-1.patch579 byteskleinmp

Issue fork editablefields-3247569

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

kleinmp created an issue. See original summary.

kleinmp’s picture

kleinmp commented
StatusFileSize
new editablefields-3247569-1.patch579 bytes

Patch

gilmord made their first commit to this issue’s fork.

gilmord opened merge request !2

  • gilmord committed 5340e4b on 1.0.x 
    #3247569 - fix the name of the entity operation
gilmord’s picture

gilmord
Location 🇺🇦Ukraine
commented
Status: Active » Fixed

Hi, @kleinmp
Nice catch, thank you!

I have pushed the fix to the dev version, please review and get back with the feedback.

Can I ask you to use the new flow with the forks and merge requests instead of patches for future issues?
Here is MR example: https://git.drupalcode.org/project/editablefields/-/merge_requests/2
You can use it as a patch like this:
https://git.drupalcode.org/project/editablefields/-/merge_requests/2.patch
Or this:
https://git.drupalcode.org/project/editablefields/-/merge_requests/2.diff

Thanks again!

gilmord’s picture

gilmord
Location 🇺🇦Ukraine
commented

Released beta2 with this fix

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.