View Issue Details

This bug affects 1 person(s).
 6
IDProjectCategoryView StatusLast Update
08232Bug reportsTheme editorpublic2013-10-23 14:42
Reporterintentcook Assigned Totpartner  
PrioritynormalSeverityminor 
Status closedResolutionfixed 
Fixed in Version2.05 RC 
Summary08232: progress bar only part work in all templates
Description

In 2.05 Beta 6 130924 the progress bar on partially works or just show 0% or 100% in any template.

Steps To Reproduce

Use default template and show progress bar in any simple survey.

TagsNo tags attached.
Attached Files
Basic template progess bar.PNG (20,700 bytes)   
Basic template progess bar.PNG (20,700 bytes)   
Bug heat6
Complete LimeSurvey version number (& build)2.05 beta 6 130924
I will donate to the project if issue is resolvedYes
BrowserChrome, Firefox and IE
Database type & versionmysql 5.1.70-cll
Server OS (if known)Apache version 2.2.25
Webserver software & version (if known)Apache version 2.2.25
PHP Version5.4.19

Users monitoring this issue

There are no users monitoring this issue.

Activities

tpartner

tpartner

2013-10-04 15:02

partner   ~26515

I cannot reproduce the problem.

Can you please:

1) Attach a sample survey
2) List any errors found in the JavaScript console

intentcook

intentcook

2013-10-06 03:36

reporter   ~26542

Attached sample survey there are two problems and in it is inconsistent - one as per ptx above and the second where the progress bar never gets to 100% as in the attached survey. The ptx basic now works but only get to 50% - see comment below.

I think on the second one the problem is that it counts all hidden equation questions and if you have say 20 hidden and 20 answerable questions then it only goes to 50% ??

intentcook

intentcook

2013-10-06 03:51

reporter   ~26543

Denying load of chrome-extension://obcbigljfpgappaaofailjjoabiikckk/js/lib/jquery.min.map. Resources must be listed in the web_accessible_resources manifest key in order to be loaded by pages outside the extension. /surveybeta/index.php/survey/index:13
foxclocks.utils._setMomentLang(): exception setting lang to 'en-us' utils.js:13
foxclocks.utils._setMomentLang(): failed to set lang to 'en-us' based on app locale 'en-US' utils.js:13
GET http://www.5deep.co.nz/surveybeta/scripts/jquery/jquery.keypad.min.js 404 (Not Found) index:18
GET http://www.5deep.co.nz/surveybeta/index.php/survey/jquery.keypad.alt.css 404 (Not Found) index:10
Viewport argument value "device-width;" for key "width" is invalid, and has been ignored. Note that ';' is not a separator in viewport values. The list should be comma-separated. index:30
Viewport argument value "1.0;" for key "initial-scale" was truncated to its numeric prefix. Note that ';' is not a separator in viewport values. The list should be comma-separated. index:30
Viewport argument value "1;" for key "user-scalable" was truncated to its numeric prefix. Note that ';' is not a separator in viewport values. The list should be comma-separated. index:30
Uncaught TypeError: Object [object Object] has no method 'keypad'

intentcook

intentcook

2013-10-06 03:52

reporter   ~26544

Last edited: 2013-10-06 03:54

On the second survey I am uploading 843357 the progress bar only show 0% 100% using the same template. the console example was from this survey.

tpartner

tpartner

2013-10-06 14:33

partner   ~26546

Carsten,it seems that jquery.keypad.min.js and its locale files are missing. Do we want those in /scripts or /third_party?

c_schmitz

c_schmitz

2013-10-19 23:07

administrator   ~26838

They should be placed in /third_party. Thank you!

tpartner

tpartner

2013-10-20 15:53

partner   ~26849

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

tpartner

tpartner

2013-10-20 15:55

partner   ~26850

Carsten, I wanted to add a test for the locale file but this doesn't seem to work:

if(file_exists(Yii::app()->getConfig('third_party').'jquery-keypad/i18n/jquery.ui.keypad-'.$clang->langcode.'.js')) {}

Thoughts?

c_schmitz

c_schmitz

2013-10-22 10:54

administrator   ~26895

Tony, what does not work about it?

tpartner

tpartner

2013-10-22 12:32

partner   ~26899

When I add that condition to the IF construct, and the file exist, it is not included.

For example in a German survey, when I leave that out, jquery.ui.keypad-de.js is included (as it should be) but with the condition it is not included.

So, this works:
if ($clang->langcode != 'en') {...}

But this doesn't:
if ($clang->langcode != 'en' && file_exists(Yii::app()->getConfig('third_party').'jquery-keypad/i18n/jquery.ui.keypad-'.$clang->langcode.'.js')) {...}

And since there are not locale files for all languages we really should have this filter.

c_schmitz

c_schmitz

2013-10-22 13:57

administrator   ~26900

Ah ok, Yii::app()->getConfig('third_party'). is a URL, not a file path.

tpartner

tpartner

2013-10-22 15:49

partner   ~26919

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

tpartner

tpartner

2013-10-22 15:50

partner   ~26920

Fixed in 2.05

c_schmitz

c_schmitz

2013-10-23 14:42

administrator   ~26961

2.05RC3 released

Related Changesets

LimeSurvey: 2.05 4d8e8864

2013-10-20 13:52:43

tpartner

Details Diff
Fixed issue 08232: progress bar only part work in all templates
Dev Added missing JS files
Affected Issues
08232
mod - application/helpers/common_helper.php Diff File
add - third_party/jquery-keypad/i18n/jquery.ui.keypad-de.js Diff File
add - third_party/jquery-keypad/i18n/jquery.ui.keypad-es.js Diff File
add - third_party/jquery-keypad/i18n/jquery.ui.keypad-fr.js Diff File
add - third_party/jquery-keypad/i18n/jquery.ui.keypad-nl.js Diff File
add - third_party/jquery-keypad/i18n/jquery.ui.keypad-pt-BR.js Diff File
add - third_party/jquery-keypad/jquery.keypad.min.js Diff File

LimeSurvey: 2.05 2d3bd2ea

2013-10-22 13:46:25

tpartner

Details Diff
Fixed issue 08232: progress bar only part work in all templates
Dev Added test for existence of locale file
Affected Issues
08232
mod - application/helpers/common_helper.php Diff File

Issue History

Date Modified Username Field Change
2013-10-04 01:59 intentcook New Issue
2013-10-04 01:59 intentcook File Added: Basic template progess bar.PNG
2013-10-04 15:02 tpartner Note Added: 26515
2013-10-06 03:36 intentcook Note Added: 26542
2013-10-06 03:36 intentcook File Added: CSI_limesurvey_survey_326535.lss
2013-10-06 03:51 intentcook Note Added: 26543
2013-10-06 03:52 intentcook Note Added: 26544
2013-10-06 03:53 intentcook File Added: VT_limesurvey_survey_843357.lss
2013-10-06 03:54 intentcook Note Edited: 26544
2013-10-06 14:33 tpartner Note Added: 26546
2013-10-06 14:33 tpartner Assigned To => c_schmitz
2013-10-06 14:33 tpartner Status new => assigned
2013-10-19 23:06 c_schmitz Assigned To c_schmitz => tpartner
2013-10-19 23:07 c_schmitz Note Added: 26838
2013-10-20 15:53 tpartner Changeset attached => LimeSurvey 2.05 4d8e8864
2013-10-20 15:53 tpartner Note Added: 26849
2013-10-20 15:53 tpartner Resolution open => fixed
2013-10-20 15:55 tpartner Note Added: 26850
2013-10-20 15:55 tpartner Assigned To tpartner => c_schmitz
2013-10-22 10:54 c_schmitz Note Added: 26895
2013-10-22 12:32 tpartner Note Added: 26899
2013-10-22 13:57 c_schmitz Note Added: 26900
2013-10-22 13:57 c_schmitz Assigned To c_schmitz => tpartner
2013-10-22 15:49 tpartner Changeset attached => LimeSurvey 2.05 2d3bd2ea
2013-10-22 15:49 tpartner Note Added: 26919
2013-10-22 15:50 tpartner Note Added: 26920
2013-10-22 15:50 tpartner Status assigned => resolved
2013-10-23 14:42 c_schmitz Note Added: 26961
2013-10-23 14:42 c_schmitz Status resolved => closed
2013-10-23 14:42 c_schmitz Fixed in Version => 2.05 RC
2015-12-11 14:40 c_schmitz Category Templates => Theme editor