1) I just installed this module and noticed that "Do not use Elaborations UI" does not really work. If I check this option, and then try and add a friend (one way relationship), I still get a popup with the form for adding Elaboration. Is there a way that user does not have to explain/add the elaboration when adding a user as a friend.

2) Is there a way to send mail to users when someone becomes their friend (One way relationship which can be reciprocated). I had a look at the User Relationship Mailer options and it seems that mails are only sent when either user has to approve the request etc but there was no option to send a mail when a one way relationship is created between users

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

vegeneric’s picture

I ran into your 1) issue as well. I tracked it down in the code, here's a patch that worked for me against 6.x-1.0.

vegeneric’s picture

Status: Active » Needs review

Status: Needs review » Needs work

The last submitted patch, user_relationship_elaborations.patch, failed testing.

vegeneric’s picture

Version: 6.x-1.0-rc4 » 6.x-1.0
Status: Needs work » Needs review
Berdir’s picture

Version: 6.x-1.0 » 6.x-1.x-dev
vegeneric’s picture

Version: 6.x-1.x-dev » 6.x-1.0
FileSize
1.1 KB

re-uploading with appropriate version tag.

Berdir’s picture

Berdir’s picture

Status: Needs review » Needs work

The patch has the wrong file path, instead of "a/sites/all/modules/contrib/user_relationships/user_relationship_elaborations/user_relationship_elaborations.module", it just need to be "a/user_relationship_elaborations/user_relationship_elaborations.module". You can either edit the patch file manually or use the --prefix option of git diff.

vegeneric’s picture

Oops, looks like we were both commenting at once. Here's an updated version with corrected path.

Berdir’s picture

Status: Needs work » Needs review
mrf’s picture

Issue tags: +6.x-1.1

Looking at issues to include in 6.x-1.1

mrf’s picture

Version: 6.x-1.0 » 7.x-1.x-dev
Component: Code » Elaborations

Tested and committed to 6.x-1.x dev.

Thanks for your help!

Has anyone checked if this same issue occurs on 7.x? I'll try and check later if no one else has confirmed.

Berdir’s picture

Status: Needs review » Fixed

Looks like we do, commited.

Status: Fixed » Closed (fixed)
Issue tags: -6.x-1.1

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