- Issue created by @freelock
- ๐จ๐ดColombia camilo.escobar
camilo.escobar โ made their first commit to this issueโs fork.
- Assigned to camilo.escobar
- Status changed to Needs review
about 1 year ago 4:47am 11 January 2024 - ๐บ๐ฆUkraine Oleksiy Lutsk ๐บ๐ฆ
I encountered the same error, but in my case, I'm requesting JSONAPI with a filter like
/jsonapi/node/{content_type}?filter[title]=Test title
. So this issue also occurs when using the default Node entity type.The provided patch helped to solve it. Thank you!
Drupal: 10.2.2
Field Permissions: 8.x-1.3 - Issue was unassigned.
- ๐ซ๐ฎFinland kekkis Pirkkala
Confirming fix working on Drupal core 10.1.8, field_permissions 8.x-1.3.
Not sure if the module has tests (didn't check) or if the fix actually is logically correct, so leaving as NR instead of moving to RTBC.
- First commit to issue fork.
- Status changed to RTBC
about 1 year ago 2:03pm 19 March 2024 - ๐ซ๐ฎFinland sokru
I tried to rebase @camilo.escobar's MR, but apparently created a new one. MR21 was failing because #b1a0d27, so rebasing resolved the issue. Since related issue ๐ TypeError: Drupal\field_permissions\FieldPermissionsService::fieldGetPermissionType(): Argument #1 ($field) must be of type Drupal\field\FieldStorageConfigInterface Fixed didn't contain tests either and this fixes the issue, I set this RTBC. I also raise the priority since this cause fatal errors on many applications using json:api.
- ๐ง๐ทBrazil fadonascimento
If anyone would like to apply the changes using the patch, please check out the link below:
- https://git.drupalcode.org/project/field_permissions/-/merge_requests/21... - ๐บ๐ธUnited States mrweiner
Confirming that MR !25 fixes the issue for me.
- First commit to issue fork.
- Status changed to Closed: outdated
9 months ago 6:19pm 10 July 2024