Will Domain Access 6.x-2.0 be Views 2 compatible?
| Comment | File | Size | Author |
|---|---|---|---|
| #28 | 319876-source.patch | 8.35 KB | agentrickard |
| #27 | 319876-source.patch | 9.25 KB | agentrickard |
| #26 | 319876-source.patch | 8.02 KB | agentrickard |
| #24 | 319876_domain_views.patch | 17.8 KB | agentrickard |
| #22 | 319876_domain_views.patch | 16.14 KB | agentrickard |
Comments
Comment #1
nirvanajyothi commentedI am using Views 2 latest rc and would like to use Domain Views.But it is not possible with DA 6.x-1.2,rite?
Comment #2
agentrickardThe final release will be Views 2 compatible. The current release is not.
Assistance (patches) would be appreciated.
Comment #3
nonsiealso see http://drupal.org/node/287337#comment-994521
Comment #4
agentrickardThis is now in testing in HEAD. Very unstable.
See also #320257: Move $domain->user data to own table and #320064: Move the default domain into the database which both make Domain Access more 'Views friendly.'
Comment #5
nonsieI've started to work on Views 2, at the moment there's no support for anything "current" (similar to current user's default language for node->language). I'm not sure what to name these filters, any help would be appreciated.
Comment #6
agentrickard@nonsie
How is that relevant to DA? And are you working along with the code in HEAD or doing something else?
Comment #7
nonsieI checked out the code from HEAD based on #4.
Let me rephrase my question - at the moment domain_id filter is a numeric field while old DA Views allowed user's to choose one or more specific domains or current domain (dynamic value based on the domain the view is rendered on). What I'm most interested in is being able to filter nodes that are published to:
1. current domain
2. current domain or all affiliates
3. any domain
Views used to have node translation in node group, now it has been moved to a separate group called node translation. Should DA use a different grouping for filters/fields dealing with dynamic values (current domain, current user's domain etc)?
For my own purposes I could create it any way I could possibly come up with but I'd rather write something everyone can use.
Comment #8
agentrickardThose are the questions where I stopped, since Views uses OO code for custom handlers now, and I have to learn how to make that work correctly.
My suggestion is to submit patches that get the functionality that you are after. Then I/we will standardize and generalize them for release.
Comment #9
nonsieAdded currently active domain and any domain to domain_id and gid filters. Also added an additional filter for content available on currently active domain (either published to the domain or all affiliates).
Still no support for arguments.
Edit: for some reason include files were left out
Comment #10
agentrickardComment #11
agentrickardOK. Patch applies and things seem to work. I have not tested heavily but will commit and release rc5 so we can get more feedback.
Comment #12
nonsieCould you update Domain Views description on the project page? rc5 is compatible with Views 6.x.2.0.
Comment #13
agentrickardDone.
Comment #14
Crell commentedI can confirm that the "in current domain" filter at least appears to work in RC5. I've not tested the other bits yet.
Comment #15
agentrickardStill more to be done here....
Comment #16
agentrickardA patch which removes the extraneous filter by null (not needed in Views 2) and adds stub support for {domain_editor}.
Comment #17
nwilets commentedDomain and Domain ID arguments are missing in Views 2.
It would be great to have these back when you guys have the chance. Thanks.
Comment #18
ericpughis this patch applied to rc6?
Comment #19
agentrickardThat latest patch is against HEAD, but should apply to rc6.
Comment #20
nonsiePatch from #16 plus:
- argument handling
- output sitename field as single or multiple values (see #405084: "Publish to" as a field in Views). Similar approach should probably also exist for all other fields. Optionally link this output to the node on default domain or the domain the node is published to).
I am not sure if the approach to handle multiple values is indeed the correct way of doing this. I've based my code on taxonomy multiple term handling and CCK views support.
Comment #21
nonsieComment #22
agentrickardSmall note -- the patch file had Windows line breaks. It still applied, but this is something we will have to watch carefully.
Attached is a reformatted version of the patch -- no changes, just the creation of the new files inside the 'includes' directory where they belong, and with Unix linebreaks.
Nothing seems to be broken by the patch, but I have not had a chance to review thoroughly.
Comment #23
agentrickardhook_theme() requires that you list the file where the theme function resides if it is not in the same file as the hook. We might as well put that one theme function directly in the module file.
http://api.drupal.org/api/function/hook_theme
Comment #24
agentrickardClears up some confusion by changing group names. Needs some documentation.
I say commit and move on. Let people test.
Comment #25
agentrickardThis last patch has been committed to HEAD. More work still, but this is much improved!
Comment #26
agentrickardHere is a patch that adds Domain Source support, and allows domain_id fields to be output as links.
Comment #27
agentrickardRevised to remove the old 'Any domain' option.
Comment #28
agentrickardRevised version of the patch. Simpler. Maybe better?
Comment #29
burningdog commentedI couldn't get the patch doesn't apply against rc6. At least, all of it applies except for the last change to domain_views.views.inc to function domain_views_views_handlers()
The contents of domain_views/domain_views.views.inc.rej are
Comment #30
agentrickardIt is a patch against HEAD.
Comment #31
burningdog commentedFair enough - thought I could apply it because in #19 you said it should apply against rc6 too.
Comment #32
agentrickardThat only applied to the earlier (pre #19 patch)
Comment #33
agentrickardPatch in #28 committed to HEAD.
Comment #34
nonsiePatch from # seems to work as expected.
Comment #35
drecute commented@agentrickard and @nonsie
I don't know if you can help me out on this. I have installed the domain module and domain block module, but It seems the domain module does not work well with views on my drupal installation. My problem is that, when I administer my content with views(with a block display), I dont seem to see the views(which has a block entry on the block page) displayed on my page. But when I create content and is promoted to front page plus including domain specific display of the content, It works fine.
Is it that domain module does not work with views or what? It also seems to me that the domain module does not work with the panel module. What I did is that I added to a mini panel the content of the views which was created earlier. And when assigned to a block region, nothing is displayed.
Please I need urgent help on this. My deadline is Friday. Please @agentrickard and @nonsie
Comment #36
agentrickard@drecute
You need to open a new issue and give more details. DA works fine with both Views 2 and Panels 2.
Nesting content (a view inside a panel inside a block) may not work as expected, especially if the content is not visible on the current domain.
Comment #37
drecute commented@agentrickard
Thanks for the response.
I have opened a new issue at http://drupal.org/node/461576. Please kindly look through. Thanks.
Comment #38
mrgoltra commentedsubscribing.
Comment #39
agentrickardI'm going to close this one and deal with other issues separately.
Comment #40
elliottbrown commented@drecute - i think your views issue is the same as mine http://drupal.org/node/965554
any resolution?
thanks all in advance
btw - i had your issue with panels but solved it with a hack - which is on drupal.org. did you solve it too ?
Comment #41
drecute commented@elliottbown
Sure I did fix it. Though I can't remember how, but I know I did a couple of head banging to get this to work.