Opened 18 years ago
Closed 17 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 |
|---|---|---|---|
| Priority: | low | Milestone: | 3.0 |
| Component: | Feeds | Version: | 2.5.1 |
| Severity: | minor | Keywords: | has-patch |
| Cc: | Focuses: |
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
@
18 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
@
17 years ago
- Milestone Future Release
- Resolution → wontfix
- Status new → closed
filtering on the_generator gets the job done already: It contains the context.
#8
@
17 years ago
- Milestone → Future Release
- Resolution wontfix
- Status closed → 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
@
17 years ago
- Cc added
- Keywords has-patch added; needs-patch removed
- Milestone Future Release → 3.0
@
17 years ago
Move the generator output to *_head actions for feeds, opml and app. Add opml_head action.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
I agree. Let's see the patch!