Commit message (Collapse) | Author | Age | Lines | |
---|---|---|---|---|
* | Add template debug panel. | Matthew Somerville | 2017-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 Somerville | 2017-08-29 | -89/+287 |
|\ | ||||
| * | Use our fork of Devel::Cover::Report::Codecov. | Matthew Somerville | 2017-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 Somerville | 2017-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 Somerville | 2017-08-29 | -3/+3 |
| | | ||||
* | | Merge branch 'move-report-states-to-database-part-1' | Matthew Somerville | 2017-08-29 | -86/+30 |
|\ \ | |/ |/| | ||||
| * | Drop council_states and $visible_states. | Matthew Somerville | 2017-08-29 | -50/+6 |
| | | ||||
| * | Show "Fixed" for fixed states only when warranted. | Matthew Somerville | 2017-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 Somerville | 2017-08-25 | -2/+2 |
| | | ||||
| * | Make sure gettext domain specified in all tests. | Matthew Somerville | 2017-08-25 | -20/+2 |
| | | ||||
| * | Stop admin_reportextrafields.t info messages. | Matthew Somerville | 2017-08-25 | -0/+4 |
| | | ||||
| * | [Zurich] Move Zurich specific setup to its cobrand. | Matthew Somerville | 2017-08-25 | -5/+6 |
| | | ||||
* | | Redirect to original URL after inspecting | Dave Arter | 2017-08-24 | -22/+69 |
|/ | | | | For mysociety/fixmystreetforcouncils#204 | |||
* | Merge remote-tracking branch 'origin/empty-alert-urls' | Matthew Somerville | 2017-08-23 | -2/+81 |
|\ | ||||
| * | Fix missing URLs in alert emails | Dave Arter | 2017-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 Somerville | 2017-08-23 | -2/+7 |
|\ \ | ||||
| * | | Fix redisplay of uploaded photo thumbnails. | Matthew Somerville | 2017-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 Somerville | 2017-08-23 | -5/+29 |
|\ \ | |/ |/| | ||||
| * | Join response_priority if needed, and reorder template conditional to not ↵ | Matthew Somerville | 2017-08-23 | -3/+12 |
| | | | | | | | | check if no permission. | |||
| * | Use a schema cache to remember body lookups. | Matthew Somerville | 2017-08-23 | -2/+17 |
| | | ||||
* | | [Travis] Run code coverage on 5.24. | Matthew Somerville | 2017-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/new | Dave Arter | 2017-08-21 | -1/+46 |
|/ | ||||
* | Merge branch 'speed-up-reports' | Matthew Somerville | 2017-08-18 | -12/+51 |
|\ | ||||
| * | Update changelog. | Matthew Somerville | 2017-08-18 | -3/+6 |
| | | ||||
| * | Cache Problem->bodies and prefetch the body areas. | Matthew Somerville | 2017-08-18 | -7/+15 |
| | | ||||
| * | Faster /reports for inspectors, reduce db queries. | Dave Arter | 2017-08-18 | -2/+30 |
| | | ||||
* | | Merge branch 'zurich-upgrade-openlayers' | Matthew Somerville | 2017-08-18 | -1398/+660 |
|\ \ | |/ |/| | ||||
| * | [Zurich] Bring all reports in line with upstream. | Matthew Somerville | 2017-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 Somerville | 2017-08-18 | -1377/+625 |
| | | ||||
| * | Allow co-ords to be supplied to fixture script. | Matthew Somerville | 2017-08-18 | -6/+15 |
| | | ||||
* | | Reduce queries on /report when loading defect types/priorities | Dave Arter | 2017-08-18 | -2/+6 |
|/ | ||||
* | Make sure /around permalink/redirect have js param | Matthew Somerville | 2017-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 Somerville | 2017-08-17 | -98/+63 |
|\ | ||||
| * | Do not fetch pins server side if JS enabled. | Matthew Somerville | 2017-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 Somerville | 2017-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 Somerville | 2017-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 Somerville | 2017-08-17 | -22/+94 |
|\ \ | ||||
| * | | show all report updates, even if no public text | Struan Donald | 2017-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 Somerville | 2017-08-17 | -0/+105 |
|\ \ \ | ||||
| * | | | allow body users to filter on all problem categories | Struan Donald | 2017-08-17 | -0/+105 |
| |/ / | | | | | | | | | | | | | | | | Not just New/Fixed/Closed but the Investigating etc Fixes #1790 | |||
* | | | Merge branch 'edit-defecttypes-tests' | Matthew Somerville | 2017-08-17 | -0/+193 |
|\ \ \ | |/ / |/| | | ||||
| * | | tests for the defect types admin interface | Struan Donald | 2017-08-17 | -0/+193 |
| | | | ||||
* | | | [fixmystreet.com] Encode ampersands in link hrefs | Zarino Zappia | 2017-08-17 | -8/+8 |
|/ / | ||||
* / | [fixmystreet.com] Donation link on success pages | Zarino Zappia | 2017-08-17 | -20/+15 |
|/ | | | | Part of #1384. | |||
* | [Travis] Ignore perllib/Email. | Matthew Somerville | 2017-08-17 | -1/+1 |
| | ||||
* | Merge branch '1743-report-extras-editing' | Dave Arter | 2017-08-17 | -31/+883 |
|\ | ||||
| * | Add site-wide extra fields for reports, and admin UI to manage | Dave Arter | 2017-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 compilation | Dave Arter | 2017-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 Somerville | 2017-08-17 | -3/+47 |
|\ | ||||
| * | Store duplicate reverse lookups upon dupe setting. | Matthew Somerville | 2017-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. |