double translation in course group form class
double (nested) translations:
method `build_header ($header_title)` translates the $header_title, using the common/libraries resource file. However, other methods calling this build_header method already translated the title (using the local resource file). First (and correct) translations:
in add_top_fields() on line 89
in build_creation_form() on lines 994 and 998
This seems to be intentional and for the time being harmless, but might lead to issues later on: if ever the translated string is itself a key in a common/libraries resource i18n file (especially the english version), the value will get retranslated into a possibly different string.
Doubly translated strings are marked with [CDA] tags in the screenshot.
Updated by Anonymous over 6 years ago
resolved in file application/weblcms/tool/course_group/php/lib/form/course_group_form.class.php
removed translation from line 106, in build_header, as every not every $header in the call to the build_header method needs translation: e.g. build_basic_editing_form on line 630 builds a header with the group name (untranslated) and then with some other fields like title, description, etc. (translated). Moreover, the build_header method looks for translations in the common\libraries context, whereas the calling methods look in the default conext application\weblcms\tool\course_group, which seems to me to be more correct.