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

bradjones1’s picture

Status: Active » Needs review
StatusFileSize
new651 bytes

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

fluxrider’s picture

worked for me . Thanks for the patch and @hasrossell , thanks for the fix.

joelpittet’s picture

That's the fix, thanks bradjones1

noslokire’s picture

Works for me as well

devin carlson’s picture

Status: Needs review » Reviewed & tested by the community

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

devin carlson’s picture

Status: Reviewed & tested by the community » Fixed

Committed to 7.x-2.x so that this issue can be closed out.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.