WordPress.org
Get WordPress

Make WordPress Core

Opened 11 years ago

Closed 11 years ago

#31835 closed defect (bug) (wontfix)

Merge the subdomain/subdirectory translation strings

Reported by: ramiy Owned by:
Priority: normal Milestone:
Component: I18N Version:
Severity: normal Keywords: has-patch
Cc: Focuses: administration, multisite

Description

See the attached patch.

I've merged the two translation strings to one string, and I've moved the logic to the code.

Merging the strings makes the PO/MO file smaller. And if you see the new code - it's just more logical.

Attachments (1)

31835.patch​ (1.3 KB ) - added by ramiy 11 years ago.

Download all attachments as: .zip

Change History (8)

@ramiy
11 years ago

#1 follow-ups: @SergeyBiryukov
11 years ago

  • Component General → Networks and Sites
  • Focuses administration multisite added

I think someone might want to translate site1/site2.

#2 in reply to: ↑ 1 @ramiy
11 years ago

  • Version → trunk

Sergey, it's your call.

From cost effectiveness point of view I prefer a smaller PO/MO file, to make my site faster few more micro-seconds.

#3 @ramiy
11 years ago

  • Keywords has-patch added

#5 @SergeyBiryukov
11 years ago

  • Component Networks and Sites → I18N

#6 @DrewAPicture
11 years ago

  • Version trunk

#7 in reply to: ↑ 1 @ocean90
11 years ago

  • Milestone Awaiting Review
  • Resolution → wontfix
  • Status new → closed

Replying to SergeyBiryukov:

I think someone might want to translate site1/site2.

Currently it's translated in 30 languages. Closing as wontfix.

Note: See TracTickets for help on using tickets.