Skip to content
This repository has been archived by the owner on Jan 19, 2022. It is now read-only.

Commit

Permalink
Fixes 662857, removes socialShare from closure; collided with WT impl…
Browse files Browse the repository at this point in the history
…mentation.
  • Loading branch information
Stew Mckinney committed Jun 9, 2011
1 parent 2ffe00f commit 39800b7
Show file tree
Hide file tree
Showing 3 changed files with 130 additions and 122 deletions.
7 changes: 4 additions & 3 deletions static/js/vendor/jquery.socialShare.js
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,8 @@
* - related: a related account. Can also format it with a title like 'adammiller: People who dislike social media buttons'
*
*/
( function( $ ) {
//removing this from the closure
//( function( $ ) {

$.socialShare = {
cfg: {
Expand Down Expand Up @@ -81,5 +82,5 @@
$.socialShare.fn.init( this, options );
} );
};
} )( jQuery );
//removing this from a closure, fixes 662857
//} )( jQuery );
15 changes: 11 additions & 4 deletions static/js/vendor/webtrends.js
Original file line number Diff line number Diff line change
Expand Up @@ -576,8 +576,15 @@ function dcsDebug(){

Function.prototype.wtbind = function(obj){
var method=this;
var temp=function(){
return method.apply(obj,arguments);
};
var temp;
//
//Ugh.
// if(method.apply){
// 662857 was actually likely due to the socialShare plugin not jiving with the webTrends
//
temp=function(){
return method.apply(obj,arguments);
};
// }
return temp;
}
}

0 comments on commit 39800b7

Please sign in to comment.