Since #28337: Add permissions to disable RSS feeds has been pushed to Drupal 8, some people will need this module for Drupal 7. Attached is a first go at this. Needs work since the taxonomy part of it doesn't quite work (Access denied on the HTML pages, not just the RSS feeds, for example).

CommentFileSizeAuthor
#14 rss_permissions.code-standards-945462.patch2.04 KBsillygwailo
#5 rss_permissions-7.x-0.2.tar_.gz8.45 KBsillygwailo
#4 rss_permissions-7.x-0.1.tar_.gz8.49 KBMonzer Emam
rss_permissions_7.tar_.gz8.25 KBsillygwailo

Comments

phpdiva’s picture

phpdiva commented
Assigned: Unassigned » phpdiva

Thanks for taking a stab at this. I'll look into the taxonomy issue.

cestmoi’s picture

cestmoi commented

subscribing

xibun’s picture

xibun commented

+1

Monzer Emam’s picture

Monzer Emam commented
Assigned: phpdiva » Unassigned
Status: Needs work » Needs review
StatusFileSize
new rss_permissions-7.x-0.1.tar_.gz8.49 KB

attached is a new D7 port with following changes

  1. using 'hook_theme_registry_alter' to override 'theme_feed_icon'
  2. using 'html_head_alter' to change the header

Also I did only tested the home page header and icon only.

@phpdiva: please commit that change as starting point ofr D7 porting.

sillygwailo’s picture

sillygwailo
Location Toronto, ON
commented
Status: Needs review » Needs work
StatusFileSize
new rss_permissions-7.x-0.2.tar_.gz8.45 KB

Still needs work, in that the regular expression to test the URLs doesn't always catch the feed URLs. For example, it looks like feed URLs are now taxonomy/term/1/feed in D7 instead of taxonomy/term/1/0/feed. Also, the line looking for blog/feed won't match because url('blog/feed') resolves to /blog/feed (note the slash at the beginning. I think I have taxonomy feeds working, so that's attached.

+1 on committing something so we can work from version control instead of attaching tarballs.

Attached version is based on Monzer Emam's work (had to manually construct the attachment URL), with some coding standards fixes as well.

phpdiva’s picture

phpdiva commented
Version: 6.x-1.x-dev » 7.x-1.x-dev

Thank you both for providing this code. I've created the Drupal-7 dev branch with the latest attachment from this thread. I think this also needs the latest changes ported from #954790: Suppress feed icon in themes that override theme_feed_icon().

doronin’s picture

doronin commented

Extremely useful for me. Thank you, Richard and Monzer .

BenK’s picture

BenK commented

Keeping track of this

Monzer Emam’s picture

Monzer Emam commented

Why still D7 version is not in project page

njardim’s picture

njardim commented

hi, could you please post the full .tgz also with the supressing of the rss feed icon patch included for D7 core?

thanks,
njardim

jm.federico’s picture

jm.federico commented

subs

Starminder’s picture

Starminder commented

+1

jonasdowney’s picture

jonasdowney commented

subscribe

sillygwailo’s picture

sillygwailo
Location Toronto, ON
commented
StatusFileSize
new rss_permissions.code-standards-945462.patch2.04 KB

A minor patch to remove unused comments and fix code standards (indenting).

ibandyop’s picture

ibandyop commented

subscribe

phpdiva’s picture

phpdiva commented

I again apologize to everyone for not addressing the issue in a timely manner. I am changing career paths, and am unable to continue maintaining this module. This module is very small and rather uncomplicated, but it plugs a very important shortcoming in the Drupal core (until that is fixed). Please contact me if you would like to take over this module.

sillygwailo’s picture

sillygwailo
Location Toronto, ON
commented
Assigned: Unassigned » sillygwailo
Status: Needs work » Needs review

I've committed the code standards patch, made the developmental snapshot listed on the project page, and will now set this to 'needs review'. If you can spare a moment for testing (the change I just made will appear in the snapshot about 12 hours after I post this), let me know how it goes and I'll make a 1.0 release for Drupal 7.

sillygwailo’s picture

sillygwailo
Location Toronto, ON
commented
Status: Needs review » Fixed

Calling this fixed. Please file new issues that are in the Drupal 7 branch as separate issues. I plan to make an official release soon but the developmental branch, at this writing, is stable as far as I know.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.