Commit message (Collapse) | Author | Age | Lines | |
---|---|---|---|---|
* | Hide hidden outgoing messages in download. | Louise Crow | 2013-09-16 | -4/+0 |
| | ||||
* | 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/+1 |
| | | | | | | 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 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 |
| | ||||
* | Add new code and specs for hiding attachments. | Louise Crow | 2013-09-16 | -0/+4 |
| | ||||
* | 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/+4 |
| | ||||
* | Adding prominence_reason to IncomingMessage. | Louise Crow | 2013-09-16 | -0/+1 |
| | ||||
* | Refactor common logic about prominence and access. | Louise Crow | 2013-09-16 | -7/+1 |
| | | | | 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 |
| | ||||
* | 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 | -7/+6 |
| | ||||
* | 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. | |||
* | 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. | |||
* | When extracting attachments for an incoming message and getting the body of ↵ | Louise Crow | 2013-05-29 | -7/+16 |
| | | | | the main part in order to look for uuencoded text, make sure that we're getting that main part from the reparsed attachments, and not getting an obsolete attachment. Fixes #958. | |||
* | Check that display_filename matches URL part number or fallback | Mark Longair | 2013-05-24 | -3/+22 |
| | | | | | | | | | | | If the display_filename of the attachment found from the URL part number doesn't match the passed in display_filename then the email may have been reparsed, causing a reordering. In that case, look to see if there is another attachment that uniquely matches that filename, and, if so, return that other attachment. If no matching uniquely matching filename is found, redirect to the incoming message, rather than returning a 404. | |||
* | Refactor IncomingMessage.get_attachment_by_url_part_number | Mark Longair | 2013-05-24 | -6/+1 |
| | ||||
* | Move the mapi requires to where they're really needed | Mark Longair | 2013-05-22 | -2/+0 |
| | | | | | | | Handling of outlook-packed attachments would fail from rake tasks or in the console without requiring 'mapi/msg' and 'mapi/convert' beforehand. Instead, require them in the source file where they're actually used. | |||
* | Avoid an FoiAttachment validation failure under Rails 3 | Mark Longair | 2013-05-16 | -0/+1 |
| | | | | | | | | | | | Under Rails 3, the uudecoded FoiAttachment in this test fails validation at the self.save! in IncomingMessage.parse_raw_email, although the FoiAttachment has been correctly created and saved to the database in _uudecode_and_save_attachments. Forcing a reload=true on self.foi_attachments fixes this. Thanks to Louise Crow for finding the fix for this problem. | |||
* | Move "require 'alaveteli_file_types'" into the initializer | Mark Longair | 2013-05-16 | -1/+0 |
| | ||||
* | Change email address in header of source code to hello@mysociety.org | Matthew Landauer | 2013-03-26 | -1/+1 |
| | ||||
* | Replace use of Iconv in Ruby 1.9 to get rid of deprecation warnings | Henare Degan | 2013-03-16 | -7/+23 |
| | ||||
* | We need to reload here in Rails 3.1 (caching?) | Henare Degan | 2013-03-03 | -0/+3 |
| | ||||
* | Rename Configuration class to avoid conflict with ActiveSupport::Configurable | Henare Degan | 2013-03-03 | -4/+4 |
| | ||||
* | In Ruby 1.9.3 we need to explicitly require iconv because it's not ↵ | Henare Degan | 2013-02-28 | -1/+1 |
| | | | | implicitly loaded via Mail | |||
* | Fix calculation of attachment sizes in Ruby 1.9.3 | Henare Degan | 2013-02-27 | -3/+0 |
| | | | | | | | | force_encoding is generally not a good idea and I can't see what it's trying to achieve here so... Revert "Use the character set of the attachment to encode the string that we're pulling out of the file before converting it to our default encoding." This reverts commit 3729f2053d4c04396d440a9c368bed174e9c9605. | |||
* | Merge branch 'develop' into rails-3-spike | Henare Degan | 2013-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 | |||
| * | Rescue an invalid character exception when handling what appears to be badly ↵ | Louise Crow | 2013-02-06 | -1/+1 |
| | | | | | | | | encoded data. | |||
* | | Merge branch 'rails_xss' into rails-3-spike | Matthew Landauer | 2013-01-25 | -0/+1 |
|\| | | | | | | | | | | | | | | | Conflicts: Gemfile Gemfile.lock config/environment.rb lib/i18n_fixes.rb | |||
| * | Merge remote-tracking branch 'mysociety/develop' into rails_xss | Matthew Landauer | 2013-01-15 | -386/+90 |
| |\ | ||||
| | * | Handle case where info request doesn't have a user_name | Louise Crow | 2013-01-07 | -0/+1 |
| | | | ||||
* | | | Activerecord 3 small api change for find_or_create_by | Matthew Landauer | 2013-01-04 | -1/+1 |
| | | | ||||
* | | | Merge branch 'rails_xss' into rails-3-spike | Matthew Landauer | 2013-01-04 | -1/+1 |
|\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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 | |||
| * | | html content of messages and comments are html safe | Matthew Landauer | 2013-01-03 | -1/+1 |
| | | | ||||
* | | | We don't need to specify the attribute again | Henare Degan | 2012-12-12 | -1/+1 |
| |/ |/| | | | | | Makes FoiAttachment model specs pass | |||
* | | Add alternative sanitization for Ruby 1.9 | Louise Crow | 2012-12-06 | -23/+57 |
| | | ||||
* | | Convert url in comment to spec. | Louise Crow | 2012-12-06 | -1/+0 |
| | | | | | | | | | | | | Conflicts: lib/mail_handler/backends/tmail_backend.rb | |||
* | | Update comments to reflect the fact that text is coming from an foi ↵ | Louise Crow | 2012-12-06 | -3/+4 |
| | | | | | | | | attachment model, and hasn't necessarily been handled by TMail. | |||
* | | Reformat for line length. | Louise Crow | 2012-12-06 | -1/+3 |
| | | ||||
* | | Rename _get_attachment_text_internal_one_file to ↵ | Louise Crow | 2012-12-06 | -2/+3 |
| | | | | | | | | get_attachment_text_one_file as it is now an externally-accessed method of the mail handler module. | |||
* | | Move methods for getting the text out of attachments to the mail handler module. | Louise Crow | 2012-12-06 | -100/+2 |
| | | ||||
* | | Standard indent. | Louise Crow | 2012-12-06 | -4/+4 |
| | | ||||
* | | Write to the temp file in binary mode - we may be writing binary content, ↵ | Louise Crow | 2012-12-05 | -0/+1 |
| | | | | | | | | such as a PDF file. | |||
* | | Use the character set of the attachment to encode the string that we're ↵ | Louise Crow | 2012-12-05 | -0/+3 |
| | | | | | | | | pulling out of the file before converting it to our default encoding. | |||
* | | Move mail-specific stuff to mail handler. | Louise Crow | 2012-12-05 | -46/+1 |
| | | ||||
* | | Separate mail-specific stuff from creation/update of model instances. | Louise Crow | 2012-12-05 | -8/+12 |
| | | ||||
* | | Add a wrapper method for get_attachment_leaves. | Louise Crow | 2012-12-05 | -4/+1 |
| | |