Commit message (Collapse) | Author | Age | Lines | |
---|---|---|---|---|
* | Don't pass a page param of less than one through to Xapian, treat it as a ↵ | Louise Crow | 2012-08-21 | -3/+6 |
| | | | | param of 1. Fixes #557. | |||
* | Merge branch 'wdtk' into develop | Louise Crow | 2012-08-16 | -0/+3 |
|\ | | | | | | | | | | | | | Conflicts: app/controllers/admin_request_controller.rb config/httpd.conf spec/models/info_request_spec.rb | |||
| * | Method spacing. | Louise Crow | 2012-08-16 | -0/+3 |
| | | ||||
* | | Set locale even for errors | Robin Houston | 2012-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! | |||
* | | Factor popup banner out into own partial, for easier setting/unsetting ↵ | Seb Bacon | 2012-07-13 | -0/+4 |
|/ | | | | (including from themes). Fixes #524 | |||
* | Fix admin debug method | Robin Houston | 2012-06-04 | -3/+0 |
| | | | | | | | | | | | > 10) admin debug (https://secure.mysociety.org/admin/foi/debug) - "you > are" - sentence that isn't finished > > 11) admin debug (https://secure.mysociety.org/admin/foi/debug) - > versioning incorrect & URLs to github malformed: > Alaveteli version: 0.5.2-297-g4ec7f6e > Alaveteli branch: (no > Alaveteli commit: 4ec7f6edbf2c00abc8464eafc889047f43533f90 | |||
* | File.dirname(__FILE__) is not necessarily absolute | Robin Houston | 2012-06-03 | -1/+1 |
| | | | | | | | It’s weird that there’s so much code here that implicitly assumes File.dirname(__FILE__) is an absolute path, because really in general it very much is not! This assumption was invalidated by the recent bundler-awareness changes. | |||
* | Merge branch 'develop' into feature/fix-404-in-themes | Seb Bacon | 2012-05-15 | -16/+16 |
|\ | ||||
| * | Remove trailing whitespace (to make a cleaner forthcoming merge with ↵ | Seb Bacon | 2012-05-15 | -16/+16 |
| | | | | | | | | wombleton:feature/440_sparkly_admin_css) | |||
* | | Add a comment to explain why we're calling `set_view_paths` in the ↵ | Seb Bacon | 2012-05-15 | -2/+7 |
| | | | | | | | | controller when the method doesn't exist in Alaveteli code. See [this comment](https://github.com/sebbacon/alaveteli/issues/320#issuecomment-5711523) for discussion. | |||
* | | Allow themes to effect the design of error pages | Seb Bacon | 2012-05-15 | -0/+7 |
|/ | ||||
* | More changes and refactoring to make purges work. | Seb Bacon | 2012-04-20 | -10/+1 |
| | ||||
* | use Rails.root, not RAILS_ROOT | James McKinney | 2012-04-02 | -1/+1 |
| | ||||
* | Fix the "log in as" function | Robin Houston | 2012-03-20 | -2/+2 |
| | | | | | | | | | | | | | | | Previously the "log in as" function after 3b6e5a692b852a88f55b21a7210f60a6f7cfc24b would attempt to log the admin user out before issuing the redirect. Unfortunately this approach does not work on WhatDoTheyKnow, where the admin pages are served via a different domain (secure.mysociety.org) and so do not share session information with the rest of the site. This commit changes it to mark the PostRedirect with circumstance == "login_as", which signals the user controller to log out the previous user even if they are an admin. In other words, the user is logged out on the main site rather than the admin site, skirting this problem. Closes #450. | |||
* | No code changes | Robin Houston | 2012-03-13 | -3/+5 |
| | | | | Just trailing whitespace & one comment. | |||
* | Admin users clicking confirmation links | Robin Houston | 2012-03-13 | -0/+1 |
| | | | | Fixes #446. See issue for details. | |||
* | Close xapian db before opening it again | Robin Houston | 2012-01-31 | -1/+4 |
| | | | | | | This *ought* to fix the problem with the alert-tracks daemon opening more and more copies of the xapian db till it exhausts the available file descriptors. | |||
* | Irrelevant change | Robin Houston | 2012-01-29 | -3/+1 |
| | | | | | So, well, I found this change when I ran git diff; and the truth is that I just prefer it aesthetically. Is that so sinful? | |||
* | More test data and a new test | Robin Houston | 2012-01-27 | -4/+3 |
| | | | | | This is a test for what I thought issue #370 might be. However this test is passing, so it isn’t that. | |||
* | Logging for fragment cache | Robin Houston | 2012-01-21 | -0/+2 |
| | ||||
* | Record URL and PID *before* a request is processed | Robin Houston | 2012-01-19 | -0/+1 |
| | | | | | | | (If DEBUG_RECORD_MEMORY is enabled) This is useful for tracking down bugs that cause Rails to go into an infinite or very long loop, as the recent Xapian wildcard bug does. | |||
* | Merge branch 'wdtk' of github.com:sebbacon/alaveteli into wdtk | Seb Bacon | 2012-01-19 | -8/+24 |
|\ | ||||
| * | Merge branch 'release/0.5' into wdtk | Robin Houston | 2012-01-19 | -3/+6 |
| |\ | | | | | | | | | | | | | Conflicts: app/views/public_body/_search_ahead.rhtml | |||
| | * | Catch timeouts and other extra errors when connection to 3rd party websites ↵ | Seb Bacon | 2012-01-18 | -1/+1 |
| | | | | | | | | | | | | fails. | |||
| | * | Fix paging bug for type ahead search. This wasn't caught because the spec ↵ | Seb Bacon | 2012-01-18 | -2/+5 |
| | | | | | | | | | | | | didn't call "integrate_views", so the rendering part was mocked by RSpec instead of executed. | |||
| * | | Limit Xapian wildcard expansions | Robin Houston | 2012-01-19 | -5/+18 |
| |/ | | | | | | | | | | | | | | | | | | | | | Wildcard searches in Xapian can expand uncontrollably, consuming all available RAM and crashing the server. This has been a real problem on WhatDoTheyKnow.com. The underlying issue is tracked in http://trac.xapian.org/ticket/350 This changeset imposes a limit on wildcard expansion. The type-ahead search will first try a wildcard query, and if that fails because of too much expansion will fall back to a plain non-wildcard search. | |||
* / | Don't choke on very long filenames when caching attachments. Fixes #349. | Seb Bacon | 2012-01-19 | -1/+4 |
|/ | ||||
* | Actually do a proper ORed and partial match query, rather than fix parsing ↵ | Seb Bacon | 2012-01-17 | -10/+18 |
| | | | | errors ad hoc as I find them that result from the workaround code. Fixes #328 (for good, I hope). | |||
* | Fail silently if third party services are broken or unavailable. Fixes #354. | Seb Bacon | 2012-01-17 | -1/+11 |
| | ||||
* | Fix bug introduced in cec2c545e0a10e0641c4ee67839c88d872b394b8, related to ↵ | Seb Bacon | 2012-01-13 | -1/+1 |
| | | | | issue #343. | |||
* | Show internal_review (and some other) items in the "unresolved" list. Fixes ↵ | Seb Bacon | 2012-01-13 | -1/+1 |
| | | | | #344. | |||
* | Ensure we show "all requests" in order that they were last updated. FIxes #343. | Seb Bacon | 2012-01-13 | -1/+1 |
| | ||||
* | Correction to commit 4808347cb65556756d38b60b25fa9761f92c4513 | Seb Bacon | 2012-01-13 | -1/+0 |
| | ||||
* | Fix further special character searches as per issue #328 | Seb Bacon | 2012-01-13 | -0/+1 |
| | ||||
* | Further fix for issue #328. | Seb Bacon | 2012-01-12 | -2/+2 |
| | ||||
* | Return 403 when attachment "folders" are spidered. Fixes #340 | Seb Bacon | 2012-01-12 | -1/+5 |
| | ||||
* | Fix problem with typeahead searches containing " - " characters and similar. ↵ | Seb Bacon | 2012-01-11 | -0/+21 |
| | | | | Closes #328 | |||
* | Ignore last-seen-item variables for the purposes of varnish caching. Fixes ↵ | Seb Bacon | 2012-01-09 | -4/+4 |
| | | | | issue #324. | |||
* | Send email notifications on exceptions | Seb Bacon | 2012-01-09 | -1/+5 |
| | ||||
* | Remove temporary patch, found a way of adding code from themes without new ↵ | David Cabo | 2011-09-23 | -14/+0 |
| | | | | controller classes | |||
* | Ugly hack to fix _() in production when themes define controllers (kind of) | David Cabo | 2011-09-20 | -1/+15 |
| | ||||
* | Only store last visited public body for logged in users. Otherwise it ↵ | Seb Bacon | 2011-09-16 | -5/+9 |
| | | | | breaks caching rather a lot, for little benefit. | |||
* | Merge branch 'develop' of github.com:sebbacon/alaveteli into develop | David Cabo | 2011-09-03 | -3/+13 |
|\ | ||||
| * | Store user's locale against profile, so we can send them localised track ↵ | Seb Bacon | 2011-09-03 | -3/+13 |
| | | | | | | | | emails. Also internationalize more strings at the same time. Fixes #163. | |||
* | | Merge branch 'asktheeu-new-design' into develop (update New Request workflow ↵ | David Cabo | 2011-09-01 | -1/+3 |
|\ \ | |/ |/| | | | to match new wireframes) | |||
| * | Merge branch 'master' of github.com:sebbacon/alaveteli into asktheeu | David Cabo | 2011-08-12 | -8/+17 |
| |\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: locale/cy/app.po locale/de/app.po locale/es/app.po locale/fr/app.po locale/sq/app.po locale/sr/app.po locale/sr@latin/app.po | |||
| * | | Move log-in point to before Preview and fix modal sign-in and sign-up redirects | David Cabo | 2011-08-11 | -1/+3 |
| | | | ||||
* | | | Merge branch 'develop' into feature/add-recaptcha | Seb Bacon | 2011-08-30 | -2/+9 |
|\ \ \ | ||||
| * | | | Additional changes omitted from commit ↵ | Seb Bacon | 2011-08-30 | -2/+9 |
| | | | | | | | | | | | | | | | | 9d8388c03d0faeaca29d233a340c58bd65f28a97 (distinguish 404s and 500s), fixes #161. | |||
* | | | | Present a reCaptcha on the signup form to foreign visitors (judging from ↵ | Seb Bacon | 2011-08-30 | -0/+11 |
|/ / / | | | | | | | | | | their IP address). Fixes #157 (at least as a starter). |