aboutsummaryrefslogtreecommitdiffstats
path: root/app
Commit message (Collapse)AuthorAgeLines
* Don't send comment alerts for external requests.Louise Crow2012-08-22-3/+4
|
* Allow external requests to be hidden from the admin interface.Louise Crow2012-08-22-6/+10
|
* 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
|\
| * 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-3/+6
|/ | | | param of 1. Fixes #557.
* Fail with NotFound if request slug doesn't existRobin Houston2012-08-20-25/+17
| | | | Closes #554.
* Do not send email for external requestsRobin Houston2012-08-20-3/+3
|
* Fix testsRobin Houston2012-08-20-1/+1
| | | | | | | | | 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.
* Merge branch 'wdtk' into developLouise Crow2012-08-16-19/+46
|\ | | | | | | | | | | | | Conflicts: app/controllers/admin_request_controller.rb config/httpd.conf spec/models/info_request_spec.rb
| * When displaying attachments, only cache them out to files if they are ↵Louise Crow2012-08-16-3/+12
| | | | | | | | 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-8/+17
| |
| * Add method all_can_view? which can be used to determine whether it is ok to ↵Louise Crow2012-08-16-8/+14
| | | | | | | | cache the associated objects for an info request in the file cache which will be served up without authentication.
| * 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)
* | 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
| |
* | Helpers to support external requestsRobin Houston2012-08-10-1/+13
| |
* | Merge branch 'release/0.6.3' into developRobin Houston2012-08-04-122/+122
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: locale/bs/app.po locale/ca/app.po locale/cs/app.po locale/cy/app.po locale/de/app.po locale/es/app.po locale/eu/app.po locale/fr/app.po locale/gl/app.po locale/hu_HU/app.po locale/id/app.po locale/pt_BR/app.po locale/sq/app.po locale/sr@latin/app.po
| * | Fix up whitespaceSeb Bacon2012-07-17-122/+122
| | |
| * | Oh dear, a "revert revert revert" that reverts the "revert revert" that ↵Seb Bacon2012-07-17-9/+2
| | | | | | | | | | | | | | | | | | | | | | | | should never have been pushed in the first place. Sorry! Revert "Revert "Revert "In the API, when parsing posted responses, assume all multipart mail parts that are Tempfiles are attachments""" This reverts commit 49ff1a1c0304cd292d3eae80dc0b91b2f83727b9.
* | | Revert "Revert "Revert "In the API, when parsing posted responses, assume ↵Robin Houston2012-08-01-9/+2
| | | | | | | | | | | | | | | | | | all multipart mail parts that are Tempfiles are attachments""" This reverts commit 49ff1a1c0304cd292d3eae80dc0b91b2f83727b9.
* | | Time out pdftohtml after 30 secondsRobin Houston2012-08-01-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)
* | | Move variable restore to an ensure blockRobin Houston2012-08-01-8/+12
| | | | | | | | | | | | | | | | | | I think this whole thing is redundant anyway, since the locale is set at the start of each request, but if we’re going to save/restore this value then we might as well do so correctly.
* | | Set locale even for errorsRobin Houston2012-08-01-0/+7
|/ / | | | | | | | | | | | | | | | | | | | | The locale was not being set correctly for error pages, causing some order-dependent test failures, e.g.: script/spec spec/controllers/services_controller_spec.rb spec/integration/errors_spec.rb was failing. Many many many thanks to Louise Crow for tracking this one down!
* | Revert "Revert "In the API, when parsing posted responses, assume all ↵Seb Bacon2012-07-17-2/+9
| | | | | | | | | | | | multipart mail parts that are Tempfiles are attachments"" This reverts commit d4a700da1760fc2ba09cf19613a995569e4965ea.
* | Merge branch 'feature/public-body-api-2a' into developRobin Houston2012-07-16-43/+64
|\ \
| * | Revert "In the API, when parsing posted responses, assume all multipart mail ↵Robin Houston2012-07-16-9/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | parts that are Tempfiles are attachments" This change makes the tests fail, I think, and doesn’t have (IMHO) a desperately strong rationale, so let’s try it without for now. This reverts commit 4490482cedf362390b25efe453232ac1b7dfce99.
| * | In the API, when parsing posted responses, assume all multipart mail parts ↵Seb Bacon2012-07-11-2/+9
| | | | | | | | | | | | that are Tempfiles are attachments
| * | Fix bug introduced in commit 049198bSeb Bacon2012-07-09-1/+1
| | |
| * | Calls to API that are expected to return JSON are always expected to return ↵Seb Bacon2012-07-09-2/+3
| | | | | | | | | | | | something, even when there are no errors.
| * | Ensure (at least in the most basic cases) that views support recent ↵Seb Bacon2012-07-09-42/+62
| | | | | | | | | | | | "external request" changes (mainly that InfoRequests no longer necessarily have a User).
* | | Style the search button with CSS so it can be overridden in themesSeb Bacon2012-07-13-1/+1
| | |
* | | Cleanup mixtures of tabs and spacesSeb Bacon2012-07-13-105/+104
| | |
* | | Indentation fixesSeb Bacon2012-07-13-16/+14
| | |
* | | Misc fixes to get tests to pass - notably a MonkeyPatch to work around ↵Seb Bacon2012-07-13-1/+1
| | | | | | | | | | | | problem testing template results
* | | Don't fail when a banner is completely absent.Seb Bacon2012-07-13-1/+1
| | |
* | | Factor out showing user info so it can easily be specialised in a themeSeb Bacon2012-07-13-19/+21
| | |
* | | Some missing i18n strings (and whitespace changes)Seb Bacon2012-07-13-30/+30
| | |
* | | Fix bug that only occurs when loading sample data - sometimes an InfoRequest ↵Seb Bacon2012-07-13-1/+1
| | | | | | | | | | | | doesn't have a PublicBody at the time its slug is being calculated.
* | | Factor popup banner out into own partial, for easier setting/unsetting ↵Seb Bacon2012-07-13-15/+18
| | | | | | | | | | | | (including from themes). Fixes #524
* | | Factor out stylesheet includes into separate partial (allows easier ↵Seb Bacon2012-07-13-23/+23
| | | | | | | | | | | | overriding in themes). Fixes #523
* | | Support regular expressions in CensorRules; also support 'global' ↵Seb Bacon2012-07-13-28/+18
| |/ |/| | | | | CensorRules that aren't attached to a User or Request or Public Body (but don't expose this in the admin UI). Fixes #33
* | Include request_id as well as event_id in the JSONRobin Houston2012-07-05-0/+1
|/
* JSON new events feedRobin Houston2012-07-04-21/+48
|
* Atom feed of request eventsRobin Houston2012-07-04-33/+40
| | | | | | | | | | We need not only new requests, but new outgoing correspondence of any sort. The idea is that this feed will contain any event that would have triggered an email to be sent to the public body, so can be used as an alternative, equivalent way to stay up-to-date with happenings on WDTK (or the Alaveteli installation of choice).
* Use fully-qualified URLs in feedRobin Houston2012-07-04-1/+6
| | | | The <link> tags in the feed <entry>s ought to be fully-qualified URLs.
generated by cgit v1.2.3 (git 2.25.1) at 2025-09-20 10:53:16 +0000