Merge pull request #216 from TreinaDev/feat/admin-deletes-reported-profile

Feat/Admin remove conta de usuário denunciado

Author: Luckvc

app/controllers/reports_controller.rb

@@ -3,9 +3,9 @@ class ReportsController < ApplicationController
   before_action :set_reportable_for_new, only: :new
   before_action :set_reportable_for_create, only: :create
   before_action :redirect_unless_published_post
-  before_action :authorize!, only: %i[index show]
   before_action :redirect_if_self_report, only: :create
-  before_action :set_report, only: %i[reject show remove_content]
+  before_action :authorize!, only: %i[index show reject remove_content remove_profile]
+  before_action :set_report, only: %i[reject show remove_content remove_profile]
 
   def new
     set_offences
@@ -37,6 +37,13 @@ def remove_content
     redirect_to @report, notice: t('.success')
   end
 
+  def remove_profile
+    @report.reportable.update removed: true
+    @report.reportable.inactive!
+    @report.granted!
+    redirect_to @report, notice: t('.success')
+  end
+
   private
 
   def set_reportable_for_new

app/models/user.rb

@@ -76,6 +76,14 @@ def delete_user_data
     destroy
   end
 
+  def active_for_authentication?
+    super && !profile.removed?
+  end
+
+  def inactive_message
+    I18n.t('reports.removed_account')
+  end
+
   private
 
   def subscribe_likes_mailer_job

app/views/reports/show.html.erb

@@ -1,3 +1,6 @@
+<div class="container">
+  <%= link_to t('return_btn'), reports_path, class: 'btn btn-secondary' %>
+</div>
 <div class="d-flex">
   <div class="container d-flex flex-column w-50">
     <div class="mt-3">
@@ -35,22 +38,31 @@
           <p class="card-text"><%= @report.message %></p>
 
           <p class="card-subtitle mb-2 text-muted">
-            <%= I18n.t('.reporting_profile') %>: <%= link_to @report.profile.full_name, profile_path(@report.profile) %>
+            <%= t('.reporting_profile') %>: <%= link_to @report.profile.full_name, profile_path(@report.profile) %>
           </p>
           <% if @report.pending? %>
             <div class="d-flex flex-row justify-content-center mt-5 gap-5">
-              <% unless @report.reportable.is_a? Profile %>
-                <%= button_to t('.remove_content_btn'), remove_content_report_path(@report), class:'card-btn flex-column btn btn-danger btn-lg' %>
+              <% if @report.reportable.is_a? Profile %>
+                <%= button_to t('.remove_profile_btn'), remove_profile_report_path(@report),
+                              class:'card-btn flex-column btn btn-danger btn-lg',
+                              data: { turbo_confirm: t('.remove_profile_warning') } %>
+              <% else %>
+                <%= button_to t('.remove_content_btn'), remove_content_report_path(@report),
+                              class:'card-btn flex-column btn btn-danger btn-lg',
+                              data: { turbo_confirm: t('.remove_content_warning') } %>
               <% end %>
               <%= button_to t('.reject_btn'), reject_report_path(@report), class:'card-btn flex-column btn btn-secondary btn-lg ml-2' %>
             </div>
           <% else %>
             <div class="d-flex flex-row justify-content-center mt-5">
               <h4 class= "card-subtitle mb-2 text-dark">
-                <%= I18n.t('reports.index.rejected') if @report.rejected? %>
+                <%= t('reports.index.rejected') if @report.rejected? %>
               </h4>
               <h4 class= "card-subtitle mb-2 text-danger">
-                <%= I18n.t('reports.index.granted') if @report.granted? %>
+                <% if @report.granted? %>
+                  <%= t('.profile_removed') if @report.reportable.is_a? Profile %>
+                  <%= t('.content_removed') unless @report.reportable.is_a? Profile %>
+                <% end %>
               </h4>
             </div>
           <% end %>

app/views/settings/index.html.erb

@@ -28,7 +28,7 @@
 		<%= form_with(url: deactivate_profile_path, method: :patch) do |form| %>
 			<%= form.submit t('.arquive_btn'),
 				class: "btn btn-danger",
-				data: { turbo_confirm: I18n.t('.turbo_arquive_warning') }
+				data: { turbo_confirm: I18n.t('settings.index.turbo_arquive_warning') }
 			%>
 		<% end %>
 	</div>
@@ -37,7 +37,7 @@
 		<%= form_with(url: delete_account_path, method: :delete) do |form| %>
 			<%= form.submit t('.delete_btn'),
 				class: "btn btn-danger",
-				data: { turbo_confirm: I18n.t('.turbo_delete_warning') }
+				data: { turbo_confirm: I18n.t('settings.index.turbo_delete_warning') }
 			%>
 		<% end %>
 	</div>

config/locales/models/reports.pt-BR.yml

@@ -11,6 +11,7 @@ pt-BR:
         offence_type: Tipo de ofensa
   reports:
     report_btn: Denunciar
+    removed_account: Sua conta foi removida pelos adminitradores
     create:
       self_report: Você não pode denunciar sí mesmo ou o próprio conteúdo.
       success: Sua denúncia foi registrada
@@ -21,12 +22,19 @@ pt-BR:
       success: Denúncia rejeitada com sucesso
     remove_content:
       success: Conteúdo removido com sucesso
+    remove_profile:
+      success: Perfil removido com sucesso
     comment:
       see_post: Ver publicação
     show:
       reporting_profile: Denunciado por
       reject_btn: Rejeitar denúncia
       remove_content_btn: Remover conteúdo
+      remove_profile_btn: Remover perfil
+      remove_profile_warning: O usuário perderá o acesso à conta dele permanentemente. Tem certeza que deseja continuar?
+      remove_content_warning: Esse conteúdo será deletado. Tem certeza que deseja continuar?
+      profile_removed: Perfil removido
+      content_removed: Conteúdo removido
     index:
       pending: Pendente
       granted: Conteúdo removido
@@ -43,3 +51,4 @@ pt-BR:
     spam: Spam
     disturbin_content: Conteúdo pertubador
     harassment: Abuso/Perseguição
+

config/routes.rb

@@ -22,7 +22,11 @@
   end
 
   resources :reports, only: %i[index new create show] do
-    post 'reject', 'remove_content', on: :member
+    member do
+      post 'reject'
+      post 'remove_content'
+      post 'remove_profile'
+    end
   end
 
   resources :posts, only: %i[] do

db/migrate/20240214134106_add_removed_to_profile.rb

@@ -0,0 +1,5 @@
+class AddRemovedToProfile < ActiveRecord::Migration[7.1]
+  def change
+    add_column :profiles, :removed, :boolean, default: false
+  end
+end

db/schema.rb

@@ -91,5 +91,21 @@
       end
       expect(page).to have_content 'E-mail ou senha inválidos'
     end
+
+    it 'e seu perfil foi removido pelos administradores' do
+      user = create(:user)
+      user.profile.inactive!
+      user.profile.update removed: true
+
+      visit root_path
+      click_on 'Entrar'
+      within '#new_user' do
+        fill_in 'E-mail', with: user.email
+        fill_in 'Senha', with: user.password
+        click_on 'Entrar'
+      end
+
+      expect(page).to have_content 'Sua conta foi removida pelos adminitradores'
+    end
   end
 end

spec/system/authentication/visitor_logs_in_spec.rb

@@ -16,7 +16,7 @@
     expect(page).to have_content 'Denúncia rejeitada com sucesso'
   end
 
-  it 'decidindo remover o conteúdo' do
+  it 'decidindo remover o post' do
     admin = create(:user, :admin)
     post = create(:post, :published)
     create(:report, reportable: post)
@@ -25,10 +25,52 @@
     login_as admin
     visit reports_path
     page.all('.see_more').to_a.first.click
-    click_on 'Remover conteúdo'
+    accept_prompt do
+      click_on 'Remover conteúdo'
+    end
 
     expect(Report.first).to be_granted
     expect(Report.second).to be_pending
+    expect(post.reload).to be_removed
     expect(page).to have_content 'Conteúdo removido com sucesso'
   end
+
+  it 'decidindo remover o comentário' do
+    admin = create(:user, :admin)
+    comment = create(:comment)
+    create(:report, :for_post)
+    create(:report, reportable: comment)
+
+    login_as admin
+    visit reports_path
+    page.all('.see_more').to_a.second.click
+    accept_prompt do
+      click_on 'Remover conteúdo'
+    end
+
+    expect(Report.first).to be_pending
+    expect(Report.second).to be_granted
+    expect(comment.reload).to be_removed
+    expect(page).to have_content 'Conteúdo removido com sucesso'
+  end
+
+  it 'decidindo remover o perfil' do
+    admin = create(:user, :admin)
+    user = create(:user)
+    create(:report, :for_post)
+    create(:report, reportable: user.profile)
+
+    login_as admin
+    visit reports_path
+    page.all('.see_more').to_a.second.click
+    accept_prompt do
+      click_on 'Remover perfil'
+    end
+
+    expect(page).to have_content 'Perfil removido com sucesso'
+    expect(Report.first).to be_pending
+    expect(Report.second).to be_granted
+    expect(user.reload.profile).to be_inactive
+    expect(user.profile.removed).to be true
+  end
 end