View Issue Details

This bug affects 1 person(s).
 8
IDProjectCategoryView StatusLast Update
06840Bug reportsTheme editorpublic2012-11-13 09:32
ReporterMazi Assigned Toc_schmitz  
PrioritynormalSeveritypartial_block 
Status closedResolutionfixed 
Product Version2.00+ 
Target Version2.00+Fixed in Version2.00+ 
Summary06840: When copying a shipped template the "images" folder is missing
Description

When creating a customized template based on a copy of a shipped template, files are missing at the created copy. For the shipped templates there is an "images" folder which is not copied.

Same problem shows up when exporting and importing a shipped template: The exported template is missing the "images" folder and even if you add it to the archive and then import the ZIP file, the "images" folder gets lost on import.

Steps To Reproduce

Copy a shipped template

Additional Information

I assume the problem is caused by the copy/import function which might not allow sub-folders. If this is right, we should either add all the files from the "images" folder directly to the main template folder or overwork the import/copy function.

TagsNo tags attached.
Bug heat8
Complete LimeSurvey version number (& build)121106
I will donate to the project if issue is resolvedNo
BrowserChrome
Database type & versionMySQL 5
Server OS (if known)Windows 7
Webserver software & version (if known)Apache 2.2
PHP Version5.3.1

Users monitoring this issue

c_schmitz

Activities

tpartner

tpartner

2012-11-08 14:49

partner   ~22019

Back to you Mazi, I'm afraid this is out of my area of expertise.

Mazi

Mazi

2012-11-08 15:20

updater   ~22020

Tony, if we go for the simple solution to just place all the files from the "images" folder into the template's root folder, then probably some of the CSS has to be re-written, too. Is that correct?

tpartner

tpartner

2012-11-08 15:34

partner   ~22021

Yup, the paths to background images will be broken. Also if JavaScript inserts images, that will be broken.

I'm not up on all of the security implications but ideally the template should be imported as-is.

c_schmitz

c_schmitz

2012-11-08 21:35

administrator   ~22028

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

c_schmitz

c_schmitz

2012-11-08 21:36

administrator   ~22029

Subdirectory are now properly handled in import/export/copy

c_schmitz

c_schmitz

2012-11-09 00:07

administrator   ~22037

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

c_schmitz

c_schmitz

2012-11-13 09:32

administrator   ~22101

New 2.00+ build released

Related Changesets

LimeSurvey: master 490460a4

2012-11-08 21:20

c_schmitz


Details Diff
Fixed issue 06840: When copying a shipped template the "images" folder is missing
Dev Remove phpzip library for good
Affected Issues
06840
mod - application/controllers/admin/export.php Diff File
mod - application/controllers/admin/templates.php Diff File
mod - application/helpers/admin/template_helper.php Diff File
rm - application/libraries/admin/Phpzip.php Diff
mod - application/views/admin/templates/importuploaded_view.php Diff File

LimeSurvey: 2.1 b9791de2

2012-11-09 00:06

c_schmitz


Details Diff
Fixed issue 06840: When copying a shipped template the "images" folder is missing
Dev Remove phpzip library for good
Affected Issues
06840
mod - application/controllers/admin/export.php Diff File
mod - application/controllers/admin/templates.php Diff File
mod - application/helpers/admin/template_helper.php Diff File
rm - application/libraries/admin/Phpzip.php Diff
mod - application/views/admin/templates/importuploaded_view.php Diff File

Issue History

Date Modified Username Field Change
2012-11-08 14:44 Mazi New Issue
2012-11-08 14:44 Mazi Status new => assigned
2012-11-08 14:44 Mazi Assigned To => tpartner
2012-11-08 14:44 Mazi Issue Monitored: c_schmitz
2012-11-08 14:49 tpartner Assigned To tpartner => Mazi
2012-11-08 14:49 tpartner Note Added: 22019
2012-11-08 15:00 c_schmitz Assigned To Mazi => c_schmitz
2012-11-08 15:20 Mazi Note Added: 22020
2012-11-08 15:34 tpartner Note Added: 22021
2012-11-08 21:35 c_schmitz Changeset attached => LimeSurvey master 490460a4
2012-11-08 21:35 c_schmitz Note Added: 22028
2012-11-08 21:35 c_schmitz Resolution open => fixed
2012-11-08 21:36 c_schmitz Note Added: 22029
2012-11-08 21:36 c_schmitz Status assigned => resolved
2012-11-08 21:36 c_schmitz Fixed in Version => 2.00+
2012-11-09 00:07 c_schmitz Changeset attached => LimeSurvey 2.1 b9791de2
2012-11-09 00:07 c_schmitz Note Added: 22037
2012-11-13 09:32 c_schmitz Note Added: 22101
2012-11-13 09:32 c_schmitz Status resolved => closed
2015-12-11 14:40 c_schmitz Category Templates => Theme editor
2021-08-04 08:18 guest Bug heat 6 => 8