aboutsummaryrefslogtreecommitdiffstats
path: root/perllib/Open311/PopulateServiceList.pm
Commit message (Collapse)AuthorAgeLines
* Fix Open311 category group bugsDave Arter2018-11-07-11/+24
| | | | | | | - Group wasn’t being set correctly by open311-populate-service-list as cobrand attribute not being updated for each body. - Extra metadata was being persisted to the DB every time even if nothing had changed, causing lots of duplicate entries in contacts_history.
* Only set Open311 category group if cobrand has groups enabledDave Arter2018-11-02-10/+20
|
* Don’t delete email contacts on devolved bodies.Matthew Somerville2018-06-06-21/+8
| | | | | If we assume that any body with can_be_devolved set is probably a mix of Open311/email, we can ignore any email address contacts for such a body.
* [Bromley] Add asset/PROW ID meta to attributes.Matthew Somerville2018-06-04-0/+19
|
* [Open311] Factor cobrand overrides to functions.Matthew Somerville2018-06-04-14/+28
| | | | [Stevenage] Make sure Other included, like East Herts.
* [BANES] Don’t delete non-Open311 categoriesDave Arter2018-04-10-2/+3
|
* [Buckinghamshire] Send report_url attribute with Open311 service requestsDave Arter2018-04-03-1/+1
|
* Store server_set automated, but ignore for input.Matthew Somerville2018-03-21-2/+0
| | | | | We do need to store them, so that the sending knows which fields are being requested, but we do not want them output to the client at all.
* Deal with 'automated' Open311 service attributes.Struan Donald2018-03-15-0/+2
| | | | | | This is part of the integration with our open311-adapter. The automated property indicates that it's an expected value but either it's going to be set by the server, or appear as a hidden field.
* [UK] Re-include Bristol in open311-populate-service-listDave Arter2018-01-18-1/+2
| | | | | | | | | | | | Bristol's Open311 endpoint still seems to be returning empty metadata for some services that claim to have metadata. They have made changes to their published services since the change was made to exclude their endpoint, which is causing issues for new reports. Rather than exclude their endpoint entirely from being updated, this commit just silences the noisy error message for Bristol. Reverts the change made in 491eb26e4.
* Store group on Contacts in Open311 PopulateServiceListDave Arter2017-12-05-1/+13
|
* [East Herts] Preserve 'Other' category in PopulateServiceListDave Arter2017-11-29-0/+8
|
* Move FixMyStreet::DB to FixMyStreet::DB::Schema.Matthew Somerville2017-08-09-1/+1
| | | | Store a schema object on FixMyStreet::DB instead.
* Add inactive state to categories.Matthew Somerville2017-07-14-8/+5
| | | | | A new 'state' column replaces confirmed and deleted, allowing categories to be unconfirmed, confirmed, deleted or inactive.
* Automatically spot Open311 co-ord/ID attributes.Matthew Somerville2016-08-24-6/+6
|
* [Bristol] Bristol City Council cobrand frontendDave Arter2016-06-17-3/+4
| | | | For mysociety/FixMyStreet-Commercial#766.
* Improve Open311 handling of bad attribute data.Matthew Somerville2016-06-17-1/+1
| | | | | We have seen e.g. an endpoint that is returning 'serviceAttributes' as the child of 'attributes', rather than 'attribute'.
* Fix Open311 service code check.Matthew Somerville2016-06-17-1/+1
| | | | The service code could be "0".
* Fail nicely if Open311 response lacks service codeMatthew Somerville2016-06-15-6/+10
|
* Tidy up Open311 XML handling.Matthew Somerville2016-06-15-10/+2
| | | | | Parse the XML in such a way other parts of the code do not have to worry about single-value folding or the like.
* Remove some unused cobrands.Matthew Somerville2016-04-20-1/+0
|
* Remove many uses of mySociety::Config.Matthew Somerville2015-12-02-1/+1
|
* Move from Moose to Moo in non-App code.Matthew Somerville2015-12-02-1/+1
| | | | And create default cobrand class without all of Moose.
* Reduce use of FixMyStreet::App.Matthew Somerville2015-12-02-8/+4
| | | | Command line scripts don't need a full blown app, just database.
* Remove ms_current_timestamp() override function.Matthew Somerville2015-08-13-3/+3
| | | | | This override is not used anywhere, it dates from a very old long-gone test script, and causes issues with database query planning.
* Add Extra role to ease use of {extra} field.Hakim Cassimally2015-03-20-5/+1
| | | | | | | | | Historically, the extra field has been used in two different ways by different cobrands, both as a list (e.g. Open311 category fields) and a hash (e.g. the Zurich cobrand). This commit consolidates usage, adding an API to make use of the field easier and always returning a hash for the code to use. Fixes #1018.
* Better error if Open311 returns nil service list.Matthew Somerville2014-11-21-8/+10
| | | | | | | | If an Open311 server responded with e.g. '<services></services>', you got a strange hash error as the code treated it as one undef service. Spot this at the same point we check anything was returned. Also remove a couple of warnings due to lc of a maybe-undef variable.
* [Warwickshire]. PopulateServiceList deletion tweakHakim Cassimally2014-10-16-4/+15
| | | | | | as WCC is (during beta) mixed email (manually updated) and Open311, override the behaviour to delete contacts not returned by service list, to ignore ones that look like email addresses.
* Open311: FMS <-> Open311 Endpoint End to end testsHakim Cassimally2014-10-16-0/+1
| | | | | Uses LWP::Protocol::PSGI to avoid having to spawn external processes in order to test these end to end.
* Open311 Warwick (Exor) IntegrationHakim Cassimally2014-10-16-5/+18
| | | | | | | | | | | | | | | | | | | | | | | ::Integration::Warwick subclasses ::Integration::Exor refactor request_class and new_request Exor service Warwickshire updates retrieval, with datetimes stubbing out of Oracle constants, for local testing We also edit FMS's core PopulateServiceList routine to hide system fields from FMS: Bromley/Warwickshire send metadata in their services/FOO.xml advising that you can pass, e.g. attributes[easting]. FMS by default shows all of these to the user to fill in, however we don't *want* the user to supply these, rather they are added by the cobrand. Bromley had an exception for this (keyed by $body->areas->id). We write this more generally for Warwickshire too, keying instead by $body->name (as this is far less likely to be overridden for installs using global or custom Mapit's)
* Replaced hardcoded Mapit-URL in error message to use variable from config ↵Rikard2013-08-30-1/+2
| | | | instead
* Check for only one result, fixes #359.Matthew Somerville2013-03-18-1/+4
|
* Merge remote branch 'origin/zurich'Matthew Somerville2013-02-04-30/+31
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: bin/open311-populate-service-list bin/send-comments bin/update-all-reports conf/crontab.ugly db/schema.sql perllib/FixMyStreet/App/Controller/Admin.pm perllib/FixMyStreet/App/Controller/Report/New.pm perllib/FixMyStreet/App/Controller/Reports.pm perllib/FixMyStreet/Cobrand/Default.pm perllib/FixMyStreet/Cobrand/LichfieldDC.pm perllib/FixMyStreet/DB/Result/Open311conf.pm perllib/FixMyStreet/DB/Result/Problem.pm perllib/FixMyStreet/DB/ResultSet/Problem.pm perllib/FixMyStreet/SendReport.pm perllib/FixMyStreet/SendReport/Email.pm perllib/FixMyStreet/SendReport/Open311.pm perllib/Open311/GetServiceRequestUpdates.pm perllib/Open311/PopulateServiceList.pm t/app/controller/report_new.t t/app/controller/rss.t templates/web/bromley/report/display.html templates/web/default/admin/council_contacts.html templates/web/default/common_header_tags.html templates/web/default/dashboard/index.html templates/web/default/front/stats.html templates/web/default/report/_main.html templates/web/default/report/update-form.html templates/web/emptyhomes/index.html templates/web/emptyhomes/report/display.html templates/web/emptyhomes/report/new/councils_text_all.html templates/web/emptyhomes/reports/body.html templates/web/emptyhomes/reports/index.html templates/web/fixmystreet/report/new/fill_in_details_form.html templates/web/fixmystreet/report/update-form.html web/cobrands/fixmystreet/fixmystreet.js web/js/fixmystreet.js
| * Allow bodies to span multiple area IDs, and areas can be covered by more ↵Matthew Somerville2012-12-19-6/+6
| | | | | | | | than one body.
| * Convert PopulateServiceList to use body IDs.Matthew Somerville2012-12-15-29/+30
| |
| * Rename area_id on Contacts table to body_id.Matthew Somerville2012-12-15-3/+3
| | | | | | | | Add foreign key constraint from contacts to body.
| * Rename 'open311conf' database table to 'body'.Matthew Somerville2012-12-15-1/+1
| |
* | don't do Open311 service discovery for departments in FixMyBarangay project ↵Dave Whiteland2013-01-29-1/+1
|/ | | | (yet)
* do not fall over if there is unexpectedly no meta data for a service codeStruan Donald2012-11-23-0/+8
|
* Merge branch 'stevenage'Struan Donald2012-11-19-0/+6
|\ | | | | | | | | | | | | | | | | | | Conflicts: .gitignore bin/make_css conf/crontab.ugly perllib/FixMyStreet/Cobrand/Default.pm perllib/Open311.pm templates/web/fixmystreet/around/postcode_form.html
| * handle updates to meta data as well as meta data when a contact is createdStruan Donald2012-10-18-0/+6
| |
* | Improve error message.Matthew Somerville2012-10-04-1/+2
| |
* | Caseinsensitive match.Matthew Somerville2012-10-04-1/+1
| |
* | Add warning option to Open311 populate, to show warnings.Matthew Somerville2012-10-04-7/+10
| |
* | Merge remote branch 'origin/master' into fmb-read-onlyDave Whiteland2012-05-29-6/+37
|\| | | | | | | | | | | | | | | | | | | | | Conflicts: bin/send-reports perllib/FixMyStreet/Cobrand/Default.pm perllib/FixMyStreet/Cobrand/FixMyStreet.pm templates/web/fixmystreet/alert/index.html templates/web/fixmystreet/around/display_location.html web/cobrands/fixmystreet/_layout.scss web/js/map-OpenLayers.js
| * do not put empty arrays into extra field of contacts and ignore them if they ↵Struan Donald2012-05-25-1/+5
| | | | | | | | are there
| * add verbose option and apply to print statementsStruan Donald2012-04-23-4/+5
| |
| * correct function nameStruan Donald2012-04-23-1/+1
| |
| * more populate tests and also cope with bromley non display fieldsStruan Donald2012-03-22-0/+26
| |
* | need to check for 'open311' send_method because (naughtily) there may be ↵Dave Whiteland2012-05-01-0/+1
|/ | | | services in there that are not open311 (later: rename table?)
generated by cgit v1.2.3 (git 2.25.1) at 2025-08-22 21:31:03 +0000