WordPress.org

Make WordPress Core

Opened 11 years ago

Closed 11 years ago

#11679 closed enhancement (fixed)

Mark 2nd param of load_plugin_textdomain() as deprecated

Reported by: sirzooro Owned by: nacin
Milestone: 3.0 Priority: normal
Severity: normal Version: 3.0
Component: General Keywords: has-patch commit
Focuses: Cc:

Description

Mark 2nd param of load_plugin_textdomain() as deprecated.

Attachments (1)

11679.diff (570 bytes) - added by nacin 11 years ago.

Download all attachments as: .zip

Change History (5)

#1 @nacin
11 years ago

  • Owner set to nacin
  • Status changed from new to accepted

@nacin
11 years ago

#2 @nacin
11 years ago

  • Keywords has-patch commit added; needs-patch removed

We normally check for empty() but since the function is very particular about checking for $abs_rel_path !== false, I patched it on !== false.

Commit candidate.

#3 @westi
11 years ago

I would prefer to put the check inside where the argument is used.

That way we only warn people who just use this argument rather than those who supply both so as to enable the plugin to function with old WordPress version.

#4 @westi
11 years ago

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

(In [12594]) Mark 2nd param of load_plugin_textdomain() as deprecated. Fixes #11679 props nacin.

Note: See TracTickets for help on using tickets.