Huawei device with Android 9 throws java.lang.NoSuchFieldException while calling reflect()

[Navneet7k]

try {
      reflect(editor::class, "mSelectHandleCenter").set(editor, centerDrawable)
      reflect(editor::class, "mSelectHandleLeft").set(editor, leftDrawable)
      reflect(editor::class, "mSelectHandleRight").set(editor, rightDrawable)
    } catch (e : NoSuchFieldException) {
      Log.e("Huawei",e.message)
    }

This produces the following exception :

java.lang.NoSuchFieldException: No field mSelectHandleCenter in class Lhuawei/com/android/internal/widget/HwEditor; (declaration of 'huawei.com.android.internal.widget.HwEditor' appears in /system/framework/hwEmui.jar)

[saket]

Oh wow! Did not expect OEMs to modify the Editor class, but then relying on reflection wasn't supposed to be foolproof. It might be a better idea to set android:colorControlActivated from a xml theme and intercept the color at runtime.

[saket]

Wanna send a PR with a fix @Navneet7k, along the lines of how the cursor color is overridden?

[saket]

Fixed along with #8 in a7b3c92. Thanks for reporting!

[saket]

Whoops my bad. Your device is running on Android 9 so this issue isn't fixed yet.

[Navneet7k]

hi, couldn't reply yesterday, is this fixed

[Navneet7k]

wrapping it with a try catch gave the following result in huawei

[saket]

That's… strange. How are the handles getting colored if the code fails to run? 😮

Can you share a screenshot of the cursor? This is what I see on vanilla Android:

[Navneet7k]

maybe because the handler drawable is not getting set(which we wrapped in try) , but the drawable color is later set to accent color.

screenshot of cursor :

[saket]

That sounds very suspicious 😄

It looks like there might be a way to set the handles from Xml: https://stackoverflow.com/a/15133928/2511884. Press could intercept the Resources#getDrawable() calls and tint them. I'll take a stab at this sometime.

[TheDanileron]

Look at Danil. B and Jared Rummler answers here: https://stackoverflow.com/questions/15133926/how-to-change-color-appearance-of-edittext-select-handle-anchor/63121585#63121585
It shows how to do it from the code

[saket]

That looks exactly what I'm doing currently: https://github.com/saket/press/blob/master/androidApp/src/main/java/press/theme/TextViewCompat.kt

[saket]

Can someone check if they're still running into this crash with Press v1.7.0?