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

Мысли, пожелания по модулю #1

Open
pchelk1n opened this issue Jan 23, 2013 · 13 comments
Open

Мысли, пожелания по модулю #1

pchelk1n opened this issue Jan 23, 2013 · 13 comments
Assignees

Comments

@pchelk1n
Copy link

Есть модуль https://github.com/kloopko/kohana-pagination
Не хватает переопределения шаблона с использованием другого шаблонизатора.
Что еще ?

@ghost ghost assigned biakaveron, aktuba and pchelk1n Jan 23, 2013
@aktuba
Copy link

aktuba commented Jan 23, 2013

Раскрой мысль, plz. Что значит "переопределения шаблона с использованием другого шаблонизатора"?
По мне, если используется шаблонизатор, он должен подменять собой класс View. Иначе, при смене шаблонизатора на готовом проекте придется весь код править.

@pchelk1n
Copy link
Author

Согласен. Но для тех кто не переопределяет View, такой функционал был бы полезен. Правда пока еще не думал над реализацией.

@biakaveron
Copy link
Member

Вероятно, таким людям следует просто переопределить метод Pagination::render() под свои нужды. Для бОльшего удобства можно вынести все, что завязано на рендеринг шаблона, в отдельный метод _render(). То есть в render() останутся проверки, общие для любого шаблонизатора, а в _render() можно будет переопределить чисто свои хотелки.

@pchelk1n
Copy link
Author

Ага согласен. Чего тогда не хватает в том модуле ?

@pchelk1n
Copy link
Author

Посмотрел пагинацию в Yii. В шаблонах используют вывод через виджет.

@biakaveron
Copy link
Member

Это простой, несложный модуль. Не надо к нему прикручивать никаких лишних наворотов. ИМХО, на данный момент вполне достаточно довести до ума kohana-pagination (версии для 3.2 и 3.3), актуализировать стандартные шаблоны для пагинатора (взять за основу самые популярные + какие-нить привычные из соцсетей + подсмотреть в других фреймах).

Сперва нужна основа - просто работающий модуль. Далее будем собирать фидбеки, нужно будет что-то допилить - пользователи сами скажут об этом.

@Fer0x
Copy link

Fer0x commented Feb 11, 2013

Приветствую. Касаемо Pagination обычно все что включает релизация этого модуля - вывод самой пагинации. Ни у кого не было идеи чтобы этот модуль так же выполнял работу с данными? Т.е. на входе получал array($orm, $per_page, $current_page) и возвращал array($results, $total_pages)

@Fer0x
Copy link

Fer0x commented Feb 11, 2013

Еще один момент, на текущий момент чтобы kohana-pagination корректно работал, необходимо чтобы для был отдельный роут для текущего контроллера. В админке с общем роутом под неё, kohana-pagination берет только первый сегмент url - /admin/.

@biakaveron
Copy link
Member

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

Зачем завязываться на определенный модуль (который еще и не является обязательным)? Тогда уж пусть модель ORM подготавливает данные для пагинатора.

Еще один момент, на текущий момент чтобы kohana-pagination корректно работал, необходимо чтобы для был отдельный роут для текущего контроллера.

Можно поподробнее? А лучше - в отдельный тикет.

@Fer0x
Copy link

Fer0x commented Feb 11, 2013

Да, на счет ORM согласен.

Еще предложение ввести число максимального кол-во дополнений, для того чтобы на последнюю страницу не приходилось 1-2 результата. Подробнее тут: http://habrahabr.ru/qa/3198/

@biakaveron
Copy link
Member

Идею понял. Тикет, please ))

@Fer0x
Copy link

Fer0x commented Feb 12, 2013

Зачем завязываться на определенный модуль (который еще и не является обязательным)? Тогда уж пусть модель ORM подготавливает данные для пагинатора.
И все таки по поводу этой штуки. Как это лучше организовать, как отдельный класс наследующий ORM в рамках модуля?

@biakaveron
Copy link
Member

Можно сделать абстрактную модель ORM, от нее наследовать свои Model_Article, Model_News и тд

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

No branches or pull requests

4 participants