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

Fix unknown function vl:raise-login #7

Merged
merged 3 commits into from
Sep 21, 2024

Conversation

tribut
Copy link
Contributor

@tribut tribut commented Jul 8, 2023

Not sure if this is the correct way to fix it, but while trying to query Stadtbibliothek Chemnitz using xidel, I got an error:

$ xidel -e '$username:="XXX",
          $password:="XXX",
          $server:="https://opac2.stadtbibliothek-chemnitz.de/webOPACClient/",
          $startparams:="?Login=opsb"
    ' --template-action connect,update-all  \
      --module mockvidelibri.xqm --xmlns videlibri=https://www.videlibri.de --dot-notation=on \
      --template-file data/libraries/templates/sisis/template
[...]
Error:
err:XPST0017: unknown function: vl:raise-login #0

Did you mean: 
  In module math: 	(namespace url: "http://www.w3.org/2005/xpath-functions/math")
    log #1:  (double?) as double?


in line 1 column 5
 vl:raise-login() 
    ^^^^^^^^^^^^^  error occurs around here

Multipage action trace:
update-all
login <call>
(pattern of action:login page:{$request})

The problem appears to be that vl:raise-login is called without argument from data/libraries/templates/sisis/template.

convert-repo and others added 2 commits July 7, 2023 04:13
`vl:raise-login` is called without parameters from
`data/libraries/templates/sisis/template`.
@benibela benibela merged commit 9fa9e84 into benibela:master Sep 21, 2024
1 check failed
@benibela
Copy link
Owner

benibela commented Oct 6, 2024

It took a while to merge, because I was trying to figure out how to deal with the "Verfified" label

If I merge it on Github, it does not synchronize with the sourceforge/gitlab mirrors.

There is no verified label in the associated patch: https://github.com/benibela/videlibri/pull/7.patch (and a hgtags file. that comes from the mirroring, but it should not be in the patch)

Now I have merged it on Github, but the "Verified" label got lost anyways

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants