Sutty/panel
5
0
Fork 0
mirror of https://0xacab.org/sutty/sutty synced 2024-11-26 15:56:21 +00:00
Code Issues Projects Releases Wiki Activity

refactor: eliminar relaciones

Browse source 
(cherry picked from commit b81e38e9ea)
This commit is contained in:
f 2023-10-26 16:27:01 -03:00
parent ce299cc9d3
commit fabcabc100
No known key found for this signature in database
1 changed files with 15 additions and 13 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

28
app/services/post_service.rb
View file

@ -195,9 +195,7 @@ PostService = Struct.new(:site, :usuarie, :post, :params, keyword_init: true) do
case metadata.type case metadata.type
when 'has_and_belongs_to_many' when 'has_and_belongs_to_many'
associated_posts(value_was - value).each do |remove_post| associated_posts(value_was - value).each do |remove_post|
remove_post[inverse_attribute].value.delete(post.uuid.value) remove_relation_from(remove_post[inverse_attribute], post.uuid.value)
associated_posts_to_save << remove_post
end end
associated_posts(value - value_was).each do |add_post| associated_posts(value - value_was).each do |add_post|
@ -208,17 +206,12 @@ PostService = Struct.new(:site, :usuarie, :post, :params, keyword_init: true) do
end end
when 'has_many' when 'has_many'
associated_posts(value_was - value).each do |remove_post| associated_posts(value_was - value).each do |remove_post|
# @todo por qué no podemos usar nil para deshabilitar un valor? remove_relation_from(remove_post[inverse_attribute], '')
remove_post[inverse_attribute].value = ''
associated_posts_to_save << remove_post
end end
associated_posts(value - value_was).each do |add_post| associated_posts(value - value_was).each do |add_post|
associated_posts(add_post[inverse_attribute].value_was).each do |remove_post| associated_posts(add_post[inverse_attribute].value_was).each do |remove_post|
remove_post[attribute].value.delete(add_post.uuid.value) remove_relation_from(remove_post[attribute], add_post.uuid.value)
associated_posts_to_save << remove_post
end end
add_post[inverse_attribute].value = post.uuid.value add_post[inverse_attribute].value = post.uuid.value
@ -228,9 +221,7 @@ PostService = Struct.new(:site, :usuarie, :post, :params, keyword_init: true) do
when 'belongs_to' when 'belongs_to'
if value_was.present? if value_was.present?
associated_posts(value_was).each do |remove_post| associated_posts(value_was).each do |remove_post|
remove_post[inverse_attribute].value.delete(value_was) remove_relation_from(remove_post[inverse_attribute], value_was)
associated_posts_to_save << remove_post
end end
end end
@ -244,4 +235,15 @@ PostService = Struct.new(:site, :usuarie, :post, :params, keyword_init: true) do
end end
end end
end end
# @todo por qué no podemos usar nil para deshabilitar un valor?
def remove_relation_from(metadata, value)
case metadata.value
when Array then metadata.value.delete(value)
when String then metadata.value = ''
end
associated_posts_to_save << metadata.post
nil
end
end end