Bug #3594
yoxview triggers js message in Dutch or extended language
Description
yoxview generates an error message when loading a document page (with some content) in Dutch or in any sub-language.
For the sublanguage, we fixed the problem in CKeditor already by using the same ISO code for the parent and the children languages. For the Dutch problem, however, it boils down to something else.
It's been reported on the forum: http://www.chamilo.org/fr/node/1429#comment-2478 and for now I have suggested the quick-patch of commenting the code that triggers the error message. This is not a long-term solution though.
In my opinion, we need to get a reason for the error (the language file path is correct and can be loaded in a browser; and the Dutch language file is syntactically correct too).
Files
Associated revisions
Undo js error comment + fixing NL yoxview file see #3594
History
Updated by Julio Montoya over 8 years ago
Updated by - File english.png english.png added
- File nl.png nl.png added
I found a weird problem about break spaces see the differences with spanish and nl
Updated by Julio Montoya over 8 years ago
- Status changed from New to Assigned
- Assignee set to Julio Montoya
Updated by Julio Montoya over 8 years ago
- Status changed from Assigned to Needs more info
- Assignee changed from Julio Montoya to Yannick Warnier
- % Done changed from 0 to 90
Removing the /* */ from the yoxview.min file and fixing the NL file
you could just replace your bad nl file with this one:
Updated by Bas Wijnen over 8 years ago
Updated by Your new file still has HelpText and SlideShow on one line. I think the problem is only about the final }, so it might work anyway (I can't test it now; my web host caches the old file, so I need to wait several hours).
Updated by Yannick Warnier over 8 years ago
Updated by Bas Wijnen wrote:
Your new file still has HelpText and SlideShow on one line. I think the problem is only about the final }, so it might work anyway (I can't test it now; my web host caches the old file, so I need to wait several hours).
Bas, any chance you would have verified this?
Updated by Yannick Warnier over 8 years ago
- Status changed from Needs more info to Bug resolved
Closing for lack of feedback, but could be re-opened or further commented if needed.
Temporarily commented error messaging code in language file loader (in any case it defaults to English) - refs #3594