WordPress.org

Make WordPress Core

Opened 5 years ago

Closed 5 years ago

#13114 closed defect (bug) (fixed)

Deprecated functions in Akismet

Reported by: johnbillion Owned by: matt
Milestone: WordPress.org Priority: low
Severity: minor Version:
Component: Plugins Keywords: deprecated
Focuses: Cc:

Description

Akismet uses __ngettext() which is deprecated and should be replaced with _n()

Line 932 and line 942 of akismet.php. I won't patch it as Akismet is an external.

Attachments (1)

akismet-notices-nacin.diff (12.1 KB) - added by nacin 5 years ago.

Download all attachments as: .zip

Change History (9)

comment:1 @johnbillion5 years ago

  • Summary changed from Deprecated function in Akismet to Deprecated functions in Akismet

Oh we've got _c() in there too (replaced by _x()). Line 951.

comment:2 @johnbillion5 years ago

Actually it's just ocurred to me that these are probably in here to maintain backwards compatibility. Feel free to close as wontfix.

comment:3 @dd325 years ago

  • Milestone changed from Unassigned to WordPress.org site

This should be reported to the Plugins trac, It would however be nice, if someone was to handle the tickets created there for Akismet

comment:4 @nacin5 years ago

A few weeks ago, I submitted a patch to ryan that updates Akismet to use 3.0 code (while maintaining back compat to the earliest of versions). I believe it was passed onto the Akismet team at Automattic.

comment:5 follow-up: @jacobsantos5 years ago

Really? I submitted a ticket to use file level docs a year ago and if it hasn't been committed, then I'll be darned. Probably should email it to Ryan instead of depending on the Plugins Trac.

comment:6 in reply to: ↑ 5 @jacobsantos5 years ago

Replying to jacobsantos:

Really? I submitted a ticket to use file level docs a year ago and if it hasn't been committed, then I'll be darned. Probably should email it to Ryan instead of depending on the Plugins Trac.

LOL http://plugins.trac.wordpress.org/ticket/920 I guess 19 months ago.

comment:7 @solarissmoke5 years ago

  • Cc solaris.smoke@… added

comment:8 @scribu5 years ago

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

I think this is fixed now.

Note: See TracTickets for help on using tickets.