Steps to reproduce:
- Install Drupal in another language (for example with "drush si standard --locale=nb")
- Download reroute_email
- Enable reroute_email (for example with "drush en reroute_email"
- Refresh any page on the site.
This results in
Fatal error: Call to a member function getPath() on null in /Users/user/Sites/site/drupal/core/includes/install.inc on line 942
The reason for this is that the module defines a constant using the t function, and the translation expects to map this to a request uri. And this does not work so well when it is defined as a constant.
Anyway, some refactoring should probably be done here also, but as it is right now, this makes the module unusable with multilingual sites. So here is a quick patch to just remove the t call for now. Coming up
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | fix_fatal_error_if-2659960-2.patch | 544 bytes | eiriksm |
Comments
Comment #2
eiriksmPatch attached. By the way, there is no 8.x version to choose in the dropdown for issue metadata.
Comment #4
eiriksmHeh, no wonder it does not apply to 7.x-dev. There really should be a 8.x version :p
Comment #5
dydave commentedComment #7
rfayCommitted, thanks!