View Issue Details

Related ChangesetsJump to NotesJump to History
This bug affects 1 person(s).
 4
IDProjectCategoryView StatusDate SubmittedLast Update
12977Bug reportsSurvey takingpublic2017-11-29 16:472018-05-23 11:27
Reporterollehar Assigned ToLouisGac 
PrioritynoneSeverityminor 
Status closedResolutionfixed 
Product Version3.0.0-rc.x 
Target Version3.0.x 
Summary12977: The image poweredby.png creates asset folder, making fruity package not being published
Description

As title.

Check this line in Yii core: https://github.com/LimeSurvey/LimeSurvey/blob/master/framework/web/CAssetManager.php#L244

If the folder already exists, the package will not be copied. For some reason, and only in the travis docker image, the file poweredby.png published the package once, leading to Yii NOT copying/publishing the other files.

Steps To Reproduce

Start up docker etc. Ask Olle for details, or see this file:

https://github.com/LimeSurvey/LimeSurvey/blob/develop/tests/data/travis/travis_setup.sh

TagsNo tags attached.
Bug heat4
Complete LimeSurvey version number (& build)latest dev
I will donate to the project if issue is resolvedNo
Browser-
Database type & version-
Server OS (if known)-
Webserver software & version (if known)-
PHP Version-

Users monitoring this issue

There are no users monitoring this issue.

Activities

ollehar

ollehar

2017-11-29 16:54

administrator   ~45202

Changing

                {{ image("poweredby.png", "LimeSurvey Survey Software") }}

to

                {{ image("./files/poweredby.png", "LimeSurvey Survey Software") }}

resolves this bug.
ollehar

ollehar

2017-11-29 17:01

administrator   ~45203

To be clear, the bug appears when the following happens:

  1. poweredby.png is in theme root
  2. Twig layout include image("poweredby.png", "LimeSurvey Survey Software") }}
  3. This leads to the package being "pre-published"
  4. The ACTUAL package is never published since the folder ALREADY exists (the Twig line above triggers the publishing of the package, for some reason, but if the image is put in files/, it does not trigger this).
LouisGac

LouisGac

2018-05-23 11:27

developer   ~47741

new asset engine after RC

Related Changesets

LimeSurvey: develop 87d517f5

2017-11-29 17:06:02

ollehar

Details Diff 		Dev: Move powerby.png to files/ folder (see bug 12977) Affected Issues
12977
mod - themes/survey/fruity/views/layout_survey_list.twig Diff File

Issue History

Date Modified Username Field Change
2017-11-29 16:47 ollehar New Issue
2017-11-29 16:54 ollehar Note Added: 45202
2017-11-29 17:01 ollehar Note Added: 45203
2017-11-29 17:06 ollehar Changeset attached => LimeSurvey develop 87d517f5
2018-05-23 11:27 LouisGac Assigned To => LouisGac
2018-05-23 11:27 LouisGac Status new => closed
2018-05-23 11:27 LouisGac Resolution open => fixed
2018-05-23 11:27 LouisGac Note Added: 47741