aboutsummaryrefslogtreecommitdiffstats
path: root/app/controllers/request_controller.rb
Commit message (Collapse)AuthorAgeLines
* Rename ALL THE TEMPLATES!!1!!!one!!1!!Henare Degan2013-02-27-2/+2
| | | | .rhtml is deprecated in favour of .erb in Rails 3
* file_name is passed as a string not an arrayHenare Degan2013-02-26-3/+3
|
* create_ is deprecatedHenare Degan2013-02-25-1/+1
|
* Update to new mail sending APIHenare Degan2013-02-25-1/+1
|
* Cast to a string so it's not saved to the DB as an object under Ruby 1.9.3Henare Degan2013-02-21-1/+1
|
* Remove deprecated constants - #589Henare Degan2013-02-20-1/+1
|
* Merge branch 'develop' into rails-3-spikeHenare Degan2013-02-15-1/+1
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: Gemfile Gemfile.lock app/controllers/general_controller.rb app/controllers/track_controller.rb app/models/outgoing_message.rb app/models/public_body.rb app/models/user.rb app/views/general/frontpage.rhtml config/environment.rb config/initializers/inflections.rb config/initializers/mime_types.rb db/migrate/094_remove_old_tags_foreign_key.rb lib/timezone_fixes.rb spec/models/request_mailer_spec.rb spec/views/request/list.rhtml_spec.rb
| * Mark flash string with markup in it as html safe.Louise Crow2013-02-08-1/+1
| |
| * Remove interlock as it's not compatible with Rails 3 and is unmaintainedHenare Degan2013-01-18-16/+11
| |
| * Merge remote-tracking branch 'mysociety/develop' into rails_xssMatthew Landauer2013-01-15-11/+33
| |\
* | | Another hack to manually stitch together a fake email using tmailMatthew Landauer2013-02-01-1/+27
| | |
* | | There's really no need to test the internals of RailsMatthew Landauer2013-01-31-3/+7
| | |
* | | render_for_text doesn't exist anymore. Replace with render :textMatthew Landauer2013-01-29-3/+2
| | |
* | | Merge remote-tracking branch 'mysociety/develop' into rails-3-spikeMatthew Landauer2013-01-04-11/+33
|\ \ \ | | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: Gemfile.lock Rakefile app/controllers/request_controller.rb lib/tasks/rspec.rake spec/controllers/services_controller_spec.rb spec/spec_helper.rb spec/views/request/_after_actions.rhtml_spec.rb
| * | Limit pagination on similar pages in line with new upper limit on page offset.Louise Crow2012-12-17-0/+2
| | |
| * | Only serve up 'similar' pages up to the offset we use for list.Louise Crow2012-12-17-0/+5
| | |
| * | Put download zips in a predictable location - sharded folders based on ↵Louise Crow2012-12-13-1/+4
| | | | | | | | | | | | request ID, rather than distributing them across the download directories by the generated SHA. Preserve the uniqueness of the subdirectory.
| * | Use helper method for download path, and use Rails.root instead of file ↵Louise Crow2012-12-13-1/+1
| | | | | | | | | | | | location.
| * | Check that a request is publicly visible before generating a download link.Louise Crow2012-12-13-9/+15
| | |
| * | Don't offer or allow viewing of an HTML version of a response attachment if ↵Louise Crow2012-12-13-0/+6
| | | | | | | | | | | | the request is hidden, or requester_only. Google docs viewer won't be able to access it, and our own conversion process currently can produce image files that will then be publicly viewable directly from the webserver (see config/httpd.conf). If necessary we can revisit this code to enable admins and requesters to view the HTML version created by our own conversion without adding these files to a path that is served directly by the web server.
* | | Merge branch 'rails_xss' into rails-3-spikeMatthew Landauer2013-01-04-3/+3
|\ \ \ | | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: Gemfile Gemfile.lock app/views/request/_hidden_correspondence.rhtml app/views/request/hidden.rhtml app/views/request/new_please_describe.rhtml app/views/user/show.rhtml lib/i18n_fixes.rb
| * | Escaping fixes involving the unpleasant mix of translations, html and string ↵Matthew Landauer2013-01-03-3/+3
| |/ | | | | | | interpolation
* | The with_locale has been removed in Globalize3Henare Degan2012-12-11-3/+3
| | | | | | | | | | | | I think Globalize is supposed to pick up the locale from I18n anyway so I don't know if these are needed. I think I still haven't done the right thing but it's time to move on.
* | Remove interlock as it's not compatible with Rails 3 and is unmaintainedHenare Degan2012-12-11-16/+11
| |
* | Switch to new method nameHenare Degan2012-12-11-3/+3
|/
* Handle the default blank string passed by Configuration moduleLouise Crow2012-10-25-1/+1
|
* Handle a blank convert command more explicitly.Louise Crow2012-10-25-1/+1
|
* Don't keep recalculating the request statusLouise Crow2012-10-23-14/+15
|
* Remove svn tags that are out of date as we are now using gitMatthew Landauer2012-10-09-2/+0
|
* Inline ConfigHelper force_registration_on_new_requestMatthew Landauer2012-09-25-1/+1
|
* Extract configuration with defaults into one moduleMatthew Landauer2012-09-25-7/+6
|
* Record a status update by someone other than the request owner as a request ↵Louise Crow2012-09-11-6/+6
| | | | classification.
* Don't allow status updates on external requests from the front end interface ↵Louise Crow2012-08-23-1/+14
| | | | | | (they can still be changed from the admin interface). Final part of fix for #562.
* Don't error on a request to followup an external request.Louise Crow2012-08-23-0/+7
| | | | Partial solution to #562.
* Fail with NotFound if request slug doesn't existRobin Houston2012-08-20-19/+9
| | | | Closes #554.
* Do not send email for external requestsRobin Houston2012-08-20-1/+1
|
* When displaying attachments, only cache them out to files if they are ↵Louise Crow2012-08-16-3/+12
| | | | visible to everyone, so can be served up from a file cache without authentication.
* Merge branch 'develop' of github.com:sebbacon/alaveteli into developRobin Houston2012-06-23-4/+9
|\ | | | | | | | | | | | | | | Conflicts: Gemfile.lock script/handle-mail-replies script/handle-mail-replies.rb spec/controllers/request_controller_spec.rb
| * Only extend the cache time on InfoRequest views for sites with Varnish ↵Seb Bacon2012-06-20-1/+7
| | | | | | | | purging set up
| * Increase the time a page is cached for sites that are using Varnish. ↵Seb Bacon2012-06-20-1/+1
| | | | | | | | Relatedly, document the consequences of getting the Varnish config wrong; and make it easier to grep for problems with Varnish in the log.
| * Revert "Set a "Content-Disposition: attachment" header when downloading ↵Seb Bacon2012-06-15-2/+1
| | | | | | | | | | | | | | | | files. Closes #428" This reverts commit 506af7a640f63b17000ccfc5e1344bbc3039c913. See issue #428 for a discussion
| * Make emails requesting admin attention appear to come from the user who ↵Seb Bacon2012-06-12-1/+1
| | | | | | | | requested the attention, not the user who made the original request.
* | Improve explanationRobin Houston2012-06-06-2/+4
| | | | | | | | | | I struggled to understand this comment. I hope the new wording is clearer.
* | Redirect numeric request URLs in JSON mode tooRobin Houston2012-06-06-1/+1
|/ | | | | | URLs such as http://localhost:3000/en/request/106.json should be redirected to the corresponding named URL, including the .json on the end.
* Prevent spiders from reporting requestsRobin Houston2012-06-03-0/+6
| | | | | | | Make the "report request" button submit a POST rather than a GET request. Also restrict this action to logged-in users. Fixes #501.
* File.dirname(__FILE__) is not necessarily absoluteRobin Houston2012-06-03-1/+1
| | | | | | | It’s weird that there’s so much code here that implicitly assumes File.dirname(__FILE__) is an absolute path, because really in general it very much is not! This assumption was invalidated by the recent bundler-awareness changes.
* Test for reporting functionalitySeb Bacon2012-05-24-1/+3
|
* First stab at new feature. Adds new box in sidebar for reporting an issue, a ↵Seb Bacon2012-05-24-0/+11
| | | | | | new "needs admin attention"-type state to InfoRequests, a flag indicating that a request has ever been marked as needing admin attention, and a controller method and route for setting this state & flag. Also adds the reason something needs admin attention to the subject of the email that gets sent to administrators. Neeeds tests.
* Remove trailing whitespace (to make a cleaner forthcoming merge with ↵Seb Bacon2012-05-15-50/+50
| | | | wombleton:feature/440_sparkly_admin_css)
* Don't cache results from pages with different locales using the same key. ↵Seb Bacon2012-05-04-1/+1
| | | | Fixes #454