Uploaded image for project: 'CMS'
  1. CMS
  2. CMS-5421

DefaultPage does not handle null parameter in method orderBefore

    XMLWordPrintable

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • None
    • 3.4.7, 3.5.4, 3.7
    • None
    • None

    Description

      According javadoc orderBefore method for a MoveableAmetysObjet should be handle a null parameter.

      /**
           * Order a node before another sibling node (or as the last node)
           * @param siblingNode The node that will be the next sibling node of the current node. Must have the same parent as the current node. Can be null to set the current node as the last node.
           * @throws AmetysRepositoryException if an error occurs.
           */
          public void orderBefore(AmetysObject siblingNode) throws AmetysRepositoryException;
      

      If we call orderBefore with a null parameter on a DefaultPage, there is a NPE

      Attachments

        Activity

          People

            Unassigned Unassigned
            laurence Laurence Aumeunier
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: