Ticket #716 (closed defect: fixed)

Opened 20 months ago

Last modified 19 months ago

ContainerHierarchicalWrapper MethodInvocationException

Reported by: Artur Signell Owned by: Jani Laakso
Priority: blocker Milestone: 4.0.2
Component: Server-side framework Version: 4.0.1
Keywords: Cc:
Known Issue description: Relocating an item to root node is not possible with ContainerHierarchicalWrapper.
Hours estimate: Deadline (dd.mm.yyyy):
Known Issue version (since): 4.0.0 Known Issue title: ContainerHierarchicalWrapper setParent at root level issue
Hours done: Depends to:
Affects documentation: no
Known Issue workaround: If you need to relocate an item to root node, create new Container.
Affects release notes: yes Contract:

Description

setParent fails at the following line if newParentId is null

        // Check if no change is necessary
        if ((newParentId == null && oldParentId == null)
                || newParentId.equals(oldParentId))
            return true;

so it is impossible to relocate an item to root level!

Simple fix:

        if ((newParentId == null && oldParentId == null)
                || (newParentId != null && newParentId.equals(oldParentId)))
            return true;

Change History

Changed 20 months ago by Jani Laakso

  • status changed from new to closed
  • version set to 4.0.1
  • resolution set to fixed
  • milestone set to 4.1.0-rc

Fix seems good. Fixed in [1408] as Artur suggested.

Changed 20 months ago by Jani Laakso

  • milestone changed from 4.1.0-rc to 4.0.2

Changed 19 months ago by Jani Laakso

  • known_issue_title set to ContainerHierarchicalWrapper setParent at root level issue
  • known_issue_desc modified (diff)

Changed 19 months ago by Jani Laakso

  • known_issue_workaround modified (diff)
  • known_issue_desc modified (diff)

Changed 19 months ago by Jani Laakso

  • known_issue_since set to 4.0.0
Note: See TracTickets for help on using tickets.