Make WordPress Core

Opened 9 years ago

Closed 9 years ago

Last modified 9 years ago

#34015 closed defect (bug) (fixed)

Improve method documentation for WP_Nav_Menu_Widget

Reported by: drewapicture's profile DrewAPicture Owned by: leemon's profile leemon
Milestone: 4.4 Priority: normal
Severity: normal Version:
Component: Widgets Keywords: good-first-bug has-patch
Focuses: docs Cc:

Description

Note: This is part of the effort in #34013 to improve method documentation to all core widget classes

Class: WP_Nav_Menu_Widget
File: wp-includes/widgets/class-wp-nav-menu-widget.php

Attachments (3)

34015.patch (2.1 KB) - added by leemon 9 years ago.
Improve method documentation
#34015.2.patch (2.2 KB) - added by leemon 9 years ago.
Updated method descriptions
34015.3.patch (2.2 KB) - added by leemon 9 years ago.
Adding correct version

Download all attachments as: .zip

Change History (9)

#1 @DrewAPicture
9 years ago

  • Summary changed from Improve method documentation for `WP_Nav_Menu_Widget` to Improve method documentation for WP_Nav_Menu_Widget

@leemon
9 years ago

Improve method documentation

#2 @leemon
9 years ago

Added patch to improve the method docs for the widget(), update(), form(), and __construct() methods in the WP_Nav_Menu_Widget class.

34015.patch

#3 @DrewAPicture
9 years ago

  • Owner set to leemon
  • Status changed from new to assigned

Hi @leemon,

Thanks for the patch!

  • For this particular widget, the class and methods were introduced in 3.0.0, so that version should be used instead of 2.8.0

Also, as mentioned in #34027:

Just a couple of notes:

  • Let's go with the vernacular of something like "the current {type} widget instance" instead of "a particular instance of a widget". Being general in the WP_Widget base class is fine, but in this context, we can be more specific. For example:
 * Handles updating settings for the current Text widget instance.
  • Notice I inserted the type of widget, in this case "Text", and more specifically mentioned updating the settings which is what the update() method does.
  • The same sort of advice goes along with any other method in the class. The idea is to avoid simple copy pasta and actually get specific :-)

@leemon
9 years ago

Updated method descriptions

#4 @leemon
9 years ago

  • Keywords has-patch added; needs-patch removed

@leemon
9 years ago

Adding correct version

#5 @DrewAPicture
9 years ago

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

In 34621:

Docs: Improve documentation for the __construct(), widget(), update(), and form() methods in WP_Nav_Menu_Widget.

Props leemon.
Fixes #34015. Fixes #34013.

#6 @DrewAPicture
9 years ago

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