Please add support for Brazilian Portuguese to locale.inc. Both Portugal and Brazil Portuguese translations have been contributed, but there's only one Portuguese slot available to load them.

A sample patch is attached, but of course the Portuguese strings should be updated by someone who actually knows Portuguese...

CommentFileSizeAuthor
#3 locale_pt-br.patch739 bytesChrisKennedy
#2 locale_inc1.patch541 bytesChrisKennedy
locale.inc_0.patch691 bytesmikeryan

Comments

Bèr Kessels’s picture

Bèr Kessels commented

+1.
its a small addition and makes sense. WE have a large braziilian audience, and this will help them a lot.
Also: we already do the same for chinese, so there should no be a problem.

From what I read on wikipedia brazillain and portugese-portugese are very different.

ChrisKennedy’s picture

ChrisKennedy commented
Status: Needs work » Needs review
StatusFileSize
new locale_inc1.patch541 bytes

Looks good to me - re-rolled against CVS and with -u -p.

ChrisKennedy’s picture

ChrisKennedy commented
Version: x.y.z » 5.x-dev
StatusFileSize
new locale_pt-br.patch739 bytes

Syncing with cvs.

dries’s picture

dries commented
Status: Needs review » Fixed

Committed to CVS HEAD.

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)