Webpack не связывает все функции в моем файле JS - PullRequest
Новая
       15

Webpack не связывает все функции в моем файле JS

1 голос
/ 18 апреля 2019

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

enter image description here

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

enter image description here

Итак, я проверил сгенерированный пакет и понял, что определение переменной 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 && 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);
  }
};

1 Ответ

4 голосов
/ 18 апреля 2019

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

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

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

...