Remove possibly empty JS tag

trsteel88 / TrsteelCkeditorBundle

Symfony2 bundle for easy integration of the CKEditor WYSIWYG

97 stars 58 forks source link

Remove possibly empty JS tag #71

Closed Max101 closed 10 years ago

Max101 commented 10 years ago

Remove not needed JS tag.

trsteel88 commented 10 years ago

The base path has to be defined before the javascript include.

Max101 commented 10 years ago

Well in that case the "if" could be set around the JS tag in order to prevent unnecessary code. Best wishes, Mitja

trsteel88 commented 10 years ago

Around what tag? I think it's unnecessary.

Max101 commented 10 years ago

{% autoescape true js %} script type="text/javascript" var CKEDITOR_BASEPATH = '{{ app.request.basePath ~ '/' ~ base_path }}'; /script {% endautoescape %}

It does not matter that much but Just in case...