Opened 11 years ago

Closed 11 years ago

#16076 closed New feature (fixed)

Updated locale formats for hungarian language

Reported by: slink Owned by: nobody
Component: Internationalization Version: dev
Severity: Normal Keywords: locale, formats
Cc: Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

The formats module for the hungarian language was very poor, so I updated it.

Attachments (1)

formats.patch​ (1.4 KB) - added by slink 11 years ago.
Patch for hungarian locale formats module

Download all attachments as: .zip

Change History (7)

comment:1 Changed 11 years ago by anonymous

Triage Stage: Unreviewed → Accepted

comment:2 Changed 11 years ago by Julien Phalip

Needs tests: set
UI/UX: unset

Thanks for the report. Could you provide some tests to go along with your fix?

comment:3 Changed 11 years ago by Russell Keith-Magee

Easy pickings: unset
Needs tests: unset
Triage Stage: Accepted → Ready for checkin

@julien Formats don't require tests; all you'd be testing is that the format string works as expected. It would be like adding tests for translations. We have tests to validate that the format keys work, but we don't need to test that the definitions for each language work as expected.

comment:4 Changed 11 years ago by Jannis Leidel

Is the first entry to TIME_INPUT_FORMATS supposed to have a comma?

Changed 11 years ago by slink

Attachment: formats.patch​ added

Patch for hungarian locale formats module

comment:5 Changed 11 years ago by slink

@jezdez it was mispelled, thanks for the report. Just uploaded the fixed patch.

comment:6 Changed 11 years ago by Jannis Leidel

Resolution: → fixed
Status: new → closed

In [16429]:

Fixed #16076 -- Updated Hungarian l10n formats. Thanks, slink.

Note: See TracTickets for help on using tickets.
Back to Top