Changeset 3880:e6d1f6d9d7df for admin/js/_post.js

Timestamp:

09/18/18 20:22:10 (7 years ago)

Author:

franck <carnet.franck.paul@…>

Branch:

default

Message:

Use let and const rather than var (ES2015/ES6), use template string where is more efficient

File:

• admin/js/_post.js (modified) (6 diffs)

admin/js/_post.js

@@ -8 +8 @@
   }
 
-  var commentId = $(line).attr('id').substr(1);
-  var tr = document.getElementById('ce' + commentId);
-  var spam = (e.metaKey || $(line).hasClass('sts-junk'));
+  const commentId = $(line).attr('id').substr(1);
+  const lineId = `ce${commentId}`;
+  let tr = document.getElementById(lineId);
+
+  // If meta key down or it's a spam then display content HTML code
+  const clean = (e.metaKey || $(line).hasClass('sts-junk'));
 
   if (!tr) {
@@ -18 +21 @@
         // Content found
         tr = document.createElement('tr');
-        tr.id = 'ce' + commentId;
-        var td = document.createElement('td');
+        tr.id = lineId;
+        const td = document.createElement('td');
         td.colSpan = $(line).children('td').length;
         td.className = 'expand';
@@ -32 +35 @@
     }, {
       ip: false,
-      clean: spam
+      clean: clean
     });
   } else {
@@ -42 +45 @@
 $(function() {
   // Post preview
-  var $preview_url = $('#post-preview').attr('href');
+  let $preview_url = $('#post-preview').attr('href');
   if ($preview_url) {
 
     // Make $preview_url absolute
-    var $a = document.createElement('a');
+    let $a = document.createElement('a');
     $a.href = $('#post-preview').attr('href');
     $preview_url = $a.href;
@@ -78 +81 @@
 
     // Add date picker
-    var post_dtPick = new datePicker($('#post_dt').get(0));
+    const post_dtPick = new datePicker($('#post_dt').get(0));
     post_dtPick.img_top = '1.5em';
     post_dtPick.draw();
@@ -140 +143 @@
     $('a.attachment-remove').click(function() {
       this.href = '';
-      var m_name = $(this).parents('ul').find('li:first>a').attr('title');
+      const m_name = $(this).parents('ul').find('li:first>a').attr('title');
       if (window.confirm(dotclear.msg.confirm_remove_attachment.replace('%s', m_name))) {
         var f = $('#attachment-remove-hide').get(0);