Underscore: 「bindAllには関数名を渡す必要があります」

作成日 2013年03月01日  ·  3コメント  ·  ソース: jashkenas/underscore

v1.4.4より前では、「_。bindAll(this);」を使用していました。 期待どおりに機能し、ドキュメントに従って、「methodNamesが指定されていない場合、オブジェクトのすべての関数プロパティがそれにバインドされます。」

v1.4.4では「_.bindAll(this);」 エラーをスローします-「bindAllには関数名を渡す必要があります」

1.4.3と1.4.4の差分(https://gist.github.com/ahomu/4700664)の107行目で、これが変更されました。

  • if(funcs.length == 0)funcs = _.functions(obj);
  • if(funcs.length === 0)throw new Error( "bindAllには関数名を渡す必要があります");

これは変更ログになく、ドキュメントは古くなっています

興味深いことに、なぜこれが変更されたのですか? 1.4.4を更新しようとすると、私のコードの多くが壊れてしまいました。

question

最も参考になるコメント

少なくともいくつかの下位互換性を尊重していただけませんか?

http://semver.org/を引用する

バージョン番号MAJOR.MINOR.PATCHを指定して、以下をインクリメントします。

  1. 互換性のないAPIを変更した場合のメジャーバージョン、

全てのコメント3件

モーニン@markvr! _.bindAllは最近変更されましたが、バージョン1.4.4には含まれていません。 変更はマスターブランチでのみ行われます。 bf657be243a075b5e72acc8a83e6f12a564d8f55の変更についていくつかの議論があります。 それをチェックして、あなたの考えを私たちに知らせてください。

「bindAll」という名前を付けるのはおそらく誤解を招くでしょう。

少なくともいくつかの下位互換性を尊重していただけませんか?

http://semver.org/を引用する

バージョン番号MAJOR.MINOR.PATCHを指定して、以下をインクリメントします。

  1. 互換性のないAPIを変更した場合のメジャーバージョン、
このページは役に立ちましたか?
0 / 5 - 0 評価

関連する問題

acl0056 picture acl0056  ·  5コメント

arieljake picture arieljake  ·  4コメント

githublyp picture githublyp  ·  3コメント

marcalj picture marcalj  ·  5コメント

ksullivan picture ksullivan  ·  9コメント