Commit message (Collapse) | Author | Age | Lines | |
---|---|---|---|---|
* | Exclude hidden responses when calculating old_unclassified | Louise Crow | 2013-09-16 | -4/+14 |
| | | | | | Make old_unclassified_params method consistent with last_public_response_event and associated methods. | |||
* | Only include public messages in who_can_followup_to | Louise Crow | 2013-09-16 | -0/+2 |
| | ||||
* | Add public criteria for message event access methods | Louise Crow | 2013-09-16 | -16/+18 |
| | | | | | | | get_last_response_event and get_last_outgoing_event are used in various places to determine which events to link to, use in queries etc. Restrict them to refer to the last publicly visible event of the relevant type, and rename them to make that clear. | |||
* | Hide hidden outgoing messages in download. | Louise Crow | 2013-09-16 | -5/+3 |
| | ||||
* | InfoRequestEvent.indexed_by_search consults OutgoingMessage. | Louise Crow | 2013-09-16 | -0/+3 |
| | ||||
* | Move indexed_by_search to MessageProminence | Louise Crow | 2013-09-16 | -4/+0 |
| | | | | Add some tests that it's working on the outgoing message model. | |||
* | Add hidden messages for outgoing message. | Louise Crow | 2013-09-16 | -1/+4 |
| | | | | | | Conflicts: app/views/request/_incoming_correspondence.html.erb | |||
* | Create MessageProminence module. | Louise Crow | 2013-09-16 | -6/+2 |
| | | | | | | At least some of the logic for incoming and outgoing message prominence is going to be identical, so move it to a module they can both include and use. | |||
* | Move some download methods to InfoRequest. | Louise Crow | 2013-09-16 | -0/+34 |
| | | | | | | Use send_file to send zips. Also adds 'all_can_view_all_correspondence?' - is this request completely cachable, or do we need to cache different versions for different levels of privilege? | |||
* | Move display text generation to outgoing message. | Louise Crow | 2013-09-16 | -2/+8 |
| | ||||
* | Move a couple of complex conditionals to helper methods. | Louise Crow | 2013-09-16 | -0/+9 |
| | | | | We're about to reuse them for the text view. | |||
* | Normalize method spacing. | Louise Crow | 2013-09-16 | -0/+3 |
| | ||||
* | Extract calculation of last update hash | Louise Crow | 2013-09-16 | -0/+4 |
| | ||||
* | Add new code and specs for hiding attachments. | Louise Crow | 2013-09-16 | -0/+4 |
| | ||||
* | Add a possible event type 'edit_incoming'. | Louise Crow | 2013-09-16 | -1/+1 |
| | | | | This will cover changes in prominence to incoming messages. | |||
* | Make prominence states a constant so we can access them elsewhere. | Louise Crow | 2013-09-16 | -5/+2 |
| | ||||
* | Don't index hidden and requester_only incoming messages. | Louise Crow | 2013-09-16 | -0/+8 |
| | ||||
* | Adding prominence_reason to IncomingMessage. | Louise Crow | 2013-09-16 | -0/+1 |
| | ||||
* | Refactor common logic about prominence and access. | Louise Crow | 2013-09-16 | -14/+2 |
| | | | | Move it into the Ability module. | |||
* | Add a method to ask whether a user can view an incoming message. | Louise Crow | 2013-09-16 | -0/+10 |
| | ||||
* | Rename method to make it a bit more generic | Louise Crow | 2013-09-16 | -5/+5 |
| | | | | A super user will be able to see all hidden things, not just requests. | |||
* | Add response_event helper | Louise Crow | 2013-09-16 | -0/+6 |
| | | | | | Add a convenience method for getting the 'response' event associated with an incoming message. | |||
* | Add prominence to incoming message. | Louise Crow | 2013-09-16 | -0/+7 |
| | ||||
* | Re-annotate models with database fields | Louise Crow | 2013-09-16 | -143/+124 |
| | ||||
* | Remove prominence. | Louise Crow | 2013-09-16 | -23/+0 |
| | | | | | | | Add a migration to remove the unused column 'prominence' from info_request_events, remove validation of prominence, remove unused InfoRequestEvent.user_can_view? method. Remove references to InfoRequestEvent.prominence. | |||
* | Prevent erroneous internal error messages with multibyte characters under ↵ | Louise Crow | 2013-07-25 | -2/+2 |
| | | | | ruby 1.8 by using mb_chars for length comparison - it counts multibyte characters. | |||
* | Merge remote-tracking branch ↵ | Louise Crow | 2013-07-23 | -11/+1 |
|\ | | | | | | | | | | | | | 'openaustralia_github/tests_set_described_state' into rails-3-develop Conflicts: spec/models/info_request_spec.rb | |||
| * | Now record a state_update event *whenever* a user classifies a request | Matthew Landauer | 2013-04-20 | -10/+0 |
| | | ||||
| * | How we would handle event states if we added a state_update event whenever a ↵ | Matthew Landauer | 2013-04-20 | -1/+1 |
| | | | | | | | | user classifies a request | |||
| * | Simple solution to bugs shown up in tests in last 2 commits | Matthew Landauer | 2013-04-18 | -1/+1 |
| | | ||||
* | | Split the string interpolation and translation into two steps so that the ↵ | Louise Crow | 2013-06-24 | -2/+5 |
| | | | | | | | | translation gets picked up by rake gettext:find. Add a bit more explanation. | |||
* | | Merge branch 'hotfix/0.11.0.14' into rails-3-develop | Louise Crow | 2013-06-19 | -1/+1 |
|\ \ | ||||
| * | | Allow the HTML display of the body of an email to handle the case where the ↵0.11.0.14hotfix/0.11.0.14 | Louise Crow | 2013-06-19 | -1/+1 |
| | | | | | | | | | | | | body is only quoted text and there is no subject. | |||
* | | | Merge branch 'hotfix/0.11.0.7' into rails-3-develop | Louise Crow | 2013-06-10 | -1/+1 |
|\| | | | | | | | | | | | | | | Conflicts: spec/models/info_request_spec.rb | |||
| * | | Cope with replying to a message with a missing or empty Subject | Mark Longair | 2013-06-07 | -1/+1 |
| | | | | | | | | | | | | | | | | | | Conflicts: spec/models/info_request_spec.rb | |||
* | | | Merge branch 'hotfix/0.11.0.4' into rails-3-develop | Louise Crow | 2013-06-10 | -1/+3 |
|\| | | ||||
| * | | Merge branch 'release/0.11'0.11.0.3 | Louise Crow | 2013-06-04 | -1174/+281 |
| |\ \ | ||||
| * | | | Return a string from user.name always to avoid any complications with other ↵hotfix/0.9.0.5 | Louise Crow | 2013-05-28 | -1/+3 |
| | | | | | | | | | | | | | | | | functions calling gsub on the result. SafeBuffer won't work with gsub until Rails 3.2 https://github.com/rails/rails/commit/b4a6e2f8442ceda118367f9a61c38af754be1cbf | |||
| * | | | Bugfix for the case where a suspended user sends a contact message. Their ↵hotfix/0.9.0.2 | Louise Crow | 2013-05-07 | -1/+1 |
| | | | | | | | | | | | | | | | | name (with the '(account suspended)' suffix) is marked by the translation call as HTML safe, so escapes the angled brackets when they are appended to it in the contact mailer code. Use string interpolation instead. | |||
| * | | | Mark ban text as html safe | Louise Crow | 2013-04-29 | -1/+1 |
| | | | | ||||
* | | | | Handle request for json for an external request. Closes #973. | Louise Crow | 2013-06-07 | -1/+5 |
| | | | | ||||
* | | | | Merge remote-tracking branch ↵ | Louise Crow | 2013-06-04 | -0/+15 |
|\ \ \ \ | | | | | | | | | | | | | | | | 'openaustralia_github/more_info_on_report_request' into rails-3-develop | |||
| * | | | | Reasons that request can be reported now stored in model | Matthew Landauer | 2013-03-22 | -0/+6 |
| | | | | | ||||
| * | | | | Wrap in a transaction | Matthew Landauer | 2013-03-22 | -3/+5 |
| | | | | | ||||
| * | | | | Extract method | Matthew Landauer | 2013-03-22 | -0/+7 |
| | | | | | ||||
* | | | | | Merge remote-tracking branch ↵ | Louise Crow | 2013-06-04 | -26/+0 |
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | 'openaustralia_github/inline_search_method_refactor' into rails-3-develop | |||
| * | | | | | Remove unused method | Matthew Landauer | 2013-03-25 | -11/+0 |
| | | | | | | ||||
| * | | | | | Inline method InfoRequest.full_search | Matthew Landauer | 2013-03-25 | -15/+0 |
| |/ / / / | ||||
* | | | | | Merge remote-tracking branch ↵ | Louise Crow | 2013-06-04 | -19/+4 |
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | 'openaustralia_github/small_email_hash_refactor' into rails-3-develop | |||
| * | | | | | Inline method and simplify logic | Matthew Landauer | 2013-03-29 | -19/+4 |
| | |_|_|/ | |/| | | |