Implementing #683 - sending a broadcast intent on upload success or failure.
These are (respectively) net.wigle.wigleandroid.UPLOAD_COMPLETE and net.wigle.wigleandroid.UPLOAD_FAILED.
As a part of this change, I've also moved the OkFileUploader core call contents to WiGLEApiManager, our API implementation class. This necessitated numerous changes and begged certain cleanups, including:
making the global state api manager instance public, since we reference it outside of its package
cleanups of some literal strings and statuses, including two non-enumerated/conflicting integer states
eliminating some unused, duplicative code
a few safety checks that really should have been there from the start
parsing and inclusion of the upload results, including transIds
state handling cleanup - this is still not perfect, but should be cleaner
Some of the changes here may benefit from further refactoring, but this change set is getting large, and further changes (e.g. combining our various progress interaction implementations) would increase risk.
Implementing #683 - sending a broadcast intent on upload success or failure. These are (respectively)
net.wigle.wigleandroid.UPLOAD_COMPLETEandnet.wigle.wigleandroid.UPLOAD_FAILED.As a part of this change, I've also moved the
OkFileUploadercore call contents to WiGLEApiManager, our API implementation class. This necessitated numerous changes and begged certain cleanups, including:Some of the changes here may benefit from further refactoring, but this change set is getting large, and further changes (e.g. combining our various progress interaction implementations) would increase risk.