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

Errors in exist.log when opening app #19

Open
duncdrum opened this issue Feb 14, 2018 · 3 comments
Open

Errors in exist.log when opening app #19

duncdrum opened this issue Feb 14, 2018 · 3 comments

Comments

@duncdrum
Copy link
Contributor

duncdrum commented Feb 14, 2018

exist.log reports many lines of:

2018-02-14 18:50:50,030 [qtp1616515493-80] ERROR (FunError.java [eval]:139) - Not found: (http://exist-db.org/apps/site-utils#NOT_FOUND):Not found 

exist 4.0.0
fundocs 0.9
java 8
macOS 10.13.3

@joewiz
Copy link
Member

joewiz commented Feb 15, 2018

I can confirm each request to http://localhost:8080/exist/apps/fundocs/index.html adds 17 instances of this error. This issue is actually caused by the app's templates/page.html template's calls to site:expand-links, which tries to dynamically expand all of that page's links to other apps. References to missing apps like {demo} trigger the error at https://github.com/eXist-db/shared-resources/blob/master/content/siteutils.xql#L65. Installing the demo apps—which used to be included in the installer until last March—reduces the number of errors to 4. I haven't tracked down these remaining culprits but wanted to report these findings for now.

@joewiz
Copy link
Member

joewiz commented Feb 15, 2018

The remaining 4 errors are due to a mismatch between the {doc} shorthand reference and the documentation app's expath-pkg.xml @abbrev value: exist-documentation. The function that performs the lookup of these abbreviations, apputil:resolve-abbrev(), relies on this value from expath-pkg.xml's package/@abbrev.

Generally, an app's @abbrev should correspond to the name of the target collection into which it is installed. Since exist-documentation is used nowhere else—either in the app's target collection name or in the repository—I would favor changing its @abbrev to "doc".

(I actually prefer the app's original name, "docs," which was changed to "doc" only to accommodate the development of Loren Cahlander's replacement for the original app.)

Also, since we have a graceful 404 functionality that advises people who click on links to install the missing apps via Dashboard, perhaps the error here should be demoted to a warning.

@joewiz joewiz changed the title function docs are not generated Errors in exist.log when opening app Feb 15, 2018
@adamretter
Copy link
Member

It seems the function documentation was never updated, i.e. it does not use exist-apps-parent I will send a PR.

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

No branches or pull requests

3 participants