aboutsummaryrefslogtreecommitdiffstats
path: root/perllib
Commit message (Collapse)AuthorAgeLines
* Fix issues with Open/Closed translated too early.v2.2Matthew Somerville2017-09-15-3/+9
| | | | | | | | | | | | | | It was possible for _hardcoded_states to be reached with no locale selected (e.g. bin/update-all-reports), causing an error trying to translate Open/Closed. But translating those there wasn't correct, because that would then be cached and returned even if a different language was being used. Caching was ignored in testing, which did not help. We no longer translate the state names in their objects, only upon display at the point that we will know the locale. We can't simply return the translation of Open/Closed because there may be entries in the translation table as well.
* [Zurich] Fix Open311 querying unconfirmed reports.Matthew Somerville2017-09-14-1/+2
|
* Merge branch 'version-2.2'Matthew Somerville2017-09-14-1/+1
|\
| * Version 2.2.Matthew Somerville2017-09-13-1/+1
| | | | | | | | | | Couple of missed strings for translation, and prevent a couple of warnings in tests.
* | [Oxfordshire] Use local time for rdi_processed, not UTCDave Arter2017-09-13-3/+5
|/
* Merge branch 'issues/forcouncils/224-default-priority'Struan Donald2017-09-13-2/+11
|\
| * select default response priority in inspect panelStruan Donald2017-09-13-0/+6
| |
| * edit and display default response priority in adminStruan Donald2017-09-13-0/+1
| |
| * add is_default column to response_prioritiesStruan Donald2017-09-13-2/+4
| |
* | Merge branch '1836-slash-category-bug'Matthew Somerville2017-09-11-71/+10
|\ \
| * | Fix issue editing category with '/' in it.Matthew Somerville2017-09-11-35/+7
| | | | | | | | | | | | | | | | | | Simplify chaining of body/category admin URLs so that all categories are treated the same, with `/` or without, and the template will then always have its CSRF token.
| * | Fix bug if first page after restart is admin.Matthew Somerville2017-09-11-36/+3
| |/ | | | | | | | | | | | | | | If the first page looked at after server launch was an admin one, the User object was getting a different schema attached than the one used by everything else (so e.g. the cobrand was not then available to it, causing a crash on a body page). Using auto instead of begin prevents this from happening, as the setup_request auto always runs first.
* / [Oxfordshire] Log when problems sent by RDI email.Dave Arter2017-09-11-4/+25
|/ | | | | | | | The timestamp of when a problem was included in an RDI is shown on the problem’s inspector form. Also check back and include any inspected reports that for some reason haven't been instructed yet. Fixes mysociety/fixmystreetforcouncils#223.
* sort body dashboard CSV export by report confirmed timeStruan Donald2017-09-07-1/+1
| | | | | And update the tests to avoid a random failure caused by inconsistent ordering.
* Merge branch '866-alerts-ux-simpler'Matthew Somerville2017-09-06-8/+8
|\
| * Simpler alerts signup page.Zarino Zappia2017-09-06-8/+8
| | | | | | | | | | | | Alert options on the /alert/list page are now in an order that is easier to parse, with a slightly clearer message for users in two-tier locations, and a more prominent email subscription CTA.
* | [Borsetshire] Use Oxfordshire pin images.Matthew Somerville2017-09-06-4/+8
|/
* include easting/northing in dashboard CSV exportStruan Donald2017-09-04-0/+6
| | | | Fixes mysociety/fixmystreetforcouncils#221
* include ward in dashboard csv exportStruan Donald2017-09-04-0/+8
| | | | | | | Adds a Ward column to the council dashboard CSV export with the name of the ward. Fixes mysociety/fixmystreetforcouncils#220
* Merge branch 'template-panel'Matthew Somerville2017-09-01-0/+33
|\
| * Add template debug panel.Matthew Somerville2017-09-01-0/+33
| | | | | | | | | | Use a small subclass of (our) P::M::D::Template to have better display of our template path names.
* | Superusers only have permissions available on the current cobrandDave Arter2017-09-01-0/+10
| |
* | Make current cobrand available on DB::SchemaDave Arter2017-09-01-0/+4
| |
* | Merge branch 'move-report-states-to-database'Matthew Somerville2017-08-31-105/+307
|\ \
| * | Hide things needing fixed state if no fixed state.Matthew Somerville2017-08-31-0/+4
| | | | | | | | | | | | | | | Front page stats, update form question, list filtering, plus do not send any questionnaires.
| * | Admin interface for editing report states.Matthew Somerville2017-08-31-0/+103
| | |
| * | Remove hardcoded states from templates.Matthew Somerville2017-08-31-67/+67
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | State display names are now got from the database wherever they are displayed, including admin dropdowns, list filters, and update meta statements. This also covers the open/closed/fixed 'groups'. This also fixes a bug whereby if e.g. an update has problem_state investigating, the next update has no problem_state, and the last update has investigating again, it was previously showing a state change to investigating on that third update.
| * | Remove hardcoded states from Problem model.Matthew Somerville2017-08-31-40/+97
| | | | | | | | | | | | | | | We keep the internal states hardcoded, plus the core open (confirmed) and closed ones, but the remainder are moved to the database.
| * | Add state table to database.Matthew Somerville2017-08-29-0/+38
| | |
* | | Add Extra->get_extra_field_valueDave Arter2017-08-31-0/+16
| | | | | | | | | | | | Allows easy display of Open311 field values for problems in templates
* | | Add SIGNUPS_DISABLED config flagDave Arter2017-08-31-0/+20
| | |
* | | Add LOGIN_REQUIRED config keyDave Arter2017-08-31-6/+45
| |/ |/| | | | | If set to 1, this restricts all pages on the site to logged-in users.
* | Merge branch 'some-dev-improvements'Matthew Somerville2017-08-29-83/+135
|\ \
| * | Add debug toolbar middleware.Matthew Somerville2017-08-29-83/+135
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If debug is enabled, using the CATALYST_DEBUG/FIXMYSTREET_APP_DEBUG environment variables, add a debug toolbar to the output, including request/response details and a database query log. This uses Plack middleware, so works by switching our dev server to use Starman with plack directly, rather than via the script runner. We remove the GZip compression as this interferes, and take a local copy of the QueryLog::AdoptPlack trait as it needs a tweak to work. Make sure the CSP header is not output in debug mode, as that would prevent the toolbar JavaScript from running.
* | | Merge branch 'move-report-states-to-database-part-1'Matthew Somerville2017-08-29-58/+17
|\ \ \ | |/ / |/| / | |/
| * Drop council_states and $visible_states.Matthew Somerville2017-08-29-50/+6
| |
| * Show "Fixed" for fixed states only when warranted.Matthew Somerville2017-08-29-2/+2
| | | | | | | | | | There were a couple of places where the three fixed states were all displaying as "Fixed" in a dropdown.
| * Sort languages in template.Matthew Somerville2017-08-25-1/+1
| |
| * Make sure gettext domain specified in all tests.Matthew Somerville2017-08-25-0/+2
| |
| * [Zurich] Move Zurich specific setup to its cobrand.Matthew Somerville2017-08-25-5/+6
| |
* | Redirect to original URL after inspectingDave Arter2017-08-24-21/+28
|/ | | | For mysociety/fixmystreetforcouncils#204
* Merge remote-tracking branch 'origin/empty-alert-urls'Matthew Somerville2017-08-23-1/+1
|\
| * Fix missing URLs in alert emailsDave Arter2017-08-22-1/+1
| | | | | | | | | | | | | | | | | | If a staff user changed a problem’s state and an empty update was generated, it was possible for alert emails to be sent with missing URLs because the empty string in the update text was falsy and Alert.pm took the wrong path. This fixes the problem by changing the test to defined() and includes a regression test.
* | Merge branch 'speeed-up-reports'Matthew Somerville2017-08-23-4/+28
|\ \ | |/ |/|
| * Join response_priority if needed, and reorder template conditional to not ↵Matthew Somerville2017-08-23-2/+11
| | | | | | | | check if no permission.
| * Use a schema cache to remember body lookups.Matthew Somerville2017-08-23-2/+17
| |
* | [Bristol] Fix 500 error on /report/newDave Arter2017-08-21-1/+1
|/
* Merge branch 'speed-up-reports'Matthew Somerville2017-08-18-9/+40
|\
| * Cache Problem->bodies and prefetch the body areas.Matthew Somerville2017-08-18-7/+15
| |
| * Faster /reports for inspectors, reduce db queries.Dave Arter2017-08-18-2/+25
| |