cvs commit: jahia/core/src/java/org/jahia/content/comparators ContainerRankingComparator.java

[Serge Huber]

shuber      2004/09/29 16:24:09 CEST

  Modified files:
    core/src/java/org/jahia/content/comparators 
                                                ContainerRankingComparator.java 
  Log:
  Fix NullPointerException in container comparison if for example we are not allowed 
to display a container.
  
  Revision  Changes    Path
  1.2       +8 -0      
jahia/core/src/java/org/jahia/content/comparators/ContainerRankingComparator.java
http://jahia.mine.nu:8080/cgi-bin/cvsweb.cgi/jahia/core/src/java/org/jahia/content/comparators/ContainerRankingComparator.java.diff?r1=1.1&r2=1.2&f=h
  
  
  
  Index: ContainerRankingComparator.java
  ===================================================================
  RCS file: 
/cvs/jahia/core/src/java/org/jahia/content/comparators/ContainerRankingComparator.java,v
  retrieving revision 1.1
  retrieving revision 1.2
  diff -u -r1.1 -r1.2
  --- ContainerRankingComparator.java   30 Mar 2004 14:10:23 -0000      1.1
  +++ ContainerRankingComparator.java   29 Sep 2004 14:24:09 -0000      1.2
  @@ -36,6 +36,14 @@
               JahiaContainer rightJahiaContainer = rightContainer.
                                                    getJahiaContainer(paramBean,
                   paramBean.getEntryLoadRequest());
  +            if (leftJahiaContainer == null) {
  +                logger.warn("Left JahiaContainer in comparison is null");
  +                return -1;
  +            }
  +            if (rightJahiaContainer == null) {
  +                logger.warn("Right JahiaContainer in comparison is null");
  +                return 1;
  +            }
               return new Integer(leftJahiaContainer.getRank()).compareTo(new 
Integer(rightJahiaContainer.getRank()));
           } catch (JahiaException je) {
               logger.error("Error while accessing Jahia containers for entry load 
request " + paramBean.getEntryLoadRequest(), je);