Ticket #159 (closed enhancement: fixed)

Opened 4 years ago

Last modified 4 years ago

Support for free type of mediafile

Reported by: MC-arjen Owned by: MC-arjen
Priority: major Milestone: MediaMosa 1.7
Component: Core Version: 1.7.1
Keywords: Cc:
MoSCoW: Estimated time after impact analysis:
Related to project: Rich Media Tested:
Accepted: yes Estimated Hours:

Description

We need a mechanisme to identify types of mediafiles. We would like to have a free useable field for mediafiles. The mediafiles can be searched for example with

/asset/id/mediafile ?tag=

modify by:
/mediafile/id [POST] ?tag

The upload call should also support this parameter.

Q: do we need support in the playrequest call for this tag?

Change History

Changed 4 years ago by forgacs

Other search:
/mediafile/id ?tag=

Changed 4 years ago by forgacs

Because we support multiple stills per asset, and we store the stills (and other contents) as mediafile, these REST calls have the following behaviour:

This REST calls response all of the mediafile types (video, still etc.) under an asset or mediafile:
/asset/id/mediafile ?tag=
/mediafile/id ?tag=

This REST call can modifies all of the mediafile types (video, still etc.):
/mediafile/id [POST] ?tag=

Is it a good behaviour?

Changed 4 years ago by forgacs

I have changed this behaviour for backward compatibility:
I have added an additional parameter: type={ALL, VIDEO, STILL ...}
If the type parameter doesn't exist, then it get all the videos. If type=ALL, then it gets all of the mediafile type. The others (VIDEO, STILL ...) give the appropriate mediafile type.

Changed 4 years ago by forgacs

  • owner changed from forgacs to robert
  • status changed from new to assigned

The development is ready. Robert has to review the code.

Changed 4 years ago by Frans

Just curious if the code has been reviewed already....

Changed 4 years ago by robert

  • status changed from assigned to closed
  • resolution set to ready for review

Changed 4 years ago by robert

  • status changed from closed to reopened
  • resolution ready_for_review deleted

Changed 4 years ago by robert

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

Changed 4 years ago by Michiel.Schok

Waiting for complete documentation on mediamosa.org/api to test this new functionality.

Changed 4 years ago by Frans

  • version set to 1.7.0

Changed 4 years ago by forgacs

The documentation for the 'tag':

/mediafile/{mediafile_id} [POST]
http://mediamosa.org/node/3578

/asset/{asset_id}/mediafile [GET]
http://mediamosa.org/node/3554

/mediafile/{mediafile_id} [GET]
http://mediamosa.org/node/3577

Changed 4 years ago by Frans

This documentation solution does not work. Would like to see development release documentation also under the documentation API. Will contact Peter B. about this.

Changed 4 years ago by Michiel.Schok

  • status changed from closed to reopened
  • resolution fixed deleted

/mediafile/{mediafile_id} [POST] is working
/asset/{asset_id}/mediafile [GET] is working for tags

/asset/{asset_id}/mediafile [GET] is not working for type

/asset/Xt3WSGyJXBymSeSjen7bdN5i/mediafile?type=STILL [GET]

gives me all video's, just like ?type=ALL or no param at all.

/mediafile/{mediafile_id} [GET]
--> I can't think of a useful scenario for this call. It suggests that I already know the mediafile-id, and then can filter on a tag? But it's only one mediafile which has one tag. Besides, it doesn't work.

/mediafile/{id}
... <tag>preview</tag>...

/mediafile/{id}?tag=test
... i get the same mediafile
/mediafile/{id}?tag=preview
... i get the same mediafile

Changed 4 years ago by forgacs

  • owner changed from robert to MC-arjen
  • status changed from reopened to assigned

Changed 4 years ago by MC-arjen

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

The tag option for /mediafile/{mediafile_id} [GET] is not added to the api doc.

'type' does not work anymore, it is changed to is_still=true/false.

please retest in 1.7.1

Changed 4 years ago by MC-arjen

(The 'type' was introduced in 1.7, so no backward issues here...)

Changed 4 years ago by Frans

  • version changed from 1.7.0 to 1.7.1

[Core] #159: tag support bij mediafiles; changed the type parameter to
'is_still'.

Changed 4 years ago by Michiel.Schok

  • accepted changed from no to yes

Calls work as documented.

Further enhancements are possible. Please make a decision whether
1. this should be picked up in MediaMosa 1.7.2 or later;
2. they should be placed in other tickets. Feel free to copy text below.

For now, this issue is accepted.


In /asset/{id}/mediafile response when is_still=true, in the XML response, no still ticket is given. When we want to display the stills, additional REST-calls are necessary.

When using is_still=false, playlinks for mediafiles (ega_play_url) are given.

Note: See TracTickets for help on using tickets.