Opened 2 years ago

Closed 2 years ago

Last modified 2 years ago

#16806 closed defect (bug) (fixed)

Duplicate strings in XML-RPC

Reported by: pavelevap Owned by:
Priority: normal Milestone: 3.2
Component: I18N Version: 3.1
Severity: normal Keywords: has-patch
Cc: pavelevap@…

Description

In class-wp-xmlrpc-server.php file there are 2 almost same strings:

"You are not allowed to upload files to this site."

"You are not allowed to upload files on this site."

Maybe they should be the same and we can save one string?

Attachments (1)

16806.diff (568 bytes) - added by ericmann 2 years ago.
Normalize strings

Download all attachments as: .zip

Change History (8)

I'd agree. Any chance you can save us a lookup and reference line numbers?

Sorry, I wanted to create patch but was not sure which variant is right.

Lines 1525 and 2969.

Now that I've looked through the file again, there are a lot of duplicate strings ... not just those two. What's the value of reducing the number of duplicates? What's the damage in leaving them?

No damage, I guess.

But easier work for translators and smaller .mo file (and so less memory used).

Normalize strings

  • Keywords has-patch added
  • Version set to 3.1

I misread your original ticket :-( I agree with you 100% and have posted a patch. Since both sections are checking the same permission, they should output the same error message:

You are not allowed to upload files to this site.

  • Resolution set to fixed
  • Status changed from new to closed

(In [17554]) Change this string to one that already exists. props ericmann, fixes #16806.

  • Milestone changed from Awaiting Review to 3.2
Note: See TracTickets for help on using tickets.