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.
* 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
|\
| * Do not fetch pins server side if JS enabled.Matthew Somerville2017-08-17-0/+9
| | | | | | | | | | | | The page currently fetches the pins again client-side (as we don't know the extent of the map at the server point). I realise this isn't ideal, but should speed up the normal case quite a bit.
| * Fix replaced sidebar losing hover behaviour.Matthew Somerville2017-08-17-3/+4
| | | | | | | | | | If the list was replaced by JavaScript, the event handlers were being dropped. Switch the handlers to the parent that remains present.
| * Refactor /around list code to share with others.Matthew Somerville2017-08-17-95/+50
| | | | | | | | | | Both /reports and /my share an ID and a /reports/ajax function, use these also on /around (and share ajax/non-ajax code).
* | Merge branch 'issues/forcouncils/201-show-all-report-updates'Matthew Somerville2017-08-17-22/+94
|\ \
| * | show all report updates, even if no public textStruan Donald2017-08-17-22/+94
| | | | | | | | | | | | | | | | | | | | | | | | Create a comment entry for all updates to reports regardless of whether there is any text part. Also adds an entry to extra if a defect was raised and includes that in the update status line. Fixes mysociety/fixmystreetforcouncils#201
* | | Merge branch 'issues/forcouncils/187-all-reports-filter-by-substates'Matthew Somerville2017-08-17-0/+105
|\ \ \
| * | | allow body users to filter on all problem categoriesStruan Donald2017-08-17-0/+105
| |/ / | | | | | | | | | | | | | | | Not just New/Fixed/Closed but the Investigating etc Fixes #1790
* | | Merge branch 'edit-defecttypes-tests'Matthew Somerville2017-08-17-0/+193
|\ \ \ | |/ / |/| |
| * | tests for the defect types admin interfaceStruan Donald2017-08-17-0/+193
| | |
* | | [fixmystreet.com] Encode ampersands in link hrefsZarino Zappia2017-08-17-8/+8
|/ /
* / [fixmystreet.com] Donation link on success pagesZarino Zappia2017-08-17-20/+15
|/ | | | Part of #1384.
* [Travis] Ignore perllib/Email.Matthew Somerville2017-08-17-1/+1
|
* Merge branch '1743-report-extras-editing'Dave Arter2017-08-17-31/+883
|\
| * Add site-wide extra fields for reports, and admin UI to manageDave Arter2017-08-17-31/+877
| | | | | | | | | | | | | | - Also provides an editor for the extra Open311 fields on contacts. - Adds .btn--small class for small buttons Fixes #1743.
| * Add make_css flag to run script after compilationDave Arter2017-08-17-0/+6
|/ | | | | | | | | In conjunction with the --watch flag, the new --script flag can be used to run a given command after compilation caused by files changing. E.g. you could reload your browser automatically upon hitting save in your editor, with something like https://gist.github.com/davea/07a7456da15eeb8c47f1fb66e4b96c54
* Merge branch 'duplicate-reverse'Matthew Somerville2017-08-17-3/+47
|\
| * Store duplicate reverse lookups upon dupe setting.Matthew Somerville2017-08-17-3/+47
| | | | | | | | | | | | This avoids the need to search the whole table to find a report's duplicates. And is easier than constructing an index or upgrading PostgreSQL.