GemsTracker / gemstracker-library

GEneric Medical Survey Tracker, main library
http://gemstracker.org
BSD 3-Clause "New" or "Revised" License
6 stars 2 forks source link

SelectModel drops sort columns silently #672

Closed mddejong closed 3 years ago

mddejong commented 3 years ago

MUtil SelectModel drops sort columns silently if it is not sure the column exists.

This behaviour should be changed so that if a column is named in a sort statement, that column should be supposed to exist. Then if the code tries to sort on a non-existing column the model will generate an SQL error during execution - which means we know there is something wring in the code - while as long as column exists it will always be used for a sort.