LonamiWebs / Stringlate

Android application to help in strings.xml translation
https://lonamiwebs.github.io/stringlate/index
MIT License
106 stars 46 forks source link

Fails with 'Atarashii!' #68

Closed drac0300 closed 7 years ago

drac0300 commented 7 years ago

This app crashes when trying to download sources ofhttps://github.com/AnimaSA/Atarashii I think it might be a string problem (caused by the '!' In the app name but I am not sure since I've not tested a lot) Anyway, thanks for the cool app, and hoping this bug is resolved soon !

Lonami commented 7 years ago

The AnimeNeko/Atarashii repository works just fine on both the v0.9.5 (available on F-Droid) and on the current master branch. What problem do you have exactly? Could you please be a bit more specific?

drac0300 commented 7 years ago

Edited and cleaned up, sorry for the inconvenience

Well, back to subject I do have 9.5 (from F-droid) but when I try to add the atarashii repository, it crashes at step "searching for strings.xml file"

drac0300 commented 7 years ago

But if you don't get this error, it might be due to my esoteric setup... I just find it strange that it only is this repo. Is there a way for me to get the logs ?

licaon-kter commented 7 years ago

Get Forkhub ;)

On topic: do detail your setup maybe.

drac0300 commented 7 years ago

Hm hm, well I have Resurrection remix on a oneplus X with no Gapps but microG (root of course since I flashed a Rom). I also have Xposed with the Xprivacy module, but I don't think I've denied your app any permission... As you see, not really vanilla android..

Didn't know about forkhub, looks quite good ! I will try it when I'm on wifi

licaon-kter commented 7 years ago

Try to clear this app's settings in Xprivacy so you are asked again. Try to whitelist this app in Xprivacy, so it's not restricted at all.

Lonami commented 7 years ago

Try to clear this app's settings in Xprivacy so you are asked again. Try to whitelist this app in Xprivacy, so it's not restricted at all.

Stringlate doesn't even require external storage access, it does it all in its own directory. If every other repository works, then it also has internet access. Does this repository work for you @licaon-kter?

Edit: Just tried again, it did crash. That's awkward. Did I even use the same repository? Edit 2: Yes indeed. AnimeNeko/Atarashii works but AnimaSA/Atarashii does not. Where did I take the first repository from?

licaon-kter commented 7 years ago

Did not get a chance to test, :(

/LE I can confirm the crash

java.lang.NullPointerException: Attempt to invoke virtual method 'org.json.JSONArray org.json.JSONObject.getJSONArray(java.lang.String)' on a null object reference
    at io.github.lonamiwebs.stringlate.utilities.RepoHandler$2.onPostExecute(RepoHandler.java:232)
    at io.github.lonamiwebs.stringlate.utilities.RepoHandler$2.onPostExecute(RepoHandler.java:221)
    at android.os.AsyncTask.finish(AsyncTask.java:636)
    at android.os.AsyncTask.access$500(AsyncTask.java:177)
    at android.os.AsyncTask$InternalHandler.handleMessage(AsyncTask.java:653)
    at android.os.Handler.dispatchMessage(Handler.java:102)
    at android.os.Looper.loop(Looper.java:211)
    at android.app.ActivityThread.main(ActivityThread.java:5373)
    at java.lang.reflect.Method.invoke(Native Method)
    at java.lang.reflect.Method.invoke(Method.java:372)
    at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:1020)
    at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:815)
    at de.robv.android.xposed.XposedBridge.main(XposedBridge.java:102)
drac0300 commented 7 years ago

I tried after disabling xprivacy for stringlate and it still fails ><

Lonami commented 7 years ago

If you try going to https://github.com/AnimaSA/Atarashii from your browser, it will redirect you to https://github.com/AnimaNeko/Atarashii. This is the reason. Stringlate tries to reach to https://api.github.com/repos/AnimeSA/Atarashii, which doesn't exist. But for some reason, the API then follows the redirect and I don't update the entered URL anywhere else, it's considered to be correct, thus failing.

It would be nice if you could reach out to them and ask them to update their source code URL on F-Droid to reflect the name change, because eventually it will stop pointing to the correct place.

The repository is cloned correctly on the current master branch, so I will close this issue. I can't rely on the entered repository being from GitHub anymore (although these are still treated specially), and since these cases should not happen too often (I could maybe see if it's a GitHub repository and then make sure the entered URL is correct), I won't add any more code to handle it. Maybe I could explain these cases under the /Help Activity (any ideas?).

Lonami commented 7 years ago

By the way, if you want to translate the application from the v0.9.5, again, use https://github.com/AnimaNeko/Atarashii instead :)

Thanks for using my app!

licaon-kter commented 7 years ago

It would be nice if you could reach out to them and ask them to update their source code URL on F-Droid to reflect the name change, because eventually it will stop pointing to the correct place.

https://gitlab.com/fdroid/fdroiddata/merge_requests/1955

Lonami commented 7 years ago

Thanks @licaon-kter :)