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.