Блог

Недавно я интегрировал Webpack в свой проект для создания пакетов для файлов JS на моем веб-сайте. После исправления нескольких небольших проблем я смог создать пакет. При проверке в браузере один из кодов Javascript выдал приведенную ниже ошибку.

После проверки я понял, что проблема была вызвана AddComment.moveform.

Итак, я проверил сгенерированный пакет и понял, что определение переменной AddComment не было перенесено в пакет. Есть ли причина, по которой Javascript, написанный как показано ниже, не будет включен в пакет? Без того, чтобы Webpack выдавал ошибку по этому поводу?

  /**
 * 'Comment Reply' to each comment.
 * This script moves the Add Comment section to the position below the appropriate comment.
 * Modified from WordPress https://core.svn.wordpress.org/trunk/wp-includes/js/comment-reply.js
 * Released under the GNU General Public License - https://wordpress.org/about/gpl/
 */
var addComment = {
  moveForm: function(commId, parentId, respondId, postId) {
    var div,
      element,
      style,
      cssHidden,
      t = this,
      comm = t.I(commId),
      respond = t.I(respondId),
      cancel = t.I("cancel-comment-reply-link"),
      parent = t.I("comment-replying-to"),
      post = t.I("comment-post-slug"),
      commentForm = respond.getElementsByTagName("form")[0];

    if (!comm || !respond || !cancel || !parent || !commentForm) {
      return;
    }

    t.respondId = respondId;
    postId = postId || false;

    if (!t.I("sm-temp-form-div")) {
      div = document.createElement("div");
      div.id = "sm-temp-form-div";
      div.style.display = "none";
      respond.parentNode.insertBefore(div, respond);
    }

    comm.parentNode.insertBefore(respond, comm.nextSibling);
    if (post amp;amp; postId) {
      post.value = postId;
    }
    parent.value = parentId;
    cancel.style.display = "";

    cancel.onclick = function() {
      var t = addComment,
        temp = t.I("sm-temp-form-div"),
        respond = t.I(t.respondId);

      if (!temp || !respond) {
        return;
      }

      t.I("comment-replying-to").value = "0";
      temp.parentNode.insertBefore(respond, temp);
      temp.parentNode.removeChild(temp);
      this.style.display = "none";
      this.onclick = null;
      return false;
    };

    /*
     * Set initial focus to the first form focusable element.
     */
    document.getElementById("comment-form-message").focus();

    /*
     * Return false so that the page is not redirected to HREF.
     */
    return false;
  },

  I: function(id) {
    return document.getElementById(id);
  }
};
 

Вероятно, это связано с устранением мертвого кода, известного как встряхивание дерева: если Webpack замечает, что определенная функция не используется, он просто оставляет ее для создания меньшего пакета. Но Webpack знает только о функциях, вызываемых из JavaScript, а не из ваших обработчиков событий, которые жестко запрограммированы в HTML.

Самый простой способ исправить это — играть по правилам Webpack и вместо этого подключать обработчики событий через JavaScript. В любом случае, это лучшая практика по разным причинам.

Если вам нужно передать данные в обработчик событий (как вы делаете здесь), вы можете использовать атрибуты данных для элемента и считывать данные в своей функции обработчика событий.