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

getGlideCacheFile的路径不完全正确 #122

Closed
silentpride opened this issue Nov 23, 2020 · 4 comments
Closed

getGlideCacheFile的路径不完全正确 #122

silentpride opened this issue Nov 23, 2020 · 4 comments

Comments

@silentpride
Copy link

如果app配置了Glide的缓存路径,getGlideCacheFile取的文件目录就会有问题

图片加载的缓存目录
image

getGlideCacheFile取文件目录
image

这样就导致点了【查看原图】,且原图顺利下载到本地,退出页面在查看图片,还会显示【查看原图】的按钮。
此时点了【查看原图】,进度会卡在1%一直不动。

@SherlockGougou
Copy link
Owner

如果app配置了Glide的缓存路径,getGlideCacheFile取的文件目录就会有问题

图片加载的缓存目录
image

getGlideCacheFile取文件目录
image

这样就导致点了【查看原图】,且原图顺利下载到本地,退出页面在查看图片,还会显示【查看原图】的按钮。
此时点了【查看原图】,进度会卡在1%一直不动。

有按照README配置GlideModule吗?

@silentpride
Copy link
Author

如果app配置了Glide的缓存路径,getGlideCacheFile取的文件目录就会有问题
图片加载的缓存目录
image
getGlideCacheFile取文件目录
image
这样就导致点了【查看原图】,且原图顺利下载到本地,退出页面在查看图片,还会显示【查看原图】的按钮。
此时点了【查看原图】,进度会卡在1%一直不动。

有按照README配置GlideModule吗?

有的
image

@silentpride
Copy link
Author

我强行把AppGlideModule配置里的的默认缓存目录换成这个
File(context.cacheDir, DiskCache.Factory.DEFAULT_DISK_CACHE_DIR)
就好了。

这里建议还是不要写死,ImagePreviewAdapter的onLoadFailed里,saveDir可能是变的;getGlideCacheFile里判断是否有缓存读的是File(context.getCacheDir(), DiskCache.Factory.DEFAULT_DISK_CACHE_DIR),容易出现缓存判断不准确的问题。

另外:
#52 (comment)
#109 (comment)

这两个问题特别影响国际化的app,建议抽时间修复。

@SherlockGougou
Copy link
Owner

国际化已增加。

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

No branches or pull requests

2 participants