ZF-11961: Amelioration class Zend_Http_UserAgent_Features_Adapter_WurflApi

Issue Type: Patch Created: 2011-12-30T15:04:27.000+0000 Last Updated: 2012-02-26T07:42:07.000+0000 Status: Open Fix version(s): Reporter: Cédric CRISPIN (blackmennewstyle) Assignee: Business&Decision / Interakting (interakting) Tags: - Zend_Http_UserAgent

Related issues: Attachments:


On line 83 :

Replace :

$wurflConfig->wurflFile($c['wurfl']['main-file']) ->wurflPatch($c['wurfl']['patches']) ->persistence($c['persistence']['provider'], $c['persistence']['dir']);

With :

$wurflConfig->wurflFile($c['wurfl']['main-file']) ->wurflPatch($c['wurfl']['patches']) ->persistence($c['persistence']['provider'], $c['persistence']['params']) ->allowReload($c['allow-reload']);

I do not understand why it requires a directory as a parameter so that we can pass any provider...


Posted by Adam Lundrigan (adamlundrigan) on 2012-02-26T07:03:20.000+0000

What problem does this code solve?

Posted by Cédric CRISPIN (blackmennewstyle) on 2012-02-26T07:42:07.000+0000

@Adam Lundrigan

With this code, you can easily use any persistance like memcached with WURFL because the original code "->persistence($c['persistence']['provider'], $c['persistence']['dir']);" forces you to use a directory which is not required with the others persistance methods allowed. Sometimes it cause a bug with some specifics apache server configurations.

Have you found an issue?

See the Overview section for more details.


© 2006-2018 by Zend, a Rogue Wave Company. Made with by awesome contributors.

This website is built using zend-expressive and it runs on PHP 7.