The l() function creates hyperlinks, and the documentation states that "If element 'class' is included, it must be an array". This is because l() will compare the given URL against the current page and, if they're the same, it appends the string "active" to the 'class' array.
There are several places throughout Commerce Kickstart which are passing strings instead of arrays for 'class'. This causes fatal errors if the array append is attempted (ie. if the link points to the current page). My error logs are filling up with these messages.
The simple solution is to wrap each offending string in array(). As an example, I've spotted several instances of this in commerce_kickstart_user_form_alter. I'll add more as I find them.
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | commerce_kickstart-fix-l-calls-1990958-2.patch | 3.07 KB | jsacksick |
Comments
Comment #1
jsacksick commentedComment #2
jsacksick commentedThe attached patch should fix the issue.
Comment #3
jsacksick commentedHere's the gerrit commit : https://code.drupalcommerce.org/#/c/677/