Coroico / AdvSearch

Dynamic content search add-on for MODx Revolution that supports results highlighting, faceted search and search in custom packages
21 stars 14 forks source link

Empty script tag included when addJs=0 // possible fix #67

Closed vierkantemeter closed 10 years ago

vierkantemeter commented 10 years ago

I don't want to include Js, JQuery and CSS, so I added:

&addCss=0 &addJs=0 &addJQuery =0

This works as it should, except an empty script tag is added to the bottom of the document, before the closing body tag. I think the problem is line 175 in core/components/advsearch/model/advsearch/advsearchform.class.php. I commented the following lines out and now the empty tag is no longer included: //else //$this->modx->regClientScript($jsHeader);

goldsky commented 10 years ago

You are right. Thanx for reporting.