Re: Re: Session: deprecating create_sid() method and add createSid()?
| From: | Yasuo Ohgaki | Date: | Wed, 19 Mar 2014 02:15:50 +0000 |
| Subject: | Re: Re: Session: deprecating create_sid() method and add createSid()? | ||
| References: | 1 2 3 4 5 6 7 8 9 10 11 | Groups: | php.internals |
| Request: | Send a blank email to internals+get-73293@lists.php.net to get a copy of this message | ||
Hi Andrey,
On Wed, Mar 19, 2014 at 10:55 AM, Andrey Andreev <narf@devilix.net> wrote:
> Hi,
>
> On Wed, Mar 19, 2014 at 3:49 AM, Yasuo Ohgaki <yohgaki@ohgaki.net> wrote:
> > Hi Andrey,
> >
> > On Wed, Mar 19, 2014 at 10:26 AM, Andrey Andreev <narf@devilix.net>
> wrote:
> >>
> >> On Wed, Mar 19, 2014 at 3:22 AM, Yasuo Ohgaki <yohgaki@ohgaki.net>
> wrote:
> >> >> > SessionHandler does not use new
> >> >> > SessionUpdateTimestampHandlerInterface,
> >> >> > so
> >> >>
> >> >> Um ... what's the point of having this interface? Nothing uses it.
> >> >
> >> >
> >> > You may use it like
> >> >
> >> > class MySessionHandler implements SessionHandlerInterface,
> >> > SessionUpdateTimestampHandlerInterface
> >> >
> >> > I think the name is too long.
> >> > Any suggestion?
> >>
> >> It is too long, and also useless. :)
> >> You may, or you may not call it. If it's not enforced anywhere, nobody
> >> needs it and nobody is going to use it. I suggest removing it, it's
> >> pointless.
> >
> >
> > It's intended for user use.
> > This is the normal use of interface.
>
> Eh ... I know how an interface is used, but an interface is supposed
> to enforce a certain set of methods to be implemented.
> If you're not enforcing usage of the interface itself (which we can't
> do, because of BC), it becomes optional and therefore enfoces nothing.
>
I can enforce users to implement all of
SessionUpdateTimestampHandlerInterface when it is used.
I cannot enforce users to implement SessionIdInterface for
histrical/compatibility reason, though.
Regards,
--
Yasuo Ohgaki
yohgaki@ohgaki.net