View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
19932 | Bug reports | Other | public | 2025-01-15 12:32 | 2025-01-22 13:51 |
Reporter | tibor.pacalat | Assigned To | tibor.pacalat | ||
Priority | none | Severity | block | ||
Status | resolved | Resolution | fixed | ||
Product Version | 6.6.x | ||||
Summary | 19932: 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
Expected result(Write here what you expected to happen) Actual result(Write here what happened instead) | ||||
Tags | No tags attached. | ||||
Attached Files | |||||
Bug heat | 6 | ||||
Complete LimeSurvey version number (& build) | 6.10.0+250106 | ||||
I will donate to the project if issue is resolved | No | ||||
Browser | |||||
Database type & version | . | ||||
Server OS (if known) | |||||
Webserver software & version (if known) | |||||
PHP Version | . | ||||
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 ? |
|
the problem here is the sprintf, not our function. |
|
OK :) We don't use https://github.com/LimeSurvey/LimeSurvey/blob/master/application/helpers/common_helper.php#L52 ? ngT ? Can you give me the link to this translation ? |
|
Yes, but the exception is triggered by sprintf, as there are 2 placeholder "%d" in the string, and only one parameter. |
|
I mean the link to the sprintf ;) |
|
Oups : https://github.com/LimeSurvey/LimeSurvey/blob/master/application/models/User.php#L485 |
|
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. |
|
@DenisChenu sorry I don't understand what do you propose here? Should we write some code that would more gracefully handle this or? |
|
I mean : we need to fix translation, PHP code didn't have any issue. |
|
On both Arab and Italian |
|
Italian also looks good to me, maybe someone changed it in the meantime. |
|
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 |