Closed Cypher01 closed 7 years ago
But why tho? 2.0.1 and 2.0.2 are exactly the same. There is no change in code apart from pointing to the second .json file...
Are you sure? I didn't study the source code in detail, but I realized that I get Ads with v2.0.2, but not with v2.0.1.
Yes I am sure... I've updated it myself and apart from changing direct link from old .json to the new one, there is nothing else changed. Your ads may be because of other module/app...
This module does nothing more apart from adding background playback. It does not integrate in ads - so it does not add them nor remove them...
On Sat, Jun 17, 2017 at 2:48 PM Cypher01 notifications@github.com wrote:
Are you sure? I didn't study the source code exactly, but I realized that I get Ads with v2.0.2, but not with v2.0.1.
— You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/pylerSM/YouTubeBackgroundPlayback/issues/329#issuecomment-309213257, or mute the thread https://github.com/notifications/unsubscribe-auth/AMxxbF1EoFxcdTFM75YQpUy24Wjh4AIYks5sE8sogaJpZM4N9KWK .
The hooks-2.json file does not have the versions that the hooks-1.json has. I will try and get around to feeding in two JSON files so that we can do the <= 12.17.54
versions the old file has.
@Cypher01, in the meantime, keep using the 2.0.1.
Pushed 20eb10ef895a2821eecc1d5c2bc366a5b88dc866 but I don't have any devices running on anything older than N right now and I do not want to mess around with testing this on an emulator, so @th3an7 will have to test it out.
I was planning on going the route of reading in two JSON files, but the main issue I faced was deciding how to deal with the fallback
key of the JSONs. Most notably, whether differences in those should be handled in some fancy way or just one of them should be picked or whatever else should happen. So instead, we have one massive JSON which has all the multipliers and then under them the version codes the way they were prior to this change.
Me neither... I have 7.1.2... I can build new apk but can't test it :P
@Cypher01 can you please test if this works? https://drive.google.com/file/d/0B8j7vCsf6RXcNFpGdXU0ZUxweFE/view?usp=sharing
Ideally it would be if you could test this in both YouTube app versions - old one which is working on 2.0.1 and latest one which is working on 2.0.2
Also, if this apk for some reason cannot be installed (system shows errors or sth), try uninstalling previous module version (I may or may not fucked up apk signature... again)
@th3an7 I'm going to test it on Genymotion, when I have time. Is this based on the actual state of your fork?
This apk is build with changes that @sigv pushed...
Also my fork is just a copy of this one and this one is a master - modules apks are made from sources from this fork... My fork has only one extra branch with modified json file that can make playback controls visible only when app is in background and not all the time. But apart from that, it's exact the same :)
Might want to do a rebuild which includes 4a4c1cb87305ca39ef021952be5e17ac30e12d52 for testing since I realized today that I forgot to traverse into the multiplier keys of the hooks-3.json
file when I wrote the code yesterday. 🙂
And 685a5e5932408e90c089d796a00786949cb30a98.. sigh
OK, so here it is - updated apk with latest changes
@th3an7: works
So if it works, I'm adding that apk to xposed repo
v2.0.2 doesn't work with YouTube <12.19.* anymore. The YouTube version should be checked and the old code should stay for backward compatibility. (I intentionally use an older YouTube version)