fluttercandies / saver_gallery

MIT License
15 stars 15 forks source link

兼容Android 分区存储 #3

Closed pye52 closed 1 year ago

zhangruiyu commented 1 year ago

我这边会做些修改, 目前发现如果没有Permission.storage权限,会导致查询是否文件是否存在一直返回false,后续我加入选项吧

pye52 commented 1 year ago

ScopedStorage让Android的文件读写变得复杂了 现在分了Android Q版本前,requestLegacyExternalStorage的状态,等等各种场景都需要针对性的请求权限 我个人觉得由用户自己去抉择会更好些

zhangruiyu commented 1 year ago

嗯 我这边也做了点改动 是这样的 ScopedStorage 如果没有storage权限是判断不了文件是否存在的,我这边新加了个参数,让用户自己选择是否当文件存在时候不保存

zhangruiyu commented 1 year ago

增加了androidExistNotSave参数 做了存在判断

zhangruiyu commented 1 year ago

image

zhangruiyu commented 1 year ago

path: ^1.8.3没法升级flutter_test写死1.8.2了

zhangruiyu commented 1 year ago

saver_gallery: ^2.0.0 发布了 可以试试

zhangruiyu commented 1 year ago

真的感谢帮我改这块代码 我之前也是copy的 随便改吧改吧就推了

pye52 commented 1 year ago

我觉得还有优化空间,例如精简方法的参数,androidRelativePath和androidExistNotSave这两个参数是否去掉比较好?和平台绑定的参数似乎容易让开发者迷惑. 还是这个特性有实际应用需求呢?

zhangruiyu commented 1 year ago

我觉得还有优化空间,例如精简方法的参数,androidRelativePath和androidExistNotSave这两个参数是否去掉比较好?和平台绑定的参数似乎容易让开发者迷惑. 还是这个特性有实际应用需求呢?

有的,我这边就有需求,比如把一组图片存到download下特定目录,或者你把一个电视剧存到一个特定目录而不是根目录,如果存在就不存,太正常的需求了

pye52 commented 1 year ago

我觉得还有优化空间,例如精简方法的参数,androidRelativePath和androidExistNotSave这两个参数是否去掉比较好?和平台绑定的参数似乎容易让开发者迷惑. 还是这个特性有实际应用需求呢?

有的,我这边就有需求,比如把一组图片存到download下特定目录,或者你把一个电视剧存到一个特定目录而不是根目录,如果存在就不存,太正常的需求了

这个需求我可以理解,但是existnotsave这个参数在iOS上的表现不一致,我想针对这部分尝试做一下适配

zhangruiyu commented 1 year ago

我觉得还有优化空间,例如精简方法的参数,androidRelativePath和androidExistNotSave这两个参数是否去掉比较好?和平台绑定的参数似乎容易让开发者迷惑. 还是这个特性有实际应用需求呢?

有的,我这边就有需求,比如把一组图片存到download下特定目录,或者你把一个电视剧存到一个特定目录而不是根目录,如果存在就不存,太正常的需求了

这个需求我可以理解,但是existnotsave这个参数在iOS上的表现不一致,我想针对这部分尝试做一下适配

可以 不过ios我也不太了解 不知道能不能判断相册里是否存在