Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix Ignored RootNS #1367

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Fix Ignored RootNS #1367

wants to merge 3 commits into from

Commits on Nov 7, 2024

  1. Fix Ignored RootNS 

    - currently we find the root element using only the local name of an element, this can result in ambiguous roots and an error if you have an element with no namespace and one with a namespace. This PR fixes that by searching using the qname instead
- add search for qname option if no namespace is specified, because there might still be an unambiguous root element within a namespace, the user just opted not to specify it

DAFFODIL-2947
    @olabusayoT
    olabusayoT committed Nov 7, 2024
    Configuration menu
    Copy the full SHA
    a706c0c View commit details
    Browse the repository at this point in the history

  2. fixup! Fix Ignored RootNS 

    - add doc comment
- update test names

DAFFODIL-2947
    @olabusayoT
    olabusayoT committed Nov 7, 2024
    Configuration menu
    Copy the full SHA
    aa7ba35 View commit details
    Browse the repository at this point in the history

Commits on Nov 8, 2024

  1. fixup! fixup! Fix Ignored RootNS 

    - use Schemas.searchGlobalElementDecl instead of duplicating effort

DAFFODIL-2947
    @olabusayoT
    olabusayoT committed Nov 8, 2024
    Configuration menu
    Copy the full SHA
    0edf99d View commit details
    Browse the repository at this point in the history