Author: mfranklin
Date: Mon Jun 11 15:02:22 2012
New Revision: 1348886

URL: http://svn.apache.org/viewvc?rev=1348886&view=rev
Log:
Fixed page lookup error (RAVE-625)

Modified:
    
rave/branches/model_interfaces/rave-components/rave-core/src/main/java/org/apache/rave/portal/service/impl/DefaultPageService.java

Modified: 
rave/branches/model_interfaces/rave-components/rave-core/src/main/java/org/apache/rave/portal/service/impl/DefaultPageService.java
URL: 
http://svn.apache.org/viewvc/rave/branches/model_interfaces/rave-components/rave-core/src/main/java/org/apache/rave/portal/service/impl/DefaultPageService.java?rev=1348886&r1=1348885&r2=1348886&view=diff
==============================================================================
--- 
rave/branches/model_interfaces/rave-components/rave-core/src/main/java/org/apache/rave/portal/service/impl/DefaultPageService.java
 (original)
+++ 
rave/branches/model_interfaces/rave-components/rave-core/src/main/java/org/apache/rave/portal/service/impl/DefaultPageService.java
 Mon Jun 11 15:02:22 2012
@@ -98,9 +98,12 @@ public class DefaultPageService implemen
 
     @Override
     public Page getPageFromList(long pageId, List<Page> pages) {
-        Page pageToFind = new PageImpl(pageId);
-        int index = pages.indexOf(pageToFind);
-        return index == -1 ? null : pages.get(index);
+       for(Page page: pages) {
+           if(page.getId().equals(pageId)){
+               return page;
+           }
+       }
+       return null;
     }
 
     @Override


Reply via email to