You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by mc...@apache.org on 2008/12/16 22:36:30 UTC

svn commit: r727163 - /myfaces/trinidad/branches/1.2.10.1-branch/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/agent/AgentFactoryImpl.java

Author: mcooper
Date: Tue Dec 16 13:36:30 2008
New Revision: 727163

URL: http://svn.apache.org/viewvc?rev=727163&view=rev
Log:
TRINIDAD-1350 iPod Safari not identified as "iPhone" as intended in documentation

Now treats the iPod touch version of Safari just like the iPhone version of Safari.

Modified:
    myfaces/trinidad/branches/1.2.10.1-branch/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/agent/AgentFactoryImpl.java

Modified: myfaces/trinidad/branches/1.2.10.1-branch/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/agent/AgentFactoryImpl.java
URL: http://svn.apache.org/viewvc/myfaces/trinidad/branches/1.2.10.1-branch/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/agent/AgentFactoryImpl.java?rev=727163&r1=727162&r2=727163&view=diff
==============================================================================
--- myfaces/trinidad/branches/1.2.10.1-branch/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/agent/AgentFactoryImpl.java (original)
+++ myfaces/trinidad/branches/1.2.10.1-branch/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/agent/AgentFactoryImpl.java Tue Dec 16 13:36:30 2008
@@ -724,7 +724,11 @@
 
     if (agent.indexOf("iPhone") > 0)
     {
-      // At the moment, all known iPhone and iPod touch agent strings contain "iPhone"
+      agentObj.setPlatform(Agent.PLATFORM_IPHONE);
+    }
+    else if (agent.indexOf("iPod") > 0)
+    {
+      // At the moment, the iPod touch version of this browser matches iPhone's
       agentObj.setPlatform(Agent.PLATFORM_IPHONE);
     }
     else if (agent.indexOf("Win") > 0)