WordPress.org

Making WordPress.org

Opened 18 months ago

Closed 18 months ago

Last modified 17 months ago

#3411 closed defect (maybelater)

"Installation Instructions" string is imported for plugins that don't have it

Reported by: SergeyBiryukov Owned by:
Milestone: Priority: normal
Component: Plugin Directory Keywords: needs-patch
Cc:

Description (last modified by SergeyBiryukov)

Since [5253], plugin FAQ section is prefixed with "Installation Instructions" string.

That string ends up imported into GlotPress project for the plugin, see an example.

If the string does not exist in the plugin readme, it should not be imported into its GlotPress project.

Change History (6)

#1 @SergeyBiryukov
18 months ago

  • Description modified (diff)

#2 follow-up: @obenland
18 months ago

So you're saying it should offer "Installation" as a translatable string rather than "Installation Instructions"?

#3 follow-up: @ocean90
18 months ago

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

#4 in reply to: ↑ 3 @SergeyBiryukov
18 months ago

Replying to ocean90:

All strings on the plugin page (https://ru.wordpress.org/plugins/display-active-plugins-first/) are parsed again and directly searched in GlotPress so the string needs to be in the project.

But the string is the same for all plugins, why does it have to be translated in each project over and over again? The Consistency tool shows 11 different translations for ru_RU.

Version 0, edited 18 months ago by SergeyBiryukov (next)

#5 in reply to: ↑ 2 @SergeyBiryukov
18 months ago

Replying to obenland:

So you're saying it should offer "Installation" as a translatable string rather than "Installation Instructions"?

"Installation" is already offered for translation in the Plugin Directory project.

"Installation Instructions" is a part of the same UI, but due to the current implementation it ends up being translated in each plugin project separately. As noted in comment:4, I don't see a point in that, it just adds a bit more work for translators :)

Last edited 18 months ago by SergeyBiryukov (previous) (diff)

#6 @SergeyBiryukov
17 months ago

  • Keywords needs-patch added
  • Resolution changed from worksforme to maybelater

Still thinking the readme importer should skip this string per the comments above.

Marking as maybelater pending a patch.

Note: See TracTickets for help on using tickets.