ns.ViewCollection которая зависит от модели коллекции и ещё от какой-то невалидной модели - boom

[chestozo]

Есть модель коллекция типа такой:

ns.ViewCollection.define('mega-view-collection', {
    models: {
        'model-collection': false,
        'model-that-can-be-invalid': false
    },
    split: {
        byModel: 'model-collection',
        intoViews: 'collection-item-view'
    }
});

Шаги:

• первая отрисовка
  • валидная непустая model-collection
  • невалидная model-that-can-be-invalid
  • получаем отрисованную error моду для коллекции
• вторая отрисовка через вызов view.update() у mega-view-collection
  • получаем ошибку типа Error: [ns.View] Can't find node for 'collection-item-view'

Тут либо править проверку внутри ns.ViewCollection, либо не давать ей зависеть от других моделей - т.е. чтобы коллекция отрисоввывала только модель-коллекцию.

[vitkarpov]

Кажись, проще обернуть коллекцию в другую вьюшку с зависимостью от той, другой модели, чем чинить :) Хотя, по логике, конечно, коллекция наследник обычной вьюшки и должна все уметь. ср, 30 авг. 2017 г. в 11:18, chestozo <notifications@github.com>:

…

Есть модель коллекция типа такой: ns.ViewCollection.define('mega-view-collection', { models: { 'model-collection': false, 'model-that-can-be-invalid': false }, split: { byModel: 'model-collection', intoViews: 'collection-item-view' }, }); Шаги: - первая отрисовка - валидная непустая model-collection - невалидная model-that-can-be-invalid - получаем отрисованную error моду для коллекции - вторая отрисовка через вызов view.update() у mega-view-collection - получаем ошибку типа Error: [ns.View] Can't find node for 'collection-item-view' Тут либо править проверку внутри ns.ViewCollection, либо не давать ей зависеть от других моделей ) т.е. чтобы коллекция отрисоввывала только модель-коллекцию. — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#654>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AC3Nv6kOS2b7ti5H01MldDJFsGTU14Ufks5sdRq8gaJpZM4PHBnJ> .

[chestozo]

Я скорее про то, что ничто не мешает в зависимости коллекции добавить ещё моделей и словить багу )

[vitkarpov]

Ну да, нужно варнинг кидать какой-нибудь :) ср, 30 авг. 2017 г. в 11:41, chestozo <notifications@github.com>:

…

Я скорее про то, что ничто не мешает в зависимости коллекции добавить ещё моделей и словить багу ) — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#654 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AC3Nv7-iYWzw0QKVSBW11YI7jIC-3sHqks5sdSBUgaJpZM4PHBnJ> .