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

Метод генерации урла ns.router.generateUrl #154

Merged
merged 6 commits into from
Sep 12, 2013

Conversation

chestozo
Copy link
Member

@chestozo chestozo commented Sep 5, 2013

ns.router.generateUrl(page, params) и тесты к нему.
/сс @doochik @edoroshenko

@chestozo
Copy link
Member Author

chestozo commented Sep 5, 2013

Will Fix #123

@chestozo
Copy link
Member Author

chestozo commented Sep 5, 2013

ща, попробую схлопнуть всё в один коммит )

@chestozo
Copy link
Member Author

chestozo commented Sep 5, 2013

Готово, можно review-ить / merge-ить

@chestozo
Copy link
Member Author

chestozo commented Sep 5, 2013

Замёрж меня )

@doochik
Copy link
Contributor

doochik commented Sep 10, 2013

Добавил падающий тест

@chestozo
Copy link
Member Author

Ready to merge

@doochik
Copy link
Contributor

doochik commented Sep 11, 2013

Может добавим еще поддержку rewriteUrl ?

Есть роут

routes = {
  rewriteUrl: {
     '/shortcut': '/page/1'
  },
  route: {
     '/page/{id:int}': 'page'
  }

}

Сейчас ns.router.generateUrl('page', {id: 1}) сгенерит /page/1, а хочется /shortcut
По сути надо всего лишь после генерации прогнать урл через обратный хеш от rewriteUrl

@chestozo
Copy link
Member Author

Договорились о следующем:

  1. ns.router.generateUrl(id, params) генерит урл, все параметры, которые не попали в урл сохраняются отдельным списком (см. дальше)
  2. сгенерённый урл прогоняется через rewriteUrl (в цикле, пока не останется rewrite-ов для текущего урла)
  3. к полученному урлу дописывают параметры из п.1 в виде querystring

@chestozo
Copy link
Member Author

Ммм... а у нас ведь ещё есть rewriteParams, его не трогаем? Это ведь функция? )

@chestozo
Copy link
Member Author

И снова всё готово )
Только нужно прояснить вопрос про rewriteParams )

@chestozo
Copy link
Member Author

Откатили и будем дорабатывать в #160

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

Successfully merging this pull request may close these issues.

2 participants