Ticket #455 (closed defect: ready_for_review)

Opened 2 years ago

Last modified 2 years ago

[workflow] node delete

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

Current situation: I have upload a video to a node. This result in a asset + mediafiles. Then I edit the node and set the checkbox "Delete the Video" and save the node. I will edit the node again and delete it. The overall result will be that the node is gone but the asset + mediafiles still exist.

Problem: A part of the problem is #454. the other part is that the asset_id is not saved with the node.

Desired situation: Delete the node results in deletion of the asset. After executing #410 we know which the relation between the node and asset. This relation needs to be used to delete the right asset.

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;

  • Do not store the asset_id on the node (as you might think its asked here in ticket). Store the node_id in the reference_id of the asset. So there is only a connection from asset to node. This forces you to delete the asset if the node is deleted. When asset is throw away outside drupal, then recreate the asset when needed.

Changed 2 years ago by forgacs

  • owner changed from Robert to robert

Node delete was fixed.

@Robert: Needs review.

@Peter, Robert: It is not clear for me, why we store the node_id in the asset. But I left the code as it is.

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.