Strings beginning with space

Re: Strings beginning with space

Séverin Terrier -
Erantzun kopurua: 6

I must admit i'm not sure if all spaces are needed, but for sure some are.

You can look at screenshots in the french discussion i pointed to better understand: the problems are not double spaces (ignored by HTML rendering) but missing spaces.

Choice has been made to integrate them in strings rather than in Moodle code, and i wonder if it's the right way of doing it, and if there are recommandations about that.

In fact, when looking at translations, some (rare) have beginning space, and most not. If you launch, from moodledata/lang folder the command grep criteria_descr_bydate */badges.php | grep "'] = ' " you'll see that de and hr languages have beginning space, but most languages not.

I just don't know if AMOS doesn't allow beginning space, or if it just don't see the difference beetween previous existing string without space and the new one with space. If it's the case, perhaps it could be possible that the language maintener modify the string, and validate it, to be able to modify it again, adding space.

Even if it's possible that way, i still wonder if it shouldn't better be directly encoded.

Séverin

Séverin Terrier(e)ri erantzunda

Re: Strings beginning with space

koen roggemans -
Language pack maintainers-ren irudia

It's a no brainer: there shouldn't be any leading or trailing spaces required in translation because it's too error prone.

koen roggemans(e)ri erantzunda

Re: Strings beginning with space

Nicolas Martignoni -
Language pack maintainers-ren irudia

I completely agree. I suggest to amend the Coding Style documentation page to clarify this and add an example under "Language strings > Structure": https://docs.moodle.org/dev/Coding_style#Language_strings.

Is there some procedure to follow (e.g. "policy" issue to open, as this is a quite official document) or can I do it directly?

Nicolas Martignoni(e)ri erantzunda

Re: Strings beginning with space

David Mudrák -
Language pack maintainers-ren irudia

Yes, there is a procedure - please report such a suggestion as a new MDL issue, component "Policy".

I agree it is a bad practice to design strings with trailing and heading whitespace and I believe it should be considered a bug in the original English strings.

Meanwhile, you should be able to insert strings with whitespaces into AMOS though - https://lang.moodle.org/mod/forum/discuss.php?d=5908#p6932

David Mudrák(e)ri erantzunda

Re: Strings beginning with space

Séverin Terrier -

Thanks David for (pointing to) the tip : Maintaining CTRL before clicking on text editing place to avoid AMOS filtering irribarrea

Here's a command to search for english files containing trailing space (launch from Moodle dir) :

for i in `find . -type f -name '*.php' | grep 'lang/en/'`
  do grep -Hne " ';$" $i | grep string
done | sort | cut -c 3-

There are more than for begining space. I suppose my search won't find multiple lines strings...

Séverin

David Mudrák(e)ri erantzunda

Re: Strings beginning with space

Nicolas Martignoni -
Language pack maintainers-ren irudia

Just a notice about AMOS: it's impossible to propagate strings with leading spaces to other versions (or at least I didn't find how to).

Is it worth opening an tracker issue?