Summary: | Loading svc/cataloguing/framework bottlenecks advanced cataloging editor | ||
---|---|---|---|
Product: | Koha | Reporter: | Phil Ringnalda <phil> |
Component: | Cataloging | Assignee: | Bugs List <koha-bugs> |
Status: | RESOLVED WORKSFORME | QA Contact: | Testopia <testopia> |
Severity: | normal | ||
Priority: | P5 - low | CC: | lauren_denny, lthorrat, m.de.rooy, nick |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
See Also: |
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36461 https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36463 https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36531 |
||
Change sponsored?: | --- | Patch complexity: | --- |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Circulation function: |
Description
Phil Ringnalda
2024-01-12 20:29:19 UTC
Actually, since we die with an uncaught exception with no notification and nothing the user can do but reload or close the tab, there's no reason why we should be using "waitSeconds: 30" when 0 (== infinite) exists. Moved setting waitSeconds: 0 to bug 36461, so it can be about making it possible to load the editor, and this can be about making it pleasant to load it. Filed bug 36463 which improves size of loading subsequent frameworks, however, the initial load is not compressed for some reason - I think because of the loading by requireJS? Not sure why, but the initial framework load is served as text/javascript, so it's missing compression from that. Oh, heh. That's what &callback= does, https://git.koha-community.org/Koha-community/Koha/src/branch/master/C4/Service.pm#L56 serves it as 'js' rather than 'json', and although we deflate application/javascript, we don't serve that, https://git.koha-community.org/Koha-community/Koha/src/branch/master/C4/Output.pm#L250 maps 'js' to text/javascript. And because the names of mimetypes are names, and thus a Hard Problem, rfc9239 says text/javascript is correct and application/javascript is an obsolete alias, so the correct fix is to deflate text/javascript. Filed bug 36531 for compressing the initial load. So snappy now! |