As every other plugin exposed_form::pre_render should get $values as parameter, too.

Beside there is perhaps a need for this, the api symetry is better with this patch :)

CommentFileSizeAuthor
views-exposed_form-pre_render-values.patch2.82 KBdawehner
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

bojanz’s picture

bojanz CreditAttribution: bojanz commented
Status: Needs review » Reviewed & tested by the community

Applies, makes sense.

merlinofchaos’s picture

merlinofchaos CreditAttribution: merlinofchaos commented
Status: Reviewed & tested by the community » Needs work

The documentation in help/api-handlers.html needs to be updated for this change.

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented
Status: Needs work » Fixed

This part seemed to be commited already to 7.x-3.x, but the docs were missing.

Changed this and commited to 6.x-3.x with the documentation patch.

Status: Fixed » Closed (fixed)

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