Skip to content
This repository has been archived by the owner. It is now read-only.

Навести порядок в модулях #809

Closed
ivnish opened this issue Nov 11, 2017 · 15 comments

Comments

@ivnish
Copy link
Member

commented Nov 11, 2017

Необходимо разобраться с модулями:

  1. Модули из каталога sites/all/modules/contrib перенести в каталог sites/all/modules
  2. Модули из каталога sites/all/modules/github перенести в каталог profiles/drupalru/modules
  3. Удалить каталоги с модулями, которые сейчас не используются и не будут использоваться в будущем на сайте

Это немного облегчит порог вхождения в разработку, а также немного ускорит поиск файлов, в которых необходимо вносить изменения

@mbaev

This comment has been minimized.

Copy link
Contributor

commented Nov 11, 2017

Да, это моя задача, она немного глубже, но спасибо, что начал эту линию. Я буду заниматься ей, но не знаю когда.

@mbaev mbaev self-assigned this Nov 11, 2017
@mbaev mbaev added the Improvement label Nov 11, 2017
@mbaev

This comment has been minimized.

Copy link
Contributor

commented Dec 3, 2017

Из трёх пунктов верный только третий.

@ivnish

This comment has been minimized.

Copy link
Member Author

commented Dec 3, 2017

Вроде как на орге уже давно отказались от каталога contrib. И модули по умолчанию ставятся именно в sites/all/modules
Каталог sites/all/modules/github тоже какая-то архаичность старого д.ру

Напиши подробнее почему ты не согласен с первыми пунктами

@mbaev

This comment has been minimized.

Copy link
Contributor

commented Dec 3, 2017

Нет, не отказались и многие компании, из тех, что пишут модули для своего проекта, именно так используют распределение модулей по папкам.
https://www.drupal.org/node/2621480

[Optional] /sites/all/modules/contrib: Often Drupal users separate in this sub-folder the contrib-modules. That means modules which are officially released as a full project on Drupal.org.

Папка GitHub тоже имеет место быть, для понимания того, откуда взялся модуль и можно ли его править, т.к. модули оттуда не относятся ни к contrib ни к custom модулям.

@mbaev

This comment has been minimized.

Copy link
Contributor

commented Dec 3, 2017

Тем не менее я согласен с тем, что их нужно объединить и положить в единое место, например, sites/all/modules и удалить неиспользуемые.

@ivnish

This comment has been minimized.

Copy link
Member Author

commented Dec 3, 2017

Ты меня запутал :)

@ivnish

This comment has been minimized.

Copy link
Member Author

commented Dec 3, 2017

Распиши также по пунктам, что ты будешь делать и что куда переносить

@mbaev

This comment has been minimized.

Copy link
Contributor

commented Dec 3, 2017

Я буду

  • переносить модули из папки нашего профиля profiles/drupalru/modules в sites/all/modules. Contrib модули в папку contrib, custom'ные модули в папку custom, модули скачанные с GitHub в папку github.
  • Удалю неиспользуемые модули.

Так понятно?

@ivnish

This comment has been minimized.

Copy link
Member Author

commented Dec 3, 2017

Да, так понятно :)

@mbaev

This comment has been minimized.

Copy link
Contributor

commented Dec 3, 2017

Отлично! 👍

@ivnish

This comment has been minimized.

Copy link
Member Author

commented Dec 3, 2017

Я просто думал, что в github лежат тоже модули д.ру. altpager, alttracker, drupal_deploy... Почему бы их тоже в каталог custom не перенести?

@mbaev

This comment has been minimized.

Copy link
Contributor

commented Dec 3, 2017

Нет, там лежат модули, которых нет на орге, но они хостятся на гитхабе. Их, как и контрибные модули, нужно править через тикеты, патчи, ПРы.

@mbaev

This comment has been minimized.

Copy link
Contributor

commented Dec 3, 2017

Это отдельные прокеты, не drupal.ru

@ivnish

This comment has been minimized.

Copy link
Member Author

commented Dec 3, 2017

А, ну тогда конечно не нужно их путать с custom. Спасибо, что разъяснил :)

@ivnish

This comment has been minimized.

Copy link
Member Author

commented Feb 14, 2018

Готово!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
2 participants
You can’t perform that action at this time.