wasdennnoch / AndroidN-ify

[Discontinued] An Xposed module to use features introduced in Android N on your LP+ Device
265 stars 93 forks source link

Can't write lock screen PIN #1349

Closed licaon-kter closed 7 years ago

licaon-kter commented 7 years ago

Xperia 5.1.1

Description Install/update module, restart, swipe to enter PIN, pressing any number does not actually write the number in the box, although you can see the animation on touch.

Module version (build version) Started in build 828 (I guess), present in 829 and 833 (did not bother to test in between). Build 827 works ok.

Xposed log / logcat https://gist.github.com/licaon-kter/115e85d9a6e144d33d9784f6dbe76fea

I guess one of these fixes broke it: https://github.com/wasdennnoch/AndroidN-ify/commit/6f79766b05770f8313f63ddf69b4eb43ec1c58e8 (why weren't these added one by one ?)

@numeprenume Any ideas?

rvictorr commented 7 years ago

Which experimental features have you enabled? Can you record a video and send me your SystemUI.apk? (deodexed if it's possible) I know I should have commited them one by one, but I worked on this for a long time, initially without thinking about releasing it, and by the time I decided to release it the harm was already done (my Git skills are basically non-existent). Additionally, there was a lot of trial and error involved because all the commits were essentially fixing the bugs (by backporting Nougat code) that were caused by reconfiguring the layout hierarchy. Anyway, long story short, the only thing that could cause bugs on 5.1.1 is the updated experimental stack scroll algorithm, which has been reworked to fix many bugs existent in the previous implementation.

licaon-kter commented 7 years ago

Experimental: Fix huge space under qs header and Experimental N notifications behavior

rvictorr commented 7 years ago

Okay, try disabling Experimental N notifications behavior and see if it fixes it.

licaon-kter commented 7 years ago

Nope, and neither does cleaning up the app settings (using default settings only). Unfortunately this is not my system to mess with all the time so if you have ideas/commits I will test them but not super fast (ex. unable to unlock I need to: uninstall Xposed, re-optimize all apps, uninstall N-ify, reinstall xposed, re-optimize all apps; these take a while)

rvictorr commented 7 years ago

You don't have to mess with Xposed. You can just delete the apk in /data/app/tk.wasdennnoch/androidn_ify in recovery and then reboot. Anyway, I still need your SystemUI apk to see what might cause it.

licaon-kter commented 7 years ago

There is no access to encrypted /data from TWRP on Xperia 5.1.1, it's a known issue on their side. :(

breadone commented 7 years ago

This might sound crazy but i recently watched POTC so everyrhing i say sounds crazy: try entering your normal pin and clicking enter and see if it works

rvictorr commented 7 years ago

If you have USB debugging enabled you can run adb shell su touch /data/data/de.robv.android.xposed.installer/conf/disabled and it will disable Xposed, or just run rm /data/app/tk.wasdennnoch.androidn_ify after the first 2 commands to uninstall N-ify.

licaon-kter commented 7 years ago

@PradyunS There is no ENTER key ( not sure if by default of the making of another Xposed module, I'll getback to you on this:) )

@numeprenume Which part of "device can't be unlocked" did I misspell? :)

wasdennnoch commented 7 years ago

That's why you enable usb debugging and grant it su permissions before you install Xposed ;)

And adb is accessible before you unlock the device.

licaon-kter commented 7 years ago

Did not know about that. Thanks

licaon-kter commented 7 years ago

Stock SystemUI.apk/.odex I've read about de-odexing but I did not manage to do it. Are these enough or you need some framework apks too?

rvictorr commented 7 years ago

Yeah, I'll need the framework folder too (can't remember exactly which specific files are needed)

licaon-kter commented 7 years ago

Hope these are it: framework-res and SemcGenericUxpRes

rvictorr commented 7 years ago

I meant the whole folder, but more specifically /framework/arm and /framework/oat

licaon-kter commented 7 years ago

This is /framework

rvictorr commented 7 years ago

Okay, try this and see if it works. Make sure you test with different options selected (fix space under qs header, experimental N notifications behavior).

thesims2customs commented 7 years ago

It happened to me too... I should re-flash my ROM and restore data. Well, this time I turn off emergency info on lockscreen and it works. Only that option relates to lockscreen beside "reply on lockscreen".

licaon-kter commented 7 years ago

(I did not get a chance to test the new build yet)

FYI, I had "emergency info" OFF too, but it did not help.

licaon-kter commented 7 years ago

Updated to build 838 and it's fixed (no settings changed). Thanks.