Bug #3368

Mobile/NonMobile.php dose not implement makeDisplay()

Details

Submitted	2005-02-03 08:00 UTC
From	roux at dot-wired dot org
Assigned	kuboa
Status	Closed
Package	Net_UserAgent_Mobile
PHP Version	4.3.10
OS	Debian
Roadmaps	(Not assigned)

Comments

[2005-02-03 08:00 UTC] roux at dot-wired dot org

Description:
------------
Mobile/NonMobile.php dose not implement makeDisplay().
So, non mobile UA detected (Mobile/NonMobile.php required), died by getDisplay() method calls, because of Mobile/Common.php 's makeDisplay() method.
Accoding to its API document, this case should return PEAR error object.

http://pear.php.net/package/Net_UserAgent_Mobile/docs/0.21.0/Net_UserAgent_Mobile/Net_UserAgent_Mobile_Common.html#methodgetDisplay

Reproduce code:
---------------
<?php
require_once "Net/UserAgent/Mobile.php";
$obj = Net_UserAgent_Mobile::factory("Mozilla/4.0 (compatible; MSIE 6.0; Windows
NT 5.1; SV1; .NET CLR 1.1.4322)");
$display = $obj->getDisplay();
var_dump($display); // Nothing occured because Died at Common.php
?>

[2005-02-03 08:41 UTC] kubo at isite dot co dot jp

Thank you for your bug report. This issue has been fixed
in the latest released version of the package, which you can download at
http://pear.php.net/get/Net_UserAgent_Mobile

[2005-02-03 08:57 UTC] kubo at isite dot co dot jp

This bug has been fixed in CVS.

In case this was a documentation problem, the fix will show up at the
end of next Sunday (CET) on pear.php.net.

In case this was a pear.php.net website problem, the change will show
up on the website in short time.

Thank you for the report, and for helping us make PEAR better.