Ticket #454 (closed defect: ready_for_review)

Opened 2 years ago

Last modified 2 years ago

[workflow] node edit & checkbox delete video

Reported by: peter Owned by: robert
Priority: major Milestone: WLE modules into D6
Component: WLE Version:
Keywords: Cc:
MoSCoW: Should Have Estimated time after impact analysis:
Related to project: none Tested: no
Accepted: no Estimated Hours: 0

Description (last modified by MC-arjen) (diff)

Current situation: the checkbox "delete video" is checked, the node is saved but nothing happens in mediamosa

Problem: pollution of mediamosa

Desired situation:

1) The checkbox "Delete the Video" should also result in an empty asset.
2) The position of the checkbox should be in the fieldset "Attach a media file to upload MediaMosa".

3) When the checkbox has been set the mediafiles & stills should be removed. The asset should remain because of the metadata that can be available.

How to implement:

You can not use /asset/delete here. Because you delete the asset, and the problem is that we remove the asset while the node is still active. Also deleting the asset will remove the metadata.

Use /asset/$asset_id/mediafile call to get the asset mediafiles and figure out the mediafiles that are under the asset. Also delete the still(s) as well, so all we want to delete all mediafiles under this asset.

Loop the found mediafiles.

Delete the mediafile with /mediafile/$mediafile_id/delete and use parameter 'delete' with value 'cascade'. Could be that if you try to delete a mediafile and its already deleted, it will give back a failure. Then ignore the failure; the mediafile is gone anyway. Remember you need to provide the owner of the mediafile you want to delete.

Change History

Changed 2 years ago by peter

  • owner set to Robert
  • status changed from new to assigned

Changed 2 years ago by robert

My notes on implementation of this ticket;

None.

Changed 2 years ago by peter

  • owner changed from Robert to t1mm1

Changed 2 years ago by peter

  • owner changed from t1mm1 to Robert

Changed 2 years ago by MC-arjen

  • owner changed from Robert to t1mm1

Changed 2 years ago by MC-arjen

  • description modified (diff)

Changed 2 years ago by t1mm1

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

1. Done.
2. Done.
3. Done.

Changed 2 years ago by forgacs

  • status changed from closed to reopened
  • resolution ready_for_review deleted

Changed 2 years ago by forgacs

  • owner changed from t1mm1 to robert
  • status changed from reopened to assigned

@Robert: It is ready. Please, review the code.
You can find the latest version here:
 http://freya/hg/module/mediamosa/ck/6/

Changed 2 years ago by forgacs

  • status changed from assigned to closed
  • resolution set to ready_for_review
Note: See TracTickets for help on using tickets.