It should really be checking for wurfl-config.xml, then parsing where to check for the "main-file" attribute of that config file, since it can be a zip file or have any name at all (the WURFL PHP API comes with a main-file called "wurfl-regression.zip").

I'm not sure if this error is present in the 2.x version.

As a separate, but related, issue:
The cache folder is also defined in the wurfl-config.xml file, but for some reason, it's hard coded that the wurfl_settings function check at ""$library/cache", which doesn't correspond to the example resources dir provided by the WURFL PHP API (which has the cache folder INSIDE the resources folder).

Comments

twom’s picture

twom commented
Status: Active » Needs work

Hi,

Thanks for the remarks. The module indeed can be more generic... And parsing the wurfl file seems like a good idea. This will be for a next big release.

In fact the whole configuration could be created programmatically instead of using the wurfl-config file.

Any patches are highly appreciated!