You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@rave.apache.org by mf...@apache.org on 2012/06/11 17:02:23 UTC
svn commit: r1348886 -
/rave/branches/model_interfaces/rave-components/rave-core/src/main/java/org/apache/rave/portal/service/impl/DefaultPageService.java
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