WordPress.org

Make WordPress Core

Opened 16 months ago

Last modified 9 months ago

#44424 new defect (bug)

PHPDoc comments and @return tags missing at many places

Reported by: subrataemfluence Owned by:
Milestone: Future Release Priority: normal
Severity: normal Version:
Component: I18N Keywords: has-patch needs-refresh
Focuses: docs, coding-standards Cc:
PR Number:

Description

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 16 months ago.
Proposed patch

Download all attachments as: .zip

Change History (5)

@subrataemfluence
16 months ago

Proposed patch

#1 @netweb
16 months ago

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

#2 @pento
12 months ago

  • Milestone changed from 5.0 to 5.1

#3 @pento
9 months 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
9 months ago

  • Milestone changed from 5.1 to Future Release
  • Version trunk deleted
Note: See TracTickets for help on using tickets.