aboutsummaryrefslogtreecommitdiffstats
path: root/app/controllers/comment_controller.rb
Commit message (Collapse)AuthorAgeLines
* 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