Closed (outdated)
Project:
Domain
Version:
7.x-3.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
28 Aug 2009 at 10:16 UTC
Updated:
3 Sep 2024 at 22:32 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
Ghostthinker commentedmymodule/plugins/access/domain.inc
Comment #2
agentrickardThe current access filter is universal, not per-user.
http://drupal.org/patch/create
Please patch against HEAD.
Comment #3
Ghostthinker commentedI made a patch. It (should) simply add a new file plugins/access/domain_editor.inc
At the moment I am using my own module (that does the same than domain_ctools plus the ability to control access per user) but I would drop it if this goes into yours.
Comment #4
agentrickardNice. Hopefully I can review and add this in.
Comment #5
hefox commentedTesting patch at the moment.
Initial thoughts: small detail, but in general patches are done via the root of the module, not the directory the file is in.
The views access plugin has these joined together, what are the reasons for having them separate in ctools, other than the added complexity? Perhaps domain_ctools and.. domain_views? could share some functions on checking access so to avoid duplication.
To the patch itself, eyeballing the patch looks good other than some random spacing, brief testing looks good :)
Comment #6
hefox commentedHm looking at the domain_views and domain_ctools, and looking at my own use case my preference would be
Create/move two functions into the main domain module for easy reusability:
(My use case is a custom field permission module that checks if the user and node is part of the current domain before letting em view certain fields, so I could use the first function using node's domains and member = true.)
Comment #7
hefox commentedThrowing it to domain; throw it back here if prefer the earleir approach.
Patch 1 for domain/domain views puts 2 functions into domain module, and has the views access plugin use them; the second is for domain ctools.
Comment #8
agentrickardI think it's too late to clean this up for D6, and these modules are stand-alone in D7.
Comment #9
agentrickardMoving versions.
Comment #10
agentrickardThis is totally out of date and I have forgotten the original use-case.
Comment #11
dqdI'll close this issue due to inactivity for 13 years and upcoming EOL of Drupal 7 in January 2025. While the project's version for Drupal 7 will surely keep providing support for security issues and website breaking bug fixes for Drupal 7, we should try to minimize the open issues for Drupal 7 in the queue as much as possible. A big thanks around for the report and all the efforts in here. Feel free to re-open if you can provide a fix or found an additional related problem which increases the issue priority.