added better error handling when article has been cleared from cache

[asl97]

For feed if for some reason cache was cleared.

Ideally, at least the non full article shouldn't be written into the cachedir as it is a key part of the whole "Offline reading" mentioned in Features.

https://developer.android.com/training/data-storage/app-specific#internal-remove-cache

Caution: When the device is low on internal storage space, Android may delete these cache files to recover space. So check for the existence of your cache files before reading them.

I did accidentally put a huge git repo recently into a syncthing folder which I deleted afterwards but that might had cause the article to get deleted due to running very low on storage space

This hotfix is not ideal but at least it's something, as I doubt it would be possible to retrieve the lost data. screenshot:

I would appreciate if you could fix the tag on #443 too
Having an situation where the feed get lost ('Found nothing to fetch') is a bug.

[spacecowboy]

So the problem here is that you are removing the feedback to the user that something went wrong.

The current situation is certainly not the best, but this will lead to confusing bug reports where sometimes the snippet is shown instead of the real text.

[spacecowboy]

Instead, could you perhaps update where the articles are saved?

Feeder/app/src/main/java/com/nononsenseapps/feeder/util/FilePathProvider.kt

Line 37 in d69df4c

override val articleDir: File = cacheDir.resolve("articles")

[asl97]

Instead, could you perhaps update where the articles are saved?

Feeder/app/src/main/java/com/nononsenseapps/feeder/util/FilePathProvider.kt

Line 37 in d69df4c

override val articleDir: File = cacheDir.resolve("articles")

This is just a quick hot fix, and changing the location wouldn't help for already deleted saved article.

Not to mention migrating exisiting articles from the cachedir to another location is a bit out of my league, where the maintainer of this project with more knowledge would be more suited to do.

I do not want to the cause for even more article to get deleted because I mess up the migration.

[asl97]

So the problem here is that you are removing the feedback to the user that something went wrong.

The current situation is certainly not the best, but this will lead to confusing bug reports where sometimes the snippet is shown instead of the real text.

IMO, Found nothing to fetch is not much of a feedback, but if you want a more clear feedback for the user, sure.

[spacecowboy]

@asl97 just run ./gradlew ktlintformat to fix the ktlint stuff