Fix: Inform user if there's no internet connection while trying to ac…

[balaji-ramavathu]

…cess media files

Fixes #447

Summary

1. For all file types, a toast is displayed that there's no internet connection and goes back to files list page.
2. For video files, if internet is disconnected while the video is playing, EndReached Event is fired and hence the check for internet connection is implemented there.

Please Add Screenshots If any UI changes.

For audio (same for all other types) :

If internet goes off while playing video :

[codedentwickler]

Great idea. It is good to have such indicator messages on the app. Maybe we can do better with the messages and the checks though.

There is a possibility the user is running a local server which means he won't need to be connected to the internet to access his files. (Basically accessing files on a local server via a Wifi network.)

This messages might come up at the wrong time in this case. However there is still a good chance, the Wifi network gets disconnected and the files on the local server cant be access.

We will need to have appropriate checks and messages for these situations.

[balaji-ramavathu]

hey @codedentwickler how do I check if the user is running a local server?

[codedentwickler]

Yes, An easy way is to do a check like in the snippet below. You can understand how auto, remote and local connections is used by observing how this shared preference key preference_key_server_connection is modified.

private boolean isConnectionLocal() { SharedPreferences preferences = PreferenceManager.getDefaultSharedPreferences(getActivity()); String preferenceConnection = preferences.getString(getString(R.string.preference_key_server_connection), null); return preferenceConnection.equals(getString(R.string.preference_key_server_connection_local)); }

[codedentwickler]

@csoni111 @cpg what do you think about this feature.

[balaji-ramavathu]

@codedentwickler thanks! I will add these checks :-)

[balaji-ramavathu]

@codedentwickler I've added checks and messages for the local server case.
Can you have a look once? : )

[Akash-Ramjyothi]

This is a really good feature and works fine on testing on both local as well as on remote connections, I recommend @cpg to Merge this PR.

[chirag-jn]

Something wrong happened. Some issues while merging instead closed the PR.