Moodle

Language editing interface terms

Details

  • Type: Sub-task Sub-task
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: 2.0.1, 2.1, 2.2
  • Fix Version/s: 2.1.5, 2.2.2
  • Component/s: Language
  • Labels:
  • Environment:
    en language pack
  • Database:
    MySQL
  • Testing Instructions:
    Hide

    This is a no clear cut test for this one.

    1. Open the admin/tool/customlang/lang/en/tool_customlang.php (or admin/report/customlang/lang/en/report_customlang.php for 2.2) in patched version
      1. Make sure all variations of the word customisation use the Australian spelling of the word (an s instead of a z)
      2. Check that the descriptions are friendly to non developers
    Show
    This is a no clear cut test for this one. Open the admin/tool/customlang/lang/en/tool_customlang.php (or admin/report/customlang/lang/en/report_customlang.php for 2.2) in patched version Make sure all variations of the word customisation use the Australian spelling of the word (an s instead of a z ) Check that the descriptions are friendly to non developers
  • Affected Branches:
    MOODLE_20_STABLE, MOODLE_21_STABLE, MOODLE_22_STABLE
  • Fixed Branches:
    MOODLE_21_STABLE, MOODLE_22_STABLE
  • Pull from Repository:
  • Pull Master Branch:
    m_MDL-25996_making_language_editing_terms_user_friendly
  • Pull Master Diff URL:

Description

Language customization: Should be Language customisation, "z" is for en_us.

Checkout strings into translator and Save and check in strings into files: These terms are less than useful for users who have no knowledge of the underlying technology. This is developer speak that needs to be replaced.

Suggestions:

Open language pack for editing
Update language pack

The first of these suggestions should be redundant in an ideal world as selecting the language should take the user directly to this page without the additional step - why else would you select the language from the drop-down?

Issue Links

has been marked as being related by

Improvement - An enhancement to an existing Moodle feature. MDL-31582 Random step in language customiser which involves clicking a button and nothing else.

  • Minor - Minor loss of function where workaround is possible
  • Open - The issue is open and ready for the assignee to start work on it.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Mary Cooch added a comment -

I absolutely agree!

Show
Mary Cooch added a comment - I absolutely agree!
Hide
Permalink
Koen Roggemans added a comment -

English langpack is for moodle.com - Helen will take care of it.

Show
Koen Roggemans added a comment - English langpack is for moodle.com - Helen will take care of it.
Hide
Permalink
Helen Foster added a comment -

Ray, thanks for your report and apologies for not yet getting around to amending the language strings as suggested. I'm currently going through open language string issues and adding them as subtasks to MDL-29160 for an HQ developer to easily fix.

Show
Helen Foster added a comment - Ray, thanks for your report and apologies for not yet getting around to amending the language strings as suggested. I'm currently going through open language string issues and adding them as subtasks to MDL-29160 for an HQ developer to easily fix.
Hide
Permalink
Gerard Caulfield added a comment -

Hi Ray

I also agree.

Regarding the ideal world you mention, I have submitted an issue so that we can start working towards it MDL-31582 That extra step baffles me too.

I'm currently working on the other language issues.

Thank you for your input and time

Show
Gerard Caulfield added a comment - Hi Ray I also agree. Regarding the ideal world you mention, I have submitted an issue so that we can start working towards it MDL-31582 That extra step baffles me too. I'm currently working on the other language issues. Thank you for your input and time
Hide
Permalink
Dan Poltawski added a comment -

Looks good to me!

Show
Dan Poltawski added a comment - Looks good to me!
Hide
Permalink
Gerard Caulfield added a comment -

Thanks for reviewing this Dan

Show
Gerard Caulfield added a comment - Thanks for reviewing this Dan
Hide
Permalink
Eloy Lafuente (stronk7) added a comment -

The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week.

TIA and ciao

Show
Eloy Lafuente (stronk7) added a comment - The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week. TIA and ciao
Hide
Permalink
Gerard Caulfield added a comment -

rebased.

Show
Gerard Caulfield added a comment - rebased.
Hide
Permalink
Eloy Lafuente (stronk7) added a comment -

Uhm... it surprised me to find that:

  • the 22_STABLE branch had 4 less lines modified. Seems that the strings 'checkin' and 'savecontinue' have been NOT modified there.
  • the 21_STABLE branch has (adds) a trailing white line not present in other branches.
  • The 'savecheckin' string has different translations in master/22_STABLE

Perhaps we could, simply, prepare the master branch and cherry-pick it both to 22 and 21 stables? Just to make them 100% the same fix (unless there is some reason to keep them different).

Holding this for some hours....ciao

Show
Eloy Lafuente (stronk7) added a comment - Uhm... it surprised me to find that: the 22_STABLE branch had 4 less lines modified. Seems that the strings 'checkin' and 'savecontinue' have been NOT modified there. the 21_STABLE branch has (adds) a trailing white line not present in other branches. The 'savecheckin' string has different translations in master/22_STABLE Perhaps we could, simply, prepare the master branch and cherry-pick it both to 22 and 21 stables? Just to make them 100% the same fix (unless there is some reason to keep them different). Holding this for some hours....ciao
Hide
Permalink
Gerard Caulfield added a comment -

Looking at this now

Show
Gerard Caulfield added a comment - Looking at this now
Hide
Permalink
Gerard Caulfield added a comment -

Fixed the mentioned issues. Thanks Eloy.

Show
Gerard Caulfield added a comment - Fixed the mentioned issues. Thanks Eloy.
Hide
Permalink
Eloy Lafuente (stronk7) added a comment -

Integrated (21, 22 & master), thanks!

Show
Eloy Lafuente (stronk7) added a comment - Integrated (21, 22 & master), thanks!
Hide
Permalink
Rajesh Taneja added a comment -

Looks Good,
Checked "Language customisation" and all string appear fine.

Few Questions:

  1. Do we need to change string keys to reflect en-au ? like $string['filtercustomized'] ?
  2. Should we break long strings in multiple lines with strong tags, so they appear nicely on confirmation screen.
Show
Rajesh Taneja added a comment - Looks Good, Checked "Language customisation" and all string appear fine. Few Questions: Do we need to change string keys to reflect en-au ? like $string ['filtercustomized'] ? Should we break long strings in multiple lines with strong tags, so they appear nicely on confirmation screen.
Hide
Permalink
Eloy Lafuente (stronk7) added a comment -

Some changes to Moodle should be milestones in the project by themselves.

This is not the case and your fix is not so important, but your collaboration is highly appreciated, thanks!

Closing as fixed, ciao

Show
Eloy Lafuente (stronk7) added a comment - Some changes to Moodle should be milestones in the project by themselves. This is not the case and your fix is not so important, but your collaboration is highly appreciated, thanks! Closing as fixed, ciao

People

Vote (3)
Watch (5)

Dates

  • Created:
    Updated:
    Resolved: