Corrections for Fix bootstraps for webview and service_only (recently merged)

[opacam]

We merged by mistake the mentioned pr #1541 with some minor issues that should had been fixed in there. We solve that in here.

¡¡¡Thanks @AndreMiras!!!

[AndreMiras]

Now I'm thinking we may go even more concise with:

args.services = getattr(args, 'services', [])

What do you think?

[ghost]

yes that looks exactly how the default value of getattr is supposed to be used like 👍

[ghost]

oh wait, actually this was supposed to also apply if args.services is None, right? which it won't, so it's not the same thing

[ghost]

This would work, but it looks uglier: args.services = getattr(args, 'services', None) or [] (or alternatively, args.services = getattr(args, 'services', []) or [] but that's super redundant)

[AndreMiras]

True, the semantic would not be the same for the args.services is None use case. But is that even an use case with add_argument('--service', dest='services', action='append')? I'm really wondering

[ghost]

hmm only if the default is wrong, I suppose. if we checked that the default has a proper value and never None then it would be fine

[opacam]

Well, thanks to @KeyWeeUsr that noticed that the --service argument was disabled.

This is related with the problematic line discussed above, which I think that all the discussion started because of the --service argument being disabled (which leaded me to change the discussed line to avoid compilation errors)...so...I restored the problematic line as it was originally and enabled the --service argument again....I think that this should close the above discussion...

I also modified the testapp_service in order that the test made in there matches one of the options explained in the python-for-android's docs to create a service (thanks again to @KeyWeeUsr 😄...who pointed me to the right direction via discord)

This changes has been tested for python2 and python3 with the service's test app (the one I modify in here)

Note: While testing the new app, I realised that the file Service.tmpl.java needed a slight modifications to make it work with the new python core (a wrong path and a couple of missing imports) so I also fixed it.

[AndreMiras]

Great changeset again, thanks!