Ticket #232 (closed defect: fixed)

Opened 3 years ago

Last modified 3 years ago

[Core] Master/Slave: mediafiles are visible

Reported by: Michiel.Schok Owned by: arjen
Priority: major Milestone: MediaMosa 2.1
Component: Core Version: 2.1.2
Keywords: Cc:
MoSCoW: Must Have Estimated time after impact analysis:
Related to project: none Tested: yes
Accepted: yes Estimated Hours:

Description

When an asset contains multiple mediafiles, and one or more of those mediafiles are master/slaved to another application, it is possible to perform an [get] /asset/{id} call in the slave app.

Regardless of the individual master/slave setting on the mediafiles, ALL of the mediafiles are visible in the /asset/{id} call. The mere visibility of one of them is enough to show them all.

e.g. when looking at NIBG content which contains (sometimes) .sami mediafile, we see 4 mediafiles, while only three of them have a master/slave relation.

/asset/18611

<item id="1">
      <asset_id>18611</asset_id>
      <app_id>3</app_id>
      <provider_id></provider_id>
      <owner_id>nibg-admin</owner_id>
      ...
      <mediafiles>
        <mediafile id="1">
          <mediafile_id>31239</mediafile_id>
...
        </mediafile>
        <mediafile id="2">
          <mediafile_id>EzGxXtkk3zFhQnB5qJgotgKv</mediafile_id>
...
        </mediafile>
        <mediafile id="3">
          <mediafile_id>LF73NslrskFZj7BcGIaiJoWL</mediafile_id>
...        </mediafile>
        <mediafile id="4">
          <mediafile_id>PqdlNc6t8ALzGYqmLmbXMPUn</mediafile_id>
...
          <filename>26652_klokhuis_gerrit_rietveld.sami</filename>
...
          <ega_download_url>http://www.surfmedia.nl/app/video/18611/play?format_id=PqdlNc6t8ALzGYqmLmbXMPUn&amp;mode=download</ega_download_url>
          <ega_stream_url></ega_stream_url>
        </mediafile>
      </mediafiles>
    </item>

Change History

Changed 3 years ago by Frans

  • milestone changed from MediaMosa X.X to MediaMosa 2.1

Changed 3 years ago by robert

  • status changed from new to closed
  • resolution set to ready_for_review

2.1 release

Changed 3 years ago by robert

  • status changed from closed to reopened
  • resolution ready_for_review deleted

Changed 3 years ago by robert

  • status changed from reopened to closed
  • resolution set to fixed

Changed 3 years ago by Michiel.Schok

  • status changed from closed to reopened
  • tested changed from no to yes
  • resolution fixed deleted

I removed the aut_app ACLs on mediafile PqdlNc?6t8ALzGYqmLmbXMPUn

An /asset/18611 now shows 3 mediafiles (correct),
An /asset/18611/mediafile show 4 mediafiles (not correct)
I'm not able to play, download or view details of the fourth mediafile (correct).

Changed 3 years ago by robert

  • owner set to forgacs
  • status changed from reopened to assigned

Changed 3 years ago by forgacs

  • owner changed from forgacs to robert

Changed 3 years ago by forgacs

  • owner changed from robert to arjen

Changed 3 years ago by MC-arjen

  • status changed from assigned to closed
  • version set to 2.1.2
  • resolution set to fixed

/asset/{id}/mediafile was originaly forgotten. This patch also implements the same behaviour as /asset/{id}

Changed 3 years ago by Michiel.Schok

  • accepted changed from no to yes

Test: OK
3 mediafiles on a /asset/{id}/mediafile

Note: See TracTickets for help on using tickets.