Uploaded image for project: 'CiviCRM'
  1. CiviCRM
  2. CRM-21166

WP-CLI commands fail due to whitespace in regex

    Details

    • Type: Bug
    • Status: Done/Fixed
    • Priority: Major
    • Resolution: Fixed/Completed
    • Affects Version/s: 4.7.24
    • Fix Version/s: 4.7.25
    • Component/s: WordPress Integration
    • Labels:
      None
    • Versioning Impact:
      Patch (backwards-compatible bug fixes)
    • Documentation Required?:
      None
    • Funding Source:
      Contributed Code
    • Verified?:
      No

      Description

      https://github.com/civicrm/civicrm-wordpress/pull/110 fixed https://issues.civicrm.org/jira/browse/CRM-20148 for 4.7.19

      Part of the PR was a cleanup to match CiviCRM coding standards.   However, this did introduce whitespace in regex statements.

      https://github.com/civicrm/civicrm-wordpress/pull/114 corrected the whitespace for wp-cli api calls, we need to fix the remaining regex.

      In 4.7.24 to reproduce:

      wp cv sql-dump > ~/bkup/cvdb-20170910.sql
      Warning: preg_match(): Compilation failed: unmatched parentheses at offset 20 in /home/user/public_html/wp-content/plugins/civicrm/wp-cli/civicrm.php on line 1138
      Warning: preg_match(): Compilation failed: unmatched parentheses at offset 23 in /home/user/public_html/wp-content/plugins/civicrm/wp-cli/civicrm.php on line 1165

       

       

       

        Attachments

          Activity

            People

            • Assignee:
              kcristiano Kevin Cristiano
              Reporter:
              kcristiano Kevin Cristiano
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: