Before I saw this module I did a function and used with php format in header. Not the best way :p
Youre solution is supercool. I made this modification becouse I wanted to use this on header area for creating administrative links and restrict access per role. This is also tested with and worked for me.
Sorry I couldnt make patch.. Dont know why it didnt work for me.
This package doesnt work I deleted one row my mistake :S Ill make new one.

CommentFileSizeAuthor
#2 views_linkarea.tar_.gz2.13 KBkaido.toomingas
#2 add_role_based_access_multiple_1445560-5622656.patch2.68 KBkaido.toomingas
#1 add_role_based_access-1445560.patch2.41 KBkaido.toomingas
#1 views_linkarea.tar_.gz2.12 KBkaido.toomingas
views_linkarea.tar_.gz2.15 KBkaido.toomingas

Comments

kaido.toomingas’s picture

kaido.toomingas commented
Issue summary: View changes

.

kaido.toomingas’s picture

kaido.toomingas commented
Issue summary: View changes

.

kaido.toomingas’s picture

kaido.toomingas commented
Priority: Normal » Major
Status: Active » Patch (to be ported)
StatusFileSize
new views_linkarea.tar_.gz2.12 KB
new add_role_based_access-1445560.patch2.41 KB

Added changes witch I described before + added changed wich has been submitted http://drupal.org/files/issues/996974-viewslinkarea-query-class-array.patch changes becouse I had the same problem again.

kaido.toomingas’s picture

kaido.toomingas commented
StatusFileSize
new add_role_based_access_multiple_1445560-5622656.patch2.68 KB
new views_linkarea.tar_.gz2.13 KB

Mutliple access role patch and version for download.

jantimon’s picture

jantimon commented

Wouldn't it be cleaner to use the role machine names instead of the ids?
This would help people who export views with features.

Bevan’s picture

Bevan commented

This related issue has a simpler patch that handles many of the use cases this patch also handles: #1451266: Links should not render if user does not have access

kaido.toomingas’s picture

kaido.toomingas commented

I do agree!

geek-merlin’s picture

geek-merlin
Primary language German
Location Freiburg, Germany
commented
Status: Patch (to be ported) » Closed (duplicate)

access should *never* be done via roles, the proper way is to use permissions.
see: #1445494: block.module maintains visibility via roles thereby establishing a hidden permission system

so marking as dup of #1451266: Links should not render if user does not have access

geek-merlin’s picture

geek-merlin
Primary language German
Location Freiburg, Germany
commented
Issue summary: View changes

Review