Make WordPress Core

Opened 14 years ago

Closed 14 years ago

#11679 closed enhancement (fixed)

Mark 2nd param of load_plugin_textdomain() as deprecated

Reported by: sirzooro's profile sirzooro Owned by: nacin's profile 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 14 years ago.

Download all attachments as: .zip

Change History (5)

#1 @nacin
14 years ago

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

@nacin
14 years ago

#2 @nacin
14 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
14 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
14 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.