[NFR] Phalcon\Session\AdapterInterface::destroy() extra $session_id param

[anggiaj]

It will be nice to have Phalcon\Session\AdapterInterface::destroy() compatible with
php SessionHandlerInterface::destroy(string $session_id) :D

Ex:

use Phalcon\Session\Adapter;
use Phalcon\Session\AdapterInterface;
use Phalcon\Session\Exception;


class MySessionAdapter extends Adapter implements AdapterInterface, SessionHandlerInterface
{
    public function __construct($options = null) 
    {
        session_set_save_handler($this, true);
    }

    // phalcon version
    // public function destroy(){}
    // session handler interface version
    // public function destroy($session_id){}
}

Note: tested under phalcon 1.2.3, php 5.4.x

CMIIW

[pgasiorowski]

+1. This might be useful for functionality such as logging out someone else when we want to

• swap current session adapter to something else
• take application offline for maintenance
• logout an user after suspicious activity

[ghost]

PR submitted

[ghost]

I guess this one can be closed now?