Updated: Comment #5

Commit/Message

Issue #2095917 by penyaskito, e0ipso: Move Date format localization to language module.

Problem/Motivation

Move the Date format localization feature to the locale (Interface Translation) module.
META: #2052193: [META] Date format localisation is a huge mess, conflicts, does not work, regressed.

Proposed resolution

Move this functionality to language module, so we can adapt date formats to the different languages.

Remaining tasks

- Debate about if the right place should be locale or language.
- Patch review needed.

User interface changes

None.

API changes

None.

(A list of related issues.)

Files: 
CommentFileSizeAuthor
#4 date-format-locale-2095917-1.patch8.31 KBpenyaskito
PASSED: [[SimpleTest]]: [MySQL] 58,866 pass(es). View
#4 date-format-locale-2095917.1-2.txt7.25 KBpenyaskito
#4 date-format-locale-2095917-2.patch8.66 KBpenyaskito
FAILED: [[SimpleTest]]: [MySQL] 58,541 pass(es), 3 fail(s), and 0 exception(s). View
#3 date-format-locale-2095917.1-2.txt7.25 KBpenyaskito
#2 date-format-locale-2095917.1-2.txt7.25 KBpenyaskito

Comments

penyaskito’s picture

Assigned: Unassigned » penyaskito
penyaskito’s picture

Status: Active » Needs review
FileSize
7.25 KB

Moved the date format localize menu callbacks and forms to locale module.

penyaskito’s picture

Title: Move Date format localization to locale module » Move Date format localization to language module
FileSize
7.25 KB

While doing the patch before, I thought that maybe the right place is the language module.
Pairing in Prague with e0ipso, so credit to him too.

penyaskito’s picture

FileSize
8.66 KB
FAILED: [[SimpleTest]]: [MySQL] 58,541 pass(es), 3 fail(s), and 0 exception(s). View
7.25 KB
8.31 KB
PASSED: [[SimpleTest]]: [MySQL] 58,866 pass(es). View

The previous patch is only interdiff, sorry. Uploading full patches again.

date-format-locale-2095917-1.patch moves functionality to locale.
date-format-locale-2095917-2.patch moves functionality to language.

Status: Needs review » Needs work

The last submitted patch, date-format-locale-2095917-2.patch, failed testing.

Jose Reyero’s picture

I think moving it to language module doesn't make any sense since this is a 'localization' feature....

Anyway, there was this one in case it is any help, #2059835: Implement translation interface for date formats as part of locale UI

penyaskito’s picture

Status: Needs work » Closed (duplicate)

Closing as duplicated then, and updating META.

Gábor Hojtsy’s picture

Issue tags: -sprint
Gábor Hojtsy’s picture

Very sticky tag.

Gábor Hojtsy’s picture

Issue summary: View changes

Issue template.

penyaskito’s picture

Assigned: penyaskito » Unassigned
Issue summary: View changes