AM
Please do it, OK?
Size: a a a
2020 June 05
T
T
Почему в
Например те же
ApplicationParameters не все параметры по умолчанию указаны? может привести к ошибке.Например те же
string $name, string $logo - хоть и задаются в config/web.php, но они не являются обязательными частями конструктора👍
JD
Интересный вопрос...
Думаю отказываться полностью не стоит. Должна быть возможность регистрации различных конфигов под разные страницы
Мб стоит сделать что то вроде AssetManager::register('path/to/config.json')
или
$assetConfig = [
'site' => 'path/to/site_config.json',
'site_custom_page' => 'path/to/site_custom_page.json',
];
$assetManager = new AssetManager;
$assetManager->setConfig($assetConfig);
...
в layout $assetManager->register('site')
и на какой то специальной странице сайта
$assetManager->register('site_custom_page')
Мб стоит сделать что то вроде AssetManager::register('path/to/config.json')
или
$assetConfig = [
'site' => 'path/to/site_config.json',
'site_custom_page' => 'path/to/site_custom_page.json',
];
$assetManager = new AssetManager;
$assetManager->setConfig($assetConfig);
...
в layout $assetManager->register('site')
и на какой то специальной странице сайта
$assetManager->register('site_custom_page')
В
Думаю отказываться полностью не стоит. Должна быть возможность регистрации различных конфигов под разные страницы
Мб стоит сделать что то вроде AssetManager::register('path/to/config.json')
или
$assetConfig = [
'site' => 'path/to/site_config.json',
'site_custom_page' => 'path/to/site_custom_page.json',
];
$assetManager = new AssetManager;
$assetManager->setConfig($assetConfig);
...
в layout $assetManager->register('site')
и на какой то специальной странице сайта
$assetManager->register('site_custom_page')
Мб стоит сделать что то вроде AssetManager::register('path/to/config.json')
или
$assetConfig = [
'site' => 'path/to/site_config.json',
'site_custom_page' => 'path/to/site_custom_page.json',
];
$assetManager = new AssetManager;
$assetManager->setConfig($assetConfig);
...
в layout $assetManager->register('site')
и на какой то специальной странице сайта
$assetManager->register('site_custom_page')
Насколько я понял, там речь о том, чтобы отказаться именно от конфигов в виде классов, перенеся объявление ассетов в общий конфиг. А на нужных страницах, да, по-прежнему вызывать
И да, мне эта идея нравится, голосую "за" 👍 Ведь, по сути, текущие
register, но передавать туда уже ключ из конфига, а не класс Asset.И да, мне эта идея нравится, голосую "за" 👍 Ведь, по сути, текущие
Asset-ы именно конфигами и являются, это небольшой, но оверхед.T
Насколько я понял, там речь о том, чтобы отказаться именно от конфигов в виде классов, перенеся объявление ассетов в общий конфиг. А на нужных страницах, да, по-прежнему вызывать
И да, мне эта идея нравится, голосую "за" 👍 Ведь, по сути, текущие
register, но передавать туда уже ключ из конфига, а не класс Asset.И да, мне эта идея нравится, голосую "за" 👍 Ведь, по сути, текущие
Asset-ы именно конфигами и являются, это небольшой, но оверхед.You should also keep in mind that with assetmanager you can run sass, gulp, etc?
Д
это нормально?
Installing npm dependencies
npm WARN deprecated popper.js@1.16.1: You can find the new Popper v2 at @popperjs/core, this package is dedicated to the legacy v1
npm notice created a lockfile as package-lock.json. You should commit this file.
npm WARN yii3app No repository field.
В
TradersVE
You should also keep in mind that with assetmanager you can run sass, gulp, etc?
I don't mean to exclude any functionality, just to move configuration to array instead of current classes.
AM
это нормально?
Installing npm dependencies
npm WARN deprecated popper.js@1.16.1: You can find the new Popper v2 at @popperjs/core, this package is dedicated to the legacy v1
npm notice created a lockfile as package-lock.json. You should commit this file.
npm WARN yii3app No repository field.
T
это нормально?
Installing npm dependencies
npm WARN deprecated popper.js@1.16.1: You can find the new Popper v2 at @popperjs/core, this package is dedicated to the legacy v1
npm notice created a lockfile as package-lock.json. You should commit this file.
npm WARN yii3app No repository field.
It is necessary to update the dependency.
Д
понял)
T
это нормально?
Installing npm dependencies
npm WARN deprecated popper.js@1.16.1: You can find the new Popper v2 at @popperjs/core, this package is dedicated to the legacy v1
npm notice created a lockfile as package-lock.json. You should commit this file.
npm WARN yii3app No repository field.
package ?
AM
I don't mean to exclude any functionality, just to move configuration to array instead of current classes.
It may make sense. Disadvantage is that IDEs won't be able to analyze it.
T
registerJson() can be added.
AM
TradersVE
registerJson() can be added.
Having two ways of doing it is a bad thing.
T
Having two ways of doing it is a bad thing.
ok
Д
блин.. толи я туплю, толи оно как-то не так поставилось.. у меня в public/ есть только папка css/site.css
Д
а в репе на гитхабе есть и .htaccess и index.php
Д
симлинк vendor/bin/codecept тоже отсутствует