Make WordPress Core

Opened 5 years ago

Last modified 12 days ago

#44424 new defect (bug)

PHPDoc comments and @return tags missing at many places

Reported by: subrataemfluence's profile subrataemfluence Owned by:
Milestone: Future Release Priority: normal
Severity: normal Version:
Component: I18N Keywords: has-patch needs-refresh good-first-bug
Focuses: docs, coding-standards Cc:


Function/methods and property declarations are missing PHPDoc comment and @return tags at many places of src/wp-includes/pomo/translations.php

Proposed patch added.

Attachments (1)

44424.diff (3.1 KB) - added by subrataemfluence 5 years ago.
Proposed patch

Download all attachments as: .zip

Change History (8)

5 years ago

Proposed patch

#1 @netweb
5 years ago

  • Keywords has-patch added; needs-patch removed
  • Milestone changed from Awaiting Review to 5.0

#2 @pento
5 years ago

  • Milestone changed from 5.0 to 5.1

#3 @pento
4 years ago

  • Keywords needs-refresh added

Reviewing 44424.diff:

  • $entries and $headers could do with descriptions.
  • get_header() returns a header, not an element.
  • translate_entry() returns an entry, not an element.
  • select_plural_form() returns int, and the return needs a description.
  • translate_plural() only returns a string. The params and return need a description.
  • gettext_select_plural_form() returns the result of calling the callable that make_plural_form_function() returns. The callable that make_plural_form_function() returns will return an int.
  • make_plural_form_function() returns a callable.
  • All of the docblocks added to NOOP_Translations need descriptions.

#4 @pento
4 years ago

  • Milestone changed from 5.1 to Future Release
  • Version trunk deleted

#5 @swissspidy
13 days ago

  • Keywords good-first-bug added

#6 @niravsherasiya7707
12 days ago

@swissspidy Should I take ownership of the task to implement the improvements?

#7 @swissspidy
12 days ago

You can, but it's not a requirement. People reading through the comments here will see that you're working on it regardless :-)

Just remember to follow the PHP documentation standards

Note: See TracTickets for help on using tickets.