Fix #32. Escaping numeric replacements in the content fix method to m…

…ake it compatible with domains that start with a number, such as Tor domains.
straube · Dec 19, 2018 · 61c65f4 · 61c65f4
1 parent ac2a62f
commit 61c65f4
Show file tree

Hide file tree

Showing 4 changed files with 14 additions and 6 deletions.
diff --git a/README.md b/README.md
@@ -59,6 +59,10 @@ language set in the plugin config. Also notice the language may be `null`.
 
 ## Changelog
 
+### 0.8.2
+
+* Fix: #32 Image URLs not being re-written properly via Tor.
+
 ### 0.8.1
 
 * Fix: #23 Undefined index when using wp-cli.
@@ -120,7 +124,7 @@ This is the first release. It supports setting domains and an optional base URL
 Contributors: GustavoStraube, cyberaleks, jffaria  
 Tags: multiple, domains, redirect  
 Requires at least: 4.0  
-Tested up to: 4.9.6  
+Tested up to: 5.0.1  
 Stable tag: trunk  
 License: GPLv2 or later  
 License URI: http://www.gnu.org/licenses/gpl-2.0.html  
diff --git a/multiple-domain/MultipleDomain.php b/multiple-domain/MultipleDomain.php
@@ -8,7 +8,7 @@
  * @author  Clay Allsopp <https://github.com/clayallsopp>
  * @author  Alexander Nosov <https://github.com/cyberaleks>
  * @author  João Faria <https://github.com/jffaria>
- * @version 0.8.1
+ * @version 0.8.2
  * @package multiple-domain
  */
 class MultipleDomain
@@ -20,7 +20,7 @@ class MultipleDomain
      * @var   string
      * @since 0.3
      */
-    const VERSION = '0.8.1';
+    const VERSION = '0.8.2';
 
     /**
      * The current domain.
@@ -348,7 +348,7 @@ public function fixContentUrls($content)
     {
         if (array_key_exists($this->domain, $this->domains)) {
             $regex = '/(https?:\/\/)' . preg_quote($this->originalDomain) . '/i';
-            $content = preg_replace($regex, '$1' . $this->domain, $content);
+            $content = preg_replace($regex, '${1}' . $this->domain, $content);
         }
         return $content;
     }

diff --git a/multiple-domain/multiple-domain.php b/multiple-domain/multiple-domain.php
@@ -6,7 +6,7 @@
 Description: This plugin allows you to have multiple domains in a single
              WordPress installation and enables custom redirects for each
              domain.
-Version:     0.8.1
+Version:     0.8.2
 Author:      Gustavo Straube (straube.co)
 Author URI:  http://straube.co
 License:     GPLv2 or later

diff --git a/multiple-domain/readme.txt b/multiple-domain/readme.txt
@@ -2,7 +2,7 @@
 Contributors: GustavoStraube, cyberaleks, jffaria
 Tags: multiple, domains, redirect
 Requires at least: 4.0
-Tested up to: 4.9.6
+Tested up to: 5.0.1
 Stable tag: trunk
 License: GPLv2 or later
 License URI: http://www.gnu.org/licenses/gpl-2.0.html
@@ -69,6 +69,10 @@ language set in the plugin config. Also notice the language may be `null`.
 
 == Changelog ==
 
+= 0.8.2 =
+
+* Fix: #32 Image URLs not being re-written properly via Tor.
+
 = 0.8.1 =
 
 * Fix: #23 Undefined index when using wp-cli.