I personally like code to be easy to understand, but I also agree with all your other points.
not making use of language patterns like inheritance/interfaces/modules/etc, no or bad documentation.
I think a lot of this overhyped. The easiest to understand code has minimally viable abstraction.