getRow() API

I feel that the last two parameters to DB::getRow() should be reversed. Currently, it's: getRow($query, $fetchmode = DB_FETCHMODE_DEFAULT, $params = array()); You would most likely use getRow() with non-empty params much more often than specifying a $fetchmode. Most people would probably just use setFetchMode() to set the default mode and forget about it. I think we should change it to: getRow($query, $params = array(), $fetchmode = DB_FETCHMODE_DEFAULT); -Andrei It is commonly the case with technologies that you can get the best insight about how they work by watching them fail. -- Neal Stephenson