Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Hi,
The module does not work with popups. The problem is on the line 502 in hierarchical_select.js with the return statement. Removing this sentence is functioning properly.
Regards.
Comments
Comment #1
Wim LeersStrange. Yet simple fix. Some testing did not reveal any problems, so I decided it's safe to go. Thanks, committed!
http://drupal.org/cvs?commit=412118
http://drupal.org/cvs?commit=412120
Comment #3
digi24 CreditAttribution: digi24 commentedThis commit is having some side effects for me:
I use a two level hierarchy vocabulary. Before this commit the behavior was as follows:
Changing the selection on the first hierarchy level results in a AJAX request to retrieve the second level terms. Changing the second level selection is instantaneous (no AJAX call).
After this patch every selection change leads to an AJAX request, even if it is unnecessary.
Comment #4
digi24 CreditAttribution: digi24 commentedmore meaningful status
Comment #5
Wim LeersYou're absolutely right, digi24. This was fixed in #1007492: Don't always perform a callback to the server when updating.
Comment #6
Wim LeersNow also fixed for Drupal 5: http://drupal.org/cvs?commit=495496.