aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
...
* | | Merge remote-tracking branch 'origin/fix-geographic-mobile-numbers'Matthew Somerville2017-10-05-44/+47
|\ \ \
| * | | Accept +1 geographic numbers as possibly mobile.Matthew Somerville2017-10-05-44/+47
| |/ / | | | | | | | | | | | | We have no way of knowing whether a +1 geographic number is fixed line or mobile, so accept them all as potentially mobile.
* | | Return page with "No results" if no ref found.Matthew Somerville2017-10-05-15/+23
| | | | | | | | | | | | | | | | | | This lookup is often performed by inspectors with appcache switched on, which means they were getting the intercepted Glitch page rather than a 404; give them a no results page instead.
* | | [Oxfordshire] Stop crash on alert signup/update.Dave Arter2017-10-05-1/+2
| | | | | | | | | | | | | | | | | | | | | This fixes a crash when leaving an update on the Oxfordshire cobrand or signing up for an alert, as the _email_sent_extra.html template didn’t have a report to pass through to the cobrand in order to display the expected response time.
* | | Fix crash if large number given in postcode field.Dave Arter2017-10-05-2/+9
|/ / | | | | | | | | | | It was possible to trigger an SQL error by entering a value larger than that permitted by the `integer` SQL type. This commit instead returns a 404 in that case.
* | correctly handle category changes in inspector formStruan Donald2017-10-04-14/+150
| | | | | | | | | | | | | | | | | | | | | | This resolves two issues when updating the report category in the staff users inspect form: * report category is only updated if the rest of the form is valid * only one update on the report is left Previously changing the category would create an update in addition to any public update created, and the category was always updated even when validation errors occurred elsewhere in the form.
* | correctly handle average state changes of 0 days in area statsStruan Donald2017-10-04-2/+19
| | | | | | | | | | If no problems have changed status then set the average to -1 so we can cope with an average status change of 0 days.
* | move area stats page problems by status calc into databaseStruan Donald2017-10-04-9/+22
|/ | | | | | | | | | | Doing this calculation in code turns out to be much too slow. As part of this also fix an issue where if a report changed state last month but had a further comment this month that was counted as being a state change this month. Also tweak the 'last month' start date to be midnight so the stats don't change throughout the day.
* In RSS, only escape the necessary five/ctrl chars.Matthew Somerville2017-10-02-5/+15
| | | | | | By default encode_entities uses named entities, which don't exist by default in XML. It seemed nicer to switch to only escaping the minimum rather than switching to numeric entities.
* Merge branch 'phone-number-work'Matthew Somerville2017-09-30-1137/+3471
|\
| * Allow verify/change/add email/phone in profile.Matthew Somerville2017-09-30-38/+362
| | | | | | | | | | | | This expands the 'change email' functionality to work with phone number also, allow addition of missing details, and verifying of unverified details.
| * Add ability to confirm reports/updates via text.Matthew Somerville2017-09-30-332/+1181
| |
| * Update admin interface for 'phone' users.Matthew Somerville2017-09-30-183/+380
| | | | | | | | | | | | | | This includes search/flagging users, editing user on the report and update edit pages, and user adding/editing itself. Add some more tests for user adding/editing.
| * Adapt things that assume email will be present.Matthew Somerville2017-09-30-46/+103
| | | | | | | | | | | | This includes stopping some emails being sent (moderation, alert, questionnaire), dealing with Open311/email report sending, and tokenised_url.
| * Add ability to log in on /auth via text.Matthew Somerville2017-09-30-109/+522
| | | | | | | | A confirmation code is sent via Twilio to be entered on the site.
| * Add 'verified' database columns for email/phone.Matthew Somerville2017-09-30-22/+95
| | | | | | | | | | These are so we can state whether a user's email address or phone number have been verified by confirmation email/text.
| * Add Number::Phone and WWW::Twilio::API packages.Matthew Somerville2017-09-30-70/+435
| | | | | | | | These will be used to help with phone-based authentication.
| * Move some auth functions to their own controllers.Matthew Somerville2017-09-30-379/+435
|/ | | | | | Move the social auth functions to Auth::Social, and the change email/password functions to Auth::Profile. There are no actual code changes.
* [SeeSomething] Remove emails left behind.Matthew Somerville2017-09-30-37/+0
|
* Fix test calling external service.Matthew Somerville2017-09-30-1/+5
| | | | | | This test would pass offline (as query would fail without error), and online was passing because Nominatim was presumably returning no results before today?
* show nearest address for report in inspector panelStruan Donald2017-09-28-0/+53
| | | | | | | If we've stored the nearest address then display that at the top of the inspector panel. Fixes mysociety/fixmystreetforcouncils#234
* Fix bug in cookie passthrough query parameter.Matthew Somerville2017-09-27-5/+9
| | | | | For temp URLs there was not already a query string, and so "?" needs to be used rather than "&" when appending.
* characters left counter for detailed informationStruan Donald2017-09-27-2/+23
|
* set maximum length of report detailed_information in cobrandStruan Donald2017-09-27-30/+61
|
* limit detailed_information to 172 charactersStruan Donald2017-09-27-2/+43
| | | | | | | | RDI files have a 180 character limit for the description so limit the description to 172 to allow for the ID which is also included in this field. Fixes mysociety/fixmystreetforcouncils#228
* Reduce filesize of sprite.pngZarino Zappia2017-09-26-0/+1
| | | | | web/cobrands/fixmystreet/images/sprite.png is loaded by most cobrands. Optimising it reduced the filesize from 97 KB to 36 KB.
* shortlist menu item is always a linkStruan Donald2017-09-26-2/+4
| | | | | | avoids odd problems caused when using pushstate Fixes mysociety/fixmystreetforcouncils#233
* Ensure easting/northing is sent over Open311 even if map isn’t usedDave Arter2017-09-26-1/+5
|
* [Warwickshire] Don’t send questionnairesDave Arter2017-09-26-0/+2
|
* [Warwickshire] Update styling to match new council websiteZarino Zappia2017-09-26-805/+4890
| | | | Fixes mysociety/fixmystreet-commercial#920.
* [Oxfordshire] Make sure RDI file is encoded.Matthew Somerville2017-09-21-1/+2
| | | | | body_str is encoded with the provided charset/encoding, body is used as-is (and so will die in email sending if it contains wide characters).
* area stats page for staff usersStruan Donald2017-09-20-4/+546
| | | | | | | | | Admin page to show some simple summary stats for an area. If the user has been assigned to an area then they will see the stats for that area. Superusers can pick which area they want to view. For mysociety/fixmystreetforcouncils#2
* Merge branch 'fix-parallel-test-run'Matthew Somerville2017-09-20-1/+4
|\
| * Revert using memcached cache in testing.Matthew Somerville2017-09-18-1/+4
| | | | | | | | This can fail if the tests are being run in parallel.
* | [Zurich] Fix map offset bug.Matthew Somerville2017-09-20-25/+8
|/ | | | | | | | | The upgrade to OpenLayers in 31b8337e3 was not accompanied by fixing the tileOrigin manually tweaked as an old workaround. This moved the map c. 100m out of alignment. Bristol fixed this in a7f60abb by adding topLeftCorners, but this is better done by fixing origin_x/y.
* Fix issues with Open/Closed translated too early.v2.2Matthew Somerville2017-09-15-6/+33
| | | | | | | | | | | | | | 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.
* Include schema migration test.Matthew Somerville2017-09-14-0/+1
| | | | This was missed in 0998aef4.
* [Zurich] Fix Open311 querying unconfirmed reports.Matthew Somerville2017-09-14-1/+2
|
* Merge branch 'version-2.2'Matthew Somerville2017-09-14-18067/+24047
|\
| * Version 2.2.Matthew Somerville2017-09-13-18067/+24047
| | | | | | | | | | 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-5/+15
|/
* Disable email field for logged in people using JS.Matthew Somerville2017-09-13-4/+3
| | | | | They're logged in, it's only showing them as a confirmatory check, so it should not be editable.
* Merge branch 'issues/forcouncils/224-default-priority'Struan Donald2017-09-13-4/+72
|\
| * select default response priority in inspect panelStruan Donald2017-09-13-2/+20
| |
| * edit and display default response priority in adminStruan Donald2017-09-13-0/+36
| |
| * add is_default column to response_prioritiesStruan Donald2017-09-13-2/+16
| |
* | Add LWP panel to debug toolbar.Matthew Somerville2017-09-12-1/+23
| |
* | Merge branch '1836-slash-category-bug'Matthew Somerville2017-09-11-71/+11
|\ \
| * | Fix issue editing category with '/' in it.Matthew Somerville2017-09-11-35/+8
| | | | | | | | | | | | | | | | | | 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.