The following lines addition in the date_views module:

Code screenshot

cause the exposed filters element wrapper to have useless classes.

Steps to reproduce correct behaviour using Date 2.10:

1. Create a view with exposed date field
2. Assign custom identifier to the exposed date field (e.g. 'my-date').
3. Views generated HTML will use this id for form field and for the wrapper div ('edit-my-date', etc). You can now use this for styling and js.

Steps to reproduce incorrect behaviour using Date 2.11:

1. Create a view with exposed date field
2. Assign custom identifier to the exposed date field (e.g. 'my-date').
3. Views generated HTML will use uncontrollable (and randomized) id for both wrapper div and form field, as caused by the line 327

CommentFileSizeAuthor
Capture.PNG21.24 KBpawel.traczynski

Comments

pawel.traczynski created an issue. See original summary.

steinmb’s picture

Version: 7.x-2.11 » 7.x-2.x-dev

Hi
From what I understand I the change you are seeing coming from issue #2449261: expects an array, not a scalar - Fatal error: Cannot create references to/from string offsets

damienmckenna’s picture

Version: 7.x-2.x-dev » 7.x-3.x-dev
Issue tags: +Needs tests

Due to the amount of changes needed for the all-day fix, 7.x-2.12 has been changed to 7.x-3.0.

damienmckenna’s picture

Version: 7.x-3.x-dev » 7.x-2.x-dev

Can you please test the current 7.x-2.x dev snapshot, let us know if it solves the problems or if they still exist. Thank you.

damienmckenna’s picture

Status: Active » Postponed (maintainer needs more info)

Please test the latest dev snapshot and let us know if the problem persists. Thank you.

pawel.traczynski’s picture

Damien - with the latest version of Date 2.14, the problem still persists.
Commenting out this line:

$form['value']['#id'] = 'date_views_exposed_filter-' . bin2hex(drupal_random_bytes(16));

fixes the problem. I understand it might cause problem elsewhere but I did not see any on my setup.

damienmckenna’s picture

Status: Postponed (maintainer needs more info) » Active

Thanks for confirming that.