webarchitect609 / bitrix-cache Goto Github PK

Добавить поддержку тегов в AntiStampedeCacheAdapter и покрыть Unit-тестами.

Можно устанавливать path по имени класса, где используется кеш, автоматически. Например, если кешируется что-то внутри класса \Foo\Bar\Baz, то можно преобразовать это к path = /foo/bar/baz или с сохранением регистра /Foo/Bar/Baz

Импортировать команду check:all из composer-library-skeleton

Добрый день! Благодарю за библиотеку, очень полезная! Скажите как обстоят дела с поддержкой PHP 8.1, массово переезжаем с битриксом и есть проблемы с поддержкой данной библиотеки.

С уважением,
Eshi

Добавить метод \WebArch\BitrixCache\CacheItem::addIblockTag(), чтобы удобно устанавливать тег инфоблока, как это уже реализовано в \WebArch\BitrixCache\Cache::addIblockTag().

Возможно, рациональнее будет выделить функционал генерации тегов в отдельную сущность или даже библиотеку. Сейчас это будет тег инфоблока, но в будущем можно ещё какие-нибудь теги добавить. Можно назвать как-нибудь TagBuilder.

Добрый день. Вот эта строчка:

$this->maxIdLength = $this->getMaxIdLengthByEngineType(BitrixCache::getCacheEngineType());

при типе cacheenginefiles всегда (как и задумано, исходя из кода) возвращает null. Потом соответственно вываливается с Exception по длине. По моему, это не здорово. Если склонировать в свое пространство имен и убрать !is_numeric($this->maxIdLength), все становится OK.

Еще недостатки. Мой замысел - найти (не писать самому) кэшер для Битрикса, реализующий CacheItemPoolInterface (нужен для использования в связке Guzzle - middleware, Bitrix и сервис-контейнер Symfony). Если поправить вышеуказанное, то AntiStampedeCacheAdapter годится. Но не хватает возможности задавать сеттерами TTL кэша (сейчас он намертво прописывается из конструктора). Path тоже было бы неплохо (чтобы не плодить симфонические сервисы на каждый чих). Или, хотя бы, сделать protected соответствующие свойства. Т.е. речь идет о некоей гибкости конфигурирования конструкции.

Вообще было бы здорово, реализовать версию Cache (а не только AntiStampedeCacheAdapter), реализующую CacheItemPoolInterface. Был бы за такое очень признателен. В целом, отличная работа.

Для Memcache существует ограничение на длину id кеша в 250 символов. Для остальных типов кеша, поддерживаемых Битриксом, таких ограничений нет. Необходимо, чтобы \WebArch\BitrixCache\Cache производил соответствующую проверку и предотвращал указание слишком длинного id кеша, если используется Memcache.

Кэш поддерживает addIblockTag() но возможности так же почистить по ID инфоблока не даёт. Нужно либо руками собирать тэг для инфоблока, либо дёргать внутренности легаси битрикса, типа CIBlockElement::clearIblockTagCache().

Было бы удобно иметь возможность почистить кэш по инфоблоку.

Сейчас, кеш по тегу возможен только в том случае, если какой-либо тег был задан до callback, что может быть неудобно, когда ключи для тега могут быть известны только в замыкании.