aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
* Don't send comment alerts for external requests.Louise Crow2012-08-22-3/+14
|
* Re-enable testing of comment links in emails.Louise Crow2012-08-22-7/+2
|
* Clearer formatting of global fixtures list.Louise Crow2012-08-22-7/+21
|
* Fix comment typo.Louise Crow2012-08-22-2/+2
|
* Allow external requests to be hidden from the admin interface.Louise Crow2012-08-22-6/+60
|
* Use user_name instead of user.name to handle external requests.Louise Crow2012-08-22-4/+7
|
* Merge branch 'develop' of github.com:sebbacon/alaveteli into developLouise Crow2012-08-21-2/+5
|\
| * Merge branch 'develop' of github.com:sebbacon/alaveteli into developRobin Houston2012-08-21-0/+4
| |\
| * | Email admins about external requests tooRobin Houston2012-08-21-2/+5
| | | | | | | | | | | | | | | | | | | | | If someone reports an external request as needing administrator attention, we should email the administrators about it. Thanks for spotting this, @crowbot.
* | | Don't pass a page param of less than one through to Xapian, treat it as a ↵Louise Crow2012-08-21-4/+14
| |/ |/| | | | | param of 1. Fixes #557.
* | Merge branch 'develop' of github.com:sebbacon/alaveteli into developLouise Crow2012-08-20-38/+50
|\|
| * Fail with NotFound if request slug doesn't existRobin Houston2012-08-20-26/+37
| | | | | | | | Closes #554.
| * ECONNRESET is another error we might getRobin Houston2012-08-20-2/+2
| |
| * Do not send email for external requestsRobin Houston2012-08-20-3/+3
| |
| * Fix another testRobin Houston2012-08-20-1/+1
| | | | | | | | | | | | | | | | | | | | | | Is there actually any point in tests of this sort? It is not testing that anything works as intended, only that a particular SQL string is passed to the database -- and surely the only way that string could have been discovered for test purposes was essentially to copy it from the code under test. It seems to me that the only real function of a test of this sort is to make it more difficult to modify the code under test.
| * Fix testsRobin Houston2012-08-20-4/+4
| | | | | | | | | | | | | | | | | | Also make the InfoRequest#is_old_unclassified? method a little more conservative, by returning false only is the is_external? method returns true. This makes it subtly inconsistent with InfoRequest.find_old_unclassified, but it is better I think to be subtly inconsistent than to risk breaking things that used to work.
| * External requests ought not to be considered old_unclassifiedRobin Houston2012-08-20-1/+2
| | | | | | | | | | We certainly do not want to send reminder emails for such requests, for example, since we do not know the email address to send them to.
| * External requests were not sent by usRobin Houston2012-08-20-1/+1
| | | | | | | | | | | | | | Therefore we ought not to display the warning “This message has not yet been sent for an unknown reason” for such requests. Closes #548.
| * External requests need not have exim log linesRobin Houston2012-08-20-1/+1
| | | | | | | | We do not send messages for external requests. See #548.
| * Fix comment typoRobin Houston2012-08-20-1/+1
| |
* | Use find_by_url_title! to raise ActiveRecordNotFound in upload response when ↵Louise Crow2012-08-20-1/+5
|/ | | | | | no record is found using url_title. Closes #553.
* Remove whatdotheyknow-specific variables.Louise Crow2012-08-16-6/+1
|
* Merge branch 'wdtk' into developLouise Crow2012-08-16-345/+525
|\ | | | | | | | | | | | | Conflicts: app/controllers/admin_request_controller.rb config/httpd.conf spec/models/info_request_spec.rb
| * Fix loading of specific fileFOI archive email account2012-08-16-1/+4
| | | | | | | | abspath is not a standard command. Replace with a case statement.
| * Add temp task to delete the cached attachment files for requests that have ↵Louise Crow2012-08-16-10/+39
| | | | | | | | been deleted or are currently not publicly visible - having prominence requester_only or hidden.
| * When displaying attachments, only cache them out to files if they are ↵Louise Crow2012-08-16-201/+231
| | | | | | | | visible to everyone, so can be served up from a file cache without authentication.
| * Method spacing.Louise Crow2012-08-16-0/+3
| |
| * Fix typo in comment.Louise Crow2012-08-16-26/+104
| |
| * Add method all_can_view? which can be used to determine whether it is ok to ↵Louise Crow2012-08-16-107/+142
| | | | | | | | cache the associated objects for an info request in the file cache which will be served up without authentication.
| * Don't let apache serve attachments for now.Louise Crow2012-08-15-3/+3
| |
| * Time out pdftohtml after 30 secondsRobin Houston2012-07-20-8/+15
| | | | | | | | | | | | | | | | We have a PDF document that appears to send pdftohtml into a loop where it creates millions of tiny PDF files and consumes ever-increasing amounts of memory. (This document: http://www.whatdotheyknow.com/request/119267/response/296719/attach/5/Document%203.pdf)
| * Serve cached attachments directly from cacheRobin Houston2012-07-18-0/+14
| |
| * Merge branch 'develop' into wdtkRobin Houston2012-07-05-28/+170
| |\ | | | | | | | | | | | | Conflicts: doc/INSTALL.md
| * | Use 33 bytes consistentlyRobin Houston2012-07-02-1/+1
| | | | | | | | | | | | | | | Let’s use 33 bytes consistently for the API key, to avoid an = pad at the end when it’s base64-encoded.
| * | Set the API key in a much more efficient wayRobin Houston2012-07-02-6/+47
| | | | | | | | | | | | | | | If we are using PostgreSQL, populate the api_key column in a more efficient fashion.
| * | Do not pre-populate the API keyRobin Houston2012-07-02-6/+6
| | | | | | | | | | | | | | | It takes too long, and the mySociety deployment system runs these migrations while the site is down.
| * | Merge branch 'master' into wdtkRobin Houston2012-07-02-126/+840
| |\ \
| * \ \ Merge branch 'master' into wdtkRobin Houston2012-07-02-1717/+1462
| |\ \ \
| * \ \ \ Merge branch 'release/0.6' into wdtkSeb Bacon2012-06-20-4/+11
| |\ \ \ \
| * \ \ \ \ Merge branch 'release/0.6' into wdtkSeb Bacon2012-06-20-12/+26
| |\ \ \ \ \
| * \ \ \ \ \ Merge branch 'release/0.6' into wdtkSeb Bacon2012-06-20-15/+36
| |\ \ \ \ \ \
| * \ \ \ \ \ \ Merge branch 'release/0.6' into wdtkSeb Bacon2012-06-15-11/+2
| |\ \ \ \ \ \ \
| * | | | | | | | New lock file to go with previous commitSeb Bacon2012-06-15-1/+1
| | | | | | | | |
| * | | | | | | | Merge branch 'release/0.6' into wdtkSeb Bacon2012-06-15-1/+1
| |\ \ \ \ \ \ \ \
| * | | | | | | | | Revert "Remove WDTK-specific config variables"Seb Bacon2012-06-15-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 505fdbf99ee322aab2c753dc3215a263171c9463.
* | | | | | | | | | Merge pull request #542 from henare/import_csv-page-fixesLouise Crow2012-08-13-10/+14
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | import_csv page fixes
| * | | | | | | | | | Test showed this page wasn't valid HTMLHenare Degan2012-08-11-3/+5
| | | | | | | | | | |
| * | | | | | | | | | Default to dry run and don't raise an error when this page is visited normallyHenare Degan2012-08-11-7/+2
| | | | | | | | | | |
| * | | | | | | | | | Add failing test showing a bug when clicking 'Import from CSV file' on the ↵Henare Degan2012-08-11-0/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | admin public authorities page
* | | | | | | | | | | Update install doc to reflect the fact that develop is now the default ↵Louise Crow2012-08-13-5/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | branch on github.
generated by cgit v1.2.3 (git 2.25.1) at 2025-08-01 20:10:45 +0000