aboutsummaryrefslogtreecommitdiffstats
path: root/app/controllers/comment_controller.rb
Commit message (Collapse)AuthorAgeLines
* Add global protect_from_forgeryGareth Rees2014-12-18-1/+0
| | | | | | | Grepping the git logs didn’t bring up a good reason for this to be excluded. Seems like it came along after the app was initially created so it never got fully added for fear of regressions. The specs pass for this commit.
* Rename XXX comments with TODO:Gareth Rees2014-06-10-1/+1
| | | | Picks these up in `rake notes` and adds semantic meaning
* Use filter to reject if user is bannedGareth Rees2014-03-13-7/+9
| | | | | | | | | Extract checking whether a user is banned from making Comments on an InfoRequest to a filter in CommentController. Removes responsibility from the #new method. Adds a missing spec.
* Use filter to reject unless comments allowedGareth Rees2014-03-13-9/+11
| | | | | | | Extract checking whether comments are allowed on an InfoRequest to a filter in CommentController. Removes responsibility from the #new method.
* Extract create_track_thing from CommentControllerGareth Rees2014-03-13-1/+5
| | | | | Use a before_filter to make @track_thing available to all filters called on the same action and remove responsibility from the #new method
* Extract find_info_request from CommentControllerGareth Rees2014-03-13-12/+17
| | | | | Use a before_filter to make @info_request available to all filters called on the same action
* Graceful failure of new_comment routeGareth Rees2014-03-13-4/+6
| | | | | | | | | | Fixes https://github.com/mysociety/alaveteli/issues/662 If /annotate/request/:url_title is accessed when comments are disabled an exception is incorrectly thrown. Conditionals should be used for control flow, so now the action redirects to the info_request path and displays a notice.
* Make method names for finding existing objects clearerLouise Crow2013-12-04-2/+2
| | | | | They're not finding by the existing object, they're finding an existing object.
* Change email address in header of source code to hello@mysociety.orgMatthew Landauer2013-03-26-1/+1
|
* Remove svn tags that are out of date as we are now using gitMatthew Landauer2012-10-09-2/+0
|
* Add an option to disable comments on a requestRobin Houston2012-09-19-0/+7
| | | | Closes #30.
* Fail with NotFound if request slug doesn't existRobin Houston2012-08-20-1/+1
| | | | Closes #554.
* Remove trailing whitespace (to make a cleaner forthcoming merge with ↵Seb Bacon2012-05-15-3/+3
| | | | wombleton:feature/440_sparkly_admin_css)
* Changes required to get selishta (Kosovo) fork merges to pass tests. ↵Seb Bacon2011-07-06-1/+1
| | | | Includes new "REPLY_LATE_AFTER_DAYS" config option.
* I18n string for controllersFaton Selishta2011-06-30-6/+7
|
* Add CSRF protection on state changing actions. Use default handler ↵Louise Crow2011-02-28-0/+1
| | | | handle_unverified_request which clears session.
* Read only mode.Francis Irving2010-05-06-0/+1
|
* Option to ban users.francis2009-03-09-1/+8
|
* Don't subscribe you to your own requests taht you annotate.francis2008-11-17-2/+4
|
* Make it so you definitely don't get alert for the annotation that you just madefrancis2008-11-17-6/+8
|
* Have a link to the annotation form, rather than the form being on the request.francis2008-09-02-8/+15
| | | | | (Partly to reduce clutter, partly to not overemphasise annotation vs. sending followups. Not sure if this is right, but will try it a bit.)
* Do not show track field if already tracking.francis2008-08-29-1/+2
|
* By default sign people up to track requests that they annotate upon.francis2008-08-29-2/+18
|
* Prevent double posting of comments, and test code for that.francis2008-08-26-3/+3
|
* Test code for comments.francis2008-08-26-4/+3
|
* Most of comments code.francis2008-08-13-0/+54