From 962623ede4718b121dba90ab93645252abf3d1d7 Mon Sep 17 00:00:00 2001
From: f <f@sutty.nl>
Date: Wed, 10 Jul 2019 17:28:28 -0300
Subject: [PATCH] agregar o quitar privilegios usando roles

---
 app/controllers/usuaries_controller.rb | 13 ++++++-------
 1 file changed, 6 insertions(+), 7 deletions(-)

diff --git a/app/controllers/usuaries_controller.rb b/app/controllers/usuaries_controller.rb
index 7ecf727e..a937c3e7 100644
--- a/app/controllers/usuaries_controller.rb
+++ b/app/controllers/usuaries_controller.rb
@@ -1,6 +1,8 @@
 # frozen_string_literal: true
 
-# Controlador de Usuaries
+# Controlador de relación entre usuaries y sitios
+#
+# XXX: Debería llamarse SiteUsuariesController?
 class UsuariesController < ApplicationController
   include Pundit
   before_action :authenticate_usuarie!
@@ -21,6 +23,7 @@ class UsuariesController < ApplicationController
 
     @usuarie = Usuarie.find(params[:id])
 
+    # Mágicamente elimina el rol
     @usuarie.sites.delete(@site)
 
     redirect_to site_usuaries_path
@@ -32,9 +35,7 @@ class UsuariesController < ApplicationController
     authorize SiteUsuarie.new(@site, current_usuarie)
 
     @usuarie = Usuarie.find(params[:usuarie_id])
-
-    @usuarie.sites.delete(@site)
-    @site.invitades << @usuarie
+    @usuarie.rol_for_site(@site).update_attribute :rol, 'invitade'
 
     redirect_to site_usuaries_path
   end
@@ -45,9 +46,7 @@ class UsuariesController < ApplicationController
     authorize SiteUsuarie.new(@site, current_usuarie)
 
     @usuarie = Usuarie.find(params[:usuarie_id])
-
-    @usuarie.sites_as_invitade.delete(@site)
-    @site.usuaries << @usuarie
+    @usuarie.rol_for_site(@site).update_attribute :rol, 'usuarie'
 
     redirect_to site_usuaries_path
   end