ZF-2037: Zend_Session::getSaveHandler()

Issue Type:
Improvement
Created:
2007-10-03T16:06:10.000+0000
Last Updated:
2008-02-26T12:56:01.000+0000
Status:
Resolved
Fix version(s):
  • 1.0.4 (26/Feb/08)
Reporter:
Shekar Reddy (zendfw)
Assignee:
Ralph Schindler (ralph)
Tags:
Related issues:
Attachments:

Description

Zend_Session class does not store the custom save-handler inside the class so we can access it at a later time to configure it further.

Zend_Session::getSaveHandler()->setXxxx()

Thoughts...?

Comments

Posted by Shekar Reddy (zendfw) on 2007-10-04T14:12:01.000+0000

This got double-posted due to ZIRA going Yo-Yo! Please remove this issue, too - along with 2038.

Posted by Thomas Weidner (thomas) on 2007-10-15T13:37:55.000+0000

Assigned to Ralph

Posted by Darby Felton (darby) on 2007-11-06T14:55:46.000+0000

I support the idea of storing the save handler object as a static member and providing access to it via {{Zend_Session::getSaveHandler()}}. Maybe the method returns {{null}} if no custom save handler has been set.

Posted by Ralph Schindler (ralph) on 2008-02-14T11:10:48.000+0000

Shekar,

I have added the requested method. I have run the unit tests and everything looks sane. Unfortunatly, Zend_Session does not have a test for save handler, so I think if you could test this method just to make sure its doing what you want (After all it is simply an accessor), that will have to suffice for a test until after 1.5 when I have the time to dedicated some energies to Zend_Sessions tests.

Lemme know if it all works out :)

-ralph

Posted by Ralph Schindler (ralph) on 2008-02-15T14:31:34.000+0000

Marking as fixed, if there are any further issues, please reopen this one.

Posted by Shekar Reddy (zendfw) on 2008-02-16T14:24:22.000+0000

Hi Ralph,

I'll test it out over the weekend and let you know very soon.

Thanks,

Posted by Shekar Reddy (zendfw) on 2008-02-19T04:49:07.000+0000

Ralph,

I've tested the new method and it works like a charm!

Thanks!