aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
...
| * Add template debug panel.Matthew Somerville2017-09-01-1/+47
| | | | | | | | | | 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/+13
| |
* | Make current cobrand available on DB::SchemaDave Arter2017-09-01-0/+4
| |
* | Put comma in correct place in report lat/lonDave Arter2017-08-31-2/+2
| |
* | Merge branch 'move-report-states-to-database'Matthew Somerville2017-08-31-335/+760
|\ \
| * | Hide things needing fixed state if no fixed state.Matthew Somerville2017-08-31-2/+10
| | | | | | | | | | | | | | | Front page stats, update form question, list filtering, plus do not send any questionnaires.
| * | Update form asks for reopen of all closed states.Matthew Somerville2017-08-31-25/+25
| | |
| * | Admin interface for editing report states.Matthew Somerville2017-08-31-0/+246
| | |
| * | Remove hardcoded states from templates.Matthew Somerville2017-08-31-204/+142
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-46/+202
| | | | | | | | | | | | | | | 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-63/+140
| | |
* | | Stop make_css failing if run on fresh checkout.Matthew Somerville2017-08-31-1/+7
| | | | | | | | | | | | The change in 66bd45b6 did not account for being run with no CSS files.
* | | Merge branch 'issues/commercial/840-groundcontrol'Dave Arter2017-08-31-11/+315
|\ \ \ | |_|/ |/| |
| * | Add Extra->get_extra_field_valueDave Arter2017-08-31-0/+58
| | | | | | | | | | | | Allows easy display of Open311 field values for problems in templates
| * | Add ability for area outlines on /reports to be hidden by cobrandDave Arter2017-08-31-1/+4
| | |
| * | Add SIGNUPS_DISABLED config flagDave Arter2017-08-31-4/+124
| | |
| * | Add LOGIN_REQUIRED config keyDave Arter2017-08-31-6/+125
| | | | | | | | | | | | If set to 1, this restricts all pages on the site to logged-in users.
| * | [Ground Control] Add cobrand to .gitignoreMatthew Somerville2017-08-30-0/+4
|/ / | | | | | | This cobrand comes from the fixmystreet-commercial repository.
* | Merge branch 'some-dev-improvements'Matthew Somerville2017-08-29-89/+287
|\ \
| * | Use our fork of Devel::Cover::Report::Codecov.Matthew Somerville2017-08-29-1/+1
| | | | | | | | | | | | | | | This allows branches that are fully covered or marked as uncoverable to pass on Codecov, rather than count as partials.
| * | Add debug toolbar middleware.Matthew Somerville2017-08-29-85/+283
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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.
| * | Stop using File::Slurp in make_css.Matthew Somerville2017-08-29-3/+3
| | |
* | | Merge branch 'move-report-states-to-database-part-1'Matthew Somerville2017-08-29-86/+30
|\ \ \ | |/ / |/| / | |/
| * Drop council_states and $visible_states.Matthew Somerville2017-08-29-50/+6
| |
| * Show "Fixed" for fixed states only when warranted.Matthew Somerville2017-08-29-9/+10
| | | | | | | | | | 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-2/+2
| |
| * Make sure gettext domain specified in all tests.Matthew Somerville2017-08-25-20/+2
| |
| * Stop admin_reportextrafields.t info messages.Matthew Somerville2017-08-25-0/+4
| |
| * [Zurich] Move Zurich specific setup to its cobrand.Matthew Somerville2017-08-25-5/+6
| |
* | Redirect to original URL after inspectingDave Arter2017-08-24-22/+69
|/ | | | For mysociety/fixmystreetforcouncils#204
* Merge remote-tracking branch 'origin/empty-alert-urls'Matthew Somerville2017-08-23-2/+81
|\
| * Fix missing URLs in alert emailsDave Arter2017-08-22-2/+81
| | | | | | | | | | | | | | | | | | 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 '1815-fix-dropzone-thumbnail-display'Matthew Somerville2017-08-23-2/+7
|\ \
| * | Fix redisplay of uploaded photo thumbnails.Matthew Somerville2017-08-23-2/+7
| |/ | | | | | | | | Dropzone version 5 changed how createThumbnailFromUrl was called, so the upgrade in 30dd9d8 broke this.
* | Merge branch 'speeed-up-reports'Matthew Somerville2017-08-23-5/+29
|\ \ | |/ |/|
| * Join response_priority if needed, and reorder template conditional to not ↵Matthew Somerville2017-08-23-3/+12
| | | | | | | | check if no permission.
| * Use a schema cache to remember body lookups.Matthew Somerville2017-08-23-2/+17
| |
* | [Travis] Run code coverage on 5.24.Matthew Somerville2017-08-21-3/+3
| | | | | | | | | | Older perl versions can leak the locale decimal point character, leading to invalid JSON being written by Devel::Cover.
* | [Bristol] Fix 500 error on /report/newDave Arter2017-08-21-1/+46
|/
* Merge branch 'speed-up-reports'Matthew Somerville2017-08-18-12/+51
|\
| * Update changelog.Matthew Somerville2017-08-18-3/+6
| |
| * 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/+30
| |
* | Merge branch 'zurich-upgrade-openlayers'Matthew Somerville2017-08-18-1398/+660
|\ \ | |/ |/|
| * [Zurich] Bring all reports in line with upstream.Matthew Somerville2017-08-18-15/+20
| | | | | | | | | | Update the map/list if the map is zoomed/panned, and make sure the updated HTML list is returned.
| * [Zurich] Upgrade OpenLayers to master, matching others.Matthew Somerville2017-08-18-1377/+625
| |
| * Allow co-ords to be supplied to fixture script.Matthew Somerville2017-08-18-6/+15
| |
* | Reduce queries on /report when loading defect types/prioritiesDave Arter2017-08-18-2/+6
|/
* Make sure /around permalink/redirect have js paramMatthew Somerville2017-08-18-0/+7
| | | | | The change in 17e38922 would not deal with the URL construted by the map permalink code, or the redirect after inspection.
* Merge branch 'js-you-only-load-once'Matthew Somerville2017-08-17-98/+63
|\