Closed (fixed)
Project:
Drupal.org CVS applications
Component:
new project application
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
7 Dec 2009 at 20:31 UTC
Updated:
16 Apr 2023 at 18:25 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
phpdiva commentedComment #2
avpadernoComment #3
avpadernoA module made of 18 lines (where just 3 lines are the effective code) cannot be considered a module.
The purpose of the CVS application is to see if the applicant understood how to write a Drupal module / theme; the proposed code doesn't allow to verify that.
Then, there isn't the need to use such module, when everybody can create a custom module, and add those code lines.
Comment #4
phpdiva commentedThanks!
Comment #5
phpdiva commentedComment #6
dave reidThis is actually something that *lots* of people would find useful since there is a big demand for this feature to be put into core. I'd like to see this maybe have more permissions (core modules that provide feeds are aggregator, blog, and node). I know it'd still be a small module, but again, this would be a useful feature until this is fixed in core. As such, I've marked phpdiva's application back to 'Pending' (and the issue here to needs work).
Comment #7
phpdiva commented@Dave,
Thanks for re-opening this, and yes - I was thinking the same thing. This needed more features.
Attaching the module again with the following new features:
Added support for blog (individual user feeds and main blog feed), taxonomy and aggregator RSS feeds.
This is now also hiding the RSS icon if user is not allowed to see a particular feed.
Comment #8
andrewmacpherson commentedThis is starting to look like a better feature set.
I think the module now adresses the most common feature/support requests that pop up on Drupal.org, variations on "How do I turn off the RSS feeds for my site?"
Currently, the only UI is on the main permissions form. The module would benefit greatly from an implementation of hook_help() and good documentation.
The 'version' line needs to be removed from the .info file, as this will be handled by the Drupal.org packaging scripts, IIRC.
+1
(I'd likely use this module, if it were actively maintained in contrib.)
Comment #9
phpdiva commentedThanks for the feedback! Uploaded new version with these changes:
- Removed the version line from .info file
- Added hook_help() and README file
Comment #10
dave reidLooks great!
So sorry for the delay, but your CVS application has been approved. Use it wisely and responsibly!
Comment #13
avpaderno