Sorting By Incorrect Column On Duplicate Column Names

That piece of code was changed in #3059245: Rewrite main module functions to work with raw values → by @JordiK, so I'm assigning the issue to him to look at because I don't know exactly what the intent was in that change.

I don't think your workaround is the correct way to fix it though, because if the entity_id array element is not being accessed then we don't need to check to see if it's set anymore. But why was it being used in the first place? @JordiK should know ...

Upon re-reading my response in #3, I see it doesn't make sense, even to me. I think I must have posted that response in the wrong issue, because I talk about something that is not relevant to this problem. And specifically this piece of code was *not* altered or added by #3059245: Rewrite main module functions to work with raw values → like I said.

Sorry for the mistake.

I think we need a test case to demonstrate the problem, which will also prove the proposed solution. That's going to be the hold-up in getting this fixed, but this module is not very maintainable without a good set of test cases and we don't have that yet. So when something like this is identified as a bug, it's extremely important to me that we write a test case to prove that the fixed code does things correctly.

Good catch @mitchmccoy! Please check the patch.