vpx_still URL is empty when it should not be

[Michiel.Schok]

When an asset is master-slaved, and via the /asset/{id} or /asset call the details about the asset are requested *from the slave app*, the <vpx_still> URL is empty.

On VP-Core acceptatie:

Asset_id P1ZDccSWaQRNqgL5WUkOhIby (master is app_id=9, slaved to app_id=5)

from app_id=9:

/asset/P1ZDccSWaQRNqgL5WUkOhIby
...

      <vpx_still_url>http://app.acceptatie.vpcore.snkn.nl/still/vSBUnsXWdsPafZ9GREemUAtl</vpx_still_url>
      <ega_still_url>http://www.grouper.surfmedia.nl/medialibrary/item.html?id=P1ZDccSWaQRNqgL5WUkOhIby</ega_still_url>

from app_Id=5

/asset/P1ZDccSWaQRNqgL5WUkOhIby
...
      <vpx_still_url></vpx_still_url>
      <ega_still_url></ega_still_url>

A consequence of this bug is that all stills of NIBG/Academia material will not show up in SURFmedia.

[Michiel.Schok]

Peter, found this one during testing of the CSA.

[peter]

Validated that this is introduced in MM 2.3. This will be a fix in the next 2.3.x release.

[robert]

Not sure anymore now I looked at why its not working. The reason the still is not shown is because the still isn't master-slaved, although the mediafile is. Can someone tell me if this is by design or that the master-slaving on the mediafile is enough for access on the still? This is a problem for CSA, because it will only publish the mediafiles under the asset and not the stills atm.

[robert]

Also, even if we put master-slave on the still, what happens if we replace the still, regenerate, etc. Does acl_inherit work for still too? Don't think it does. So replacing the still will lose the master-slave setting too?

[Michiel.Schok]

In current production environment, when you look at asset 10060 (NIBG), it has a still with id=KOQyCn8R07IqRJyJ7z9yQ6Pd

a request as /mediafile/KOQyCn8R07IqRJyJ7z9yQ6Pd/acl?user_id=nibg-admin (from app_id=3) results in an empty set, so this still is also *not* master-slaved. And it is working correct in SURFmedia, and a vpx_still_url is given in the /asset/10060 response, when issued from app_id=5.

[robert]

Ok thanks, I already looked in the 2.2 code and its different in setup. Reason for the bug is the change we made for perm. still links in 2.3. I just wrote unittest to confirm the bug and now I'm fixing it for 2.3.5 release.

[Michiel.Schok]

OK, stills are now shown in SURFmedia.