Bluz\Session\Adapter\AbstractAdapter

Complexity

Total Complexity

4

Size/Duplication

Total Lines	71
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	prepareId()	0	3	1
A	close()	0	5	1
A	open()	0	9	1
A	gc()	0	4	1

<?php

/**
 * Bluz Framework Component
 *
 * @copyright Bluz PHP Team
 * @link      https://github.com/bluzphp/framework
 */

declare(strict_types=1);

namespace Bluz\Session\Adapter;

/**
 * Abstract session handler
 *
 * @package Bluz\Session\Adapter
 */
abstract class AbstractAdapter
{
    /**
     * @var mixed instance of Redis or Cache or some other
     */
    protected $handler = null;

    /**
     * @var string prefix for session store
     */
    protected $prefix = 'PHPSESSID:';

    /**
     * @var integer TTL of session
     */
    protected $ttl = 1800;

    /**
     * Prepare Id - add prefix
     *
     * @param  string $id
     *
     * @return string
     */
    protected function prepareId($id): string
    {
        return $this->prefix . $id;
    }

    /**
     * Initialize session
     *
     * @param  string $savePath
     * @param  string $sessionName
     *
     * @return bool
     */
    public function open($savePath, $sessionName): bool

The parameter $savePath is not used and could be removed.

    {
        $this->prefix = $sessionName . ':';
        $this->ttl = (int)ini_get('session.gc_maxlifetime');

        // No more action necessary because connection is injected
        // in constructor and arguments are not applicable.

        return true;
    }

    /**
     * Close the session
     *
     * @return bool
     */
    public function close(): bool
    {
        $this->handler = null;
        unset($this->handler);
        return true;
    }

    /**
     * Cleanup old sessions
     *
     * @param integer $maxLifetime
     *
     * @return bool
     */
    public function gc($maxLifetime): bool

The parameter $maxLifetime is not used and could be removed.

    {
        // no action necessary because using EXPIRE
        return true;
    }
}