From 7b3f49a25aecef13a7b20ba7014c7432318bbd26 Mon Sep 17 00:00:00 2001 From: Joachim Filip Ignacy Bartosik Date: Tue, 26 Jul 2011 16:15:36 +0200 Subject: Improve controller readability --- site/app/controllers/agendas_controller.rb | 2 +- site/app/controllers/voting_options_controller.rb | 8 ++++---- 2 files changed, 5 insertions(+), 5 deletions(-) (limited to 'site') diff --git a/site/app/controllers/agendas_controller.rb b/site/app/controllers/agendas_controller.rb index 669f364..f94d96c 100644 --- a/site/app/controllers/agendas_controller.rb +++ b/site/app/controllers/agendas_controller.rb @@ -14,7 +14,7 @@ class AgendasController < ApplicationController end def results - data = JSON.parse(request.env["rack.input"].read) + data = JSON.parse(request.env['rack.input'].read) Agenda.update_voting_options data['agenda'] Agenda.process_results data data['votes'] agenda = Agenda.current diff --git a/site/app/controllers/voting_options_controller.rb b/site/app/controllers/voting_options_controller.rb index 14ec89f..73a0dfb 100644 --- a/site/app/controllers/voting_options_controller.rb +++ b/site/app/controllers/voting_options_controller.rb @@ -6,7 +6,10 @@ class VotingOptionsController < ApplicationController def community_vote option = VotingOption.find(params[:id]) - unless option.nil? + if option.nil? + flash[:notice] = "No such voting option" + redirect_to :controller => :agendas, :action => :index + else if current_user.signed_up? Vote.vote_for_option(current_user, option, false) flash[:notice] = "You voted for #{option.description}" @@ -14,9 +17,6 @@ class VotingOptionsController < ApplicationController flash[:notice] = "You must be logged in to vote" end redirect_to :controller => :agenda_items, :action => :show, :id => option.agenda_item_id - else - flash[:notice] = "No such voting option" - redirect_to :controller => :agendas, :action => :index end end end -- cgit v1.2.3-18-g5258