Skip to content
This repository has been archived by the owner on Mar 25, 2019. It is now read-only.

Рефакторинг мероприятий #1068

Closed
avakorin opened this issue May 9, 2018 · 9 comments
Closed

Рефакторинг мероприятий #1068

avakorin opened this issue May 9, 2018 · 9 comments
Assignees
@avakorin @ivnish
Labels
pre-Drupal8

Comments

@avakorin
Copy link
Contributor

avakorin commented May 9, 2018

В связи с багом #1067 возник вопрос, почему вместо простого вьюса используется кастомный модуль?

Предлагаю переделать мероприятия, используя views. Преимущества очевидны: меньше кастома → меньше проблем с поддержкой, больше возможностей по расширению функциональности, больше возможностей по интеграции с другими контрибами и т. д.
@ivnish
Copy link
Member

ivnish commented May 9, 2018

Там не только представление надо создать, но и новый стандартный тип материала со стандартными полями.

@mbaev
Copy link
Contributor

mbaev commented May 10, 2018

Да, это уже где-то есть в ишьюсах. Views не использовался потому, что с 6ки на 7ку делал миграцию Егор, а его политика заключалась в неиспользовании тяжеловесного модуля Views.
И это имеет смысл, сайт действительно работает быстро, но вот только ресурсов на поддержку нужно много. Поэтому мы с полгода назад уже решили использовать Views, но решили отложить это до 8ки, ибо переписать на Views - это очень большая работа.
Как правильно заметил @itcrowd72 у нас ещё и События с Компаниями написаны не на стандартных филдах. Так что Views с полтычка не заведётся.

@ArmGono
Copy link

ArmGono commented May 16, 2018

Поддерживаю идею переделать на views. Трудозатраты не такие уж и большие. Создать паралельно тип материала с нужными полями, перенести данные, запустить views с полтычка)

@awd-studio
Copy link
Contributor

+ Мигрировать существующие.

@avakorin
Copy link
Contributor Author

Еще один гвоздь: в текущем виде невозможно создать мероприятие, которое пройдет в 2020 году (Друпалкон Миннеаполис 2020). Максимальный год при создании мероприятия в форме установлен +1, то есть сейчас это 2019.

@ivnish ivnish self-assigned this Jul 24, 2018
This was referenced Jul 27, 2018
Merged
Closed
avakorin pushed a commit that referenced this issue Jul 30, 2018
@avakorin
issue #1068: Рефакторинг мероприятий. (#1151)
c795509 
* issue #1068: Рефакторинг мероприятий. Work in progress...

* issue #1068: Рефакторинг мероприятий. Work in progress...

* issue #1068: Рефакторинг мероприятий. Work in progress...

* issue #1068: Рефакторинг мероприятий. Work in progress...

* issue #1068: Рефакторинг мероприятий.

* issue #1068: Рефакторинг мероприятий.

* issue #1068: Рефакторинг мероприятий.

* WIP. Moves code for views to external files and makes some small code corrections. Fixes placing events block. Renames field start_date to date. Renames views and tpl files accordingly.

* Adds deletion of manually created link for Events
@ivnish
Copy link
Member

ivnish commented Jul 30, 2018

Исправлено в #1151

@ivnish ivnish closed this as completed Jul 30, 2018
@ivnish ivnish mentioned this issue Jul 31, 2018
Merged
avakorin pushed a commit that referenced this issue Jul 31, 2018
@avakorin
issue #1068: Рефакторинг мероприятий. (#1169)
0626dbe
@ivnish ivnish mentioned this issue Aug 2, 2018
Closed
@avakorin avakorin reopened this Aug 3, 2018
@avakorin avakorin mentioned this issue Aug 3, 2018
Merged
@avakorin
Copy link
Contributor Author

avakorin commented Aug 3, 2018

БД на dev.drupal.ru была возвращена в исходное состояние в связи с некоторыми доработками в коде, представленными в #1173

avakorin added a commit to avakorin/drupal.ru that referenced this issue Aug 4, 2018
@avakorin
issue DrupalRU#1068: Adds image style to views and less-code for even…
f03ead6 
…ts pages
avakorin added a commit that referenced this issue Aug 6, 2018
@avakorin
issue 1068: Code cleanup and reorder. (#1173)
9ad6dda 
* issue 1068: Code cleanup and reorder.

* issue 1068: Moves importing of views from module to hook_update.

* issue #1068: Adds image style to views and less-code for events pages
@ivnish ivnish closed this as completed Aug 6, 2018
@avakorin
Copy link
Contributor Author

avakorin commented Aug 6, 2018

Осталось удалить модуль simple_events из репозитория

@avakorin avakorin reopened this Aug 6, 2018
@ivnish
Copy link
Member

ivnish commented Aug 13, 2018

Удаление модуля simple_events из репозитория отмечено в #1182

@ivnish ivnish closed this as completed Aug 13, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@avakorin avakorin

@ivnish ivnish

Labels
pre-Drupal8
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@avakorin @ArmGono @mbaev @awd-studio @ivnish