Commit message (Collapse) | Author | Age | Lines | |
---|---|---|---|---|
* | Handle case where info request doesn't have a user_name | Louise Crow | 2013-01-07 | -0/+1 |
| | ||||
* | Handle the case of a comment on an external request. | Louise Crow | 2012-12-21 | -1/+1 |
| | ||||
* | Limit pagination on similar pages in line with new upper limit on page offset. | Louise Crow | 2012-12-17 | -1/+3 |
| | ||||
* | Only serve up 'similar' pages up to the offset we use for list. | Louise Crow | 2012-12-17 | -0/+5 |
| | ||||
* | Merge branch 'hotfix/authenticate-and-expire-download-zips' into develop | Louise Crow | 2012-12-13 | -11/+39 |
|\ | ||||
| * | Remove any download zip files when a request's cached files are expired. | Louise Crow | 2012-12-13 | -0/+8 |
| | | ||||
| * | Put download zips in a predictable location - sharded folders based on ↵ | Louise Crow | 2012-12-13 | -1/+9 |
| | | | | | | | | 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 Crow | 2012-12-13 | -1/+5 |
| | | | | | | | | location. | |||
| * | Check that a request is publicly visible before generating a download link. | Louise Crow | 2012-12-13 | -9/+15 |
| | | ||||
| * | Only offer download links on publicly visible requests | Louise Crow | 2012-12-13 | -0/+2 |
| | | ||||
| * | Don't offer or allow viewing of an HTML version of a response attachment if ↵ | Louise Crow | 2012-12-13 | -8/+14 |
| | | | | | | | | 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. | |||
* | | Don't offer or allow viewing of an HTML version of a request if it is ↵ | Louise Crow | 2012-12-13 | -8/+14 |
| | | | | | | | | hidden, or requester_only. Google docs viewer won't be able to access it, and our own conversion process currently produces image files that will then be publicly viewable. 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. | |||
* | | 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 |
| | | ||||
* | | Only make a tempfile etc if we're actually going to use it. Keep the body ↵ | Louise Crow | 2012-12-06 | -20/+30 |
| | | | | | | | | | | | | | | | | call outside the Chdir block - sometimes getting the body may involve opening files too. Conflicts: app/models/foi_attachment.rb | |||
* | | 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/+17 |
| | | | | | | | | 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 |
| | | ||||
* | | Move the getting of attachment leaves to the mail handler. | Louise Crow | 2012-12-05 | -90/+1 |
| | | ||||
* | | Pass around mail object explicitly when getting attachment leaves. | Louise Crow | 2012-12-05 | -8/+8 |
| | | ||||
* | | Move part counting to the mail handler. | Louise Crow | 2012-12-05 | -53/+7 |
| | | ||||
* | | Pass explicit references to the mail when counting parts. | Louise Crow | 2012-12-05 | -13/+13 |
| | | ||||
* | | Move counters to mail object. | Louise Crow | 2012-12-05 | -5/+5 |
| | | ||||
* | | Move content type normalization to mail handler. | Louise Crow | 2012-12-05 | -27/+2 |
| | | ||||
* | | Now that the underlying methods handle the case where there is no from field ↵ | Louise Crow | 2012-12-05 | -2/+2 |
| | | | | | | | | content by returning nil, I don't think we need to rescue a NoMethodError - this would have originally been returned by calling spec on from_addrs[0] when it was nil. | |||
* | | Use the binary_output option to have the strings returned by ↵ | Louise Crow | 2012-12-05 | -8/+17 |
| | | | | | | | | AlaveteliExternalCommand be transcoded using the default internal and external encodings in Ruby 1.9. | |||
* | | Use mail handler to get the content type of each email part. | Louise Crow | 2012-12-05 | -1/+1 |
| | | ||||
* | | Rewrite spec to reflect delegation of low-level mail methods to mail ↵ | Louise Crow | 2012-12-04 | -6/+3 |
| | | | | | | | | handler, use mail handler methods. | |||
* | | Remove duplicate method that was masking what seems to be intended to be the ↵ | Louise Crow | 2012-12-04 | -30/+1 |
| | | | | | | | | real valid_to_reply_to? method - which accesses cached values. Replacing the _calculate_valid_to_reply_to logic with the code that will have actually been running. | |||
* | | Remove obsolete comment. | Louise Crow | 2012-12-04 | -5/+0 |
| | | ||||
* | | Remove redundant method. | Louise Crow | 2012-12-04 | -5/+1 |
| | | ||||
* | | Move method for getting the to, cc and envelope-to addresses of a mail to ↵ | Louise Crow | 2012-12-04 | -3/+1 |
| | | | | | | | | the mail handler. | |||
* | | Move the specifics of getting a From: name from a mail to the mail handler. | Louise Crow | 2012-12-03 | -4/+1 |
| | | ||||
* | | Delegate getting the body of a mail part to the mail handler. | Louise Crow | 2012-12-03 | -1/+1 |
| | | ||||
* | | Delegate getting the from address to the mail handler. | Louise Crow | 2012-12-03 | -1/+1 |
| | | ||||
* | | Delegate getting the from address of an incoming mail to the mail handler. | Louise Crow | 2012-12-03 | -2/+2 |
| | | ||||
* | | The absence of an API key in an API request should be a permission denied ↵ | Louise Crow | 2012-12-03 | -36/+36 |
| | | | | | | | | error, so that notification emails don't get sent. | |||
* | | Add method for getting the body of a mail part to mail handling modules. | Louise Crow | 2012-11-29 | -1/+2 |
| | | ||||
* | | Standardize slug generation for external users so that it uses the same ↵ | Louise Crow | 2012-11-29 | -1/+1 |
| | | | | | | | | underlying method as other slug generation. Fixes #567. | |||
* | | Use Rails.env as a more reliable way of determining environment - it may not ↵ | Louise Crow | 2012-11-29 | -2/+2 |
| | | | | | | | | have come from ENV. | |||
* | | Use generated urls so that they go through locale filters. | Louise Crow | 2012-11-29 | -3/+3 |
| | | ||||
* | | Move styling of popup close buttons to stylesheet so they can be more easily ↵ | Louise Crow | 2012-11-28 | -2/+2 |
| | | | | | | | | overriden by themes. | |||
* | | Merge branch 'feature/no-default-locale-in-routes' into develop | Louise Crow | 2012-11-28 | -2/+7 |
|\ \ |