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

Предложение. Создать дополнительное поле Код. #50

Open
tsukasa-mixer opened this issue Aug 20, 2015 · 9 comments

Comments

@tsukasa-mixer
Copy link

Суть.
Добавляем новый типо поля Код, и добавляем его в часто используемые модели, по которым происходит выборка например для блоков.

При заполнения поля код оно записывается в таблицу и в пределах системы этот код должен быть уникальным.

Пишется хелпер которым в который просто передаем код и получаем объект привязанный к нему.

НА правах бреда.

@max107
Copy link

max107 commented Aug 20, 2015

ContentType?

@tsukasa-mixer
Copy link
Author

all
возвращаем модель или набор моделей

@max107
Copy link

max107 commented Aug 21, 2015

Не понимаю о чем идет речь. Дай пример абстрактным кодом.

@tsukasa-mixer
Copy link
Author

code model pk
IDX_TOP_TEXT TextBlock 1
IDX_CATALOG_CATEGORY Catalog 24

$model = helper::getByCode('IDX_TOP_TEXT'); // return TextBlock::objects()->find(['id' => 1])->get();

@max107
Copy link

max107 commented Mar 24, 2016

По сути тут идет речь о content-type аналогией из Django. @qantus есть мысли?

@qantus
Copy link
Contributor

qantus commented Mar 24, 2016

@tsukasa-mixer не понял ничего. Как ты планируешь задавать этот самый код? И что планируешь получать? Давай конкретный пример для чего это нужно?

@max107 max107 modified the milestone: 2.0 Mar 25, 2016
@tsukasa-mixer
Copy link
Author

tsukasa-mixer commented Apr 15, 2016

@qantus я очень хочу избавится от привязки к ID страниц. это порочная практика.
например в шаблонах тойже гармонии куча подобных вещей
{% set pages = get_pages(2, 3, 0, ['-published_at']) %}
{% set pages = get_pages(10, 3, 0, ['-published_at']) %}
{% set page = get_page(10) %}
{% set page = get_page(2) %}
А вот теперь внимание вопрос, не заглядыва в БД - какие данные тут хочет получить программист ?

Вместо это гораздо читабельней и понятней будет выглядеть выбока по коду PAGE_NEWS, PAGE_ARTICLES, + отпадает необходимость синхронизировать БД.

уже были случаи когда клиент "случайно" удалял нужную страницу и приходилось лезти и править ID

@max107
Copy link

max107 commented Apr 15, 2016

@tsukasa-mixer

Предложения:

  1. Заблокировать возможность удаления критически важных страниц
  2. Вместо страниц использовать Block (блоки) в модуле страниц

@tsukasa-mixer
Copy link
Author

@max107 не поднодит когда ты выводишь связанные с объектом данные.

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

No branches or pull requests

3 participants