| Commit message (Collapse) | Author | Age | Lines |
... | |
|\ |
|
| |
| |
| |
| | |
This also updates the body column to match the category.
|
|/
|
|
|
|
| |
If a category had eg. a double space, the use of the filter meant it
was output in the HTML with only a single space and so woudn't match
the database entry. Switch to more explicit template space chomping.
|
| |
|
|
|
|
|
|
|
|
| |
Replaced duplicate code blocks that are responsible for sending
confirmation of problem email with a function. Should make it
easier to do two-tier conditional messages later on. Also fixes a
minor bug where the wrong confirmation email would be sent for
reports from the mobile app to unresponsive councils.
|
|
|
|
| |
The tests were running slowly because of gaze.mysociety.org's rate-limiting.
|
|
|
|
|
|
| |
If you had no session cookie, started reporting a problem, logged in
through that process, you would then get a CSRF error as the token
had been created before the session was.
|
|
|
|
|
| |
Tidy up use of absolute URLs when in the admin.
Have the 404 handler spot static files in admin as a fallback.
|
|\ |
|
| |
| |
| |
| | |
Show bodies for which details are missing, and any send method used.
|
| | |
|
|/
|
|
|
|
| |
jhead 3.00 considers 0 an invalid value for EXIF orientation and will replace it
with 1 when run with the -autorot flag. This was causing a test failure as the
SHA hash of the sample image was being changed.
|
| |
|
|\ |
|
| | |
|
| | |
|
| |
| |
| |
| |
| | |
It errored on new-style parsed XML, rather than let it through.
Add a new category to the test that highlights this.
|
|\ \ |
|
| |/
| |
| |
| |
| | |
Parse the XML in such a way other parts of the code do not have to worry
about single-value folding or the like.
|
|/
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Some UK councils with Open311 integrations (e.g. Bromley) have a custom wording
in the confirmation email sent when updates are left on reports, to make the
user aware that the update is sent to the council in question.
Bromley noticed that some emails were being sent without this wording, leading
at least one user to contact the council directly about the report.
It turns out that although the email template contains an IF clause to use the
appropriate wording for Bromley (and Stevenage) reports, the incorrect template
file was being used when updates were made via the Bromley cobrand.
This commit solves the problem by introducing a new
`Cobrand::Default::path_to_email_templates` method, which is overridden by
`Cobrand::UKCouncils` to include the `templates/email/fixmystreet` path. Paths
returned by this method are used as the `additional_template_paths` param when
templating emails. A regression test is included.
Additionally moves email templates for fixmystreet.com to a directory name
reflecting their purpose, in the same way the web templates are arranged.
|
| |
|
|
|
|
|
|
|
|
|
|
|
| |
If we don't do this, the default quoted-printable encoding will make
sure the lines are physically wrapped anyway. format=flowed could be
an alternative, but I think "=" acting as a quasi-hyphen is somewhat
nicer than every line ending with "=20".
A better solution could be quoted-printable doing soft wrapping only
at word breaks, making each line be readable though end with a " =",
but that didn't seem worth the effort involved.
|
| |
|
|
|
|
|
|
|
| |
Historically, emails sent offline (alerts, questionnaires, etc) used a
different templating system from those sent by the website (e.g. login
emails), though the newer system was also being used for the site name
and signature of offline emails.
|
|
|
|
| |
Fixes #1286.
|
|
|
|
|
| |
Even when we ask for a specific range of dates, we sometimes get given
more in the response.
|
|\ |
|
| | |
|
| |
| |
| |
| | |
And associated, e.g. nothing references fixmystreet-old-box.js any more.
|
| | |
|
| |
| |
| |
| | |
Pull in the same fix that was made in f598f56f for the Facebook tests.
|
|\ \
| |/
|/| |
|
| | |
|
|\ \ |
|
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Store image type along with hash in photo column, and use that when
outputting images / generating URLs. Make sure all public photo URL
generation goes through appropriate functions, and change temp URLs
so the filename can be output directly from the list.
|
| |/
| |
| |
| |
| | |
We check for a session's existence before looking in it, as looking in
it first would auto create a session.
|
|/
|
|
| |
Fixes #574.
|
|
|
|
|
|
|
|
|
|
| |
Test::PostgreSQL sets up a new database with initdb and empties the
generated postgresql.conf; PostgreSQL 9.2+ sets its locale/timezone
information at the initdb stage, not when it is run. It is possible
therefore for the tests to be in one timezone but the test database
to be in another (GMT) – causing a test failure when e.g. an update
is created manually. Make sure the first comment is well before the
others being tested, so there is no chance the order gets confused.
|
|\ |
|
| | |
|
| | |
|
| | |
|
|/
|
|
| |
Fixes #1266.
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The following modules had bugs that have been fixed for working in
recent perls:
* List::MoreUtils
* Guard
* PadWalker
* aliased
* URI
* Convert::NLS_DATE_FORMAT
The CGI module was removed from core in 5.20, so include it in the
snapshot (I don't think it's actually used, but is a dependency).
"{" needs to be escaped in regular expressions, and ~~ should not be
used.
Fix some tests that expect e.g. a certain hash ordering, to use sorted
output or better comparisons.
|
|\ |
|
| |
| |
| |
| |
| |
| |
| |
| | |
Includes a smiley, bright yellow homepage, and updated nav bar and
footer across the entire site; a few content changes; request for
gender information from users.
Fixes #1341.
|
|/ |
|
|
|
|
|
|
| |
This adds a new cobrand variable, report_form_extras, which contains a
list of extra fields that will be saved in the 'extra' metadata of the
report. Fields may optionally be marked as required.
|
|
|
|
|
| |
If a user is logged in, use their last report/update to set the default
anonymity state.
|