mmikkel\cacheflag\controllers\CachesController::actionInvalidate()

Complexity

Conditions	5
Paths	6

Size

Total Lines	26
Code Lines	16

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function actionInvalidate(): bool
    {
        $flags = Craft::$app->getRequest()->getParam('flags', []);

        if (\is_string($flags)) {
            $flags = \preg_replace('/\s+/', '', $flags);
            $flags = \array_filter(\explode(',', $flags));
        } else if (\is_array($flags)) {
            $flags = \array_reduce($flags, function (array $carry, string $flag) {
                $flag = \preg_replace('/\s+/', '', $flag);
                if (\strlen($flag)) {
                    $carry[] = $flag;
                }
                return $carry;
            }, []);
        }

        if (empty($flags)) {
            CacheFlag::getInstance()->cacheFlag->invalidateAllFlaggedCaches();
            return true;
        }

        $flags = \array_unique($flags);

It seems like $flags can also be of type string; however, parameter $array of array_unique() does only seem to accept array, maybe add an additional type check?

        CacheFlag::getInstance()->cacheFlag->invalidateFlaggedCachesByFlags($flags);
        return true;
    }