- ppy.squirrel is not fully .net core upgraded, so it is still showing restored using .net framework
- does not contain .net core runtime installer which is now required by Wox
- UpdateManager.RestartApp method is throwing an OSU application specific error which Wox is unable to resolve.
- Squirrel.Windows 1.5.2 still works and functionalities required by Wox are compatible. Tested RestartApp method, tested enable/disable protable mode methods and tested updating Wox to the latest version.
* Add handling of portable mode when updating
Additionally:
1. Changed getting version from internally rather than from RELEASES file
2. Added PortableDataPath as a constant so can be used to determine if portable mode is used
* Fix incorrectly wired auto update event handler
* Fix Sys plugin missing Shutdown command icon
* Add check update command to Sys plugin
* Add message if current Wox version is latest
* Add default silent when auto checking updates in background
Silent when current is still the latest version
* Move UserData folder to new version location
* Changes per review
1. Move IsPortableMode to Constant
2. Merge if statement
* Per comment- change variables to be more descriptive
UpdateInfo and UpdateManager renamed
* Per comment- Add exception handling and message if failed.
* fix typo
* make function obsolete
it is not used in the code
* rewrite the function that converts chinese chars to pinyin
1. Only difference in this rewrite is instead of returning 2D array, return as a combined single string of all the possible pinyin combination. Since fuzzy search does character matching, this shouldn't be a problem.
2. Added a function that returns a custom language converter. In this case Pinyin converter. New converters can be added.
* Use new language converter param + strip out ScoreForPinyin method
* update
* Change parameter name
* fix failing tests
* WIP
* Remove todo
There should be some distinction between score after precision filter and actual raw score derived from FuzzySearch. Although so far RawScore is used in testing, but it seems to describe the structure. Originally it was to avoid assigning score directly as it would be hard to reason about that output of FuzzySearch score is.
* Add constructors, remove default to enforce required properties
* remove setting rawscore in SearchPrecision
* Change method name to reflect intention
* Change parameter name + update comment
* update
* Remove params comment
Co-authored-by: theClueless <14300910+theClueless@users.noreply.github.com>
Flag is used in method "ShouldUsePinyin()" to avoid calling Alphabet
service. Otherwise, tests applying to StringMatcher.FuzzySearch() would
fail because the pinyin helper library fails to initialize.
As Alphabet class is static, its methods could be used without
ever calling Alphabet.initialize(_settings) beforehand which would end
in an exception. Therefor only _shouldUsePinyin settings needed will
be transferred with a given default value.
