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.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

naught101’s picture

Probably 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?

mrf’s picture

I 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.

mikran’s picture

Assigned: Unassigned » mikran

I think I could implement this during the easter holiday :)

ryan.armstrong’s picture

That 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.

mikran’s picture

Status: Active » Needs review
FileSize
66.48 KB

Okay first version is ready. It's mostly menu items and relation.admin.inc that got moved to relation_ui module.

chx’s picture

Category: feature » task

Totally. 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?

mikran’s picture

Tests are failing because of #1563678: Endpoints table formatter is displaying label only, those fails are not related to this patch.

Status: Needs review » Needs work

The last submitted patch, relation_ui-1416856-5.patch, failed testing.

chx’s picture

Status: Needs work » Needs review

#5: relation_ui-1416856-5.patch queued for re-testing.

Status: Needs review » Needs work

The last submitted patch, relation_ui-1416856-5.patch, failed testing.

mikran’s picture

Status: Needs work » Needs review
FileSize
66.49 KB
chx’s picture

Status: Needs review » Fixed

Committed thanks.

mikran’s picture

Status: Fixed » Needs review
FileSize
32.79 KB

Apparently the commit didn't include any of the new files. Here is a patch containing just those.

mikran’s picture

Status: Needs review » Fixed

And the missing files are now also committed.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.