Opened 17 years ago
Closed 15 years ago
#6947 closed enhancement (fixed)
Add the Generator element in feeds through the {rss2|atom|rdf|rss}_head hook
Reported by: | gnot | Owned by: | westi |
---|---|---|---|
Milestone: | 3.0 | Priority: | low |
Severity: | minor | Version: | 2.5.1 |
Component: | Feeds | Keywords: | has-patch |
Focuses: | Cc: |
Description
The generator metatag is added to the HTML pages through the wp_head()
hook.
I think it would be more useful to do the same in the feeds. The generator element could be added through the {rss2|atom|rdf|rss}_head
hook.
This way it would be easier for plugin authors to totally remove the generator tags for security reasons or make modifications to the generator string.
Attachments (2)
Change History (13)
#2
@
17 years ago
If you guys 'n' gals expect this patch from me, you may check this ticket again after a few months, since this requires knowledge I do not currently have :)
#7
@
16 years ago
- Milestone Future Release deleted
- Resolution set to wontfix
- Status changed from new to closed
filtering on the_generator gets the job done already: It contains the context.
#8
@
16 years ago
- Milestone set to Future Release
- Resolution wontfix deleted
- Status changed from closed to reopened
Sounds like a good idea to me.
The ticket isnt just about being able to remove it easily, Its about cleaner code.
It would however need a helper function, since its impossible to pass params via add_action (ie. add_action('rss_head', 'the_generator("rss")');
)
#9
@
15 years ago
- Cc matt@… added
- Keywords has-patch added; needs-patch removed
- Milestone changed from Future Release to 3.0
@
15 years ago
Move the generator output to *_head actions for feeds, opml and app. Add opml_head action.
I agree. Let's see the patch!