Problem/Motivation

When running reroute_email_update_8202 using drush, the double quotes in the message 'The outdated property "whitelist" has already been removed. This update is to be skipped.' throw an JSON error.
The update itself is executed fine.

If the update is run as part of a drush deploy, it will halt at the running updates-phase.

Steps to reproduce

- $drush deploy

 In ProcessBase.php line 171:
>
>   Unable to decode output into JSON: Syntax error
>
>   {
>       "0": {
>           "reroute_email": {
>               "8202": {
>                   "results": {
>                       "query": "The outdated property "whitelist" has already been removed. This update is to be skip
>   ped.",
>                       "success": true
>                   },
>                   "type": "update"
>               }
>           },
>       },
>       "1": [],
>       "2": [],
>       "3": [],
>       "4": [],
>       "drush_batch_process_finished": true
>   }

Proposed resolution

Replace double quotes with single quotes.

Remaining tasks

User interface changes

API changes

Data model changes

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

spokje created an issue. See original summary.

spokje’s picture

Status: Active » Needs review

bohart made their first commit to this issue’s fork.

  • bohart committed 34e5d288 on 2.2.x authored by spokje
    Issue #3472234 by spokje, bohart: Double quotes in...

  • bohart committed 54966da7 on 2.3.x authored by spokje
    Issue #3472234 by spokje, bohart: Double quotes in...
bohart’s picture

Status: Needs review » Fixed

This one is tricky :)
The changes have been committed to both 2.2.x and 2.3.x branches.
Thanks!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.