[jbossseam-issues] [JBoss JIRA] Closed: (JBSEAM-3940) JbossPojoCacheProvider get method implementation error

[Norman Richards (JIRA)]

     [ https://jira.jboss.org/jira/browse/JBSEAM-3940?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Norman Richards closed JBSEAM-3940.
-----------------------------------

    Fix Version/s: 2.1.2.CR1
       Resolution: Done


forgot to set fix version

> JbossPojoCacheProvider get method implementation error
> ------------------------------------------------------
>
>                 Key: JBSEAM-3940
>                 URL: https://jira.jboss.org/jira/browse/JBSEAM-3940
>             Project: Seam
>          Issue Type: Bug
>          Components: Core
>    Affects Versions: 2.1.1.GA
>         Environment: JBoss 4.2.3.GA
>            Reporter: Hakan Uygun
>            Assignee: Norman Richards
>             Fix For: 2.1.2.CR1
>
>
> JbossPojoCacheProvider get method implementation is wrong. Region parameter ignored and return Node instance :
> public Object get(String region, String key)
>    {
>       try
>       {
>          return cache.get(getFqn(key));
>       }
>       catch (CacheException e)
>       {
>          throw new IllegalStateException(String.format("Cache throw exception when trying to get %s from region %s.", key, region), e);
>       }
>    }
> It shoul be like JbossCacheProvider get method :
> public Object get(String region, String key)
>    {
>       try
>       {
>          Node node = cache.get(getFqn(region));
>          if (node != null)
>          {
>             return node.get(key);
>          }
>          else
>          {
>             return null;
>          }
>       }
>       catch (CacheException e)
>       {
>          throw new IllegalStateException(String.format("Cache throw exception when trying to get %s from region %s.", key, region), e);
>       }
>    }

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: https://jira.jboss.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira