View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
06840 | Bug reports | Theme editor | public | 2012-11-08 14:44 | 2012-11-13 09:32 |
Reporter | Mazi | Assigned To | c_schmitz | ||
Priority | normal | Severity | partial_block | ||
Status | closed | Resolution | fixed | ||
Product Version | 2.00+ | ||||
Target Version | 2.00+ | Fixed in Version | 2.00+ | ||
Summary | 06840: 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. | ||||
Tags | No tags attached. | ||||
Bug heat | 8 | ||||
Complete LimeSurvey version number (& build) | 121106 | ||||
I will donate to the project if issue is resolved | No | ||||
Browser | Chrome | ||||
Database type & version | MySQL 5 | ||||
Server OS (if known) | Windows 7 | ||||
Webserver software & version (if known) | Apache 2.2 | ||||
PHP Version | 5.3.1 | ||||
Back to you Mazi, I'm afraid this is out of my area of expertise. |
|
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? |
|
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. |
|
Fix committed to master branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=10203 |
|
Subdirectory are now properly handled in import/export/copy |
|
Fix committed to 2.1 branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=10213 |
|
New 2.00+ build released |
|
LimeSurvey: master 490460a4 2012-11-08 21:20 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 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 |
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 |