Don't use l() function in hook_requirements. [#2547401]

This causes issues when doing a fresh Drupal installation. The code path takes you down to 'drupal_path_alias_whitelist_rebuild' which throws an error saying "Base table or view not found: url_alias doesn't exist". This might not be the case when installing the module after the fact, but it definitely fails on installation when it's a dependency of an install profile.

Comment	File	Size	Author
#2	2547401-no-l-in-hook_requirements.patch	733 bytes	Sam152
#2

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

7 August 2015 at 09:03

Sam152 created an issue. See original summary.

Comment #2

Sam152 CreditAttribution: Sam152 commented 7 August 2015 at 09:05

Status:

Active

» Needs review

File	Size
2547401-no-l-in-hook_requirements.patch	733 bytes

Comment #3

jibran

he/him

Urdu

Sydney, Australia

CreditAttribution: jibran as a volunteer commented 9 August 2015 at 18:02

Status:

Needs review

» Reviewed & tested by the community

Look good to me.

Comment #4

23 September 2015 at 15:21

das-peter committed 4e29dcf on 7.x-2.x authored by Sam152

Issue #2547401 by Sam152: Don't use l() function in hook_requirements

Comment #5

das-peter CreditAttribution: das-peter as a volunteer and at Cando commented 23 September 2015 at 15:22

Status:

Reviewed & tested by the community

» Fixed

Hmm, I liked the link ;) But right now I'm to lazy to try to use drupal_installation_attempted() or something similar to make this conditional. Oh wait, wasn't there a status given for the hook?
However, fixed for now ;) Thanks for the patch and the review!