- π©πͺGermany a.dmitriiev
The changes from patch #2 are also included in Drupal 10 compatibility issue https://www.drupal.org/project/scheduled_updates/issues/3363800 π Drupal 10 compatibility Needs review , so maybe let's focus on that issue.
- π©πͺGermany Kevin Hoffmann Berlin
Kevin Hoffmann β made their first commit to this issueβs fork.
- π©πͺGermany Kevin Hoffmann Berlin
Kevin Hoffmann β changed the visibility of the branch 3228974-query-for-retrieving to hidden.
- Merge request !10Issue #3228974: Query for retrieving entities with ready updates should be preventing access check β (Merged) created by Kevin Hoffmann
- π©πͺGermany Kevin Hoffmann Berlin
Kevin Hoffmann β changed the visibility of the branch 3228974-query-for-retrieving to active.
- π©πͺGermany Kevin Hoffmann Berlin
I added a fork for version 2.x that sets the access checks to false like @hchonov suggested. This fixes an issue with node related update events - especially unpublished ones.
The problem is that we fetch all nodes with update flags first before deciding who should run the update. Setting the access check during the retrieval to TRUE means cron and drush runs will not be able to see unpublished nodes and therefore miss those completely. An alternative would be to switch to user 1 during query execution but might as well just ignore the access check then for the same result. We cannot switch to a specific user yet during that query as we dont even know if and how many updates there are and which nodes are affected.
Each specific update will then later be executed as the user specified in the modules configuration.
-
hchonov β
authored 045b339d on 2.x
Issue #3228974: Query for retrieving entities with ready updates should...
-
hchonov β
authored 045b339d on 2.x
- π©πͺGermany hchonov πͺπΊπ©πͺπ§π¬
I've just merged this one into 2.x.
- Status changed to Fixed
4 months ago 6:29pm 2 September 2024 Automatically closed - issue fixed for 2 weeks with no activity.