Неймспейсы в событиях

[chestozo]

@pasaran, что ты думаешь про неймспейсы в событиях?

Пример использования:
ns.View подписывается на изменения моделей в методе _bindModels:
https://github.com/yandex-ui/noscript/blob/master/src/ns.view.js#L658-L711
_unbindModels пока не работает
https://github.com/yandex-ui/noscript/blob/master/src/ns.view.js#L717
но мог бы (должен?) отписываться от моделей

Так вот, было бы очень удобно сделать так:

// Когда подписываемся указываем namespace (this._uniqID - уникальный идентификатор view)
model.on('ns-model-changed', function() { ... }, this._uniqID);
model.on('ns-model-destroyed', function() { ... }, this._uniqID);

// Когда нужно - отписываем все обработчики по namespace-у.
model.off(null, null, this._uniqID);

Если ты за - я могу сделать PR.

[pasaran]

Мы уже обсуждали это. Кажется еще Артур был. И issue и PR даже был вроде. Я был против.

В ноде вот нет неймспейсов и все хорошо.
То есть я понимаю, где их можно использовать, но не хочу усложнять базовый код. У меня пока не было примеров, когда это реально было бы нужно. Кажется, можно как-то заэкстендить no.Events и добавить туда какой-нибудь onNs, triggerNs ...

Sergey Nikitin
Sent with Sparrow (http://www.sparrowmailapp.com/?sig)

On Friday, 17 January 2014 г. at 21:28, chestozo wrote:

@pasaran (https://github.com/pasaran), что ты думаешь про неймспейсы в событиях?
Пример использования:
ns.View подписывается на изменения моделей в методе _bindModels:
https://github.com/yandex-ui/noscript/blob/master/src/ns.view.js#L658-L711
_unbindModels пока не работает
https://github.com/yandex-ui/noscript/blob/master/src/ns.view.js#L717
но мог бы (должен?) отписывать от моделей
Так вот, было бы очень удобно сделать так:
// Когда подписываемся указываем namespace (this._uniqID - уникальный идентификатор view) model.on('ns-model-destroyed', function() { ... }, this._uniqID); // Когда нужно - отписываем все обработчики по namespace-у. model.off(null, null, this._uniqID);

Если ты за - я могу сделать PR.

—
Reply to this email directly or view it on GitHub (#22).

[chestozo]

Эх )

Напишу сюда, потому что может быть ты посчитаешь что это ок:

var foo = no.extend( {}, no.Events );
var calls = '';
var handler = function() { calls += 'a'; };

foo.on('foo', handler);
foo.on('foo', handler);
foo.trigger('foo');
expect(calls).to.be('aa'); // ok

foo.off('foo', handler);
foo.trigger('foo');
expect(calls).to.be('aa'); // fail: calls === 'aaa' - handler удалился один только

[pasaran]

Это в смысле, что если повесить несколько одинаковых хэндлеров, то удалится только один?
Нуууу. Это, наверное, баг.

Только непонятно в чем именно. Нужно ли позволять вешать повторно один и тот же хэндлер?
Или же позволять, но и удалять все.

[pasaran]

Там просто ищется хэндлер через indexOf и удаляется первый найденный.
Я так просто не делаю никогда, поэтому и не парился.