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

Library discovery: try to load all Qiqqa.known_web_libraries files you can find as those should provide better (= original) names of the various Commercial Qiqqa Web Libraries. #109

Closed
GerHobbelt opened this issue Oct 15, 2019 · 1 comment
Labels
duplicate This issue or pull request already exists 🦸‍♀️enhancement🦸‍♂️ New feature or request 🤔question Further information is requested or this is a support question
Projects
Milestone

Comments

@GerHobbelt
Copy link
Collaborator

See 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, instead of the 'Legacy Intranet Library - XYZ' auto-named instances that will pop up otherwise.

@GerHobbelt GerHobbelt added 🦸‍♀️enhancement🦸‍♂️ New feature or request 🤔question Further information is requested or this is a support question labels Oct 15, 2019
@GerHobbelt GerHobbelt added this to the v83 milestone Oct 15, 2019
@GerHobbelt GerHobbelt added the duplicate This issue or pull request already exists label Oct 15, 2019
@GerHobbelt
Copy link
Collaborator Author

Dupe of #103. 😨

GerHobbelt added a commit to GerHobbelt/qiqqa-open-source that referenced this issue Oct 15, 2019
…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 GerHobbelt modified the milestones: v83, v82 Nov 5, 2019
@GerHobbelt GerHobbelt added this to Done in v82pre4 Nov 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate This issue or pull request already exists 🦸‍♀️enhancement🦸‍♂️ New feature or request 🤔question Further information is requested or this is a support question
Projects
No open projects
v82pre4
  
Done
Development

No branches or pull requests

1 participant