Make WordPress Core

Opened 13 years ago

Closed 13 years ago

#8742 closed defect (bug) (fixed)

Multi-Widget Example Missing Semicolon

Reported by: dcole07's profile dcole07 Owned by:
Milestone: 2.8 Priority: lowest
Severity: trivial Version: 2.7
Component: Plugins Keywords: has-patch commit
Focuses: Cc:

Description

Within wp-includes/widgets.php is an example of a multi-widget, the following code currently creates an error:
http://trac.wordpress.org/browser/trunk/wp-includes/widgets.php#L2095
Line 2095: add_action( 'widgets_init', 'widget_many_register' )

This line should have a semicolon at the end, becoming
add_action( 'widgets_init', 'widget_many_register' );

Attachments (1)

widgets.diff (259 bytes) - added by mrmist 13 years ago.
widgets.php

Download all attachments as: .zip

Change History (6)

@mrmist
13 years ago

widgets.php

#1 @mrmist
13 years ago

  • Keywords has-patch added

Attached diff against trunk. 10258

#2 @jacobsantos
13 years ago

  • Keywords commit added

WTF, the code is in a comment and most like was intentional to ensure that the developer who is copying and pasting is paying attention.

#3 @azaozz
13 years ago

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

(In [10303]) Add semicolon to Multi-Widget Example, props dcole07, fixes #8742 for 2.7

#4 @azaozz
13 years ago

  • Milestone changed from 2.7.1 to 2.8
  • Resolution fixed deleted
  • Status changed from closed to reopened

Reopen for trunk

#5 @azaozz
13 years ago

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

(In [10304]) Add semicolon to Multi-Widget Example, props dcole07, fixes #8742 for trunk

Note: See TracTickets for help on using tickets.