Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

Better Error Warning when using private attributes in EpoxyModel

See original GitHub issue

Hi there,

I wrote some attributes with the private modifier @EpoxyAttribute private lateinit var htmlText: String (conditioned reflex, my bad) and I had a weird error that took me way too much time to link to the private modifier. I don’t know if it’s possible to verify this and display a better error than: <identifier> expected and not a statement. I was searching an eventual conflict in my gradle or a problem in kapt.

Issue Analytics

State:
Created 6 years ago
Comments:6 (5 by maintainers)

Top GitHub Comments

1reaction

elihartcommented, Jan 18, 2018

Fix https://github.com/airbnb/epoxy/pull/388

The main issue was that since we didn’t have the name of the setter/getter we used “null” which made the generated code all messed up. The fix is simple. It switches the field to be accessed directly like super.html = html. This will give an error that html is private and can’t be accessed, but it is much clearer.

0reactions

elihartcommented, Jan 18, 2018

Good analysis. The reason we still try to generate code even though we log this error is that without generating the code there is the possibility for hundreds of more errors where referenced generated files don’t exist, and that makes it even harder to find the original issue.

I will try to clean up the code generated in this case.