TransactionItem::save() - Code Metrics - Inspection of "EZP-30823: [6.x] Isolated cache during transaction..." - ezsystems/ezpublish-kernel - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — 6.7 ( fb8957...f97c26 )

by André

created 2019-09-24 09:04 UTC

TransactionItem::save() A

↳ Parent: TransactionItem

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	1
nc	1
nop	0
dl	0
loc	13
rs	9.8333
c	0
b	0
f	0

<?php

/**
 * @copyright Copyright (C) eZ Systems AS. All rights reserved.
 * @license For full copyright and license information view LICENSE file distributed with this source code.
 */
namespace eZ\Publish\Core\Persistence\Cache\Adapter\Item;

use Tedivm\StashBundle\Service\CacheItem;

/**
 * Class TransactionItem.
 *
 * Custom Item class for use during transactions, routes calls to save() and clear() to clear callback.
 *
 * @internal
 */
class TransactionItem extends CacheItem
{
    /** @var callable */
    private $clearFn;

    /** @var callable */
    private $isItemDeferedClearedFn;

    public function setClearCallback(callable $clear)
    {
        $this->clearFn = $clear;
    }

    public function setIsClearedCallback(callable $isItemDeferedCleared)
    {
        $this->isItemDeferedClearedFn = $isItemDeferedCleared;
    }

    /**
     * {@inheritdoc}
     */
    public function isMiss()
    {
        // Mark any cache item which has been scheduled to be cleared as a miss.
        // We do this using callback since isHit property is private, & it will be reset on get()
        $fn = $this->isItemDeferedClearedFn;
        if ($fn($this->keyString)) {
            return true;
        }

        return parent::isMiss();
    }

    /**
     * {@inheritdoc}
     */
    public function save()
    {
        // We don't save cache during transaction (as cache is shared), & given the use of cache items within eZ
        // Platform kernel is not used across transaction boundaries items (these) given by pool can safely be assumed
        // to still be within transaction when save() is called.
        // ...
        // We do need to tell Pool that it should delete the item tough (which is done on commit if in transaction)
        // so this save will be done on-demand when needed.
        $clear = $this->clearFn;
        $clear($this->keyString);

        return false;
    }

    /**
     * {@inheritdoc}
     */
    public function clear()
    {
        $clear = $this->clearFn;
        $clear($this->keyString);

        return true;
    }
}


1			<?php
2
3			/**
4			* @copyright Copyright (C) eZ Systems AS. All rights reserved.
5			* @license For full copyright and license information view LICENSE file distributed with this source code.
6			*/
7			namespace eZ\Publish\Core\Persistence\Cache\Adapter\Item;
8
9			use Tedivm\StashBundle\Service\CacheItem;
10
11			/**
12			* Class TransactionItem.
13			*
14			* Custom Item class for use during transactions, routes calls to save() and clear() to clear callback.
15			*
16			* @internal
17			*/
18			class TransactionItem extends CacheItem
19			{
20			/** @var callable */
21			private $clearFn;
22
23			/** @var callable */
24			private $isItemDeferedClearedFn;
25
26			public function setClearCallback(callable $clear)
27			{
28			$this->clearFn = $clear;
29			}
30
31			public function setIsClearedCallback(callable $isItemDeferedCleared)
32			{
33			$this->isItemDeferedClearedFn = $isItemDeferedCleared;
34			}
35
36			/**
37			* {@inheritdoc}
38			*/
39			public function isMiss()
40			{
41			// Mark any cache item which has been scheduled to be cleared as a miss.
42			// We do this using callback since isHit property is private, & it will be reset on get()
43			$fn = $this->isItemDeferedClearedFn;
44			if ($fn($this->keyString)) {
45			return true;
46			}
47
48			return parent::isMiss();
49			}
50
51			/**
52			* {@inheritdoc}
53			*/
54			public function save()
55			{
56			// We don't save cache during transaction (as cache is shared), & given the use of cache items within eZ
57			// Platform kernel is not used across transaction boundaries items (these) given by pool can safely be assumed
58			// to still be within transaction when save() is called.
59			// ...
60			// We do need to tell Pool that it should delete the item tough (which is done on commit if in transaction)
61			// so this save will be done on-demand when needed.
62			$clear = $this->clearFn;
63			$clear($this->keyString);
64
65			return false;
66			}
67
68			/**
69			* {@inheritdoc}
70			*/
71			public function clear()
72			{
73			$clear = $this->clearFn;
74			$clear($this->keyString);
75
76			return true;
77			}
78			}
79

ezsystems / ezpublish-kernel

Push — 6.7 ( fb8957...f97c26 )

TransactionItem::save() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like