Merge pull request #343 from ocxers/master

Added the `onBeforeCardDelete` function

Author: dapi

README.md

@@ -139,6 +139,7 @@ This is the container component that encapsulates the lanes and cards
 | onDataChange        | function | Called everytime the data changes due to user interaction or event bus: `onDataChange(newData)`                                |
 | onCardClick         | function | Called when a card is clicked: `onCardClick(cardId, metadata, laneId)`                                                         |
 | onCardAdd           | function | Called when a new card is added: `onCardAdd(card, laneId)`                                                                     |
+| onBeforeCardDelete  | function | Called before delete a card, please call the `callback()` if confirm to delete a card: `onConfirmCardDelete(callback)`    
 | onCardDelete        | function | Called when a card is deleted: `onCardDelete(cardId, laneId)`                                                                  |
 | onCardMoveAcrossLanes        | function | Called when a card is moved across lanes `onCardMoveAcrossLanes(fromLaneId, toLaneId, cardId, index)`                                                                  |
 | onLaneAdd           | function | Called when a new lane is added: `onLaneAdd(params)`                                                                     |

src/controllers/BoardContainer.js

@@ -119,6 +119,7 @@ class BoardContainer extends Component {
       onDataChange,
       onCardAdd,
       onCardClick,
+      onBeforeCardDelete,
       onCardDelete,
       onLaneScroll,
       onLaneClick,
@@ -141,6 +142,7 @@ class BoardContainer extends Component {
       'onLaneDelete',
       'onLaneUpdate',
       'onCardClick',
+      'onBeforeCardDelete',
       'onCardDelete',
       'onCardAdd',
       'onLaneClick',
@@ -216,6 +218,7 @@ BoardContainer.propTypes = {
   eventBusHandle: PropTypes.func,
   onLaneScroll: PropTypes.func,
   onCardClick: PropTypes.func,
+  onBeforeCardDelete: PropTypes.func,
   onCardDelete: PropTypes.func,
   onCardAdd: PropTypes.func,
   onLaneAdd: PropTypes.func,

src/controllers/Lane.js

@@ -69,8 +69,15 @@ class Lane extends Component {
   }
 
   removeCard = cardId => {
-    this.props.onCardDelete && this.props.onCardDelete(cardId, this.props.id)
-    this.props.actions.removeCard({laneId: this.props.id, cardId: cardId})
+    if (this.props.onBeforeCardDelete && typeof this.props.onBeforeCardDelete === 'function') {
+      this.props.onBeforeCardDelete(() => {
+        this.props.onCardDelete && this.props.onCardDelete(cardId, this.props.id)
+        this.props.actions.removeCard({laneId: this.props.id, cardId: cardId})
+      })
+    } else {
+      this.props.onCardDelete && this.props.onCardDelete(cardId, this.props.id)
+      this.props.actions.removeCard({laneId: this.props.id, cardId: cardId})
+    }
   }
 
   handleCardClick = (e, card) => {
@@ -237,6 +244,7 @@ class Lane extends Component {
       onLaneScroll,
       onCardClick,
       onCardAdd,
+      onBeforeCardDelete,
       onCardDelete,
       onLaneDelete,
       onLaneUpdate,
@@ -281,6 +289,7 @@ Lane.propTypes = {
   droppable: PropTypes.bool,
   onCardMoveAcrossLanes: PropTypes.func,
   onCardClick: PropTypes.func,
+  onBeforeCardDelete: PropTypes.func,
   onCardDelete: PropTypes.func,
   onCardAdd: PropTypes.func,
   onLaneDelete: PropTypes.func,