WordPress.org
Get WordPress

Make WordPress Core

Opened 15 years ago

Closed 15 years ago

#13114 closed defect (bug) (fixed)

Deprecated functions in Akismet

Reported by: johnbillion's profile johnbillion Owned by: matt's profile 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 15 years ago.

Download all attachments as: .zip

Change History (9)

#1 @johnbillion
15 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.

#2 @johnbillion
15 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.

#3 @dd32
15 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

#4 @nacin
15 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.

@nacin
15 years ago

#5 follow-up: @jacobsantos
15 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.

#6 in reply to: ↑ 5 @jacobsantos
15 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.

#7 @solarissmoke
15 years ago

  • Cc solaris.smoke@… added

#8 @scribu
15 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.