aboutsummaryrefslogtreecommitdiffstats
path: root/spec
Commit message (Collapse)AuthorAgeLines
* Handle a request with no user in the show request function0.16.0.5hotfix/0.16.0.5Louise Crow2014-01-28-0/+12
|
* Merge branch 'feature/cleanup-popups' into release/0.16Louise Crow2014-01-14-1/+1
|\ | | | | | | | | | | | | | | Conflicts: Gemfile.lock app/views/layouts/default.html.erb config/application.rb public/admin/stylesheets/admin.css
| * Cleanup popup notices.Louise Crow2014-01-14-1/+1
| | | | | | | | | | | | | | | | Move HTML to view from controller, use same elements for other country popup and everypage - partly so they don't display on top of each other anymore. Don't position them over existing content, position them at the top of the page. Use consistent styling, and keep the javascript unobtrusive.
| * Treat any document/pdf attachment as if it were application/pdfMark Longair2013-12-05-0/+116
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is a fix for issue #1232. Richard Taylor pointed out that some PDF attachments had the non-standard content-type document/pdf, and that these weren't being treated as PDFs. (Ganesh Sittampalam discovered that all of these PDFs were generated by a Lexmark X945e, according to the PDF metadata.) This commit adds an extra case to normalise_content_type to map document/pdf to application/pdf. In fact, since the upgrade of the Mail gem in ccebe3c3d6d4dc5f81 the behaviour when handling the non-standard content-type document/pdf was much better, but this commit also means that you get the right icon for the attachment, and can be cherry-picked onto older versions to fix #1232.
| * Merge branch 'feature/restore-caching' into wdtkLouise Crow2013-11-26-75/+95
| |\
| * \ Merge branch 'release/0.15' into wdtkMark Longair2013-11-15-102/+423
| |\ \ | | | | | | | | | | | | | | | | Conflicts: lib/tasks/stats.rake
| * | | Reduce the memory used to serve /body/all-authorities.csvMark Longair2013-11-14-0/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | On WDTK, /body/all-authorities was using lots of memory - this commit reduces that by (a) fetching the public bodies in batches, rather than keeping them all in memory at one time and (b) writing the CSV to a file and then returning it with X-Sendfile (or equivalent), rather than returning the whole file from memory with send_data. There's a FIXME to do with the layout of download directories; if that's changed, the example nginx config, etc. will need to be updated too. This commit also adds a basic test for reasonable CSV being returned and switches from FasterCSV to CSV in order to fix this NotImplementedError under Ruby 1.9: Please switch to Ruby 1.9's standard CSV library. It's FasterCSV plus support for Ruby 1.9's m17n encoding engine. (The CSV version seems to still work fine under 1.8.7.)
| * | | Insert cookie stripping at correct point in middleware stack.Louise Crow2013-11-13-0/+12
| | | |
| * | | Session keys are stored as strings in Rails 3.Louise Crow2013-11-13-28/+28
| | | | | | | | | | | | | | | | Update our session-stripping code.
| * | | Merge branch 'hotfix/0.14.0.2' into wdtkLouise Crow2013-10-03-0/+40
| |\ \ \
| * \ \ \ Merge branch 'hotfix/0.14.0.1' into wdtkLouise Crow2013-09-30-0/+10
| |\ \ \ \
| * \ \ \ \ Merge branch 'release/0.14' into wdtkLouise Crow2013-09-24-440/+2398
| |\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: app/views/request/show.html.erb
| * \ \ \ \ \ Merge branch 'hotfix/0.13.0.3' into wdtkLouise Crow2013-09-18-0/+37
| |\ \ \ \ \ \
| * \ \ \ \ \ \ Merge branch 'release/0.13' into wdtkLouise Crow2013-08-20-173/+506
| |\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: locale/he_IL/app.po locale/hr_HR/app.po locale/nb_NO/app.po locale/uk/app.po
| * \ \ \ \ \ \ \ Merge branch 'release/0.12' into wdtkLouise Crow2013-06-26-6/+0
| |\ \ \ \ \ \ \ \
| * \ \ \ \ \ \ \ \ Merge branch 'release/0.12' into wdtkLouise Crow2013-06-26-261/+524
| |\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: locale/he_IL/app.po locale/hr_HR/app.po locale/nb_NO/app.po locale/uk/app.po spec/models/info_request_spec.rb
| * | | | | | | | | | Respond to a (currently unsupported) json request for a public body list ↵Louise Crow2013-06-20-1/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | with a 406, not a 500 caused by a missing template.
| * | | | | | | | | | Merge branch 'hotfix/0.11.0.14' into wdtkLouise Crow2013-06-19-0/+10
| |\ \ \ \ \ \ \ \ \ \
| * \ \ \ \ \ \ \ \ \ \ Merge branch 'hotfix/0.11.0.13' into wdtkLouise Crow2013-06-17-4/+29
| |\ \ \ \ \ \ \ \ \ \ \
| * \ \ \ \ \ \ \ \ \ \ \ Merge branch 'hotfix/0.11.0.9' into wdtkLouise Crow2013-06-11-3/+3
| |\ \ \ \ \ \ \ \ \ \ \ \
| * \ \ \ \ \ \ \ \ \ \ \ \ Merge branch 'wdtk' of ssh://git.mysociety.org/data/git/public/alaveteli ↵Louise Crow2013-06-10-0/+1
| |\ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | into wdtk Conflicts: spec/models/info_request_spec.rb
| | * | | | | | | | | | | | | Cope with replying to a message with a missing or empty Subjecthotfix/0.11.0.5Mark Longair2013-06-06-0/+12
| | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | Merge branch 'hotfix/0.11.0.7' into wdtkLouise Crow2013-06-10-0/+39
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ | | |/ / / / / / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: lib/tasks/temp.rake spec/models/info_request_spec.rb
| * | | | | | | | | | | | | | Merge branch 'hotfix/0.11.0.4' into wdtkLouise Crow2013-06-05-0/+12
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \
| * \ \ \ \ \ \ \ \ \ \ \ \ \ \ Merge branch 'hotfix/0.11.0.1' into wdtkLouise Crow2013-06-03-7/+88
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ Merge branch 'release/0.11' into wdtkLouise Crow2013-06-03-856/+1684
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ Merge branch 'master' into wdtkLouise Crow2013-05-28-1/+1
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ Merge branch 'master' into wdtkLouise Crow2013-05-28-3/+14
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ Merge branch 'hotfix/0.9.0.2' into wdtkLouise Crow2013-05-07-1/+15
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ Merge branch 'release/0.9' into wdtkLouise Crow2013-04-24-326/+288
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ Merge branch 'hotfix/0.8.0.1' into wdtkLouise Crow2013-04-02-7/+22
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ Merge branch 'release/0.8' into wdtkLouise Crow2013-03-19-175/+350
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: app/controllers/admin_request_controller.rb app/views/contact_mailer/from_admin_message.rhtml
| * \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ Merge branch 'hotfix/0.7.0.1' into wdtkLouise Crow2013-02-15-0/+21
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | | | | | | | | | | Rename spec file so that it's picked up by rake spec.Louise Crow2013-02-14-0/+0
| | | | | | | | | | | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | | | | | | | | | Merge branch 'release/0.7' into wdtkLouise Crow2013-02-14-38/+56
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ Merge branch 'release/0.6.9' into wdtkLouise Crow2013-01-15-209/+1428
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | | | | | | | | | | | | As we're validating filename with validates_presence_of, which doesn't allow ↵Louise Crow2013-01-14-1/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | blanks, ensure_filename! should populate a default filename on a blank filename, as well as on nil.
| * | | | | | | | | | | | | | | | | | | | | | | | | | Handle case where info request doesn't have a user_nameLouise Crow2013-01-07-0/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | | | | | | | | | | | Only serve up 'similar' pages up to the offset we use for list.Louise Crow2012-12-17-0/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | | | | | | | | | | | Merge branch 'hotfix/authenticate-and-expire-download-zips' into wdtkLouise Crow2012-12-13-2/+28
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | | | | | | | | | | | | | Don't offer or allow viewing of an HTML version of a response attachment if ↵Louise Crow2012-12-13-0/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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.
| * | | | | | | | | | | | | | | | | | | | | | | | | | | Merge branch 'develop' into wdtkLouise Crow2012-11-22-186/+224
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ Merge remote-tracking branch 'origin/develop' into wdtkLouise Crow2012-11-20-3/+12
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ Merge remote-tracking branch 'origin/release/0.6.8' into wdtkLouise Crow2012-11-12-164/+1020
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: config/general.yml-example
| * | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add spec - currently failing due to typo.Louise Crow2012-10-23-0/+42
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add extra checks from ↵Louise Crow2012-10-23-2/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | https://github.com/mysociety/alaveteli/commit/cdb1c9420169ed6eafdd4e57ca21e1809c95e5cb and some extra description.
| * | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Adding new spec from ↵Louise Crow2012-10-22-2/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | https://github.com/mysociety/alaveteli/commit/cdb1c9420169ed6eafdd4e57ca21e1809c95e5cb
| * | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Merge branch 'release/0.6.7' into wdtkLouise Crow2012-10-17-92/+387
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: config/general.yml-example
| * | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Handle the case where an incoming message is badly encoded and has no ↵Louise Crow2012-10-09-0/+48
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | charset on the part we're using as a main part.
| * | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Merge branch 'release/0.6.6' into wdtkLouise Crow2012-09-20-22/+76
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
generated by cgit v1.2.3 (git 2.25.1) at 2025-08-05 20:36:01 +0000