WordPress.org
Get WordPress

Make WordPress Core

Opened 17 years ago

Closed 17 years ago

#9792 closed defect (bug) (fixed)

inconsistent name spaces hooks in rss/atom feeds

Reported by: Denis-de-Bernardy Owned by: westi
Priority: normal Milestone: 2.8
Component: Feeds Version: 2.8
Severity: normal Keywords: has-patch tested commit
Cc: Focuses:

Description

  • atom: ns_atom
  • atom/comments: ns_atom
  • rss2: ns_rss2
  • rss2/comments: ns_comments_rss2 since r11279

backward compat probably prevents to use ns_comments_atom.

either way, things should be consistent.

Attachments (2)

9792.diff​ (490 bytes ) - added by Denis-de-Bernardy 17 years ago.
9792-whale.patch​ (994 bytes ) - added by peaceablewhale 17 years ago.

Download all attachments as: .zip

Change History (6)

#1 @Denis-de-Bernardy
17 years ago

  • Owner set to westi

@Denis-de-Bernardy
17 years ago

#2 @peaceablewhale
17 years ago

"atom_ns" and "rss2_ns" are general namespaces that both the general and comments feed can share while "rss2_comments_ns" may be used for adding namespaces to the comments feed only. I suggest adding "rss2_ns" to feed-rss2-comments.php and "atom_comments_ns" to feed-atom-comments.php.

@peaceablewhale
17 years ago

#3 @Denis-de-Bernardy
17 years ago

  • Keywords has-patch tested commit added

+1. that way we keep backwards compat and we don't introduce inconsistencies.

#4 @ryan
17 years ago

  • Resolution → fixed
  • Status new → closed

(In [11303]) Feed namespace hooks. Props Denis-de-Bernardy, peaceablewhale. fixes #9792

Note: See TracTickets for help on using tickets.