Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

Fix call to iterate in removeNode. #112

Merged
merged 1 commit into from

2 participants

@ingemaradahl

I had some problems with removeNode throwing exceptions at me, and this seems to fix that. However, this is the first time I've ever written something in CoffeeScript (or ever actually looked at, for that matter), so you probably want to review this one line fix though :P

@mbraak mbraak merged commit 0078b40 into mbraak:master
@mbraak
Owner

Thanks a lot! This is indeed an error.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Jan 27, 2013
  1. @ingemaradahl
This page is out of date. Refresh to see the latest.
Showing with 2 additions and 2 deletions.
  1. +1 −1  tree.jquery.coffee
  2. +1 −1  tree.jquery.js
View
2  tree.jquery.coffee
@@ -627,7 +627,7 @@ class JqTreeWidget extends MouseWidget
else
result = true
- iterate(
+ @tree.iterate(
(child) =>
if node == child
result = true
View
2  tree.jquery.js
@@ -994,7 +994,7 @@ limitations under the License.
return true;
} else {
result = true;
- iterate(function(child) {
+ _this.tree.iterate(function(child) {
if (node === child) {
result = true;
return false;
Something went wrong with that request. Please try again.