Skip to content

Conversation

@JohnJ255
Copy link

В ассет-менеджер я внёс то же самое предложение об использовании $pattern, но в более свежий код. Основывался на таком размышлении: если принято решение в необходимости компрессии, то она всё-равно будет использоваться для всех файлов, либо ни для каких. Поэтому вынес эту настройку в свойство класса (вообще-то это свойство там уже было, но не использовалось). В отдельных ситуациях придётся играться с этим свойством, что бы задать разные условия вклчюения файлов.
В методах этот аргумент заменил на $pattern. На случай глобальной работы с одним паттерном, что бы постоянно его не вводить, предусмотрел публично есвойство класса $activePattern, которое, если установлено, перебивает значения, переданные в методы. Не знаю, подойдёт ли такое решение, так как оно является потенциальным источником ошибок по невнимательности. Но, с другой стороны, его можно вообще не использовать тому, у кого есть такие опасения.

@JohnJ255
Copy link
Author

Странно, что предложилось 5 коммитов, всего должно было быть два последних...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant