The ItemCollectionWithMetadata class has no callers within the package, no tests and as far as I know no users.
My recollection is that Chad added this for a use case I suggested that has never actually come up, of wanting to pass metadata along with a collection of items rather than on individual items.
It's not clear to me at this point that the use case is in general addressable, as collections are created in various places and there's little control over which class is used for that.
I think the right thing to do here is to excise this from the API for 0.10, along with the method that references it in ItemMetadataSupport (which in any case I plan to remove from the API). If the use case comes up we can revisit this and make sure that it is implemented correctly, some time after 1.0.
The
ItemCollectionWithMetadataclass has no callers within the package, no tests and as far as I know no users.My recollection is that Chad added this for a use case I suggested that has never actually come up, of wanting to pass metadata along with a collection of items rather than on individual items.
It's not clear to me at this point that the use case is in general addressable, as collections are created in various places and there's little control over which class is used for that.
I think the right thing to do here is to excise this from the API for 0.10, along with the method that references it in ItemMetadataSupport (which in any case I plan to remove from the API). If the use case comes up we can revisit this and make sure that it is implemented correctly, some time after 1.0.