View Issue Details

This bug affects 1 person(s).
 6
IDProjectCategoryView StatusLast Update
19932Bug reportsOtherpublic2025-01-22 13:51
Reportertibor.pacalat Assigned Totibor.pacalat  
PrioritynoneSeverityblock 
Status resolvedResolutionfixed 
Product Version6.6.x 
Summary19932: Getting 500 after changing interface language to Arabic
Description

Getting 500 after changing interface language to Arabic.

This is on testing env.

Steps To Reproduce

Steps to reproduce

  1. Go to /admin/user/sa/personalsettings
  2. Change language to Arabic

Expected result

(Write here what you expected to happen)

Actual result

(Write here what happened instead)

TagsNo tags attached.
Attached Files
Bug heat6
Complete LimeSurvey version number (& build) 6.10.0+250106
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

gabrieljenik

gabrieljenik

2025-01-17 14:58

manager   ~81831

Problem here is that the Arab translation of 'At least %d number.|At least %d numbers.' doesn't have a pipe. So we just need to fix the translation

DenisChenu

DenisChenu

2025-01-17 15:50

developer   ~81832

Problem here is that the Arab translation of 'At least %d number.|At least %d numbers.' doesn't have a pipe. So we just need to fix the translation

Maybe we need to avoid send a error if debug are not set ?
Catch error if happen ?

gabrieljenik

gabrieljenik

2025-01-17 15:58

manager   ~81833

Maybe we need to avoid send a error if debug are not set ?
Catch error if happen ?

the problem here is the sprintf, not our function.
As we don't use our own function, it is very hard to catch the specific error

DenisChenu

DenisChenu

2025-01-17 16:43

developer   ~81834

OK :)

We don't use https://github.com/LimeSurvey/LimeSurvey/blob/master/application/helpers/common_helper.php#L52 ? ngT ?
Or Yii:t ?

Can you give me the link to this translation ?

gabrieljenik

gabrieljenik

2025-01-17 17:41

manager   ~81835

https://translate.limesurvey.org/projects/limesurvey-6/ar/default/?filters%5Bterm%5D=At+least+%25d+number.%7CAt+least+%25d+numbers.&filters%5Bterm_scope%5D=scope_any&filters%5Bstatus%5D=current_or_waiting_or_fuzzy_or_untranslated&filters%5Buser_login%5D=&filter=Apply+Filters&sort%5Bby%5D=priority&sort%5Bhow%5D=desc

gabrieljenik

gabrieljenik

2025-01-17 17:44

manager   ~81836

We don't use https://github.com/LimeSurvey/LimeSurvey/blob/master/application/helpers/common_helper.php#L52 ? ngT ?

Yes, but the exception is triggered by sprintf, as there are 2 placeholder "%d" in the string, and only one parameter.

DenisChenu

DenisChenu

2025-01-17 18:47

developer   ~81838

https://translate.limesurvey.org/projects/limesurvey-6/ar/default/?filters%5Bterm%5D=At+least+%25d+number.%7CAt+least+%25d+numbers.&filters%5Bterm_scope%5D=scope_any&filters%5Bstatus%5D=current_or_waiting_or_fuzzy_or_untranslated&filters%5Buser_login%5D=&filter=Apply+Filters&sort%5Bby%5D=priority&sort%5Bhow%5D=desc

I mean the link to the sprintf ;)

DenisChenu

DenisChenu

2025-01-17 18:47

developer   ~81839

Oups : https://github.com/LimeSurvey/LimeSurvey/blob/master/application/models/User.php#L485

DenisChenu

DenisChenu

2025-01-17 18:49

developer   ~81840

I'm OK with Gabriel !: it's not a code issue : translation issue.

Create a system for this, make more code than really needed since error came from po file.

tibor.pacalat

tibor.pacalat

2025-01-20 17:37

administrator   ~81861

@DenisChenu sorry I don't understand what do you propose here? Should we write some code that would more gracefully handle this or?

DenisChenu

DenisChenu

2025-01-20 17:38

developer   ~81862

I mean : we need to fix translation, PHP code didn't have any issue.

gabrieljenik

gabrieljenik

2025-01-21 14:51

manager   ~81872

On both Arab and Italian

tibor.pacalat

tibor.pacalat

2025-01-22 13:51

administrator   ~81884

Italian also looks good to me, maybe someone changed it in the meantime.

Issue History

Date Modified Username Field Change
2025-01-15 12:32 tibor.pacalat New Issue
2025-01-15 12:32 tibor.pacalat File Added: Screen-2025-01-15-122515.mp4
2025-01-15 12:33 tibor.pacalat Assigned To => gabrieljenik
2025-01-15 12:33 tibor.pacalat Status new => assigned
2025-01-15 12:41 tibor.pacalat Assigned To gabrieljenik => twilllig
2025-01-15 12:41 tibor.pacalat Description Updated
2025-01-17 14:58 gabrieljenik Note Added: 81831
2025-01-17 14:58 gabrieljenik Bug heat 0 => 2
2025-01-17 14:59 gabrieljenik Assigned To twilllig => DenisChenu
2025-01-17 14:59 gabrieljenik Status assigned => ready for code review
2025-01-17 15:50 DenisChenu Note Added: 81832
2025-01-17 15:50 DenisChenu Bug heat 2 => 4
2025-01-17 15:58 gabrieljenik Note Added: 81833
2025-01-17 16:43 DenisChenu Note Added: 81834
2025-01-17 17:41 gabrieljenik Note Added: 81835
2025-01-17 17:44 gabrieljenik Note Added: 81836
2025-01-17 18:47 DenisChenu Note Added: 81838
2025-01-17 18:47 DenisChenu Note Added: 81839
2025-01-17 18:49 DenisChenu Assigned To DenisChenu => tibor.pacalat
2025-01-17 18:49 DenisChenu Status ready for code review => ready for testing
2025-01-17 18:49 DenisChenu Note Added: 81840
2025-01-20 17:37 tibor.pacalat Note Added: 81861
2025-01-20 17:37 tibor.pacalat Bug heat 4 => 6
2025-01-20 17:38 DenisChenu Note Added: 81862
2025-01-20 18:00 tibor.pacalat Status ready for testing => resolved
2025-01-20 18:00 tibor.pacalat Resolution open => fixed
2025-01-21 14:51 gabrieljenik Note Added: 81872
2025-01-22 13:51 tibor.pacalat Note Added: 81884