# cuHyc (23.12.2019 / 20:05)
А целесообразность одна - я конечно ничего не требую ну раз уж переводите на шаблоны - где гибкость ? Элементарно я сверстаю веб тему и захочу сделать много разных информативных блоков - блок вывода н
Сейчас основная задача переписать всё на шаблоны. Комментарии на сайтах обычно имеют единый стиль. Сейчас можно задать этот единый стиль как хочешь. Добавлю ещё возможность выноса шаблонов в модуль, чтобы можно было разместить прям под страницей файла ну и таким образом сможешь задать особый стиль для какого нить другого модуля.
Но переписывать комменты пока не буду. В следующих версиях перепишем. Там на переписывание много кандидатов, не только комменты. Если я начну всё переписывать, то старый код успеет устареть и его надо будет тоже переписывать гг. Не дождётесь короче новой версии тогда
Щас основная задача всё переписать на шаблоны, а потом уже займёмся написанием новых модулей и переписыванием старых.
# kantry (23.12.2019 / 20:13)
Ну а про что еще спрашивать было,если тип в неё не тот передается?))
function _p(string $singular, string $plural, int $number, string $textDomain = 'default'): string
А в timecount тип float в неё
Ну поправь, скинь пул реквест)
Заодно можешь ещё чё нить поправить
https://scrutinizer-ci.com/g/j ... velop
Simba, Ну я тут не знаю что подойдет))
intdiv() целочисленное деление, но в меньшую сторону округляет))
# kantry (23.12.2019 / 20:36)
Simba, Ну я тут не знаю что подойдет))
intdiv() целочисленное деление, но в меньшую сторону округляет))
Ну оно может и подойдет )
# Simba (23.12.2019 / 20:17)
Ну поправь, скинь пул реквест)
Заодно можешь ещё чё нить поправить https://scrutinizer-ci.com/g/j ... velop
Кстати, насчет Скрутинизера...
Он сцуко упорно не видит константы, хотя они там есть, файл с ними подключается через Composer.
Наверняка какая-то небольшая бяка - и будет видеть.
Тогда штук 20 ложных ахтунгов сразу исправятся.
kantry, Да тупо каст к инту
# Koenig (24.12.2019 / 08:54)
kantry, Да тупо каст к инту
Ну это костыль)) Зачем там int, если деление всегда дает float?
В функции _p() надо тип изменить и все. А еще проще выкинуть вообще эту функцию))
И использовать то что уже есть return $var >= 86400
? $day . ' ' . $translator->translatePlural('Day', 'Days', $day, 'system')
: date('G:i:s', mktime(0, 0, $var));
# kantry (24.12.2019 / 09:29)
А еще проще выкинуть вообще эту функцию))
И использовать то что уже есть
.......
$translator->translatePlural('Day', 'Days', $day, 'system')
......
Можно и так, хелпер _p() используется весьма редко и в тех случаях, когда используется, можно дернуть метод плюрализации непосредственно из зендовского транслятора.
Правда придется перестроить шаблон перевода, чтоб правильно шло автоматическое извлечение фраз из кода. Без этого теряется одно из главных удобств Gettext.
Но все таки более правильно - это передавать именно int.
Выкидывать хэлпер этот нельзя. То, что он сейчас используется только где-то в датах, не значит что он не будет использоваться дальше нигде.
