Tickets :: Comment on [#6722] persistent_params not set or asked in conf.xml

* Your Email Address
* Spam protection	Enter the letters below: .___.. .._. __..__ _/ \|__\| \| (__ [__) ./__.\| \|_\|_.__)[__)
Comment	>> When using e.g. the sql session handler together with memcache > >> conf.xml is not setting or asking for the persistent_params used in > >> SessionHandler.php. > >> > >> if ($driver == 'memcached') { > >> // Trap for old driver name. > >> $driver = 'memcache'; > >> } elseif (($driver != 'memcache') && !empty($params['memcache'])) { > >> $driver = 'memcache'; > >> unset($params['memcache']); > >> $persistent_params = array('persistent_driver' => > >> $driver, 'persistent_params' => $params); > >> $params = null; > >> } > > > > Why do we need to define 'persistent_params' in conf.xml? All this > code is doing is saving the original params (for the persistent > driver) so that the memcache driver can later pass these params on to > the persistent driver. It's just a temporary holding place to allow > us to pass driver information between classes. $driver is replaced > with 'memcache' and $params is replaced with the params needed for > the 'memcache' driver, so we need a way to make sure the original > values aren't blown away. > > > > Nowhere are we trying to load 'persistent_params' from the conf.xml, > and there is absolutely no reason we would ever want to.
Attachment
Watch this ticket