Denis, Carsten is very busy so I'm assigning this one to you. Let me know if you have any questions.

You can see the broken conditions and the broken progress bar here. At this demo survey, a following question should show up if you select "yes":

• 1.92 "basic" template: http://limesurvey-templates.com/20/index.php/survey/index/sid/657996/newtest/Y/lang/en

• 2.0 "basic" shipped template: http://limesurvey-templates.com/20/index.php/survey/index/sid/382536/newtest/Y/lang/en

1.8 -> 1.9 : Condition don't work (question.pstpl + {QUESTION_ESSENTIALS})
1.91 -> 1.92 : hide_tip class), if you don't add some line : then hide-tip attribute don't work.

There a lot of incompatibility each time we make a sub update (1.7,1.8, ....), here it's a major update 1 -> 2.

Thin we can have same think for 2.0, and now : we can start to use some PHP script to compress js file and css files.

I can do a
str_replace("<script type="text/javascript" src="{TEMPLATEURL}template.js"/></script>","{TEMPLATEJS}" ) for all template, but it's for 1.92 compatibility and there are other modification to do to have compatibilty between 1.92 and 2.0 (some css for example). And it's an instruction more for compatible template.

Here, to update template: just 1 line to change.
{TEMPLATEURL}template.js -> {TEMPLATEJS}

If you remind the 1.90 at the forum, we have a lot of user with : conditoon do not work, becaus the question.pstpl.

Denis

I think we should do the string replacement or some other "automatic" solution for existing custom templates - there will be a LOT of them.

We have this problem here too. It would nice if this could be fixed as the rest of 2.0 seems very good.

Jon

Thanks for your feedback, Denis.

I absolutely understand your point that between certain releases, incompatibilities may show up.
BUT in this case, this is really a major issue which we should be able to solve for the benefit of all the users who will be starting to switch to Limesurvey 2 soon and use their own templates.
As Tony has pointed out, there will be LOTS of users running into this issue.

In general I think incompatible template changes should get their own place in the release notes. Wordpress for example always tells what was changed for template designers and plugin developers.

For this specific change... Expression Manager introduced the same kind of issues when the validation css changed and some default block had to be added to all templates.
http://docs.limesurvey.org/Upgrade+hints+for+version+1.92
Don't know if that was linked somewhere... had to search a lot to find it. So I would vote for document and make it easy to find :)

I search for an:

str_replace("<script type="text/javascript" src="{TEMPLATEURL}template.js"/></script>","{TEMPLATEJS}" ) in upload/template/*/startpage.pstpl before apply it allways.

Denis

Fix committed to master branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=9568

Docs:
http://docs.limesurvey.org/The+template+editor#CSS_and_Javascript
http://docs.limesurvey.org/The+template+editor#Standard_Pages (Note, look to put elsewhere)

Mazi: can you look if it's OK for the update with the last GIT version.

Thanks for looking into this, Denis.

Unfortunately the fix only adds the missing {TEMPLATEJS} by editing the template files during update.

This is a problem for several reasons:

1. Lots of current LS 1.9x users set up LS2 as a test system and then later switch to the new version by copying surveys and templates. In this case the templates will not be updated.

2. There are users which run several Limesurvey installations and copy surveys and templates between them. Again, the templates will not be updated by this fix.

3. Some users, especially admins of larger organisations, have lots of templates stored on their local PC and import them when necessary. In this case, the template update will also not take place.

Though your fix is great code-wise, there are some more situations where your fix might not be sufficient.
Therefor I suggest a solution which improves the replacement function. It shouldn't be that difficult to check for the {TEMPLATEJS} placeholder when processing (the startpage.pstpl of) a template. if it is not found at the current template we simply skip the replacement and instead include the necessary JS files the way we did it for Limesurvey 1.9x.

This way all the problems mentioned above will be solved, we need no new DB version number and we also wouldn't need to take care of extending the DB setup and update procedure.

Do you think you could implement it this way?

Thanks!

I i test if TEMPLATEJS are in startpage.pstpl, then it can be adding 2 times:
in startpage.pstpl and in endpage.pstpl.

Some user can want to have some survey witjout javascript etc ....

The best wayi sto make an
str_replace("<script type="text/javascript" src="{TEMPLATEURL}template.js"/></script>","{TEMPLATEJS}" ) in upload/template/*/startpage.pstpl
Before replace {TEMPLATEJS} in replacement.php.

Shnoulle,

I talked to Mazi, discussed this and we both think that your solution is fine. I like this much better than replacing this on the fly again and again during runtime. Two points are missing though.

• In the template import the same replacement should be done.
• Edit the upgrade instructions and make clear that the templates need to be copied back to the installation before the installation is started/the database upgarde is run (and so the templates are updated)

Can you add these missing points, please?

Great work, btw.

• In the template import the same replacement should be done.,

Oh yeah ! GREAT IDEA !!!

Before this idea, i think of a "error line" at template editor:

• This template seem missed {TEMPLATEJS}
• This template seem misses question.pstpl
  Please see the documentation of LimeSurvey.

Denis

Yeah, great work Denis.

But just so I'm clear on the end result:

• All templates that are in the installation that is being upgraded and all imported templates will be automatically modified.

• Templates that are simply uploaded to the upload/templates directory by those people who don't use the template editor will not be updated and will need to be modified for use.

I think you are right, Tony... but that brings us to another problem:

What about templates being uploaded by FTP instead of using the template editor?

Have the same problem here.

Well, upload by FTP was never supported by the application, anyway. In general we have to assume that people use the import function.

Something i don't understand,

If you upload via FTP, you can't take time to change one line before upload ?

Denis
Tony : user who use sftp or some other access have the ability to replace one line in startpage.pstpl.

And again, it's not the only think to change betwwen template.
This one is great : you see it very quickliy, other are question specific, and maybe it's not you seeing it, but your customer.

Denis

You are right, Denis. We can't take care of users uploading templates via FTP. In this case we should point them to the manual.

So if we take care about template import (fix needed) and have another check when updating to LS2 (already implemented), I think that is all we can do for now.

Fix committed to master branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=9610

Looks fine to me Denis, thanks.

You forgot to raise the DBVersion on the .sql files but I did that for you now ;). Thank you for your work!

At install, yes, I see that .

Thanks to you

I can confirm that Denis' fix works great!

Thanks a lot for solving this issue, Denis.

New version 2.00+ build 121005 released.