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.

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

[BUG] one-off queries cannot import modules with location hint #5530

Open
line-o opened this issue Oct 31, 2024 · 0 comments · May be fixed by #5529
Open

[BUG] one-off queries cannot import modules with location hint #5530

line-o opened this issue Oct 31, 2024 · 0 comments · May be fixed by #5529
Assignees
Labels
bug issue confirmed as bug regression xquery issue is related to xquery implementation

Comments

@line-o
Copy link
Member

line-o commented Oct 31, 2024

Describe the bug

One-off queries that import library modules with a location hint (import module namespace ... at ...;) raise error err:XQST0059 that the source could not be found regardless if the library module is available at the location or not.

Expected behavior

One-off queries evaluated in exist-db must allow imports of library modules with a location hint.

To Reproduce

  1. Evaluate the any query importing a library with a location hint in eXide or over XML-RPC (eg. xst run)

    import module namespace packages="http://exist-db.org/apps/existdb-packages" at "/db/apps/packageservice/modules/packages.xqm";
    packages:get-local-packages()"
    xst run "import module namespace functx='http://www.functx.com' at '/db/system/repo/functx-1.0.1/functx/functx.xq';functx:atomic-type(4)"
  2. The query will raise error err:XQST0059 that the module could not be found

  3. running

    xst run "util:binary-doc-available('/db/system/repo/functx-1.0.1/functx/functx.xq')"
  4. returns true()

Context (please always complete the following information)

  • Build: eXist-7.0.0-SNAPSHOT (9f881d7)
  • Java: 17.0.6 (Azul Systems, Inc.)
  • OS: Mac OS X 14.5 (aarch64)

Additional context

  • How is eXist-db installed? built from source
  • Any custom changes in e.g. conf.xml? none
@line-o line-o linked a pull request Oct 31, 2024 that will close this issue
@line-o line-o self-assigned this Nov 1, 2024
@line-o line-o added bug issue confirmed as bug regression xquery issue is related to xquery implementation labels Nov 1, 2024
@line-o line-o moved this to In review in v6.3.1 release Nov 7, 2024
line-o added a commit to line-o/exist that referenced this issue Nov 19, 2024
fixes eXist-db#5525
fixes eXist-db#5530

- add functx to autodeploy for xquery tests
- add tests for one-off queries with module imports
  - of a registered module without location hint
  - of a module with location hint
- change XQueryContext to allow imports again
- change SourceFactory to work with contextPath set to "."
line-o added a commit to line-o/exist that referenced this issue Nov 19, 2024
fixes eXist-db#5525
fixes eXist-db#5530

- add functx to autodeploy for xquery tests
- add tests for one-off queries with module imports
  - of a registered module without location hint
  - of a module with location hint
- change XQueryContext to allow imports again
- change SourceFactory to work with contextPath set to "."
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug issue confirmed as bug regression xquery issue is related to xquery implementation
Projects
Status: In review
Development

Successfully merging a pull request may close this issue.

1 participant