New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
identity_id & unit_factor issue #1100
identity_id & unit_factor issue #1100
Conversation
I updated the pull request to include just the fixes for the identity_id and unit. |
@@ -32,7 +32,7 @@ def associate_with_org_unit | |||
identity = Identity.find identity_id | |||
|
|||
if rel_type == 'service_provider_organizational_unit' | |||
if not oe.service_providers or (oe.service_providers and not oe.service_providers.map(&:id).include? identity_id) | |||
if not oe.service_providers or (oe.service_providers and not oe.service_providers.map(&:identity_id).include? identity_id) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is minor, but for efficiency, pluck(:identity_id)
would be much more efficient since #map
requires the objects to be returned from the database
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes. I changed map to pluck. and committed the changes
It is |
I updated the branch to v3.0.0 and recreated the pull request