- Issue created by @keshav.k
- 🇮🇳India keshav.k
The same issue mentioned in https://www.drupal.org/project/lightning_scheduler/issues/3297525#commen... 📌 Automated Drupal 10 compatibility fixes Needs work
- 🇨🇦Canada teknocat
The provided patch won't apply - it just times out after a while. I will try doing a re-roll of this patch, test it and then attach to this comment shortly.
- 🇮🇳India keshav.k
We can ignore the patch added in https://www.drupal.org/project/lightning_workflow/issues/3389692 🐛 Creation of dynamic property Drupal\lightning_scheduler\Time::$_serviceId is deprecated RTBC because it will be addressed by the temporary fix in patch #77 for now.
- last update
1 day ago 34 pass, 10 fail - 🇺🇸United States fsabbagh
#11 🐛 Circular reference detected for service "cache.backend.database". Active installed cleanly, brought site back up. Running additional tests, but so far so good! @sundhar →
- 🇺🇸United States greenSkin
+++ b/modules/lightning_scheduler/src/Time.php @@ -12,30 +13,59 @@ use Symfony\Component\HttpFoundation\RequestStack; + return $this->get('lightning_scheduler.request_time', parent::getRequestTime());
Should this not be calling get() on the $this->keyValueStore (patch from #11 🐛 Circular reference detected for service "cache.backend.database". Active )?
- First commit to issue fork.
- 🇮🇳India Jeya sundhar Coimbatore
Hi @greeenskin,
State service dependency creates a circular reference, so I removed that dependency and implemented the functions in the 'lightning_scheduler.datetime.time' service. I used
$this->get('lightning_scheduler.request_time', parent::getRequestTime());
in that callback.Thankyou
- 🇨🇦Canada teknocat
Hey everyone, what's the difference between the lightning scheduler sub-module of this module and the standalone lightning scheduler → module? The standalone module seems to have addressed the issue.
Why not simply make that module a dependency of this one via composer? Or remove it completely, leaving it to the person building the Drupal site to decide what scheduling module they want to use and add it explicitly to their project?
Seems silly to maintain this independently as part of the lightning workflow codebase when the other is having the same kinds of issues being addressed separately ( https://www.drupal.org/project/lightning_scheduler/issues/3297525 📌 Automated Drupal 10 compatibility fixes Needs work ).
- 🇨🇦Canada teknocat
By the way, it looks like the latest release of the standalone module has this whole problem and other related issues resolved.