Make WordPress Core

Opened 16 years ago

Closed 15 years ago

Last modified 15 years ago

#16531 closed defect (bug) (fixed)

Context is missing from some "Name" strings

Reported by: valentinas Owned by: azaozz
Priority: normal Milestone: 3.2
Component: I18N Version: 3.1
Severity: normal Keywords: has-patch 3.2-early commit
Cc: Focuses:

Description

Name of a person and name of a link, plugin, meta, etc are translated differently, therefore they need context.

Attachments (1)

add_context_name.patch (5.1 KB ) - added by valentinas 16 years ago.
Patch to add context to "Name" where refers to something else than person.

Download all attachments as: .zip

Change History (13)

@valentinas
16 years ago

Patch to add context to "Name" where refers to something else than person.

#1 @valentinas
16 years ago

  • Keywords has-patch added

#2 @pavelevap
16 years ago

  • Cc pavelevap@… added

Great, I made the same ticket some months ago, but without patch...

http://core.trac.wordpress.org/ticket/14237

#3 @valentinas
16 years ago

Would be nice to have this committed to 3.1 if thats possible. This issue prevents WP for being fully localized. You will either have wrong meaning or will need to hack the core.

#4 @ocean90
16 years ago

  • Keywords 3.2-early added
  • Milestone Awaiting ReviewFuture Release

It' too late for 3.1, because we are already in RC4 and in string freeze.

Last edited 16 years ago by ocean90 (previous) (diff)

#5 @valentinas
16 years ago

  • Cc valentinas added

#6 @pavelevap
15 years ago

Bump, any chance to commit this patch for 3.2?

#7 @ocean90
15 years ago

  • Cc nacin added

#8 @nacin
15 years ago

  • Keywords commit added
  • Milestone Future Release3.2

Fine. Does it need further refresh?

#9 @azaozz
15 years ago

  • Owner set to azaozz
  • Resolutionfixed
  • Status newclosed

In [18247]:

Add context to _e('Name'), props valentinas, fixes #16531

#10 @pavelevap
15 years ago

Great, works well. Name without context is now used only for several strings which are related to Users.

Only one note: It is needed to have one string "Name" and other string "Name:" (with colon) for only one context "html attribute"?

#11 @nacin
15 years ago

Name without context is now used only for several strings which are related to Users.

Submit a patch and we can add context to those too.

It is needed to have one string "Name" and other string "Name:" (with colon) for only one context "html attribute"?

Could you elaborate or try again? All names were given context, including Name:. There probably isn't another Name:, but regardless, this one has nice context now.

#12 @pavelevap
15 years ago

OK, I will try to add patch, but it is not important because all of these remaining strings are related to users (living things) and there is only one context (user) for them.

Yes, it is not problem, context is fine. My English is poor, so I did not explain it well. I only noted that there are 2 strings "Name" and "Name:" with the same context "html attribute" which differ by colon. Removing colon from translation (and adding it directly to script) we would need only one string to translate and not two with the same meaning...

Note: See TracTickets for help on using tickets.