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:

    Description

    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...

    Comments

    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.