Skip to content

Commit

Permalink
fixed issue #12414: exporting printable - template assets not visible…
Browse files Browse the repository at this point in the history 
… in some extractors (eg windows default) (#701)
  • Loading branch information
@TonisOrmisson @LouisGac
TonisOrmisson authored and LouisGac committed Jun 23, 2017
1 parent 6f68b9b commit 3649d43
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion application/controllers/admin/export.php
View file
Expand Up @@ -1370,7 +1370,7 @@ public function quexml($iSurveyID)
*/
private function _exportPrintableHtmls($iSurveyID,$readFile = true){
$oSurvey = Survey::model()->findByPk($iSurveyID);
$assetsDir = Template::getTemplateURL($oSurvey->template);
$assetsDir = substr(Template::getTemplateURL($oSurvey->template),1);
$fullAssetsDir = Template::getTemplatePath($oSurvey->template);
$aLanguages = $oSurvey->getAllLanguages();

Expand Down

4 comments on commit 3649d43

@Shnoulle
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Question : it's only to remove last / ? If yes : i use more rtrim($string,"/"); then if we update getTemplateURL in another release : still working :)

@Shnoulle
Copy link
Collaborator

@Shnoulle Shnoulle commented on 3649d43 Jun 23, 2017 •
edited

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I can do it, but no windows for testing. Please @TonisOrmisson can you confirm ?

@TonisOrmisson
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes it was about the slash. cant confirm now, on vacation :)

@Shnoulle
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK, thanks, i update monday :)

Please sign in to comment.