On relation add/edit forms, the description under the "Directional" checkbox at first glance seems to make sense to me, but I think perhaps the wording "A directional relation is one that does not imply the reverse relation" mislead me a little.

It sounds like there would be *no* relation going back the other way, but that's not right; I think it means the SAME relation does not go back the other way. Would you consider something like "A directional relation is one where the same relation cannot be said in the reverse direction. The reverse direction may be described by a different relation."

And perhaps use a parent-child example instead of (or in addition to) "likes"? The "parent" relation is directional because the child is not the parent. But that's not to say there's no implied reverse relation.

Anyway, thanks for this awesome module — it's been my favourite module of the month all year so far :) Anytime a functional requirement comes up I already know the answer... relation module lol

Comments

bunthorne’s picture

bunthorne commented

How about the following:

A directional relation does not use the same description in each direction; a different description is used instead in the opposite direction to explain the reciprocal relationship. For example, a "parent-child" relation is directional because the description "is a parent of" only applies in one direction. The other direction uses a reciprocal description "is a child of."

jeff h’s picture

jeff h commented

That does actually help me more than the original. Thanks!

If others agree, can we get this into the module? I'm happy to write the patch if needed.

bunthorne’s picture

bunthorne commented

I suggest writing the patch and see what happens. I doubt that anyone else will respond to this minor Documentation issue until there is action requiring response.

mikran’s picture

mikran commented

Oh yea, patches are always welcome. We had some talks about UI when 1:n directional relations were introduced but time seems to be a limiting factor so often. Also I have relations with just 1 endpoint on my TODO list and that'll need some UI work as well but this one can be solved with a simple string change.

naught101’s picture

naught101 commented
Status: Active » Fixed

Fixed in commit 6ffbbb7.

Changed `A directional relation is one that does not imply the reverse relation.` to `A directional relation is one that does not imply the same relation in the reverse direction.`

I hope that's clear enough, and the example that follows that sentence is even clearer.

dpi’s picture

dpi
Location Perth, Australia
commented

Picked to 8.x branch. Commit #9b8832ca

Status: Fixed » Closed (fixed)

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