aboutsummaryrefslogtreecommitdiffstats
path: root/perllib/FixMyStreet
Commit message (Collapse)AuthorAgeLines
...
* cobrand option to control prefilling of report fields for inspectorsStruan Donald2017-11-06-0/+4
| | | | | | | | | | Add a `prefill_report_fields_for_inspectors` config to cobrands, defaulted to 0, to control the automatic pre-population of report fields. Also add default of 1 for UKCouncil cobrand. Fixed #1854
* Fix fetching of duplicate reports.Matthew Somerville2017-11-06-2/+3
| | | | | | c9998984 altered nearby.json to use on_map rather than around_map, to prevent needless text appearing, but did not take account of the fact that it expects Problem objects, not Nearby objects.
* Merge branch 'around-and-about-pagination'Matthew Somerville2017-11-06-79/+70
|\
| * Fix warning in contact test.Matthew Somerville2017-11-06-4/+3
| |
| * Add pagination to around page.Matthew Somerville2017-11-06-21/+49
| |
| * Remove Show all pins link and on_map_list_limit.Matthew Somerville2017-11-06-51/+11
| | | | | | | | | | /around will now show all reports all the time, but we are about to introduce pagination to stop that.
| * Have /around ajax use same format as others.Matthew Somerville2017-11-02-7/+11
| | | | | | | | | | Both /reports and /my work by using the same URL with ajax=1. We should use the same on /around for consistency.
* | Merge branch 'issues/forcouncils/248-duplicate-save-bug'Matthew Somerville2017-11-06-0/+3
|\ \
| * | Remove shortlist form from inspect duplicate list.Matthew Somerville2017-11-02-0/+3
| |/ | | | | | | | | This functionality isn't needed here, and causes an issue with the main form submitting due to a form inside a form.
* / Update Bing Maps parameter ID.Matthew Somerville2017-11-06-2/+2
|/
* always allow problems to be removed from shortlistStruan Donald2017-10-27-7/+11
| | | | | | | | | | If a user is trying to remove a problem from their shortlist we should always allow it regardless of the state of the problem. Previously if a problem wasn't displayed on the site then it could not be removed from the shortlist which was an issue with council cobrands and reports that had changed body. Fixes mysociety/fixmystreetforcouncils#243
* Merge branch 'reports-dashboard-councils'Matthew Somerville2017-10-20-100/+271
|\
| * [fixmystreet.com] Council pages only from council.Matthew Somerville2017-10-20-10/+123
| |
| * Front end way of viewing council dashboard.Matthew Somerville2017-10-20-21/+42
| |
| * update-all-reports operate per body optionally.Matthew Somerville2017-10-20-51/+97
| |
| * Allow testing override of /reports data.Matthew Somerville2017-10-16-19/+10
| |
* | Merge branch 'issues/forcouncils/227-all-filter'Struan Donald2017-10-20-0/+3
|\ \
| * | select all applicable filter states on report pageStruan Donald2017-10-18-0/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Make sure that all relevant options in the filter status drop down are checked to handle staff users who can see the full set of states and not just open, closed and fixed Also restore custom label for no states checked. Fixes mysociety/forcouncils#227
* | | Remove hidden from default staff state dropdown.Matthew Somerville2017-10-18-1/+0
| | |
* | | Tidy up report page permission checkings.Matthew Somerville2017-10-18-4/+1
| | |
* | | [Oxfordshire] Add initials/traffic to RDI desc.Matthew Somerville2017-10-18-8/+15
|/ /
* / make sure duplicate id provided when marking as duplicateStruan Donald2017-10-17-8/+8
|/ | | | | | | It was possible to mark a report as a duplicate in the inspector panel without providing an id of the duplicate. This prevents that. Fixes mysociety/fixmystreetforcouncils#236
* Redirect inspectors correctly on creation.Matthew Somerville2017-10-16-2/+6
| | | | | Do not try and redirect to a report that won't be viewable on the cobrand being used.
* Merge branch 'sms-mssid-error'Matthew Somerville2017-10-16-37/+29
|\
| * Factor the three SMS code checking fns together.Matthew Somerville2017-10-12-27/+9
| |
| * Add option to use a Twilio Messaging Service.Matthew Somerville2017-10-12-1/+7
| |
| * Spot errors coming back when sending text.Matthew Somerville2017-10-12-9/+13
| |
* | log all state changes in admin as commentsStruan Donald2017-10-16-8/+40
| | | | | | | | | | | | | | | | | | | | | | | | Create a comment on a problem when the admin is used to change the state of the problem. If only the state is changed then create a comment with blank text. If the category and state are changing then include details of the category change in the comment. Not all the state changes are displayed at the template level by default. Fixes #1846
* | Merge branch 'bit-of-yaml-tidying'Matthew Somerville2017-10-11-16/+11
|\ \ | |/ |/|
| * Simplify geocode cache read/writing.Matthew Somerville2017-10-10-9/+7
| | | | | | | | Don't think the return data ever needs decoding.
| * Upgrade commonlib to read in config as Unicode.Matthew Somerville2017-10-10-7/+4
| | | | | | | | This means variables do not need decoding in a few places.
* | Merge branch 'issues/forcouncils/231-multi-ward-reports-view'Matthew Somerville2017-10-09-18/+28
|\ \
| * | Allow ward page to show multiple wards.Matthew Somerville2017-10-06-18/+28
| | | | | | | | | | | | | | | URLs have to be manually constructed, but specifying multiple ward names separated by "|" will show all those wards and their reports on the map.
* | | Fix issue sending alerts around no-text update.Matthew Somerville2017-10-06-13/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | If there was a normal email alert, and then an alert for an update with no text (so no email should be sent), it would still try to send the blank email, and then die because no e.g. $data{cobrand} set. This moves the skip-blank-update check higher up, above any email sending code.
* | | generate update text in alerts if problem state has changedStruan Donald2017-10-05-9/+23
| | |
* | | do not send blank problem update alertsStruan Donald2017-10-05-0/+1
| | |
* | | Merge remote-tracking branch 'origin/fix-geographic-mobile-numbers'Matthew Somerville2017-10-05-40/+43
|\ \ \
| * | | Accept +1 geographic numbers as possibly mobile.Matthew Somerville2017-10-05-40/+43
| |/ / | | | | | | | | | | | | 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-5/+3
| | | | | | | | | | | | | | | | | | 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.
* | | 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-12/+21
| | | | | | | | | | | | | | | | | | | | | | 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-1/+6
| | | | | | | | | | 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/+11
|/ | | | | | | | | | | 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-2/+10
| | | | | | 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.
* Allow verify/change/add email/phone in profile.Matthew Somerville2017-09-30-16/+91
| | | | | | 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-221/+326
|
* Update admin interface for 'phone' users.Matthew Somerville2017-09-30-124/+222
| | | | | | | 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-27/+56
| | | | | | 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-50/+275
| | | | A confirmation code is sent via Twilio to be entered on the site.
* Add 'verified' database columns for email/phone.Matthew Somerville2017-09-30-17/+64
| | | | | These are so we can state whether a user's email address or phone number have been verified by confirmation email/text.