aboutsummaryrefslogtreecommitdiffstats
path: root/perllib/FixMyStreet/DB/ResultSet/Problem.pm
Commit message (Collapse)AuthorAgeLines
* Speed up admin front page.Matthew Somerville2015-10-23-1/+4
| | | | | | | | * Don't show a categories summary unless asked via a link * Don't needlessly join in questionnaire/update summaries * Switch count(distinct()) to a subselect as it turns out select count(*) from (select distinct(user_id) from problem) temp; is a lot quicker than select count(distinct(user_id)) from problem;
* [Zurich] admin workflow changes available statesHakim Cassimally2015-10-06-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | See mysociety/FixMyStreet-Commercial#690 - external_message for Extern/Wunsch states - as per PDF mockups, the "public response" field is now editable from more states, namely: - Ruckmeldung ausstehend - Extern - Zustandigkeit unbekannt - Wunsch - Nicht kontaktierbar - Unsichtbar - test fixes NB: slight hack to reorder Zurich report_display tests As these remove 'investigating' from hidden_states, these break tests for default cobrand (the FMS::App code expects to be called once on request start, not multiple times within one test) - Not contactable button and tests for this and the similar `send_back` functionality (previously untested). NB: Unsure why we need to specify form_number=2 just for these submits, but test won't find the buttons otherwise. - Make sure SDMs' "time_spent" figure is recorded. (See mysociety/FixMyStreet-Commercial#718)
* Merge branch '1070-dmarc-workaround'Matthew Somerville2015-08-19-1/+1
|\
| * Add default sender to send_email_cron.Matthew Somerville2015-08-19-1/+1
| | | | | | | | | | Matches send_email, remove unneeded setting of sender in one call, and add contact name to report sent email.
* | Simplify/consolidate body restriction db code.Matthew Somerville2015-08-19-2/+13
| | | | | | | | | | | | | | Make dashboard work properly in two-tier councils, showing reports sent to both. Create an index on the array of the bodies_str column to speed up performance, and use that throughout the code replacing all LIKE scans. This also enables a simplifying tidy of the restriction code.
* | Use ResultSet/cobrand->problems where appropriate.Matthew Somerville2015-08-19-2/+2
| | | | | | | | Fixes #893.
* | Move "missing" handling to separate column.Matthew Somerville2015-08-19-7/+12
|/
* Fix display of warning message in report sending.Matthew Somerville2015-08-13-1/+1
|
* Remove ms_current_timestamp() override function.Matthew Somerville2015-08-13-5/+5
| | | | | This override is not used anywhere, it dates from a very old long-gone test script, and causes issues with database query planning.
* Allow /ajax to be filtered by statusDave Arter2015-06-29-2/+6
| | | | | If present, the 'status' param can be set to 'open' or 'fixed' to display ensure only open or fixed reports respectively are included in the response.
* Add category filtering support to /ajax via 'category' GET paramDave Arter2015-06-29-1/+2
|
* [fixmystreet.com] Remove special NI code.Matthew Somerville2015-03-16-82/+70
| | | | | This is now handled by normal bodies in the admin. In order to remove references in send_reports, tidy up (unused) EmptyHomes code a bit.
* [UK] Output NI grid references to NI councils.Matthew Somerville2015-02-20-10/+14
|
* Switch easting/northing output to one line.Matthew Somerville2015-02-16-4/+1
| | | | | This is more friendly for e.g. copy and pasting by someone using the Dynamics CRM software. Fixes #997.
* Use same handling for cron and non-cron email.Matthew Somerville2015-01-13-4/+2
| | | | | This means that e.g. SMTP authentication is used when set up by all emails, not just non-cron ones. Fixes #988.
* [Harrogate] custom form fieldsHakim Cassimally2015-01-08-0/+4
| | | | | | | | | | | | | | Requires previous (generic) commit to prevent information only categories (e.g. Dead animals message) from displaying a spurious textbox. Also support for sending these custom fields by email: Requires a hook in RS::Problem method, as by default, only a restricted number of hard-coded fields are send in the email. (And the template language is pseudo-PHP so we can't simply pass in the whole data-structure.)
* refactor repeat code into get_email_templateHakim Cassimally2014-08-14-7/+1
|
* Merge branch '700-auto-emails'Matthew Somerville2014-05-16-2/+4
|\
| * Set site_name and signature email variables for alertsStruan Donald2014-05-16-2/+4
| | | | | | | | so we can include the name of the site non TT templated emails
* | Merge branch 'issues/commercial-503-centro-email'Steven Day2014-05-01-0/+2
|\ \ | |/ |/|
| * Fix missing subcategory_line that was breaking email sendingSteven Day2014-05-01-0/+2
| | | | | | | | | | | | subcategory_line is optional in seesomethingsaysomething, so we need to set it to something (a blank line is what I've chosen) otherwise email sending breaks when it tries to template up the email message
* | Make sure bodies returned in consistent order.Matthew Somerville2014-02-25-1/+4
| |
* | Test for admin_base_url in RS::ProblemHakim Cassimally2013-12-17-1/+6
| | | | | | | | | | Test that the slash is added in correct place (the base URL should not have its own trailing slash).
* | Rationalise trailing slashes from admin_base_urlHakim Cassimally2013-12-11-1/+1
| | | | | | | | | | | | Remove from admin_base_url (which wraps config vars like {ADMIN_,}BASE_URL) and add to places in code which aren't expecting it.
* | mechanism to stop app store tests being sentStruan Donald2013-09-12-1/+5
| | | | | | | | | | | | In order to allow the iOS app to be tested in the app store we need to allow the user to create a report but we don't want to send it. To allow this hide reports with a specific title during send_reports
* | Changed hardcoded URL in Problem.pm to be using variable from general.yml ↵Matthew Somerville2013-08-30-1/+2
| | | | | | | | instead.
* | Consolidate sending backoff for different types of sender.Matthew Somerville2013-05-02-12/+0
| |
* | Merge remote branch 'origin/zurich'Matthew Somerville2013-02-21-1/+1
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | (locale ignored, will rerun .po extraction after to get all changes.) Conflicts: locale/FixMyStreet.po locale/cy_GB.UTF-8/LC_MESSAGES/EmptyHomes.po locale/de_CH.UTF-8/LC_MESSAGES/FixMyStreet.po locale/nb_NO.UTF-8/LC_MESSAGES/FixMyStreet.po locale/nn_NO.UTF-8/LC_MESSAGES/FixMyStreet.po templates/web/fixmystreet/report/banner.html templates/web/zurich/faq/faq-de-ch.html
| * | Translate user details in Zurich external email.Matthew Somerville2013-02-20-1/+1
| | |
* | | add --debug to CronFns so can run bin/send-reports with debug/diagnosis ↵Dave Whiteland2013-02-19-9/+46
| | | | | | | | | | | | output on the command line, closes #376
* | | Couple of fixes to enable EmptyHomes to send reports again.Matthew Somerville2013-02-13-1/+2
|/ /
* | Merge remote branch 'origin/zurich'Matthew Somerville2013-02-04-62/+35
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
| * | Add some email checking to Zurich test.Matthew Somerville2013-01-31-1/+3
| | |
| * | Split the 'send reports' bit of STAGING_SITE to its own config variable, so ↵Matthew Somerville2013-01-16-1/+1
| | | | | | | | | | | | you can have a dev site that sends reports like normal.
| * | Zurich email alerts.Matthew Somerville2013-01-11-2/+6
| | |
| * | Update database fetching to fetch needed data for previous commit.Matthew Somerville2013-01-08-5/+0
| | |
| * | Front page ignoring unconfirmed, but ordering including null confirmeds.Matthew Somerville2012-12-21-2/+4
| | |
| * | Created and state columns needed for Zurich.Matthew Somerville2012-12-20-1/+1
| | |
| * | Allow unconfirmed as a visible state in Zurich, adjust front end list/item ↵Matthew Somerville2012-12-19-1/+1
| | | | | | | | | | | | templates to suit.
| * | Don't try and look up something if we have nothing.Matthew Somerville2012-12-15-1/+1
| | |
| * | Rename council column to bodies_str, and all the related code.Matthew Somerville2012-12-15-5/+5
| | |
| * | Convert report sending code to use bodies rather than areas/councils.Matthew Somerville2012-12-15-22/+20
| | |
* | | Merge branch 'centro'Struan Donald2013-01-04-0/+43
|\ \ \ | | |/ | |/| | | | | | | Conflicts: perllib/FixMyStreet/Cobrand/Default.pm
| * | add in user details field to %h so we can put sensible user details in ↵Struan Donald2012-12-18-0/+4
| | | | | | | | | | | | anonymous reports
| * | stick subcategory details in $h if there is oneStruan Donald2012-12-18-0/+4
| | |
| * | do not send report sent confirmation emails for anonymous reportsStruan Donald2012-12-18-1/+7
| | |
| * | add option to send email confirming report has been sent to councilStruan Donald2012-12-03-0/+29
| |/
* / In cases where we have two tier councils the TESTING_COUNCILSStruan Donald2012-12-18-1/+26
|/ | | | | | | mechanism could get confused as the council entry in the problem table did not match one of the entries in TESTING_COUNCILS. This code fixes that and should handle any combination of councils in that field
* Display method variables correctly.Matthew Somerville2012-10-08-1/+3
|
* More cron quietening, only show full Open311 first time, then summarise.Matthew Somerville2012-10-05-0/+13
|