Uploaded image for project: 'Tools (JBoss Tools)'
  1. Tools (JBoss Tools)
  2. JBIDE-17910

Assign and List directive fixes and tests

XMLWordPrintable

    • Icon: Task Task
    • Resolution: Done
    • Icon: Major Major
    • 4.2.0.CR1
    • None
    • freemarker
    • None

      (1) AssignmentDirective.isNestable() looks only if the directive ends with "/> which is simply wrong for assign, local and global directives. assign documentation lists several possible variants of the directive. AssignmentDirective.isNestable() should cover them all and tests need to be added.

      (2) It is clear from the usage of ItemSet.getFirstNestableItem(Stack<Item>) that it is supposed to return the last (as measured by time) item stored at the stack, not the first one. Therefore, getFirstNestableItem() needs to use a reverse order when iterating over the stack.

      These two issues should fix the improper tree presented in outline (when rendered with OutlineContentProvider.fullAstShown.

            ppalaga Peter Palaga
            ppalaga Peter Palaga
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved: