Merge pull request #556 from wenzhixin/fix/jquery

Replace jQuery utility methods with native JavaScript

Author: wenzhixin

assets/js/index.js

@@ -1,4 +1,4 @@
-const isDebug = ['localhost', '127.0.0.1'].indexOf(location.hostname) > -1
+const isDebug = ['localhost', '127.0.0.1'].includes(location.hostname)
 const { computed, createApp, onMounted, ref } = window.Vue
 
 const Utils = {
@@ -74,7 +74,7 @@ const Utils = {
     $('.view-example, .view-source').off('click').click(function () {
       if (isSource) {
         location.hash = location.hash.replace('#view-source', '')
-      } else if (location.hash.indexOf('view-source') === -1) {
+      } else if (!location.hash.includes('view-source')) {
         location.hash += '#view-source'
       }
     })

assets/js/template.js

@@ -1,6 +1,6 @@
 window._config = {
   isDebug: location.hash.slice(1) === 'is-debug' ||
-    ['localhost', '127.0.0.1', 'dev.bootstrap-table.com'].indexOf(location.hostname) > -1,
+    ['localhost', '127.0.0.1', 'dev.bootstrap-table.com'].includes(location.hostname),
   cdnUrl: 'https://cdn.jsdelivr.net/npm/bootstrap-table@1.26.0/dist/',
   localUrl: '../bootstrap-table/src/',
   testUrl: '/src/'
@@ -169,9 +169,9 @@ function _beautifySource (data) {
     }
   }
 
-  result = result.concat($.map(obj.links, _getLink))
+  result = result.concat(obj.links.map(_getLink))
   addEmptyLine()
-  result = result.concat($.map(obj.scripts, function (script) {
+  result = result.concat(obj.scripts.map(function (script) {
     return _getScript(script, true)
   }))
   addEmptyLine()
@@ -282,7 +282,7 @@ window.init = function (options_) {
   if ($('.bd-lead').length) {
     $('.bd-lead').html(window.marked(options.desc)).find('a').attr('target', '_blank')
   }
-  $.each(options.links, function (i, file) {
+  options.links.forEach(function (file) {
     _link(file)
   })
   _scripts(options.scripts, options.callback)

extensions/custom-view.html

@@ -219,7 +219,7 @@ <h3 class="mb-0">%SNIPPETS%</h3>
     const template = $('#profileTemplate').html()
     let view = ''
 
-    $.each(data, function (i, row) {
+    data.forEach(function (row) {
       view += template.replace('%NAME%', row.name)
         .replace('%IMAGE%', row.image)
         .replace('%FOLLOWER%', row.follower)

issues/395.html

@@ -52,7 +52,7 @@
       $remove.prop('disabled', !$table.bootstrapTable('getSelections').length)
     })
     $remove.click(function () {
-      const ids = $.map($table.bootstrapTable('getSelections'), function (row) {
+      const ids = $table.bootstrapTable('getSelections').map(function (row) {
         return row.id
       })
 

issues/917.html

@@ -44,8 +44,8 @@
   let selections = []
 
   window.responseHandler = res => {
-    $.each(res.rows, function (i, row) {
-      row.state = $.inArray(row.id, selections) !== -1
+    res.rows.forEach(function (row) {
+      row.state = selections.includes(row.id)
     })
     return res
   }
@@ -59,11 +59,11 @@
           rows = rowsBefore
         }
 
-        const ids = $.map(!$.isArray(rows) ? [rows] : rows, function (row) {
+        const ids = (!Array.isArray(rows) ? [rows] : rows).map(function (row) {
           return row.id
         })
 
-        const func = $.inArray(e.type, ['check', 'check-all']) > -1 ? 'union' : 'difference'
+        const func = ['check', 'check-all'].includes(e.type) ? 'union' : 'difference'
 
         selections = window._[func](selections, ids)
       })

methods/expand-collapse-all-rows.html

@@ -64,9 +64,9 @@
   window.detailFormatter = (index, row) => {
     const html = []
 
-    $.each(row, function (key, value) {
-      html.push(`<p><b>${key}:</b> ${value}</p>`)
-    })
+    for (const key in row) {
+      html.push(`<p><b>${key}:</b> ${row[key]}</p>`)
+    }
     return html.join('')
   }
 </script>

methods/expand-collapse-row-by-uniqueid.html

@@ -66,9 +66,9 @@
   window.detailFormatter = (index, row) => {
     const html = []
 
-    $.each(row, function (key, value) {
-      html.push(`<p><b>${key}:</b> ${value}</p>`)
-    })
+    for (const key in row) {
+      html.push(`<p><b>${key}:</b> ${row[key]}</p>`)
+    }
     return html.join('')
   }
 </script>

methods/expand-collapse-row.html

@@ -64,9 +64,9 @@
   window.detailFormatter = (index, row) => {
     const html = []
 
-    $.each(row, function (key, value) {
-      html.push(`<p><b>${key}:</b> ${value}</p>`)
-    })
+    for (const key in row) {
+      html.push(`<p><b>${key}:</b> ${row[key]}</p>`)
+    }
     return html.join('')
   }
 </script>

methods/remove.html

@@ -56,7 +56,7 @@
 
   function mounted () {
     $button.click(function () {
-      const ids = $.map($table.bootstrapTable('getSelections'), function (row) {
+      const ids = $table.bootstrapTable('getSelections').map(function (row) {
         return row.id
       })
 

methods/toggle-detail-view.html

@@ -56,9 +56,9 @@
   window.detailFormatter = (index, row) => {
     const html = []
 
-    $.each(row, function (key, value) {
-      html.push(`<p><b>${key}:</b> ${value}</p>`)
-    })
+    for (const key in row) {
+      html.push(`<p><b>${key}:</b> ${row[key]}</p>`)
+    }
     return html.join('')
   }
 </script>