- Issue created by @jakegibs617
- 🇺🇸United States jakegibs617
diff --git a/modules/contrib/tft/tft.module b/modules/contrib/tft/tft.module index bfb083b8f..d6d771c49 100644 --- a/modules/contrib/tft/tft.module +++ b/modules/contrib/tft/tft.module @@ -664,7 +664,7 @@ function tft_file_access(EntityInterface $entity, $operation, AccountInterface $ $fid = $entity->get('fid')->getValue(); $query = \Drupal::entityQuery('media') ->condition('tft_file', $fid[0]['value']); - $entity_id = array_values($query->execute()); + $entity_id = array_values($query->accessCheck(FALSE)->execute()); if (isset($entity_id[0])) { $media = Media::load($entity_id[0]); }this works... But not sure if maintainers want this to be set to TRUE or FALSE
- First commit to issue fork.
- 🇧🇾Belarus dakruchko
I've created a merge request: https://git.drupalcode.org/project/tft/-/merge_requests/1/diffs
Added ->accessCheck(TRUE) as this is default (current) behavior, we just need to declare it explicitly. (FYI https://www.drupal.org/node/3201242 → ) - Status changed to Needs review
about 1 year ago 3:24pm 5 October 2023 - Status changed to RTBC
11 months ago 10:08am 18 January 2024 - 🇬🇧United Kingdom catch
Looks right to me. This is an uncaught exception when trying to use TFT on Drupal 10 so bumping to critical.
- 🇦🇷Argentina andreadruiz Buenos Aires
Hi!
I'm adding more accessCheck missing from the tft.module file. I added empty, but not sure if it should be TRUE.
Please, let me know.
Thanks! - Status changed to Needs review
11 months ago 6:42pm 5 February 2024 - First commit to issue fork.
- 🇮🇳India dineshkumarbollu
Found one more entityQuery in DeleteFolderForm.php file and created MR.
- Status changed to RTBC
9 months ago 10:27am 4 April 2024 - 🇬🇧United Kingdom catch
The module shouldn't have been declared Drupal 10 compatible without fixing these, it results in fatal errors everywhere. This is ready to go. If any more have been missed, can be a new issue.