A small error on Line 25 of mobile_tools_browscap.module
/**
* Implements hook_device_groups() as provided by Mobile Tools module ().
*/
function mobile_tools_device_groups() {
return array('iphone' => 'iPhone', 'ipod' => 'iPod', 'android' => 'Android', 'opera_mini' => 'Opera Mini', 'blackberry' => 'BlackBerry');
}
should be
/**
* Implements hook_device_groups() as provided by Mobile Tools module ().
*/
function mobile_tools_browscap_device_groups() {
return array('iphone' => 'iPhone', 'ipod' => 'iPod', 'android' => 'Android', 'opera_mini' => 'Opera Mini', 'blackberry' => 'BlackBerry');
}
Comments
Comment #1
bradjones1I rolled this into a patch per best practices.
I can confirm this allows you to successfully enable the module - I'm not familiar enough to know if there's particular functionality dependent on the function in question, but I'll mark this needs review.
Comment #2
fluxrider commentedworked for me . Thanks for the patch and @hasrossell , thanks for the fix.
Comment #3
joelpittetThat's the fix, thanks bradjones1
Comment #4
noslokire commentedWorks for me as well
Comment #5
devin carlson commentedThis can either be committed to 7.x-2.x or the issue can be closed if no further development is happening to the 7.x-2.x branch.
Comment #6
devin carlson commentedCommitted to 7.x-2.x so that this issue can be closed out.