fix(b-table, b-table-lite): handle edge case with row events when table is removed from dom (fixes #4384)

tmorehouse · web-flow · commit f3334c28e5dc · 2019-11-14T12:16:15.000-04:00
diff --git a/src/components/table/helpers/mixin-tbody.js b/src/components/table/helpers/mixin-tbody.js
@@ -24,13 +24,12 @@ export default {
       // `this.$refs.itemRows` is an array of item TR components/elements
       // Rows should all be B-TR components, but we map to TR elements
       // Also note that `this.$refs.itemRows` may not always be in document order
-      const tbody = this.$refs.tbody.$el || this.$refs.tbody
-      const trs = (this.$refs.itemRows || []).map(tr => tr.$el || tr)
-      // TODO: This may take time for tables many rows, so we may want to cache
-      //       the result of this during each render cycle on a non-reactive
-      //       property. We clear out the cache as each render starts, and
-      //       populate it on first access of this method if null
-      return arrayFrom(tbody.children).filter(tr => arrayIncludes(trs, tr))
+      const refs = this.$refs || {}
+      const tbody = refs.tbody ? refs.tbody.$el || refs.tbody : null
+      const trs = (refs.itemRows || []).map(tr => tr.$el || tr)
+      return tbody && trs.length
+        ? arrayFrom(tbody.children).filter(tr => arrayIncludes(trs, tr))
+        : []
     },
     getTbodyTrIndex(el) {
       // Returns index of a particular TBODY item TR
