aboutsummaryrefslogtreecommitdiffstats
path: root/perllib/FixMyStreet
Commit message (Collapse)AuthorAgeLines
* Version 1.5.2.v1.5.2Matthew Somerville2014-12-17-24/+22
| | | | | | | Includes: * [UK] Don't show topic form field when reporting abuse. * Use token in moderation response URL to prevent hidden report leak. * Make sure successful submission page is full width.
* Add a script to always test on a clean db/config.Hakim Cassimally2014-12-12-0/+94
| | | | Fixes #786.
* By default, use area-based alerts.Matthew Somerville2014-12-12-11/+31
| | | | | | | | | The body-based alert only works if the body ID matches the MapIt ID. This fixes #959. Further work needs to be done to enable proper body-based alerts that work properly in all circumstances. Consequently, factor out fixed body IDs from many tests. Also fix a couple of tests not overriding geocoder correctly.
* Set DefaultLocale appropriately when language set.Matthew Somerville2014-12-10-7/+8
|
* Fix encoding of signature in emails.Matthew Somerville2014-12-10-0/+2
| | | | | This text was being imported encoded, which led to double-encoding when used in templates. Fixes #960 and #961.
* Pass cobrand into send_email_cron when sending SendReport emailsSteven Day2014-12-09-1/+3
| | | | | | | | | | | | | | Without this, email sending within a cobrand will get the right main email template (submit.txt) for the email - because it uses the cobrand inside get_template - but it doesn't use the cobrand for things like the email signature, because you have to pass it into send_email_cron manually to make it cobrand-aware. This probably hasn't been noticed before because most cobrands are called Something FixMyStreet, but Collideoscope is different and hence it caused a problem there. Closes mysociety/FixMyStreet-Commercial#621.
* [FiksGataMi] Update cobrand to current design.Matthew Somerville2014-11-19-16/+0
|
* Tidy up reporting sidebar notes and privacy.Matthew Somerville2014-11-19-8/+9
| | | | For #344 and #935.
* Show logged in message as success, not error.Matthew Somerville2014-11-18-3/+4
| | | | | | Fixes #357. Also consolidate almost-identical fill_in_details.html template (for #344).
* Only show text on /reports if appropriate.Matthew Somerville2014-11-14-5/+9
| | | | | Only show "select..." if more than one body, and only mention greyed out lines if there are any. Fixes #869.
* Remove unused variable.Matthew Somerville2014-11-07-1/+0
| | | | | Also, s///r isn't present in 5.10, and FixMyStreet should still install on squeeze.
* Remove liberror-perl and libsoap-lite-perl.Matthew Somerville2014-11-05-6/+7
| | | | | | Try and save other people the potential pain of installing SOAP::Lite; only UK FixMyStreet and Zurich geocoder use it, so set it as an optional feature in the cpanfile instead.
* Category should be escaped in RSS feeds.Matthew Somerville2014-10-24-1/+1
| | | | In case it contains an ampersand or somesuch.
* Allow filtering by category on All Reports page.Matthew Somerville2014-10-24-0/+6
| | | | If you know to change the URL, anyway, for now.
* [fixmystreet.com] Experiment, no pins by default.Matthew Somerville2014-10-17-10/+6
| | | | | | | | This changes the default flow to show no pins and a message about reporting, with the second tab displaying existing problems (and showing map pins). Those existing problems are the ones that would currently be on "problems on the map" followed by those currently on "problems nearby".
* [fixmystreet.com] Remove login content experiment.Matthew Somerville2014-10-17-9/+1
| | | | This gave no conclusive results, which was interesting.
* Open311 tweaks to get round-tripping workingHakim Cassimally2014-10-16-2/+8
| | | | | If https://github.com/mysociety/fixmystreet/pull/792 is accepted then this should be rewritten in terms of that.
* [Warwickshire] cobrand workDave Arter2014-10-16-0/+35
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | First Warwickshire styling - imported WCC's header and footer files - Use Warwickshire colours for page elements - Include FMS elements in header - Use correct Warwickshire colours on map - Use full-screen map on desktop. - Improve display of homepage elements - Ensure correct classes are added to body tag - Make sure site header has correct height on front page - Tidy-up of unused header content and wrappers - Improve display of map - Position map beneath other elements Improve map display - Take some CSS from Warwickshire styles to ensure its applied - Fix container widths on map page - Make sure map wrapper is the correct width - Add style overrides to match Warwickshire.gov.uk - Ensure map controls are visible Nav and mobile styling - Make sure sticky sidebar nav is usable - Improve mobile front page - Ensure map is full-screen on mobile - Improve front page display on mobile - Fix map display on desktop - Fix Warwickshire and FMS nav on mobile - Add viewport meta tag for mobile devices - Tighten up vertical spacing on front page - Fix header height on tablet size screens - Set pan/zoom position in CSS, not JS Styling tweaks - correct styling of nav for wider screens (previous hidden by media query) - Add draft Warwickshire FAQ - Fixes https://github.com/mysociety/FixMyStreet-Commercial/issues/611 - Warwickshire. remove all reports link as requested. - see https://github.com/mysociety/FixMyStreet-Commercial/issues/609 - Update WCC footer - see https://github.com/mysociety/FixMyStreet-Commercial/issues/597 Bring WCC css locally, and override - Override for https://github.com/mysociety/FixMyStreet-Commercial/issues/612 - favicon https://github.com/mysociety/FixMyStreet-Commercial/issues/616 - update contact details - custom FAQ - fix padding on mobile https://github.com/mysociety/FixMyStreet-Commercial/issues/617
* Merge branch 'issues/commercial/603/generic-problems-clause-for-two-tier'Hakim Cassimally2014-10-16-21/+9
|\
| * UK. Generic problems_clause for two-tier councilsHakim Cassimally2014-10-10-21/+9
| | | | | | | | | | | | Replace copy-pasted problems_cause declarations in Hart, Oxfordshire, and East Sussex with a generic override in UKCouncils.pm (as a bonus also overrides Stevenage)
* | Paginate admin search results.Matthew Somerville2014-10-13-22/+22
|/
* ESCC. bodies_clause specialisationHakim Cassimally2014-10-10-0/+8
| | | | | patch for https://github.com/mysociety/FixMyStreet-Commercial/issues/603 more generic fix to follow
* EastSussex: set is_two_tierHakim Cassimally2014-10-09-1/+1
| | | | | | I appear to have left this commented out. Comments from both ESCC and a district council (Wealden) on same day seem to point to this as root of error.
* [FixMyStreet.com] Add information for councils.Matthew Somerville2014-10-09-2/+2
| | | | Fixes #765.
* Validate category name/email/note in admin.Matthew Somerville2014-10-08-1/+10
| | | | This prevents the creation of a category with a blank name. Fixes #556.
* [EastSussex] pin coloursHakim Cassimally2014-09-18-0/+17
| | | | | | - yellow for open issues, green otherwise - pins > 3 months hidden from /around/ - reduced number of pins on /reports/ to 20 (as per Bromley/Hart)
* Let cobrand override geocoderHakim Cassimally2014-09-16-1/+18
| | | | | Fixing issues with Bing geocoding for East Sussex, for example "Church Street".
* [EastSussex] tweaks after comments from ESCCHakim Cassimally2014-09-16-0/+64
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - change front-page text - rename Category to Type By overriding report/new/category template snippet. This is hacked in (including substitution of the "-- Pick a category --" text, as not sure how to simply do this as a cobrand-specific translation. - hide footer on mobile view - pothole field input and display This is a temporary request, which we hope will be superseded by full integration in due course. A command is documented in the cobrand to set the "extra" on the contact info for the Potholes category. This is then displayed by the UI. The default process_extras places it in extra, for potential future processing. As this is a temporary fix, we are calling this field "detail_size", as this is automatically included in "Details" and therefore also sent in the email. (This means that we may need to update data later on to call this "pothole_size" or similar in future?) - contrast tweaks (Via request from Matt Tooke at ESCC.) Not all handled precisely as he suggests: * spacing of button nav -- more or less as suggested, but tweaks for the <span> selected button, and reduced padding on non-frontpage. * sign-in buttons: fixed using CSS for .green-btn instead of adding HTML class throughout. * mobile-footer: edited <footer> css, as suggested by Dave Arter. - revert problematic CSS (breaks map report page) - Remove spurious "FixMyStreet" button - frontpage subheading wording
* [East Sussex] map and nav cosmeticsDave Arter2014-09-16-0/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Use East Sussex-specific position_map.js - Display map correctly on desktop - Increase default zoom level and use correct grammar for postcode box - Tidy up default font size and center elements on postcode form - Correctly position report form, allow links in sidebar to be seen - Make map guidance text visible on mobile - Fix display of site header across all pages - More sidebar/header fixes - Override some text on the report sidebar to be more friendly - Remove unused images - Use breadcrumb bar for main nav on desktop - Restore correct background colour to footer - Tighten up vertical spacing on all pages - Slacken vertical spacing on home page - Stop the sidebar clobbering the footer - Tidy up menu display - Ensure form errors are shown properly - Add greeting and 'sign out' link to appropriate place - Ensure correct width is used on IE8 - Make 'home' breadcrumb a span instead of a link - Hide top 4px border on mobile - Remove breadcrumb styling from FMS nav - Use ellipsis and correct capitalization on confirm page - Don't uppercase all button labels - Hide email icon in email fields - Don't italicise input placeholder text - Use correct font family, alignment and size for various elements - Correct box shadow, fix pin alignment, various UI fixes - Ensure font size isn't too small. Fix header on wide but small screens - Don't adjust .content height for East Sussex pages
* [East Sussex] Tidy up layoutDave Arter2014-09-16-1/+1
| | | | | | | | | | | | | | | | | | | | | - Tidy up layout of a few elements on front and map pages - ensure example places are within county - match council website body width - Remove 'size2' class from body This means we don't need lots of little CSS hacks to tweak things back into position - Restore 'contact us' underline - Ensure content extends down to footer - More UI layout tidying - Make sure error and confirmation pages don't have obscured content - Improve display of frontpage header on mobile - Better CSS selector for header display on mobile - Fix display of map on mobile - Tidy up display of footer buttons on mobile - Put mobile-specific styles back into base.scss - css separate margin-top for map/twothirds pages - Fix display of languages in footer on mobile - Removed unused CSS files - East Sussex doesn't have tracking_code.html, so remove reference to it
* [East Sussex] initial work on cobrand and templatesHakim Cassimally2014-09-16-0/+38
| | | | | | | - Get layout, including map working - Identified major clash point for CSS - Move eastsussex resources locally (in part to be able to override clashing names like .container)
* refactor repeat code into get_email_templateHakim Cassimally2014-08-14-27/+5
|
* Move bromley from report hiding to moderationHakim Cassimally2014-08-14-0/+13
| | | | | | | | | The feature to hide reports is still enabled, as it is used by other `users_can_hide` bodies. As we want to roll out moderation to Bromley, in this commit we hide the functionality *only* for Bromley. Of course if we migrate those other users to this moderation method, then we will need to make equivalent change to general template, and remove the functionality from Report.pm.
* Tests for moderationHakim Cassimally2014-08-14-2/+4
|
* Report moderationHakim Cassimally2014-08-13-15/+643
| | | | | | | | | | | - redaction marked with [...] - of report and comments - stores original data - uses a single form, on the report/_main view - requires additional permissions (user_body_permissions) - Hide report functionality - Moderation notification/contact form - Moderation writes to admin_log
* Add MakeMyIsland cobrand module (really for title)Dave Whiteland2014-08-07-0/+14
| | | | | Perhaps useful as we might be adding more stuff if the project expands, e.g., geocoder disambiguation.
* Replace from_council with from_body in SeeSomething cobrandSteven Day2014-07-31-5/+6
| | | | | | | | | | | | | | | | | | | | | | 'from_council' was renamed to 'from_body' and turned into a relationship in b2aa34b Feb 2013, and the seesomething code was missed. Though the FMS::DB::Result::User object would (correctly) throw an error if an invalid method is called, *Catalyst* idea of a user object wraps this in a Catalyst::Authentication::Store::DBIx::Class which uses AUTOLOAD to delegate methods and swallows the error. e.g. this code always returns false, hence the code path is never run. (I think this is a bug and have opened: https://rt.cpan.org/Ticket/Display.html?id=97640 ) Added test case and (hopefully) corrected template, will need a check against site running copy with recent data to check that nothing looks squiffy. We also had to change the %council_details stash, which was based on the old code, not sure if suggested fix matches original intent, but looks plausible. (As discussed with @dracos @stevenday this is slightly repeated, and should arguably use ->{bodies} but that's a possible future FIXME)
* Removal old Southampton/Reading/Lichfield cobrands.Matthew Somerville2014-07-29-129/+0
|
* [Bromley] disambiguation for White Horse HillHakim Cassimally2014-07-18-0/+7
| | | | | | | | | | This street is on boundary of Bromley with Greenwich, so a simple disambiguation of $town = "chislehurst" falls foul of UKCouncils-> area_check. It may be worth overriding area_check in general, but in mean time, setting a specific postcode (on the Bromley side of the administrative boundary) allows the map to be seen.
* Add support for SMTP authentication and TLS.Matthew Somerville2014-07-14-1/+20
| | | | | | Fixes #406. Include Perl modules necessary for Email::Send::SMTP to handle TLS/SSL, and upgrade Dir::Self due to https://rt.cpan.org/Public/Bug/Display.html?id=88393
* Add fixmystreet.com reporting form A/B tests.Matthew Somerville2014-07-10-2/+13
| | | | | | Testing two things: * Putting the email box above the Yes/No, or repeating it inside. * Showing the contents of the Yes/No sections at first, or not.
* Tidy up cobrand template inheritance.Matthew Somerville2014-07-08-77/+20
| | | | | Most cobrands can just inherit from the Default function, with only those needing an exception specified.
* Rename 'default' web directory to 'base'.Matthew Somerville2014-07-08-5/+5
| | | | This should reduce confusion with the Default cobrand and override order.
* Move UK-specific text to separate templates.Matthew Somerville2014-07-08-0/+9
|
* Merge branch '571-show-local-deploys-banner'Matthew Somerville2014-07-04-0/+28
|\ | | | | | | | | Conflicts: web/cobrands/fixmystreet/base.scss
| * Display a banner to non UK FixMyStreet visitorsStruan Donald2014-07-03-0/+28
| | | | | | | | | | | | only on FMS and only if from outside the uk Fixes #571
* | Merge branch 'mysociety/798-show-reports-subset'Matthew Somerville2014-07-04-3/+24
|\ \
| * | Link through from all reports page to sub categoriesStruan Donald2014-07-01-3/+24
| |/ | | | | | | | | | | | | | | Add a type parameter to the /reports/body page to restrict list by problem category. Only link to report categories that have entries. Fixes #798
* | Update secure proxy list to include www.fixmystreet.com.Matthew Somerville2014-07-04-2/+5
| |
* | Use https for UK site where appropriateStruan Donald2014-07-03-4/+5
|/ | | | | | | Council redirects, BASE_URL comparisons, hard coded links, email signatures For #488
generated by cgit v1.2.3 (git 2.25.1) at 2025-08-06 07:40:45 +0000