Comment 12 for bug 1902965

Revision history for this message
Jeff Davis (jdavis-sitka) wrote :

Thanks for testing, Jason.

Security branch user/jeffdavis/lp1902965-filter-marc-fields-2 includes a revised version of the TPAC changes to accommodate the Read More accordion. (Please note I have force-pushed changes to this branch since it was first created a few days ago.) The previous branch should be used for 3.6 and earlier releases.

In this new branch, the accordion macro takes care of filtering, so any test that gets accordion'd shouldn't need an additional html filter.

It appears that the accordion does not exist in the Bootstrap OPAC, so there is some code drift happening between the OPACs. Some adjustments to HTML filtering will be necessary if and when the accordion does make it into Bootstrap.

It is frustrating to have to redo a security fix because it has languished while new features are committed.