Make WordPress Core

Opened 16 years ago

Closed 16 years ago

#9225 closed defect (bug) (fixed)

PHP error in Magpie RSS feed export

Reported by: jsixpack's profile jsixpack Owned by:
Milestone: 2.8 Priority: normal
Severity: normal Version: 2.7
Component: Feeds Keywords: has-patch tested commit
Focuses: Cc:

Description

Using Wordpress 2.7.1

In the apache error log are many occurrences of these two lines:

Bad arguments. in /var/www/.../wp-includes/rss.php on line 175,
The first argument, 'map_attrs', should be either NULL or a valid callback

The problem appears to be a simple coding error: the callback function should be called as array reference, array(object, user_func).

This patch solves the problem (against trunk, rev. 10641):

Index: rss.php
===================================================================
--- rss.php	(revision 10641)
+++ rss.php	(working copy)
@@ -170,7 +170,7 @@
 		{
 			// if tags are inlined, then flatten
 			$attrs_str = join(' ',
-					array_map('map_attrs',
+					array_map(array($this, 'map_attrs'),
 					array_keys($attrs),
 					array_values($attrs) ) );

Attachments (1)

9225.diff (405 bytes) - added by Denis-de-Bernardy 16 years ago.

Download all attachments as: .zip

Change History (8)

#1 @FFEMTcJ
16 years ago

  • Milestone changed from Unassigned to 2.7.2
  • Priority changed from high to normal

#2 @Viper007Bond
16 years ago

  • Keywords has-patch needs-testing added; rss rss.php feed error removed

#3 @vladimir_kolesnikov
16 years ago

  • Cc vladimir@… added

Erm...

Should be array_map(&$this, 'map_attrs') - note ampersand in front of this.

#4 @Denis-de-Bernardy
16 years ago

  • Keywords tested added; needs-testing removed
  • Milestone changed from 2.7.2 to 2.8

#5 @DD32
16 years ago

  • Keywords commit added

Note that as of 2.8, WordPress includes SimplePie, MagPie is no longer supported specifically.

This sounds like something that should be submitted upstream, however, I'm not too sure on if that support still even exists (I don't think so)

Might as well get this commited for those who end up using it, with the note that Magpie is dead.

#6 @Denis-de-Bernardy
16 years ago

nod. I figured many plugins are still using it, and will continue to use it, for a very long time.

#7 @ryan
16 years ago

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

(In [11016]) Fix PHP error in magpie. Props jsixpack. fixes #9225

Note: See TracTickets for help on using tickets.