Remove images for feedbacks, closes #1507

fairmondo · Jul 2, 2017 · 873a976 · 873a976
1 parent ab56531
commit 873a976
Show file tree

Hide file tree

Showing 7 changed files with 4 additions and 47 deletions.
diff --git a/app/mailers/feedback_mailer.rb b/app/mailers/feedback_mailer.rb
@@ -9,8 +9,6 @@ def feedback_and_help(feedback, topic)
     @last_article_id = @feedback.last_article_id
     from = feedback.from? ? feedback.from : EMAIL_ADDRESSES['default']
 
-    attachments[feedback.image.image_file_name] = File.read(Rails.root.join(feedback.image.image.path(:original))) if feedback.image
-
     if EMAIL_ADDRESSES
       mail(to: EMAIL_ADDRESSES['FeedbackMailer'][@feedback.variety][topic], from: from, subject: @feedback.subject)
     end

diff --git a/app/models/feedback.rb b/app/models/feedback.rb
@@ -6,10 +6,6 @@ class Feedback < ActiveRecord::Base
   extend Enumerize
   extend ActiveModel::Naming
 
-  # Optional image
-  has_one :image, class_name: 'FeedbackImage', foreign_key: 'imageable_id'
-  accepts_nested_attributes_for :image
-
   enumerize :variety, in: [:report_article, :get_help, :send_feedback, :become_donation_partner]
 
   enumerize :feedback_subject, in: [:dealer, :technics, :other]

diff --git a/app/models/images/feedback_image.rb b/app/models/images/feedback_image.rb
diff --git a/app/refineries/feedback_refinery.rb b/app/refineries/feedback_refinery.rb
@@ -6,8 +6,7 @@ class FeedbackRefinery < ApplicationRefinery
   def create
     [
       :from, :subject, :text, :variety, :article_id, :feedback_subject,
-      :help_subject, :forename, :lastname, :organisation, :phone, :recaptcha,
-      { image_attributes: ImageRefinery.new(Image.new, user).default }
+      :help_subject, :forename, :lastname, :organisation, :phone, :recaptcha
     ]
   end
 end
diff --git a/app/views/feedbacks/varieties/_feedback.html.slim b/app/views/feedbacks/varieties/_feedback.html.slim
@@ -10,11 +10,6 @@ h1= t('feedback.title.send_feedback')
     = f.input :subject, required: true
     = f.input :text, label: false, required: true
 
-    - f.object.build_image
-    = f.semantic_fields_for :image do |image|
-      = f.inputs do
-        = image.input :image, as: :file, required: false
-
     = f.input :feedback_subject, as: :select, required: true, include_blank: true
   .captacha
     = render '/shared/recaptcha'

diff --git a/app/views/feedbacks/varieties/_help.html.slim b/app/views/feedbacks/varieties/_help.html.slim
@@ -9,11 +9,6 @@ h1= t('feedback.title.get_help')
     = f.input :subject, required: true
     = f.input :text, label: false, required: true
 
-    - f.object.build_image
-    = f.semantic_fields_for :image do |image|
-      = f.inputs do
-        = image.input :image, as: :file, required: false
-
     = f.input :help_subject, as: :select, required: true, include_blank: true
   .captacha
     = render '/shared/recaptcha'

diff --git a/test/models/image_test.rb b/test/models/image_test.rb
@@ -6,7 +6,7 @@
 
 describe Image do
   subject { Image.new }
-  let(:image) { create(:article_image) }
+  let(:image) { build_stubbed(:article_image) }
 
   it 'has a valid Factory' do
     subject.valid?.must_equal true
@@ -28,11 +28,9 @@
   end
 
   describe 'associations' do
-    let(:article_image) { ArticleImage.new }
-    let(:feedback_image) { FeedbackImage.new }
-    let(:user_image) { UserImage.new }
+    article_image = ArticleImage.new
+    user_image = UserImage.new
     it { article_image.must belong_to :article }
-    it { feedback_image.must belong_to :feedback }
     it { user_image.must belong_to :user }
   end