Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Drupal 10. Add entity query check to Format Strawberryfield OAI-PMH #389

Closed
DiegoPino opened this issue Dec 8, 2023 · 2 comments
Closed

Drupal 10. Add entity query check to Format Strawberryfield OAI-PMH #389

DiegoPino opened this issue Dec 8, 2023 · 2 comments
Assignees
@DiegoPino
Labels
bug Something isn't working Drupal 10 Upgrade economy Sub Modules When you need more .info.yml files to keep life organized
Milestone
1.3.0

Comments

@DiegoPino
Copy link
Member

What?

Entity queries since D10 require always an explicit access check. I missed adding it to this submodule

@patdunlavey just to keep you informed this is happening

:)
@DiegoPino DiegoPino added bug Something isn't working Sub Modules When you need more .info.yml files to keep life organized Drupal 10 Upgrade economy labels Dec 8, 2023
@DiegoPino DiegoPino added this to the 1.3.0 milestone Dec 8, 2023
@DiegoPino DiegoPino self-assigned this Dec 8, 2023
@DiegoPino
Copy link
Member Author

@patdunlavey I have the feeling that the OAI-PMH module (the base one used by this submodule) has issues. The cache system is failing badly with Search API driven Views. Will ask you in slack about your experience with this later on and see if we need to make a pull against them or we can override the Cache system?

@DiegoPino DiegoPino mentioned this issue Dec 10, 2023
Merged
@DiegoPino
Copy link
Member Author

Solved via bb437d3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@DiegoPino DiegoPino

Labels
bug Something isn't working Drupal 10 Upgrade economy Sub Modules When you need more .info.yml files to keep life organized
Projects
None yet
Milestone
1.3.0
Development

No branches or pull requests
1 participant
@DiegoPino