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

Upgrade slf4j wrapper #1

Open
wants to merge 17 commits into
base: 1.0.3
Choose a base branch
from

Conversation

PhilZ-cwm6
Copy link

@PhilZ-cwm6 PhilZ-cwm6 commented Jun 30, 2022

  • Upgrade slf4j wrapper to support all current slf4j formatting options for both compatibility with third party libraries (jcifs-nlg) and for performance (accept formatting string with {})
  • Update zip4j to 2.11.1:
    • use stock upstream zip4j lib
    • fixes upstream zip4j vulnerabilities
    • drop un-needed SMBSync3 compression methods

Upgrade slf4j wrapper to support all current slf4j formatting options for both compatibility with third party libraries (jcifs-nlg) and for performance (accept formatting string with {})
- use stock upstream zip4j lib
- fixes upstream zip4j vulnerabilities
- drop un-needed SMBSync3 compression methods
When creating a Zip Sync Task, when we browse for existing files and we enter a zip file name, browsing further into an empty directory disables the Ok button even if the filename is properly set
On app start and on EditSyncTasks, SafManager was causing NPE and app/system crash to get app specific directory
Note that currently the method result is not used in code as app specific directory is hardcoded for now
Do not use hard coded references like "/storage/emulated/0".
Use built in Android methods to get relevant storage path. This ensures a better compatibility across devices with different storage path standard
On a few devices, there is an implementation bug for secondary external storage (removable sdcard) causing getExternalFilesDirs() method to only return the path for primary external storage (internal non removable storage /Storage/emulated/0)
This includes a few Lenovo devices and emulator Pixel devices for APIs 26 and 27

This caused getAppSpecificDirectory() to return null for external removable sdcard path that is now properly checked for on app start
…types

- Restore original upstream custom list preference for non compressed file types
- Fix screen rotation did not properly preserve Ok button state
- Add missing resources for edit compress file type dialogs
- Fix compression file types list sorting was only applied in adapter and not persisted in saved preferences
- Fix when editing file type entries in list, they were not sorted when back to the list view
- Support custom title dialog in ListEditPreference
- Remove unused deprecated method in ListEditPreference
For custom list preference (non compressed file types selection):
- add option to undo delete entry
- when list is modified, Ok button will only be enabled if the final resulting list is different from the original one
- on screen rotation and activity killed, properly preserve all the custom preference list dialogs state
- Upgrade all tools and plugins to latest releases
- Target SDK 33
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

Successfully merging this pull request may close these issues.

1 participant