I want to make sure that each user can follow other users, but using the permission given in the title does not seem possible.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Berdir’s picture

Status: Active » Postponed (maintainer needs more info)

Make sure that you are using the most recent 7.x-1.x-dev version and you're users have the "Request type relationships" permission.

klucid’s picture

Version: 7.x-1.x-dev » 7.x-1.0-alpha4
FileSize
56.98 KB

I'm having a similar issue (or so it seems).

I'm using 7.x-1.0-alpha4 and I believe my permissions are set properly (see attached), however when a user logs in to their requests page, the request shows up, but there is no option to accept the request under the "operations" column.

Note on screenshot: UR should be a one-way relationship, "User" to "Supplier".

klucid’s picture

Just a follow up on what I've just discovered. The operations to Approve or Decline the request appears when the "Supplier" role is given permissions to "Request follower relationships". This however does not work for my case, because I would only like an Individual to be allowed to "follow" a Supplier.

Thanks in advance! :)

tmsimont’s picture

Title: "Maintain own relationships" permission does not work » "Maintain x relationships" permission does not work without "Request x relationships"
Status: Postponed (maintainer needs more info) » Active

I also have this same problem.

I think there should be ample information here to reproduce. I discovered this is in a fresh install of this module, I have hardly any other modules installed.

Step-by-step on reproduction
1) 2 roles required: for this example Role A is Requester, Role B is Acceptor
2) At least 1 type of relationship required, in this example it's just "x"
3) Give both roles "Have x relationships, Maintain x relationships and Delete x relationships"
4) Give only Role A, Requester the "Request x relationships"
5) Log in as a Requester, and request a relationship with an Acceptor
6) Log in as the Acceptor, and go to the "Received requests" page

There are no "operations" in the operations column, which means the relationship will never happen.

tmsimont’s picture

Assigned: middleman90 » Unassigned
tmsimont’s picture

Version: 7.x-1.0-alpha4 » 7.x-1.x-dev
Status: Active » Needs review
FileSize
3.7 KB

very much related is #1898026: PHP notice "Notice: Undefined property: stdClass::$requester_id" on user_relationships_ui_check_access -- it's not the same exact issue but it's in the same boat. I'll close that out as a duplicate of this.

the attached patch should make everyone happy :)

tmsimont’s picture

Title: "Maintain x relationships" permission does not work without "Request x relationships" » Maintain, Delete and Request permissions problems

broader title to encompass other related issue

daveonaka’s picture

Version: 7.x-1.x-dev » 7.x-1.0-alpha5
Component: User interface » Code

I hope I'm posting this in the right location. The error-
Error messageNotice: Undefined property: stdClass::$requester_id in user_relationships_ui_check_access()
Occurs when one of two users is deleted. In this case the initiator of a relationship was deleted. Going to the receiver's Relationships tab showed no relationships, but the error message is displayed.
Note: I have not yet applied the patch in #6.

tmsimont’s picture

Version: 7.x-1.0-alpha5 » 7.x-1.x-dev

Yes it's the same error that was in the other issue I closed as a duplicate. Please keep the issue in the dev branch because that's the most recent branch where the error still exists. Patches are committed to dev and then new releases come from dev. Please get the dev branch and test the patch I posted. Let us know if that works for you. If it does we will be closer to committing a solution.

joshua.howell’s picture

Issue summary: View changes

Using 7.x-1.0-alpha6 and applying patch #6 resolved the error message for me.

heylookalive’s picture

I can't see an issue with the patch, but don't have enough knowledge of the UI submodule to commit, any other maintainers?

Also @joshua.howell if you're confident that the patch is resolving the issues please mark as RBTC. I'd like others to chime in but this is an old issue and could be wishful thinking.

joshua.howell’s picture

Status: Needs review » Reviewed & tested by the community
hockey2112’s picture

Thank you!!! This patch saved my sanity.