scinote-eln/scinote-web
1
1
Fork 0
mirror of https://github.com/scinote-eln/scinote-web.git synced 2025-02-23 23:35:00 +08:00
Code Issues Projects Releases Packages Wiki Activity

Fix user assigning to teams with existing team sharing records [SCI-7720] (#4879)

Browse source
This commit is contained in:
Alex Kriuchykhin 2023-01-24 15:47:53 +01:00 committed by GitHub
parent 3fdbed382e
commit 00d99ee296
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 1 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
app/models/team_shared_object.rb
View file

@ -3,7 +3,6 @@
class TeamSharedObject < ApplicationRecord
enum permission_level: Extends::SHARED_OBJECTS_PERMISSION_LEVELS.except(:not_shared)
after_create :assign_shared_inventories, if: -> { shared_object.is_a?(Repository) }
before_destroy :unassign_unshared_items, if: -> { shared_object.is_a?(Repository) }
before_destroy :unassign_unshared_inventories, if: -> { shared_object.is_a?(Repository) }

2
app/services/user_assignments/create_team_user_assignments_service.rb
View file

@ -68,7 +68,7 @@ module UserAssignments
end
def create_or_update_user_assignment(object, role = nil)
new_user_assignment = object.user_assignments.find_or_initialize_by(user: @user)
new_user_assignment = object.user_assignments.find_or_initialize_by(user: @user, team: object.team)
return if new_user_assignment.manually_assigned?
new_user_assignment.user_role = role || @user_role