Обсуждение модуля:Transclude

Материал из Википедии — свободной энциклопедии
Перейти к навигации Перейти к поиску

Разделение модуля[править код]

Участник putnik предложил запрячь {{#invoke:Transclude|npc|Строка таблицы|…}} для карточек, см. Шаблон:Карточка/Temp. Это вызовет не только огромную нагрузку на модуль, но и применение защиты, и рекомендации править пореже. В то же время нетрудно заметить, что функции настоящего модуля не имеют никакой взаимозависимости. А потому, не лучше ли будет отделить тяжело используемые функции (пока имеется в виду одна лишь M.npc) в отдельный модуль, защищённый и более тщательно поддерживаемый? Код некоторых из остальных функций, мягко говоря, весьма далёк от завершённости. Да и, независимо от мер защиты, не следует опирать широко используемую функциональность на ПО стадии альфа, тем более если куски находящиеся в стадии альфа технически не связано с тем, на что предлагается опереться. Incnis Mrsi (обс.) 08:49, 15 мая 2014 (UTC)[ответить]

  • Честно говоря, удивительно, что модуль вообще работает, — я его толком не отлаживал. Где он актуально задействован? Конечно, если тормозит, надо разделять (хотя я не уверен, что тормоза лезут именно отсюда). Ignatus 10:52, 15 мая 2014 (UTC)[ответить]
    Модуль не тормозит, проблема в другом. Техник putnik, незнамо как проведавший об этом модуле, собирается опереть миллион статей (посредством {{карточка}}) на вашу функцию M.npc. Я предложил, дабы не создавать взаимопроблем, вытащить функцию в отдельный модуль. Если никто не возражает, то так и поступим (с указанием авторства, разумеется), и будете спокойно дорабатывать остальные функции. Кому какая разница, какое слово при вызове npc писать после «#invoke:»? А поддержка кода упростится, т.к. тут не будет защит, а там — риска поломок и backlog обновления страниц с карточками после каждого обновления модуля. Incnis Mrsi (обс.) 15:00, 15 мая 2014 (UTC)[ответить]
    • Ну, не я предложил, а Seryo93 (и, видимо, с оглядкой на {{карточка/lua}} от автора модуля). В целом смысла в разделении модулей особо не вижу. Но если в данном случае остальные функции не используются, то можно и сделать отдельный модуль. — putnik 16:43, 15 мая 2014 (UTC)[ответить]

@Incnis Mrsi, Ignatus, putnik: я вынес (скопировал) функцию npc в отдельный Модуль:Карточка/Вставка. Полагаю, теперь шаблон можно сравнительно безболезненно обновить? Seryo93 (о.) 11:00, 24 июня 2014 (UTC)[ответить]