Refactoring the FileProcessor and GPSExtractor classes

[ilgazer]

Both of these classes contain old code that doesn't match how we currently use them. For example, the logic of extracting GPS data from a similar image and using them for that class is spread out between UploadMediaDetailFragment(some of the business logic seems to be totally missing) and FileProcessor. I restructuring those classes will lead to much more readable and maintainable code. What do you think?

[misaochan]

I agree, those are very old classes, we might have missed them in the overhaul (oops!). Are you interested in working on these, @ilgazer ? I guess you are already familiar with the MVP architecture requirements (since I think you did some of the Retrofit refactor too)?

[ilgazer]

Yup, I would be interested in working on that. I've already started working on it, and I'll be refactoring some other old code that is related (ie. I just created a License class instead of passing around these different keys and text etc. everywhere.)

[misaochan]

Thank you! We appreciate it. :)

[macgills]

I will be taking over this ticket as it has had no activity for a month. Hope I am not stepping on your toes @ilgazer

[ilgazer]

Hi, I was just about to make a PR with some changes. I was really busy in the past 2 weeks but I will have more time now. Is there any chance you could take up another issue?

…

On Wed, 18 Mar 2020, 16:54 Seán Mac Gillicuddy, ***@***.***> wrote: I will be taking over this ticket as it has had no activity for a month. Hope I am not stepping on your toes @ilgazer <https://github.com/ilgazer> — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#3408 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AB2EXI5CUJ2GS4YBXC5ST23RIDHBDANCNFSM4KUZJSJQ> .

[macgills]

@ilgazer I am a full time resource on the project now and this ticket has been quite beneficial for me to get to grips with the upload process. I would prefer to keep going now that I have momentum but if you insist I will hand it back over, it being your ticket and all.