Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

paramsRewrite вью-коллекции не влияет на параметры деток #632

Open
vitkarpov opened this issue Nov 9, 2016 · 3 comments

Comments

@vitkarpov
Copy link
Member

vitkarpov commented Nov 9, 2016

Есть вью-коллекция:

ns.View.define('foo', {
  models: ['m'],
  split: {
        byModel: 'foo',
        intoViews: 'foo-item'
  },
  // эта вьюшка зависит неявно от p1, потому что зависит от модели m,
  // в процессе жизни в параметрах страницы появляются разные p1,
  // но хочется, чтобы эта вьюшка смотрела только на экземпляр m с параметром p1 == '/feed'
  // и сама была в единственном экземпляре (лежит в боксе), поэтому определим p1 явно
  paramsRewrite(params) {
      params.p1 = '/feed';
      return params;
  }
})

есть детки:

ns.View.define('foo-item', {
  models: ['m']
})

Детки так же зависят от модели m и, соответственно, имеют параметр p1, однако, он будет браться из параметров страницы, а не из параметров вью-коллекции (которые переписаны) — это кажется, странным, приходится писать paramsRewrite в двух декларациях: вью-коллекции и деток

@vitkarpov
Copy link
Member Author

@chestozo похоже на баг?

@vitkarpov
Copy link
Member Author

vitkarpov commented Nov 9, 2016

Вообще, конечно, странно, что появляется желание «заставить» вьюшку смотреть всегда на модельку с одним и тем же параметром: параметры страницы меняются, другие вьюшки получают другие экземпляры, а вот эта непременно должна смотреть всегда на один и тот же.

Согласен с тем, что здесь есть архитектурная кривизна :(

@chestozo
Copy link
Member

chestozo commented Nov 9, 2016

Давай голосом реальный кейс обсудим )

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants