Merge pull request #1508 from xwp/fix/issue-1489-valueerror-class-log

Fix Uncaught ValueError in Gravity Forms and WordPress SEO connectors

.gitignore

@@ -10,6 +10,7 @@
 /stream.zip
 /stream-*.zip
 npm-debug.log
+debug.log
 package.lock
 .phpunit.result.cache
 

classes/class-connector.php

@@ -294,4 +294,14 @@ function ( $value ) {
 	public function is_dependency_satisfied() {
 		return true;
 	}
+
+	/**
+	 * Escape % characters in a string to avoid Uncaught ValueErrors in $this->log().
+	 *
+	 * @param string $value The string value to be escaped.
+	 * @return string The escaped string.
+	 */
+	public function escape_percentages( $value ) {
+		return str_replace( '%', '%%', $value );
+	}
 }

composer.json

@@ -31,6 +31,7 @@
     "wpackagist-plugin/easy-digital-downloads": "2.9.23",
     "wpackagist-plugin/jetpack": "10.0",
     "wpackagist-plugin/user-switching": "1.5.5",
+    "wpackagist-plugin/wordpress-seo": "23.0",
     "wpackagist-theme/twentytwentythree": "^1.0",
     "xwp/wait-for": "^0.0.1",
     "yoast/phpunit-polyfills": "^1.1"
@@ -77,6 +78,10 @@
       "phpunit --coverage-text",
       "php local/scripts/make-clover-relative.php ./tests/reports/clover.xml"
     ],
+    "test-one": [
+      "phpunit",
+      "WP_MULTISITE=1 phpunit"
+    ],
     "test-multisite": [
       "WP_MULTISITE=1 phpunit --coverage-text",
       "php local/scripts/make-clover-relative.php ./tests/reports/clover.xml"