They called it train wreck because the getters are chained together like the carriages in a train. I have never heard that before but I thought it was pretty funny.
They recommended changing the code to something like this
They made a good point about the method name. They said it should say something about "what the method is for" not just "how it is currently implemented".
I ran into an example of this the other day. We trying to make a conditional more expressive so we changed this code
if (property !== "query")
var isNotQuery = property !== "query";
The explaining variable what was put in to try and explain why we needed the conditional. When we looked at the code though it didn't really add much so we changed it to this
var isNotExtReservedWord = property !== "query";
Now you can see that "query" is a reserved word so it was filtered out.