Skip to content

Commit 2986f82

Browse files
author
Jacob Wenger
committed
Merge pull request #56 from timotius02/master
fix bind warning in todoApp example
2 parents ce7e9f5 + c66a926 commit 2986f82

File tree

2 files changed

+2
-4
lines changed

2 files changed

+2
-4
lines changed

examples/todoApp/js/todoAppFirebaseExplicit.js

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ var TodoList2 = React.createClass({
66
return (
77
<li key={ index }>
88
{ item.text }
9-
<span onClick={ _this.props.removeItem.bind(_this, item['.key']) }
9+
<span onClick={ _this.props.removeItem.bind(null, item['.key']) }
1010
style={{ color: 'red', marginLeft: '10px', cursor: 'pointer' }}>
1111
X
1212
</span>
@@ -50,7 +50,6 @@ var TodoApp2 = React.createClass({
5050
},
5151

5252
removeItem: function(key) {
53-
console.log('removeItem()', key);
5453
var firebaseRef = new Firebase('https://ReactFireTodoApp.firebaseio.com/items/');
5554
firebaseRef.child(key).remove();
5655
},

examples/todoApp/js/todoAppFirebaseImplicit.js

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ var TodoList3 = React.createClass({
66
return (
77
<li key={ index }>
88
{ item.text }
9-
<span onClick={ _this.props.removeItem.bind(_this, item['.key']) }
9+
<span onClick={ _this.props.removeItem.bind(null, item['.key']) }
1010
style={{ color: 'red', marginLeft: '10px', cursor: 'pointer' }}>
1111
X
1212
</span>
@@ -37,7 +37,6 @@ var TodoApp3 = React.createClass({
3737
},
3838

3939
removeItem: function(key) {
40-
console.log('removeItem()', key);
4140
var firebaseRef = new Firebase('https://ReactFireTodoApp.firebaseio.com/items/');
4241
firebaseRef.child(key).remove();
4342
},

0 commit comments

Comments
 (0)