- 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
6 months 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
3 months 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...