- Issue created by @lostcarpark
- Merge request !8Add asset.query_string dependendency and remove datetime.time as no longer... → (Merged) created by lostcarpark
Automatically closed - issue fixed for 2 weeks with no activity.
While adding tests, I discovered a deprecation warning on the method of clearing the CSS/JS cache:
1x: The 'system.css_js_query_string' state is deprecated in drupal:10.2.0. Use \Drupal\Core\Asset\AssetQueryStringInterface::get() and ::reset() instead. See https://www.drupal.org/node/3358337.
1x in NavigationExtraToolsAlterTest::testFlushCssJs from Drupal\Tests\navigation_extra_tools\Functional
This seems a cleaner way of clearing the CSS/JS cache so we should fix that.
Run testFlushCssJs
test.
Add the asset.query_string
service as a dependency, then replace the current clear CSS/JS cache logic with a call to the ->reset()
method.
Add dependency injection of asset.query_string
Add call to reset()
method.
None
None
None
Active
1.0
Code
Automatically closed - issue fixed for 2 weeks with no activity.