- last update
about 1 year ago 7 pass - 🇨🇦Canada joseph.olstad
Reroll, I'd feel more comfortable about this if there was a check wrapping this logic to ensure that the language module was installed using a condition on the moduleHandler for the language module before trying to open the configuration that might not exist and could throw a null exception.
- 🇨🇦Canada joseph.olstad
The above patch needs some extra logic to make sure it doesn't crash for those not using language negotiation.
https://www.drupal.org/project/facets_pretty_paths/releases/8.x-1.5 → - Merge request !19Issue #3328831: Domain based language negotiation stops AJAX facets from working → (Open) created by rossb89
- 🇬🇧United Kingdom rossb89 Bristol
I had completely forgotten about this!
I've re-done this against the latest 8.x-1.x with the suggested change to check the language module exists before trying to load the config.
I have opened a MR, but will attach a patch just in case that is useful for anyone.
-
joseph.olstad →
committed 1a9de863 on 2.0.x
Issue #3328831 by rossb89, joseph.olstad: Domain based language...
-
joseph.olstad →
committed 1a9de863 on 2.0.x
- 🇬🇧United Kingdom rossb89 Bristol
Ah yes, I totally should have done this against 2.x! Thanks for approving, merging in and doing a new release!
I guess this issue can be marked as fixed now? I see 1.x isn't visible on the project homepage anymore.
Automatically closed - issue fixed for 2 weeks with no activity.