Forums / National / Russian / Модули и плагины / [Release] Files Module

#39413 Alex300:

В качестве рекомендации — использовать единую установку для Bootstrap — https://github.com/macik/cot-bootstrap/blob/master/README_ru.md

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

1. Если мы не будем двигаться в сторону создания универсального решения его и не будет.

2. К тому же это не требование, а возможность интеграции с `cot-bootstrap` на уровне рекомендаций. Т.е. пишем `Recommends_plugins=bootstrap`, а в коде  (файл `files.files.tpl`) простая проверка на случай, если в системе установлен «общий» бутстрап (если его нет — ничего не измениться, используем тот что в модуле): 

<!-- IF {PHP.cfg.bootstrap} -->
<link rel="stylesheet" href="{PHP.cot_bootstrap.css_file}?{PHP.cfg.bootstrap}">
<link rel="stylesheet" href="{PHP.cot_bootstrap.theme_file}?{PHP.cfg.bootstrap}">
<!-- ELSE -->
<link rel="stylesheet" href="{PHP.cfg.modules_dir}/files/lib/bootstrap/css/bootstrap.min.css?{PHP.cot_modules.files.version}">
<link rel="stylesheet" href="{PHP.cfg.modules_dir}/files/lib/bootstrap/css/bootstrap-theme.min.css?{PHP.cot_modules.files.version}">
<!-- ENDIF -->

// …skipped

<!-- IF {PHP.cfg.bootstrap} -->
<script src="{PHP.cot_bootstrap.js_file}?{PHP.cfg.bootstrap}"></script>
<!-- ELSE -->
<script src="{PHP.cfg.modules_dir}/files/lib/bootstrap/js/bootstrap.min.js?{PHP.cot_modules.files.version}"></script>
<!-- ENDIF -->

Исправлено.

Спасибо за баг репорт ))

Проверю чуть позже.

А пока вопрос: 

В коде есть проверка на `COT_HEADER_COMPLETE`, однако нигде не нашел определения этой константы. 

А вот черные иконки мне не очень понравились

Я не предлагаю именно эти, суть в другом — на них крупно написано расширение файла и его срезу видно. А так можно подыскать и цветные.