fixed an url encoding issue (#378)

* fixed an uri encode issue * updated version/notes
PoE-TradeMacro · Jun 21, 2017 · 42b300a · 42b300a
1 parent df24f85
commit 42b300a
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 5 deletions.
diff --git a/Fallback.exe b/Fallback.exe
diff --git a/resources/Updates_Trade.txt b/resources/Updates_Trade.txt
@@ -1,5 +1,9 @@
 The following is a list of what has been updated, starting with 1.1.0
 
+2.3.4
+================================================================================================
+Fixed an url encoding issue with the new browser search.
+
 2.3.3
 ================================================================================================
 Changed the way that searches openend in your browser are functioning. They now also work when poe.trade has cloudflare protection enabled without needing to bypass this protection.

diff --git a/resources/VersionTrade.txt b/resources/VersionTrade.txt
@@ -1,2 +1,2 @@
-TradeReleaseVersion := "v2.3.3"
+TradeReleaseVersion := "v2.3.4"
 TradeAHKVersionRequired := "1.1.24.01"
diff --git a/resources/ahk/TradeMacro.ahk b/resources/ahk/TradeMacro.ahk
@@ -2185,11 +2185,9 @@ class _ParamMod {
 	mod_min := ""
 	mod_max := ""
 	ToPayload() 
-	{
-		; for some reason '+' is not encoded properly, this affects mods like '+#% to all Elemental Resistances'
-		this.mod_name := StrReplace(this.mod_name, "+", "%2B")
+	{		
 		this.mod_name := TradeUtils.UriEncode(this.mod_name)
-		p := "&mod_name=" this.mod_name "&mod_min=" this.mod_min "&mod_max=" this.mod_max
+		p := "&mod_name=" this.mod_name "&mod_min=" this.mod_min "&mod_max=" this.mod_max		
 		Return p
 	}
 }