diff options
Diffstat (limited to 'app')
| -rw-r--r-- | app/controllers/request_controller.rb | 10 | ||||
| -rw-r--r-- | app/models/request_mailer.rb | 4 | ||||
| -rw-r--r-- | app/views/request/_correspondence.rhtml | 2 | ||||
| -rw-r--r-- | app/views/request/show_response.rhtml (renamed from app/views/request/classify.rhtml) | 6 |
4 files changed, 13 insertions, 9 deletions
diff --git a/app/controllers/request_controller.rb b/app/controllers/request_controller.rb index 843446032..d1698c787 100644 --- a/app/controllers/request_controller.rb +++ b/app/controllers/request_controller.rb @@ -4,7 +4,7 @@ # Copyright (c) 2007 UK Citizens Online Democracy. All rights reserved. # Email: francis@mysociety.org; WWW: http://www.mysociety.org/ # -# $Id: request_controller.rb,v 1.31 2008-01-09 15:35:40 francis Exp $ +# $Id: request_controller.rb,v 1.32 2008-01-09 19:34:07 francis Exp $ class RequestController < ApplicationController @@ -73,12 +73,16 @@ class RequestController < ApplicationController end end - # Did the incoming message contain info? - def classify + # Show an individual incoming message, and let the user classify it. + def show_response @incoming_message = IncomingMessage.find(params[:incoming_message_id]) @info_request = @incoming_message.info_request @collapse_quotes = params[:unfold] ? false : true + if @incoming_message.info_request_id != params[:id].to_i + raise sprintf("Incoming message %d does not belong to request %d", @incoming_message.info_request_id, params[:id]) + end + if not authenticated_as_user?(@info_request.user, :web => "To view and classify the response to this FOI request", :email => "Then you can classify the FOI response you have got from " + @info_request.public_body.name + ".", diff --git a/app/models/request_mailer.rb b/app/models/request_mailer.rb index 13bb5fdab..f6b34c120 100644 --- a/app/models/request_mailer.rb +++ b/app/models/request_mailer.rb @@ -4,7 +4,7 @@ # Copyright (c) 2007 UK Citizens Online Democracy. All rights reserved. # Email: francis@mysociety.org; WWW: http://www.mysociety.org/ # -# $Id: request_mailer.rb,v 1.14 2008-01-03 18:21:30 francis Exp $ +# $Id: request_mailer.rb,v 1.15 2008-01-09 19:34:07 francis Exp $ class RequestMailer < ActionMailer::Base @@ -26,7 +26,7 @@ class RequestMailer < ActionMailer::Base def new_response(info_request, incoming_message) post_redirect = PostRedirect.new( - :uri => classify_request_url(:incoming_message_id => incoming_message.id), + :uri => show_response_url(:id => info_request.id, :incoming_message_id => incoming_message.id), :user_id => info_request.user.id) post_redirect.save! url = confirm_url(:email_token => post_redirect.email_token) diff --git a/app/views/request/_correspondence.rhtml b/app/views/request/_correspondence.rhtml index f3f39be7a..d087aa87a 100644 --- a/app/views/request/_correspondence.rhtml +++ b/app/views/request/_correspondence.rhtml @@ -20,7 +20,7 @@ <% end %> on <strong><%= simple_date(incoming_message.sent_at) %></strong> <% if not incoming_message.user_classified %> - — please <%= link_to "classify this response", classify_request_url(:incoming_message_id => incoming_message.id) %> + — please <%= link_to "classify this response", show_response_url(:id => incoming_message.info_request.id, :incoming_message_id => incoming_message.id) %> <% end %> </p> <% elsif (correspondence.class.to_s == 'InfoRequestEvent') diff --git a/app/views/request/classify.rhtml b/app/views/request/show_response.rhtml index 15789858e..a7fa90e6f 100644 --- a/app/views/request/classify.rhtml +++ b/app/views/request/show_response.rhtml @@ -1,15 +1,15 @@ -<% @title = "Classify FOI response" %> +<% @title = "View and classify FOI response" %> <%= foi_error_messages_for :incoming_message %> -<div id="classify_view"> +<div id="show_response_view"> <h2>New response to your request '<%= request_link @info_request %>'</h2> <%= render :partial => 'correspondence', :locals => { :correspondence => @incoming_message } %> </div> <% if not @incoming_message.user_classified %> - <div id="classify_action"> + <div id="show_response_classify_action"> <%= render :partial => 'classify' %> <div> <% end %> |