TreeMap drill up is broken

[stelian56]

Steps to reproduce:

1. Open dojox/treemap/tests/test.html
2. Select Group by: Mode then State
3. Double click Water to drill down to level 2
4. Double click CA to drill down to level 3
5. Double click CA to drill up to level 2

The tree map jumps all the way to level 1 instead.

Suggested fix to dojox/treemap/TreeMap.js:

refreshRendering: function(){
...

var parentItem = null;
var rootItemRenderer = this._getRenderer(rootItem);
if (rootItemRenderer) {
    parentItem = rootItemRenderer.parentItem;
}
this._buildRenderer(this.domNode, parentItem, rootItem, {

...

[cjolif]

Thanks for reporting this.

I will commit a slightly different fix (but same spirit). For the future stelian56 please make sure to sign a Dojo CLA even for fixes contributions (maybe you have one but not listed as stelian56). See: ​http://dojofoundation.org/about/claCheck and now: ​https://github.com/dojo/dojox/blob/master/CONTRIBUTING.md for more details on how we expect contributions now that we moved to github.

[Christophe Jolif <[email protected]…>]

In 8be82a34bad4f0f4592a38367a1b90fec0ea4de4/dojox:

Error: Processor CommitTicketReference failed

Unsupported version control system "git": Can't find an appropriate component, maybe the corresponding plugin was not enabled? 

[stelian56]

Christophe,

Thank you for fixing the bug.

I have just signed an individual CLA. However, I am reluctant to become a contributor. If you don't mind, I am going to suggest any future bug fixes via bug reports.

Stelian

[cjolif]

Sure. No problem there is no obligation to contribute :)