Clean up notes when removing a lane

Related to #13.

Author: bebraw

immutable_app/app/components/Lane.jsx

@@ -38,7 +38,7 @@ class Lane extends React.Component {
  value={lane.get('name')}
  onEdit={name => props.updateLane({id: laneId, name, editing: false})} />
  <div className="lane-delete">
- <button onClick={this.deleteLane.bind(this, laneId)}>x</button>
+ <button onClick={this.deleteLane.bind(this, lane)}>x</button>
  </div>
  </div>
  <Notes
@@ -49,9 +49,17 @@ class Lane extends React.Component {
  </div>
  );
  }
- deleteLane(laneId, e) {
+ deleteLane(lane, e) {
  e.stopPropagation();
 
+ const laneId = lane.get('id');
+
+ // Clean up notes
+ lane.get('notes').forEach(noteId => {
+ this.props.detachFromLane(laneId, noteId);
+ this.props.deleteNote(noteId);
+ });
+
  this.props.deleteLane(laneId);
  }
  addNote(laneId, e) {

kanban_app/app/components/Lane.jsx

@@ -38,7 +38,7 @@ class Lane extends React.Component {
  value={lane.name}
  onEdit={name => props.updateLane({id: laneId, name, editing: false})} />
  <div className="lane-delete">
- <button onClick={this.deleteLane.bind(this, laneId)}>x</button>
+ <button onClick={this.deleteLane.bind(this, lane)}>x</button>
  </div>
  </div>
  <Notes
@@ -49,9 +49,17 @@ class Lane extends React.Component {
  </div>
  );
  }
- deleteLane(laneId, e) {
+ deleteLane(lane, e) {
  e.stopPropagation();
 
+ const laneId = lane.id;
+
+ // Clean up notes
+ lane.notes.forEach(noteId => {
+ this.props.detachFromLane(laneId, noteId);
+ this.props.deleteNote(noteId);
+ });
+
  this.props.deleteLane(laneId);
  }
  addNote(laneId, e) {