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

At startup scan for libraries and *.known_web_libraries files, load all (iff possible) and update the active known_web_libraries file. #103

Closed
GerHobbelt opened this issue Oct 9, 2019 · 2 comments
Closed

At startup scan for libraries and *.known_web_libraries files, load all (iff possible) and update the active known_web_libraries file. #103

GerHobbelt opened this issue Oct 9, 2019 · 2 comments
Labels
🦸‍♀️enhancement🦸‍♂️ New feature or request
Milestone
v82

Comments

@GerHobbelt
Copy link
Collaborator

Take care to keep 'legacy' web libraries as Web Libraries in there so that the file is backwards compatible with the last commercial Qiqqa release: v79

(Goal: recovery of [local] library collections)
@GerHobbelt GerHobbelt added the 🦸‍♀️enhancement🦸‍♂️ New feature or request label Oct 9, 2019
@GerHobbelt GerHobbelt added this to the Our Glorious Future milestone Oct 9, 2019
@GerHobbelt
Copy link
Collaborator Author

Related: #76

@GerHobbelt GerHobbelt mentioned this issue Oct 15, 2019
Closed
GerHobbelt added a commit to GerHobbelt/qiqqa-open-source that referenced this issue Oct 15, 2019
@GerHobbelt
- refactoring the PDFDocument code and making its interface threadsaf…
6ef9b44 
…e (apart from the inks, annotations and highlights attribute lists). Fixes jimmejardine#96.

- improve the internal code of `CloneExistingDocumentFromOtherLibrary_SYNCHRONOUS()` to help ensure we will be able to copy/transfer all metadata from the sourcing document to the clone. Code simplification.
- code simplification/performance: remove useless LINQ overhead
- `SignalThatDocumentsHaveChanged()`: improve code by making sure that a reference PDF is only provided when there really is only a single PDF signaled! Also see the spots where this API is being used.
- belated commit of AlphaFS upgrade/refactoring for jimmejardine#106 ; a multi-pass code review has been applied to uncover all places where (file/directory) paths are manipulated/processed in the Qiqqa code. TODO: Streams.
- redesign the ADD/REMOVE/GET-password APIs for PDFDocument as the indirection in the Qiqqa made it bloody hard to make thread-safe otherwise. (Fixes jimmejardine#96 / jimmejardine#98)
- ditto for the GC-memleaking bindables of PDFDocument: all the binding stuff has been moved to the PDFDocument thread-safe wrapper class. (TODO: investigate if we can use WeakReferences more/better to prevent memleaks due to infinitely marked PDFDocument instances...)
- tweak: `AddLegacyWebLibrariesThatCanBeFoundOnDisk()`: at startup, try to load all .known_web_libraries files you can find as those should provide better (= original) names of the various Commercial Qiqqa Web Libraries. (This is running ahead to the tune of jimmejardine#109 / jimmejardine#103)
@GerHobbelt
Copy link
Collaborator Author

Fixed as per today (marker commit: 95dff9b), will be available in upcoming pre-release v82pre4.

The order Qiqqa looks for library specifications:

  1. find all the Qiqqa.known_web_libraries config files and merge those as best as possible
  2. find Intranet *.s3db + *.txt references, which should link to the local copies
  3. scan for the Qiqqa.library files themselves.

Check the AddLegacyWebLibrariesThatCanBeFoundOnDisk() code and surroundings for details. See also: https://github.com/GerHobbelt/qiqqa-open-source/blob/95dff9bc16234ecde02a9e5b0ec8e8223675bda2/Qiqqa/DocumentLibrary/WebLibraryStuff/WebLibraryManager.cs#L67

@GerHobbelt GerHobbelt modified the milestones: Our Glorious Future, v82 Nov 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
🦸‍♀️enhancement🦸‍♂️ New feature or request
Projects
None yet
Milestone
v82
Development

No branches or pull requests
1 participant
@GerHobbelt