[invalid] cms_jquery tag output and documentation

For problem reports, and discussions about the CMSMS 2.2 beta.

This board will be locked once CMSMS 2.2 is officially released.
Locked
deactivated010521

[invalid] cms_jquery tag output and documentation

Post by deactivated010521 »

While testing the cms_jquery tag I found out the output starts with:

Code: Select all

</__script>
this can break a cms page.

Code: Select all

{cms_jquery cdn='true' include_css=0}
output:

Code: Select all

</__script>
<__script__ type="text/javascript" src="https://ajax.googleapis.com/ajax/libs/jqueryui/1.10.4/jquery-ui.min.js"></__script>
<__script__ type="text/javascript" src="//cmsms.biesheuvel.lan/lib/jquery/js/jquery.mjs.nestedSortable.js"></__script>
<__script__ type="text/javascript" src="//cmsms.biesheuvel.lan/lib/jquery/js/jquery.json-2.4.min.js"></__script>
<__script__ type="text/javascript" src="//cmsms.biesheuvel.lan/lib/jquery/js/jquery-migrate-1.2.1.min.js"></__script>
documentation:

Also the documentation is not in sync with the output, the tag help file reads:

Code: Select all

>Known Scripts:

    jQuery(1.11.1)
    jQuery-UI(1.8.21)
    nestedSortable(1.3.4)
    json(2.3)
    migrate(2.3)
calguy1000
Support Guru
Support Guru
Posts: 8169
Joined: Tue Oct 19, 2004 6:44 pm
Location: Fernie British Columbia, Canada

Re: cms_jquery tag output and documentation

Post by calguy1000 »

I could not reproduce this issue.

When I used the tag you specified, I got this in my html output:

Code: Select all

<__script__ type="text/javascript" src="https://ajax.googleapis.com/ajax/libs/jquery/1.11.1/jquery.min.js"></__script>
<__script__ type="text/javascript" src="https://ajax.googleapis.com/ajax/libs/jqueryui/1.10.4/jquery-ui.min.js"></__script>
<__script__ type="text/javascript" src="//calguy1000.dynu.com/cms2_adodb/lib/jquery/js/jquery.mjs.nestedSortable.js"></__script>
<__script__ type="text/javascript" src="//calguy1000.dynu.com/cms2_adodb/lib/jquery/js/jquery.json-2.4.min.js"></__script>
<__script__ type="text/javascript" src="//calguy1000.dynu.com/cms2_adodb/lib/jquery/js/jquery-migrate-1.2.1.min.js"></__script>
Follow me on twitter
Please post system information from "Extensions >> System Information" (there is a bbcode option) on all posts asking for assistance.
--------------------
If you can't bother explaining your problem well, you shouldn't expect much in the way of assistance.
User avatar
Rolf
Power Poster
Power Poster
Posts: 7825
Joined: Wed Apr 23, 2008 7:53 am
Location: The Netherlands
Contact:

Re: cms_jquery tag output and documentation

Post by Rolf »

I added in template:

Code: Select all

<!-- ========================================================= -->

{cms_jquery cdn='true' include_css=0}

<!-- ========================================================= -->

Code: Select all

<!-- ========================================================= -->
<__script__ type="text/javascript" src="https://ajax.googleapis.com/ajax/libs/jquery/1.11.1/jquery.min.js"></__script>
<__script__ type="text/javascript" src="https://ajax.googleapis.com/ajax/libs/jqueryui/1.10.4/jquery-ui.min.js"></__script>
<__script__ type="text/javascript" src="//localhost/trunk/lib/jquery/js/jquery.mjs.nestedSortable.js"></__script>
<__script__ type="text/javascript" src="//localhost/trunk/lib/jquery/js/jquery.json-2.4.min.js"></__script>
<__script__ type="text/javascript" src="//localhost/trunk/lib/jquery/js/jquery-migrate-1.2.1.min.js"></__script>
<!-- ========================================================= -->
- + - + - + - + - + - + -
LATEST TUTORIAL AT CMS CAN BE SIMPLE:
Migrating Company Directory module to LISE
- + - + - + - + - + - + -
Image
User avatar
Rolf
Power Poster
Power Poster
Posts: 7825
Joined: Wed Apr 23, 2008 7:53 am
Location: The Netherlands
Contact:

Re: cms_jquery tag output and documentation

Post by Rolf »

function cms_get_jquery:

Code: Select all

  // Scripts to include
  $scripts['jquery'] = array('cdn'=>'https://ajax.googleapis.com/ajax/libs/jquery/1.11.1/jquery.min.js',
			     'local'=>$basePath.'/lib/jquery/js/jquery-1.11.1.min.js',
			     'aliases'=>array('jquery.min.js','jquery',));
  $scripts['jquery-ui'] = array('cdn'=>'https://ajax.googleapis.com/ajax/libs/jqueryui/1.10.4/jquery-ui.min.js',
				'local'=>$basePath.'/lib/jquery/js/jquery-ui-1.10.4.custom.min.js',
				'aliases'=>array('jquery-ui.min.js','ui'),
				'css'=>$basePath.'/lib/jquery/css/smoothness/jquery-ui-1.10.4.custom.min.css');
  $scripts['nestedSortable'] = array('local'=>$basePath.'/lib/jquery/js/jquery.mjs.nestedSortable.js');
  $scripts['json'] = array('local'=>$basePath.'/lib/jquery/js/jquery.json-2.4.min.js');
  $scripts['migrate'] = array('local'=>$basePath.'/lib/jquery/js/jquery-migrate-1.2.1.min.js');
Help text
>Known Scripts:

jQuery(1.11.1)
jQuery-UI(1.8.21)
nestedSortable(1.3.4)
json(2.3)
migrate(2.3)
- + - + - + - + - + - + -
LATEST TUTORIAL AT CMS CAN BE SIMPLE:
Migrating Company Directory module to LISE
- + - + - + - + - + - + -
Image
deactivated010521

Re: cms_jquery tag output and documentation

Post by deactivated010521 »

Excuse me, don't know what happened cleared the cache and added the tag again. Everything looks fine so mark as [false].
User avatar
Rolf
Power Poster
Power Poster
Posts: 7825
Joined: Wed Apr 23, 2008 7:53 am
Location: The Netherlands
Contact:

Re: [invalid] cms_jquery tag output and documentation

Post by Rolf »

Help text updated.
- + - + - + - + - + - + -
LATEST TUTORIAL AT CMS CAN BE SIMPLE:
Migrating Company Directory module to LISE
- + - + - + - + - + - + -
Image
Locked

Return to “[locked] CMSMS 2.2 Beta”