xoops-trackers Mailing List for XOOPS Web Application System

Patches item #3314995, was opened at 2011-06-11 16:30
Message generated for change (Tracker Item Submitted) made by adoxa
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=430842&aid=3314995&group_id=41586

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: Patches
Status: Open
Priority: 5
Private: No
Submitted By: Jason Hood (adoxa)
Assigned to: Nobody/Anonymous (nobody)
Summary: Instant preview plus sign fix

Initial Comment:
Due to the manner in which the instant preview is generated, plus signs are turned into spaces. Here is a simple patch to correct that.

diff -ur xoops-2.5.1a.orig/htdocs/include/formdhtmltextarea.js xoops-2.5.1a/htdocs/include/formdhtmltextarea.js
--- xoops-2.5.1a.orig/htdocs/include/formdhtmltextarea.js	Fri Sep 10 22:07:36 2010
+++ xoops-2.5.1a/htdocs/include/formdhtmltextarea.js	Sat Jun 11 16:25:56 2011
@@ -473,7 +473,7 @@
 function form_instantPreview(xoopsUrl, area_id, imgurl, doHtml)
 {
     var imgUrl = xoopsUrl + '/images/form';
-    var data = escape(xoopsGetElementById(area_id).value);
+    var data = escape(xoopsGetElementById(area_id).value).replace(/\+/g, '%2B');
 
     var url_request = xoopsUrl + "/include/formdhtmltextarea_preview.php";//?text=" + data;
     var args =  "text=" + data;


----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=430842&aid=3314995&group_id=41586