- Issue created by @joseph.olstad
- Status changed to Needs review
over 1 year ago 8:46pm 7 August 2023 - last update
over 1 year ago 33 pass - π¨π¦Canada joseph.olstad
This module has a very nice UI and works well for monolingual implementations, however underneath the prettiness lies a very nasty issue as described in the issue summary that this patch will resolve.
Bumping this to Critical since it leads to increasing vulnerability to denial of service, also taxes servers unnecessarily and can lead to severe performance issues. Most people who experience this probably think that the problem is Drupal it'self, very hard to diagnose and troubleshoot especially for most people.
We should just fix the glitch to help others that may be still using this module.
- π¨π¦Canada joseph.olstad
still critical, until this is fixed, use the "Scheduled Transitions" module instead
- First commit to issue fork.
- Merge request !11Issue #3379796: Performance - Prevent redundant processing back to January 1st... β (Merged) created by kaszarobert
- Status changed to RTBC
5 months ago 5:31am 3 August 2024 - π¨π¦Canada joseph.olstad
Aside from lacking the dependency injection for this:
\Drupal::state()->get('system.cron_last');
near parfect. The MR11 is satisfactory for me as-is. - π¨π¦Canada joseph.olstad
please put a checkbox next to our names at the bottom of this issue and press save
- πΈπ°Slovakia kaszarobert
@joseph.olstad I created the MR with the exact same changes you uploaded in a patch years ago. However, I had to change a few things:
- I fixed the proper DI in the class,
- With this change, we're now disabling the processing of older transitions than last cron run, I added a new test case for that
- During writing a test for this new behavior, I found out that in the patch you were using a timestamp in the SQL query for the lowestscheduled_transition_date.value
but in the database that field contains aY-m-d\TH:i:s
formatted UTC timezoned date, so the patch by default didn't change the requested behavior the way we wanted it, so I modified the code to pass a formatted date to the query.
- this a backwards breaking change, so I'm thinking about if we should set this new 'lowest date of a state transition' thing into a setting instead. But again due to how this module was architected + with the performance issues you were experiencing indicate that this should be the default behavior to not break huge sites. So maybe we have no other choice than commit this really. -
kaszarobert β
committed dfb7e52d on 8.x-1.x
Issue #3379796: Fixing time format from timestamp to an SQL timezone...
-
kaszarobert β
committed dfb7e52d on 8.x-1.x
-
kaszarobert β
committed c5472278 on 8.x-1.x
Issue #3379796: Performance - Prevent redundant processing back to...
-
kaszarobert β
committed c5472278 on 8.x-1.x
- Status changed to Fixed
5 months ago 10:12pm 3 August 2024 Automatically closed - issue fixed for 2 weeks with no activity.