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

CommentFileSizeAuthor
#2 fix_fatal_error_if-2659960-2.patch544 byteseiriksm

Comments

eiriksm created an issue. See original summary.

eiriksm’s picture

Status: Active » Needs review
StatusFileSize
new544 bytes

Patch attached. By the way, there is no 8.x version to choose in the dropdown for issue metadata.

Status: Needs review » Needs work

The last submitted patch, 2: fix_fatal_error_if-2659960-2.patch, failed testing.

eiriksm’s picture

Status: Needs work » Needs review

Heh, no wonder it does not apply to 7.x-dev. There really should be a 8.x version :p

dydave’s picture

Version: 7.x-1.x-dev » 8.x-1.x-dev

  • rfay committed a02575c on 8.x-1.x authored by eiriksm
    Issue #2659960 by eiriksm: Fix Fatal error if reroute_email is enabled...
rfay’s picture

Status: Needs review » Fixed

Committed, thanks!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.