Kodono

The Microsoft developers are not really good, and we can see it if you use IE8 in standard mode with Sharepoint 2010: the web parts don’t work anymore due to an error with the wpadder.js file.
After few hours trying to find the problem I’ve finally discovered that the WPAdder class uses the for..in statement without testing the hasOwnProperty (more about that problem), then it causes some issues…

Here is how I fixed it (this code must be added after the DOM is loaded in the master page to be called by all the pages) :

EnsureScript("WPAdderClass", undefined, function() {
  if (typeof loadWPAdderCallback === "function") {
    var _loadWPAdderCallback=loadWPAdderCallback;
    loadWPAdderCallback = function() {
      _WPAdder.prototype._layout = function (a) {
        ULSior: ;
        var l = this._saveDescriptionForLayout();
        if (this._lastLayoutWidth == -1) {
          var g = this._getCategoryContainer();
          for (var b in this._cats) {
            if (this._cats.hasOwnProperty(b)) g.appendChild(this._cats[b].el)
          }
        }
        if (this._table.clientHeight != this._lastLayoutHeight) {
          this._selCat != -1 && this._removeItemHover();
          this._layoutCategoryColumn();
          this._getNavigationTable().parentNode.colSpan = 1;
          var d = 0,
          f = 0,
          c = document.createElement("DIV"),
          g = this._itemContainerTemplate.cloneNode(true);
          c.style.whiteSpace = "nowrap";
          c.style.width = "1px";
          c.style.height = "1px";
          c.style.overflow = "auto";
          c.innerHTML = "Quick Brown Fox!
It jumped!";
          this._getFirstChild(g).appendChild(c);
          this._getItemContainer().appendChild(g);
          var j = c.scrollWidth,
          i = c.scrollHeight;
          this._removeElement(g);
          if (a && (parseInt(a[0][0]) != j || parseInt(a[0][1]) != i || parseInt(a[0][2]) != this._table.clientHeight)) a = null;
          if (a) {
            d = parseInt(a[0][3]);
            for (var b = 0; b < this._cats.length; b++) {
              if (this._cats.hasOwnProperty(b)) {
                var e = this._fillCachedItems(this._cats[b].items, this._itemContainerTemplate, a[b + 1]);
                if (e.length > f) f = e.length;
                this._cats[b].itemCols = e
              }
            }
          } else {
            a = "";
            for (var b in this._cats) {
              if (this._cats.hasOwnProperty(b)) {
                var e = [],
                h = this._fillItems(e, this._cats[b].items, this._getItemContainer(), this._itemContainerTemplate, _WPAdder_maximumItemWidth);
                if (h[1] > d) d = h[1];
                if (e.length > f) f = e.length;
                this._cats[b].itemCols = e;
                a += ";" + h[0]
              }
            }
            a = j + "," + i + "," + this._table.clientHeight + "," + d + a;
            this._getHiddenField("layout").value = this._layoutHash + ";" + a
          }
          for (var b in this._cats) {
            if (this._cats.hasOwnProperty(b)) {
              var k = this._cats[b].itemCols;
              for (var m in k) {
                if (k.hasOwnProperty(m)) this._finishColumn(k[m], d)
              }
          }
        }
        for (var b in this._dummyCols) {
          if (this._dummyCols.hasOwnProperty(b)) this._finishColumn(this._dummyCols[b], d);
        }
        this._maxCols = f;
        this._widestColumn = d
        }(this._table.clientWidth != this._lastLayoutWidth || this._table.clientHeight != this._lastLayoutHeight) && this._calculateVisibleItemColumns();
        this._restoreDescriptionAndLayout(l);
        this._lastLayoutWidth = this._table.clientWidth;
        this._lastLayoutHeight = this._table.clientHeight
      };
      _WPAdder.prototype._removeItemHover = function () {
        ULSior: ;
        var a = this._getItems();
        for (var b in a) {
          if (a.hasOwnProperty(b)) a[b].removeHover()
        }
      };
      _loadWPAdderCallback();
    }
  }
})

Actually, when we load the WPAdder class, then I override the two functions that are buggy in adding the hasOwnProperty. That’s it.