Merge pull request #41 from hyoshida/cannot-destroy-a-variant

Cannot destroy a variant
appirits · Oct 8, 2015 · 987d478 · 987d478
2 parents 43150ad + c8b27ce
commit 987d478
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/backend/app/controllers/comable/admin/variants_controller.rb b/backend/app/controllers/comable/admin/variants_controller.rb
@@ -41,7 +41,7 @@ def update
 
       def destroy
         @variant.destroy
-        redirect_to comable.admin_product_variants_path(@product), notice: Comable.t('successful')
+        redirect_to comable.admin_product_path(@product), notice: Comable.t('successful')
       end
 
       private

diff --git a/backend/app/views/comable/admin/variants/edit.slim b/backend/app/views/comable/admin/variants/edit.slim
@@ -28,4 +28,4 @@
         .panel-body
           p
             = Comable.t('admin.confirmation_to_remove_product')
-          = link_to Comable.t('admin.actions.destroy'), comable.admin_product_path(@product), method: :delete, class: 'btn btn-danger'
+          = link_to Comable.t('admin.actions.destroy'), comable.admin_product_variant_path(@product, @variant), method: :delete, class: 'btn btn-danger'
diff --git a/backend/spec/controllers/comable/admin/variants_controller_spec.rb b/backend/spec/controllers/comable/admin/variants_controller_spec.rb
@@ -115,7 +115,7 @@
     it 'redirects to the variants list' do
       variant = create(:variant, product: product)
       delete :destroy, product_id: product.to_param, id: variant.to_param
-      expect(response).to redirect_to([comable, :admin, product, :variants])
+      expect(response).to redirect_to([comable, :admin, product])
     end
   end
 end