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
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Bèr Kessels’s picture

Bèr Kessels CreditAttribution: 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 CreditAttribution: ChrisKennedy commented
Status: Needs work » Needs review
FileSize
locale_inc1.patch541 bytes

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

ChrisKennedy’s picture

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

Syncing with cvs.

Dries’s picture

Dries CreditAttribution: Dries commented
Status: Needs review » Fixed

Committed to CVS HEAD.

Anonymous’s picture

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