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.

Comments

jsacksick’s picture

Assigned: Unassigned » jsacksick
jsacksick’s picture

Status: Active » Fixed
StatusFileSize
new3.07 KB

The attached patch should fix the issue.

jsacksick’s picture

Here's the gerrit commit : https://code.drupalcommerce.org/#/c/677/

Status: Fixed » Closed (fixed)

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