New 'hide_empty_assets' parameter on /collection call

[Frans]

Currently there is nu such parameter on the /collection call. Only on assets. Meaning that the /collection call will also return empty (no mediafiles) assets.

Setting this parameter to true will suppress this.

[robert]

Done -> release 2.2.1 build 861+

[Michiel.Schok]

I'm not sure if it works.

Have uploaded an empty asset to VP-Core acceptatie.
asset_id = w24ijNDMHfaILbHdails6PBq

I've added this asset to collection with id=aUNpWKawVeKXdVVEXsZudWkV
This is the only asset in this collection.

When I perform the following queries, I *always* get <numofvideos>1</numofvideos>, and I expect them to return 0 when 'hide_empty_assets=true' is added as a parameter.

<request_uri>[GET] collection?owner_id=SURFNET.michiel&amp;title=trac</request_uri>

    <item id="1">
      <coll_id>aUNpWKawVeKXdVVEXsZudWkV</coll_id>
      <title>Trac 338</title>
      ...
      <numofvideos>1</numofvideos>
    </item>

<request_uri>[GET] collection?owner_id=SURFNET.michiel&amp;title=trac&amp;hide_empty_assets=true</request_uri>

    <item id="1">
      <coll_id>aUNpWKawVeKXdVVEXsZudWkV</coll_id>
      <title>Trac 338</title>
      ...
      <numofvideos>1</numofvideos>
    </item>

Probably, this is implemented in the /collection/{id}/assets call (which is also very good, but not what I intended when asking Frans...).

[robert]

Ok, my mistake. I wrote unit test to test the new code, the test counts the videos that are returned, but the code only returns the assets that ARE empty. The unit test passed because I only add 2 videos in the unittest. Checking for count 1 is always true because one is always empty the other one not. In both cases the result would have been 1 (1 empty, 1 not).

I changed the unit test to get the expected test case (2 empty, 1 not), and now fixed the code to pass the unit test again.

fixed in -> v2.2.2 (build 898+)

[robert]

Ow btw, the new code also /collection REST call/

[Michiel.Schok]

ack