Even better, let's just have the default orderFilter be the equivalent of '*'. If there's always a filter, then there's no longer a branch at the point it's us

Even better, let's just have the default orderFilter be the equivalent of '*'. If there's always a filter, then there's no longer a branch at the point it's used -- that needs to be tested and maintained. I used to work with a programmer who I found difficult because their code was an ever-increasing number of "if" statements as each new case came along. Conversely he would say coding is relatively easy and that I was overcomplicating a problem by thinking about it; all I needed to do is "add an if statement here".