displays error on failed update
This commit is contained in:
@@ -14,9 +14,9 @@ class Admin::SpendingProposalsController < Admin::BaseController
|
|||||||
end
|
end
|
||||||
|
|
||||||
def edit
|
def edit
|
||||||
@admins = Administrator.includes(:user).all
|
load_admins
|
||||||
@valuators = Valuator.includes(:user).all.order("description ASC").order("users.email ASC")
|
load_valuators
|
||||||
@tags = ActsAsTaggableOn::Tag.spending_proposal_tags
|
load_tags
|
||||||
end
|
end
|
||||||
|
|
||||||
def update
|
def update
|
||||||
@@ -24,6 +24,9 @@ class Admin::SpendingProposalsController < Admin::BaseController
|
|||||||
redirect_to admin_spending_proposal_path(@spending_proposal, SpendingProposal.filter_params(params)),
|
redirect_to admin_spending_proposal_path(@spending_proposal, SpendingProposal.filter_params(params)),
|
||||||
notice: t("flash.actions.update.spending_proposal")
|
notice: t("flash.actions.update.spending_proposal")
|
||||||
else
|
else
|
||||||
|
load_admins
|
||||||
|
load_valuators
|
||||||
|
load_tags
|
||||||
render :edit
|
render :edit
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
@@ -38,4 +41,16 @@ class Admin::SpendingProposalsController < Admin::BaseController
|
|||||||
params.require(:spending_proposal).permit(:title, :description, :external_url, :geozone_id, :association_name, :administrator_id, :tag_list, valuator_ids: [])
|
params.require(:spending_proposal).permit(:title, :description, :external_url, :geozone_id, :association_name, :administrator_id, :tag_list, valuator_ids: [])
|
||||||
end
|
end
|
||||||
|
|
||||||
|
def load_admins
|
||||||
|
@admins = Administrator.includes(:user).all
|
||||||
|
end
|
||||||
|
|
||||||
|
def load_valuators
|
||||||
|
@valuators = Valuator.includes(:user).all.order("description ASC").order("users.email ASC")
|
||||||
|
end
|
||||||
|
|
||||||
|
def load_tags
|
||||||
|
@tags = ActsAsTaggableOn::Tag.spending_proposal_tags
|
||||||
|
end
|
||||||
|
|
||||||
end
|
end
|
||||||
|
|||||||
@@ -423,6 +423,20 @@ feature 'Admin spending proposals' do
|
|||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
|
scenario "Errors on update" do
|
||||||
|
spending_proposal = create(:spending_proposal)
|
||||||
|
create(:geozone, name: "Barbate")
|
||||||
|
|
||||||
|
visit admin_spending_proposal_path(spending_proposal)
|
||||||
|
click_link 'Edit'
|
||||||
|
|
||||||
|
fill_in 'spending_proposal_title', with: ''
|
||||||
|
|
||||||
|
click_button 'Update'
|
||||||
|
|
||||||
|
expect(page).to have_content "can't be blank"
|
||||||
|
end
|
||||||
|
|
||||||
end
|
end
|
||||||
|
|
||||||
context 'Summary' do
|
context 'Summary' do
|
||||||
|
|||||||
Reference in New Issue
Block a user