Closed (fixed)
Project:
Domain
Version:
6.x-2.0-rc8
Component:
Miscellaneous
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
3 Jul 2009 at 09:12 UTC
Updated:
19 Jul 2009 at 15:50 UTC
Jump to comment: Most recent file
Comments
Comment #1
agentrickardUnder 'access' in the View settings. See attached.
It appears, however, that access settings are applied to all displays in a View, so you would need to create two separate Views.
Comment #2
anrikun commentedThank you very much for your help.
It would be great if access settings could be applied per display.
Comment #3
agentrickardThat is a Views feature request.
Comment #4
anrikun commentedActually, it seems that it's possible to set domain access settings only for a display and not the others.
But now I have another error:
Access is denied with a warning
warning: Invalid argument supplied for foreach() in /xxx/sites/all/modules/domain/domain_views/domain_views.module on line 75.
Might be related to another module I'm using. Will check that.
Comment #5
agentrickardTesting... Which is odd, because the plugin is in a different file, so that error means domain_views_get_grants() is returning nothing.
Are you using Domain Strict and hitting the View as an anonymous user? Do you have domains configured?
Comment #6
agentrickardI got it, I was doing some dumb variable handling in the function. Here's a patch.
Nice catch!
Comment #7
agentrickardCommitted to HEAD.
Comment #8
anrikun commentedStill got the same error after applying patch.
For instance, when accessing /user page as an anonymous user, the same warning is displayed on top of the logging form:
warning: Invalid argument supplied for foreach() in /xxx/sites/all/modules/domain/domain_views/domain_views.module on line 75.
Will try HEAD version.
Comment #9
agentrickardYou may also need #509536: Clean up error handling, documentation. which is in HEAD and cleans up some other array handling.
Comment #10
anrikun commentedYou're right. Warning is gone. Thanks! :-)
Comment #11
agentrickardYeah. I split the patches for clarity, which caused the confusion.