We still get same issue addressed in #1593230: Restrict profile2 type per roles. It required a minor change to work for us and I am wondering if #1593230: Restrict profile2 type per roles really fixed it for everyone?

I will attach the patch soon.

Comments

junaidpv created an issue. See original summary.

junaidpv’s picture

Status: Active » Needs review
Issue tags: +role, +access
StatusFileSize
new706 bytes

It just need to filter out non-selected roles from $profile_type->data['roles'].

rickj’s picture

Could you clarify exactly the problem you're encountering?

Issue #1593230 was closed 4 years ago and has been working fine in dev releases since. I recently released 7.x-1.4 which incorporates that change (as well as many others). What version are you currently running?

I use profiles restricted by role a lot, and I don't have any issues with the current build. Exact steps to reproduce your problem would be a great help.

Thanks

junaidpv’s picture

Sorry for delayed reply.

We were using patch from #1593230 with some minor modification. We upgraded profile2 to 1.4 recently and found the code change from #1593230 alone is not fixing this issue.

We are having bit heavy site with lot of modules. I will test it in more detail and also in a fresh Drupal installation, then get back you with more info.

junaidpv’s picture

Status: Needs review » Closed (works as designed)

@RickJ Inspected it in a fresh D7 installation and I see no problem in official release. We are using patch from #2683917: Profile 2 types restricted by role, show for admin users and other few patches as well. Might be specific to our installation.

Closing the issue.

rickj’s picture

Thanks for confirming that it's working OK now. I recommend using the latest dev build (7.x-1.4+6-dev) rather than 7.x-1.4. It incorporates 6 committed patches, including the one you refer to. I hope to release it soon as 7.x-1.5.