- Issue created by @capysara
- 🇺🇸United States capysara
Support ticket opened with Acquia
Case Number
01071056 - Merge request !30#3416582 – Update reset filter to reset select list (single value) to All instead of any empty string → (Open) created by capysara
- last update
10 months ago Unable to generate test groups - 🇺🇸United States capysara
I create a MR to fix the issue with the single value select list, but the multi value select list still doesn't reset correctly.
- last update
10 months ago Unable to generate test groups - 🇺🇸United States capysara
Updated the MR to address #10 from IS
Refresh the page and select 2 or more values at once (holding down CMD or SHIFT), then it just gives me zero results and no way to clear it.
Now selecting multiple values should respect the filter_operation in the views filter config.
The Clear button still doesn't work correctly. - last update
7 months ago Composer require-dev failure - last update
7 months ago Composer require-dev failure - First commit to issue fork.
- 🇮🇳India chandu7929 Pune
@Rajeshreeputra - Requested some changes, please take a look.
- 🇮🇳India rajeshreeputra Pune
Updated as per review feedback, requesting review.
- 🇮🇳India chandu7929 Pune
Verified recent changes and it looks good to me. I would request another sets of eyes.
I encountered an issue when using the patch while switching between grid and table with selected filters because the JavaScript was attempting to access the form element with
id="views-exposed-form-acquia-dam-asset-library-widget"
. However, in some cases, it seems the form didn’t exist on the page, which resulted in a JavaScript error when trying to manipulate the form’s elements.Error Message (with private information removed):
An error occurred while attempting to process https://your-site-url/admin/content/media-dam-widget/asset_library_type?media_library_opener_id=media_library.opener.editor&media_library_allowed_types[0]=documents_asset&media_library_allowed_types[1]=image_asset&...&_wrapper_format=drupal_ajax: document.getElementById(...) is null.
Root Cause:
The JavaScript was trying to interact with a form that wasn't present on the page at the time of execution, causing a null reference error.Possible Solution:
By adding an early return when the form is not found, the script avoids further execution and prevents errors:if (!form) { return; }
This ensures that the script only proceeds if the form exists, avoiding the null reference issue. It worked for me, but you all might have more context on how to properly resolve it. I’m just sharing my input in case it helps.
I’ve attached the .diff → file for context and the image of the original issue (with private information removed).