Changeset 7443

Timestamp:

10/17/2013 10:27:57 PM (12 years ago)

Author:

r-a-y

Message:

JS: Replace String.trim() method with jQuery's trim() function.

When using IE8 and when attempting to post an activity update, IE8
will throw an error and not post the update because of the usage of
String.trim(), which does not exist in IE8.

To fix this bug, we replace String.trim() with jQuery's version of
trim().

Props tomdxw for reporting the bug.

Fixes #5205.

Location:

trunk

Files:

• bp-templates/bp-legacy/js/buddypress.js (modified) (1 diff)
• bp-themes/bp-default/_inc/global.js (modified) (1 diff)

trunk/bp-templates/bp-legacy/js/buddypress.js

@@ -1555 +1555 @@
         var cookie    = allCookies[i];
         var delimiter = cookie.indexOf("=");
-        var name      = unescape( cookie.slice(0, delimiter) ).trim();
+        var name      = jq.trim( unescape( cookie.slice(0, delimiter) ) );
         var value     = unescape( cookie.slice(delimiter + 1) );
 

trunk/bp-themes/bp-default/_inc/global.js

@@ -1534 +1534 @@
         var cookie    = allCookies[i];
         var delimiter = cookie.indexOf("=");
-        var name      = unescape( cookie.slice(0, delimiter) ).trim();
+        var name      = jq.trim( unescape( cookie.slice(0, delimiter) ) );
         var value     = unescape( cookie.slice(delimiter + 1) );