Hi all,
I wanted to feel out opinion about two small, but theoretically backwards-incompatible changes: 1) The DB query get_in_bulk() must be provided with an ID list. This list can be provided _either_ as the first input argument, or in the 'id_list' kwarg (but not both). I would suggest the kwarg should be deprecated. 2) The DB query __get_date_list() must be provided with a 'kind' to operate on. Again, this can be provided _either_ as the first argument, or in the 'kind' kwarg (but not both). Again, I would suggest the kwarg should be deprecated. Strictly, these would be backwards incompatible changes. However, in both cases, the documentation is written as if the idlist/kind is just the first argument of the method; there is no mention of the kwarg method. The motivation here is that the argument handling required to support both modes of providing the same argument is inelegant, and prone to error. Any objections to me cleaning up these methods and removing the kwargs? Thanks Russ Magee %-)
