[solved] 1.6.0.7 No JS files loaded?

[Pete78]

Just did a 1.6.0.7 from 1.6.0.6 one click upgrade.

 

Is it possible that no JS files are loaded?

 

At least, I don't see any in the page sources... 

 

Anybody else examined this problem?

[NemoPS]

No js loaded at all? both in the bo and fo? Are you using a custom theme? As they are not loaded, I assume nothing works as it should, right?

[Pete78]

Custom theme, yes. FO: No js at all, just some snippets from paypal module etc.

BO: JS loaded normal.

 

Just debugging and I think in the Controller.class, method smartyOutputContent might be a problem.

 

JS Files were added to the js_path (at least I can output them all) but the call to the javascript.tpl file seems to cause some problems. I will investigate further...

[NemoPS]

What happens if you use the default theme? does it work?

[Pete78]

Actually I don't want to switch again, dont want to lose module activation and deactivation pos etc. configuration.

 

Now I just replaced in the controller.php the

 

"protected function smartyOutputContent($content)"

 

method with an older one ( 1.6.0.6.

 

and js gets loaded as expected..

 

-> there might be a problem...with the latest one.

[Pete78]

OK, got it:

There should be a variable called 

 

PS_JS_DEFER in the configuration table.

 

you can replace it with this...for a quick fix at line 417 in Controller.php

 

$defer = true; //(bool)Configuration::get('PS_JS_DEFER');

 

 

I guess, (without further investigation) this variable will be set in the install process...

Edited June 18, 2014 by Pete78 (see edit history)

[Pete78]

Sorry for confusing, now I found the real problem:

 

The PS_JS_DEFER comes from the "move javascript to the end configuration

 

if this is turned to  "YES". everything works as expected but if switched to "NO" no JS will be loaded at all.

 

actually the source code is just wrong here...

[El Patron]

Important Note:

 

before putting this code in you have option to move JavaScript to the bottom, for 1.6.0.7 and higher.

 

back office-->advanced parms-->Move JavaScript to End-->YES!

 

this is the PREFERRED way to load JavaScript, it makes your above the fold load much faster.

 

can you check your themes/yourtheme/header.tpl

 

you should see this code, which will load .js if load to bottom turned off

{if isset($js_defer) && !$js_defer && isset($js_files) && isset($js_def)}
	{$js_def}
	{foreach from=$js_files item=js_uri}
	<script type="text/javascript" src="{$js_uri|escape:'html':'UTF-8'}"></script>
	{/foreach}
{/if}

[Pete78]

Thanks for the hint El Patron, 

this one was missing in the theme file. I guess it was introduced in one of the last releases ( the leo theme I am using is pretty new though).

 

Thanks again and I marked the problem as "resolved".

 

Peter

Edited June 18, 2014 by Pete78 (see edit history)

[jsbadillo]

Hello, Leo the administrator told me I have to follow this instructions, but I'm really new using prestashop, I've looked for this code, but is not in the header.tpl file, Do I have to paste it in the file?, If yes, where should I paste it?

Thank you!

 

 

can you check your themes/yourtheme/header.tpl

 

you should see this code, which will load .js if load to bottom turned off

{if isset($js_defer) && !$js_defer && isset($js_files) && isset($js_def)}
	{$js_def}
	{foreach from=$js_files item=js_uri}
	<script type="text/javascript" src="{$js_uri|escape:'html':'UTF-8'}"></script>
	{/foreach}
{/if}

 

 

Thanks for the hint El Patron, 

this one was missing in the theme file. I guess it was introduced in one of the last releases ( the leo theme I am using is pretty new though).

 

Thanks again and I marked the problem as "resolved".

 

Peter

[dioniz]

Paste it just after this code:

{if isset($css_files)}
    {foreach from=$css_files key=css_uri item=media}
        <link rel="stylesheet" href="{$css_uri|escape:'html':'UTF-8'}" type="text/css" media="{$media|escape:'html':'UTF-8'}" />
    {/foreach}
{/if}

[El Patron]

Important Note:

 

before putting this code in you have option to move JavaScript to the bottom, for 1.6.0.7 and higher.

 

back office-->advanced parms-->Move JavaScript to End-->YES!

 

this is the PREFERRED way to load JavaScript, it makes your above the fold load much faster.

[Rom Soul]

Thanks for this brilliant tip,

 

I was afraid I would have to investigate for the week !!

 

I have been asking for days now to the theme developer AND the Prestashop Addons team WHY my expensive theme refused to work correctly after I updated from 1.6.0.5 to 1.6.0.8 ... I wish they just sent me the url of this topic earlier ! How come something like this can happen in the code ? Sometimes, the Prestashop guys seriously scare me .

 

Thanks again !

[El Patron]

Thanks for this brilliant tip,

 

I was afraid I would have to investigate for the week !!

 

I have been asking for days now to the theme developer AND the Prestashop Addons team WHY my expensive theme refused to work correctly after I updated from 1.6.0.5 to 1.6.0.8 ... I wish they just sent me the url of this topic earlier ! How come something like this can happen in the code ? Sometimes, the Prestashop guys seriously scare me .

 

Thanks again !

 

Personally the PS development team did an excellent job but the problem is getting all the developers on board to use 'best practice'.  We want to decrease 'above the fold' page render by moving JavaScript to the bottom.  If your theme does not support bottom loaded JavaScript then you can use my module JavaPro, here is link to module forum. 

 

 I did some looking at other eCommerce cms's, and specifically the featured shops with 'big names', they not only do not support JavaScript bottom but don't have CCC of .css and .js.  So in my honest opinion the ps developers continue to push the envelope in providing best in class features, we just have to wait for theme developers to catch up. (like herding cats)

[rcking]

Guys,

talking about moving JS to the end. How to do it if I need one JS stays in header? (data layer for google tag manager). No matter what I do it always goes to the end :/

[NemoPS]

I guess the only way in that case is to hardcode what you need by modifying those modules