NPM Resource Changed ? ? Pending

User tests: Successful: Unsuccessful:

avatar wilsonge
wilsonge
14 Jul 2021

Pull Request for Issue #34665 .

Summary of Changes

Fixes finder when debug mode is enabled - due to the default JS filter not allowing <dd> tags by default. @dgrammatiko tagging you in case you have a better fix - as it's from the sanitiseHtml stuff - but I think this is fine.

Testing Instructions

Go to the Joomla Configuration > System and enable the "System Debug".
Go to components Smart Search and click the "Clear Index" button.
Go to components Smart Search and click the "Index" button.

Actual result BEFORE applying this Pull Request

Finder loads the modal but never progresses to show a progress bar

Expected result AFTER applying this Pull Request

Finder index's all content as expected

Documentation Changes Required

None

avatar wilsonge wilsonge - open - 14 Jul 2021
avatar wilsonge wilsonge - change - 14 Jul 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 14 Jul 2021
Category JavaScript Repository NPM Change
avatar dgrammatiko
dgrammatiko - comment - 14 Jul 2021

This is fine as is. Another approach is to add dd and dl in the allowedElements in core.js but I prefer your solution here

avatar richard67 richard67 - test_item - 15 Jul 2021 - Tested successfully
avatar richard67
richard67 - comment - 15 Jul 2021

I have tested this item successfully on 5cd4faa

Works like a charm.

Hint for other testers: You have to use npm run build:js to rebuild the js (or npm ci), or use the package built by drone.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/34795.

avatar ChristineWk ChristineWk - test_item - 15 Jul 2021 - Tested successfully
avatar ChristineWk
ChristineWk - comment - 15 Jul 2021

I have tested this item successfully on 5cd4faa

With Prebuilt Package


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/34795.

avatar richard67 richard67 - change - 15 Jul 2021
Status Pending Ready to Commit
Labels Added: ? NPM Resource Changed
avatar richard67
richard67 - comment - 15 Jul 2021

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/34795.

avatar Quy Quy - close - 15 Jul 2021
avatar Quy Quy - merge - 15 Jul 2021
avatar Quy Quy - change - 15 Jul 2021
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2021-07-15 14:30:24
Closed_By Quy
Labels Added: ?
avatar Quy
Quy - comment - 15 Jul 2021

Thanks!

Add a Comment

Login with GitHub to post a comment