Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
The latest core updates include error messaging which can be found here. The recurly modules have an incorrect module_load_include which is causing an error to be displayed on all pages related to recurly.
Comment | File | Size | Author |
---|---|---|---|
#6 | locale-module-load-2782135-6.patch | 1.15 KB | walangitan |
#5 | locale-module-load-2782135-5.patch | 1.47 KB | walangitan |
#2 | locale-module-load-2782135-2.patch | 1.16 KB | walangitan |
recurly-module-missing-content-user-warning.png | 93.05 KB | walangitan |
Comments
Comment #2
walangitan CreditAttribution: walangitan at Chromatic for TheaterMania.com, Inc. commentedComment #3
walangitan CreditAttribution: walangitan at Chromatic for TheaterMania.com, Inc. commentedComment #4
walangitan CreditAttribution: walangitan at Chromatic for TheaterMania.com, Inc. commentedComment #5
walangitan CreditAttribution: walangitan at Chromatic for TheaterMania.com, Inc. commentedI forgot to add the locale module as a dependency for this patch. It's included here. We're only using it for the country list, so I'm not sure if we want to include the whole module in order to pull in that function via the module_load_include or if want to go another route. I'll post a patch below that doesn't require a new dependency.
Comment #6
walangitan CreditAttribution: walangitan at Chromatic for TheaterMania.com, Inc. commentedThis patch also resolves the issue, but does not introduce a new module dependency like patch #5.
Comment #7
markdorison@walangitan: #5 seems more 'best practice' than #6. Is there a precedent for the approach in #6?
If we go with #5, what does that mean for users with locale disabled?
Comment #8
walangitan CreditAttribution: walangitan at Chromatic for TheaterMania.com, Inc. commented@markdorison: I did a check for the approach in #6 and I see that the system module does use a similar pattern to call the country_get_list() function from locale, pathauto and webform are other contrib modules that uses the include_once pattern instead of adding a dependency.
I have tried applying #5 on an installation that doesn't have locale enabled, and it maintains the same error message until I either:
1) Disable recurly and re-enable the module.
2) Enable the locale module manually.
Comment #9
markdorison@walangitan: Thanks for the additional context. I am going to commit #6.
Comment #11
markdorison