aboutsummaryrefslogtreecommitdiffstats
path: root/t
Commit message (Collapse)AuthorAgeLines
* Refactor SendReport::Open311 to use cobrand hooks.Matthew Somerville2017-03-09-0/+267
| | | | | | | | | | | | | | Avoid doing cobrand-specific logic within SendReport::Open311. The hooks defined are: open311_config: set extra fields open311_pre_send: set endpoints and munge data These are tested in the new t/app/sendreport/open311.t, which does not send any actual reports, but just checks that all the data is munged into the expected format. (tests under t/open311* should already test the actual sending) Remove unused Open311 special cases.
* [Oxfordshire] Use 'days ago' format on problem listspezholio2017-02-27-1/+14
| | | | Fixes mysociety/fixmystreetforcouncils#98
* [Oxfordshire] Add Exor RDI file download featureDave Arter2017-02-15-0/+23
| | | | | | | | | | The RDI file format encapsulates information about inspections that have taken place, and can be uploaded into Exor to create defects in bulk. This commit adds a page to the Oxfordshire cobrand's admin allowing RDI files to be generated and downloaded from FMS. For mysociety/fixmystreetforcouncils#127
* Allow cobrands to control which states affect reputationDave Arter2017-02-15-3/+18
| | | | | | | | | | | | Rather than assigning +1 to a user's reputation every time a report is inspected, this allows cobrands to specify which states should increment or decrement the reputation value when a report is inspected. The default behaviour is for reputation to never be changed, but the Oxfordshire cobrand will increment the reputation when a report is marked as 'action scheduled'. For mysociety/fixmystreetforcouncils#119
* Clarify ‘inspected’ behaviourDave Arter2017-02-15-7/+8
| | | | | | A report is now considered 'inspected' if its state was changed by an inspector. Additionally, an AdminLog entry is created so the time of inspection and inspector can be recorded.
* [Oxfordshire] Old report archiving scriptpezholio2017-02-15-0/+163
| | | | | | | | | | | | | | | This script can be used for the bulk closure of old reports that haven't been updated in a long time. Reports that haven't been updated in a *really* long time (since 2014, by default) are simply closed, and those that haven't been updated since 2015 (by default) are closed and an email explaining the closure is sent to the report creator. This script can be used for other cobrands, but they'll need an archive.{txt,html} template in order for the emails to be sent. For mysociety/fixmystreetforcouncils#51
* Add Problem->tokenised_url for logging user inpezholio2017-02-15-2/+29
| | | | | | This method creates a token that logs the reporting user in, optionally with some parameters which are stored with the token for use when redirecting after login.
* Add Problem->time_ago for pretty-printed durationpezholio2017-02-15-0/+44
|
* Utils::prettify_duration improvementspezholio2017-02-15-0/+8
| | | | | - Add year and month to $nearest options - Allow prettify_duration to choose a sensible $nearest if omitted
* [Oxfordshire] Show simpler state list on inspect formDave Arter2017-02-08-4/+4
| | | | | | | | | | Limits the state dropdown to those states which will be useful to OCC inspectors, as well as removing a few from the base template on the inspect form. Some slight template refactoring has made it easier for other cobrands to override the list, as well as sharing the template with the admin report_edit form to allow all states to be selected. Fixes mysociety/fixmystreetforcouncils#129
* [Oxfordshire] Allow cobrand-specific traffic management optionsDave Arter2017-02-08-0/+10
| | | | Fixes mysociety/fixmystreetforcouncils#159
* Fix crash on reports with empty bodies_strDave Arter2017-02-07-0/+13
|
* Add shortlist filterspezholio2017-02-07-1/+50
|
* Show ‘unable to fix’ state as ‘no further action’.Dave Arter2017-02-02-1/+1
| | | | Bromley had this already in its update state change messages.
* Return JSON if `ajax` in paramspezholio2017-01-31-0/+12
|
* Open311 warn only after a couple of failures.Matthew Somerville2017-01-19-14/+23
| | | | | There are a couple of servers we currently talk to that error not that infrequently but work in the end.
* Merge remote-tracking branch ↵pezholio2017-01-17-4/+55
|\ | | | | | | 'origin/issues/forcouncils/136-show-user-details-for-council-report'
| * Show user details for users with correct permissionspezholio2017-01-17-4/+54
| |
| * Add permision to view contributor detailspezholio2017-01-17-0/+1
| |
* | Merge remote-tracking branch ↵pezholio2017-01-17-2/+28
|\ \ | | | | | | | | | 'origin/issues/forcouncils/142-allow-anonymous-council-updates'
| * | Allow users with permission to see update usernamepezholio2017-01-17-2/+28
| |/
* | Merge branch 'listshort'Matthew Somerville2017-01-12-2/+2
|\ \ | |/ |/|
| * Add shortlist buttons to report lists.Matthew Somerville2017-01-12-2/+2
| | | | | | | | | | | | | | This includes adding/removing reports from a user's shortlist, and manual reordering of a shortlist with up/down buttons. The backend code can cope with an item moving to any point in the list.
* | Move staging flags to their own config variable.Matthew Somerville2017-01-10-4/+4
|/
* Fix bad cross-year test.Matthew Somerville2017-01-03-1/+5
| | | | Make sure the tested date is in the same year as current.
* Merge branch 'issues/forcouncils/20-offline-inspecting'Matthew Somerville2016-12-16-6/+13
|\
| * Default inspect form to save with public update.Matthew Somerville2016-12-16-2/+2
| |
| * Return 400/500 for some client/server errors.Matthew Somerville2016-12-16-4/+11
| |
* | Fix flagging wording template link.Matthew Somerville2016-12-16-1/+1
| | | | | | | | | | | | f58ce6c4 changed the base contact template to include a link to a fixmystreet.com specific page. Revert this back and make the change in the correct cobrand only.
* | Merge branch 'issues/forcouncils/108-pick-up-a-priority'Matthew Somerville2016-12-15-2/+2
|\ \ | |/ |/|
| * Update priorities in inspect form on cat. change.Matthew Somerville2016-12-15-2/+2
| | | | | | | | | | Different categories may have a different list of priorities, so store them all and update as the category changes.
* | Make it clearer that report is closed when marked as duplicateDave Arter2016-12-13-8/+42
| | | | | | | | | | | | - Record state change when leaving update and marking as duplicate - Change save button wording to match problem state when inspecting - Make it clearer that updates marking a report as duplicate actually close the report
* | Signpost users to original version of duplicate reportsDave Arter2016-12-13-24/+26
| | | | | | | | This stops updates being left on duplicates.
* | If a problem has duplicates, display them on its detail pageDave Arter2016-12-13-1/+16
| |
* | Add test for correct display of duplicate reportDave Arter2016-12-13-0/+19
|/
* [Zurich] Fix 500 error when editing superusersDave Arter2016-12-06-1/+20
| | | | | | | | | | | | The Zurich::admin_type function was setting $c->stash->{body}, meaning that when trying to edit a superuser the 'fetch_contacts' function wasn't being called, causing an "Can't call method "all" on an undefined value" error as $c->stash->{live_contacts} wasn't ever being set. Fixed by changing the conditional to simply determine whether fetch_contacts needs to be called. Included a regression test that meant a small addition to the MapItZurich mock.
* Update wording on flagging unsuitable reports.Matthew Somerville2016-11-25-1/+1
|
* Pass test if NXDOMAINs are intercepted.Matthew Somerville2016-11-24-0/+8
| | | | | Mock out the DNS lookup to fail, in case the test is being run somewhere where the ISP hijacks NXDOMAIN results and redirects them elsewhere.
* Pre-filter map pages if user has categories assignedDave Arter2016-10-31-0/+17
| | | | | This pre-filters the 'all reports' link in the main nav and the /around page reached via the front page postcode form
* Error if logged-in user visits admin without permission.Matthew Somerville2016-10-26-16/+10
| | | | | This is less confusing than redirecting to /auth and then to /my (as they're logged in). Fixes #1566.
* Redirect to correct cobrand when category changed.Dave Arter2016-10-26-1/+35
|
* Resend report if changing category changes body.Dave Arter2016-10-26-10/+44
|
* Merge branch 'issues/forcouncils/67-mop-up'Matthew Somerville2016-10-24-1/+1
|\
| * Fix warning in report inspect test.Matthew Somerville2016-10-24-1/+1
| |
* | Have report page show inspect form if available.Matthew Somerville2016-10-24-10/+13
|/
* Merge branch 'issues/forcouncils/100-fix-template-list'Matthew Somerville2016-10-20-0/+14
|\
| * Fix list of response templates for a problem.Matthew Somerville2016-10-20-0/+14
| | | | | | | | | | Instead of returning all templates for the problem's body, return those for the problem's category (or with no category).
* | Add ‘description’ field to ResponsePriority modelDave Arter2016-10-19-0/+2
|/ | | | Used internally to provide more details about when a priority is applicable.
* Merge branch 'issues/forcouncils/95-multi-select-list-filters'Matthew Somerville2016-10-13-1/+1
|\
| * Allow multiple selections in report list filter.Zarino Zappia2016-10-13-1/+1
| | | | | | | | | | | | This lets people filter by multiple categories or states. It uses our jQuery multi-select plugin to turn the <select multiple>s into little overlay lists of checkboxes. HTML5 history is also supported.