I wondered why we were looking at `request.GET` the last time I looked at this code. Your change looks good, I assume that didn't break any tests ?