The use of drupal_attributes() within apachesolr_l() implies that attributes that can have multiple values (such as 'class') should be arrays. By providing a string, developers are unable to extend the classes on that link via preprocessing.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

kaidjohnson’s picture

Issue summary: View changes
Status: Active » Needs review
FileSize
667 bytes

Proposed patch...

dooug’s picture

Re-rolled patch for latest 7.x-1.x.

dooug’s picture

Status: Needs review » Reviewed & tested by the community

This aligns with coding best practice by using arrays for class attributes, making them more easily modifiable by other code. RTBC++

  • japerry committed ede9ec1 on 7.x-1.x authored by dooug
    Issue #2153347 by kaidjohnson, dooug: $vars['options']['attributes']['...
japerry’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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