I have been thinking about how to take advantage of the relation module for User Relationships and one potential issue I'm seeing is the ability to hide the relation interface and only use the API.
We will likely be keeping something very similar to our current management screens and having additional relation management screens show up for users that have create edit and delete permissions will definitely confuse them. We might even end up in a situation where we want to hide the relation type configuration screens and make a few pre-defined types that match our current types available.
The ideal situation would be similar to what we currently have with views where the UI can be disabled when not needed. If someone comes along who is knowledgeable enough to understand the power of relation they can turn on relation's UI but regular users will have less obvious ability to get themselves into trouble.
Comment | File | Size | Author |
---|---|---|---|
#13 | relation_ui-1416856-13.patch | 32.79 KB | mikran |
#11 | relation_ui-1416856-9.patch | 66.49 KB | mikran |
#5 | relation_ui-1416856-5.patch | 66.48 KB | mikran |
Comments
Comment #1
naught101 CreditAttribution: naught101 commentedProbably a good idea, we could have relation_ui.module, similar to views_ui.module.
Would we want to move all of the UI, or just parts of it? eg. Would it be worth keeping admin/structure/relation in the api module?
Comment #2
mrf CreditAttribution: mrf commentedI could see that one going either way, its protected behind the administer permission so likely won't be visible except to admins anyway, but might be clearer just to make it all or nothing.
Comment #3
mikran CreditAttribution: mikran commentedI think I could implement this during the easter holiday :)
Comment #4
ryan.armstrong CreditAttribution: ryan.armstrong commentedThat would be great mikran. I've been playing around with a module that would be built on top of the Relation module, and this is one of the things this module needs so that it can be used as an API for other modules.
Comment #5
mikran CreditAttribution: mikran commentedOkay first version is ready. It's mostly menu items and relation.admin.inc that got moved to relation_ui module.
Comment #6
chx CreditAttribution: chx commentedTotally. I hope I can get automated testing working again and if this passes the bot then i will commit it. Mikran, do you want to be co-maintainer?
Comment #7
mikran CreditAttribution: mikran commentedTests are failing because of #1563678: Endpoints table formatter is displaying label only, those fails are not related to this patch.
Comment #9
chx CreditAttribution: chx commented#5: relation_ui-1416856-5.patch queued for re-testing.
Comment #11
mikran CreditAttribution: mikran commentedComment #12
chx CreditAttribution: chx commentedCommitted thanks.
Comment #13
mikran CreditAttribution: mikran commentedApparently the commit didn't include any of the new files. Here is a patch containing just those.
Comment #14
mikran CreditAttribution: mikran commentedAnd the missing files are now also committed.