06283: Quick translation is not working - LimeSurvey bugs, patches and feature requests

(19575)
fenyvitez (reporter)
2012-06-26 22:53

I've checked it, the error is produced in IE 10, too.

(19583)
tpartner (developer)
2012-06-27 23:20

Fix committed to Yii branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=8869 [^]

(19584)
tpartner (developer)
2012-06-27 23:23

fenyvitez, I have corrected the paths to the images but can't reproduce the problem of missing form elements.

Can you please retest when the next RC is released (or grab the latest from GitHub).

(19585)
fenyvitez (reporter)
2012-06-28 02:05

OK, I took the new translate.php. It fixed the broken image links but my page still does not load. I will look around to see what specificity of my installation causes this bug or look for further info that is needed to replicate the bug. Thanks anyway.

(19593)
fenyvitez (reporter)
2012-07-01 01:43

Hi tpartner,

I continued my search and when setting debugging level to two, I have received some clues what goes wrong. I made screenshot of the error messages, I am not sure I can upload it here but the first one basically says:
"Strict standards: Non-static method CHttpRequest::getParam() should not be called statically, assuming $this from incompatible context in C:\wamp\www\survey\application\controllers\admin\translate.php on line 28"

As a result in line 28 of the translate file $tolang is not set. And later on in line 49 it is not set again because I have only one additional language, and the condition on $langs requires more than one. (I guess more than 0 would be sufficient).

I hope this helps.

(19602)
tpartner (developer)
2012-07-03 22:48

Fix committed to Yii branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=8887 [^]

(19603)
tpartner (developer)
2012-07-03 22:52

fenyvitez, thanks for the follow-up.

Yes, the condition should be greater than 0 because it refers to the number of additional languages, not total languages.

(19834)
c_schmitz (administrator)
2012-07-19 23:34

2.00RC6 released.

Related Changesets
LimeSurvey: Yii 70ee2772 Timestamp: 2012-06-27 14:19:48 Author: tpartner Committer: tpartner [ Details ] [ Diff ]	Fixed issue 06283: Quick translation is not working Dev Corrected up path to admin theme images
	mod - application/controllers/admin/translate.php	[ Diff ] [ File ]

LimeSurvey: Yii dab518a4 Timestamp: 2012-07-03 13:47:37 Author: tpartner Committer: tpartner [ Details ] [ Diff ]	Fixed issue 06283: Quick translation is not working Dev Corrected minimum "additionalLanguages" condition
	mod - application/controllers/admin/translate.php	[ Diff ] [ File ]

Issue History
Date Modified	Username	Field	Change
2012-06-26 22:44	fenyvitez	New Issue
2012-06-26 22:53	fenyvitez	Note Added: 19575
2012-06-27 23:20	tpartner	Changeset attached	=> LimeSurvey Yii 70ee2772
2012-06-27 23:20	tpartner	Note Added: 19583
2012-06-27 23:20	tpartner	Assigned To	=> tpartner
2012-06-27 23:20	tpartner	Resolution	open => fixed
2012-06-27 23:23	tpartner	Note Added: 19584
2012-06-27 23:23	tpartner	Status	new => feedback
2012-06-28 02:05	fenyvitez	Note Added: 19585
2012-06-28 02:05	fenyvitez	Status	feedback => assigned
2012-07-01 01:43	fenyvitez	Note Added: 19593
2012-07-01 01:44	fenyvitez	File Added: ps_translate_error.jpg
2012-07-03 22:48	tpartner	Changeset attached	=> LimeSurvey Yii dab518a4
2012-07-03 22:48	tpartner	Note Added: 19602
2012-07-03 22:52	tpartner	Note Added: 19603
2012-07-03 22:52	tpartner	Status	assigned => resolved
2012-07-05 22:32	c_schmitz	Fixed in Version	=> 2.00RC5
2012-07-19 23:34	c_schmitz	Note Added: 19834
2012-07-19 23:34	c_schmitz	Status	resolved => closed

Notes
(19575) fenyvitez (reporter) 2012-06-26 22:53	I've checked it, the error is produced in IE 10, too.

(19583) tpartner (developer) 2012-06-27 23:20	Fix committed to Yii branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=8869 [^]

(19584) tpartner (developer) 2012-06-27 23:23	fenyvitez, I have corrected the paths to the images but can't reproduce the problem of missing form elements. Can you please retest when the next RC is released (or grab the latest from GitHub).

(19585) fenyvitez (reporter) 2012-06-28 02:05	OK, I took the new translate.php. It fixed the broken image links but my page still does not load. I will look around to see what specificity of my installation causes this bug or look for further info that is needed to replicate the bug. Thanks anyway.

(19593) fenyvitez (reporter) 2012-07-01 01:43	Hi tpartner, I continued my search and when setting debugging level to two, I have received some clues what goes wrong. I made screenshot of the error messages, I am not sure I can upload it here but the first one basically says: "Strict standards: Non-static method CHttpRequest::getParam() should not be called statically, assuming $this from incompatible context in C:\wamp\www\survey\application\controllers\admin\translate.php on line 28" As a result in line 28 of the translate file $tolang is not set. And later on in line 49 it is not set again because I have only one additional language, and the condition on $langs requires more than one. (I guess more than 0 would be sufficient). I hope this helps.

(19602) tpartner (developer) 2012-07-03 22:48	Fix committed to Yii branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=8887 [^]

(19603) tpartner (developer) 2012-07-03 22:52	fenyvitez, thanks for the follow-up. Yes, the condition should be greater than 0 because it refers to the number of additional languages, not total languages.

(19834) c_schmitz (administrator) 2012-07-19 23:34	2.00RC6 released.

Relationships