Needs review
Project:
Video Upload
Version:
6.x-2.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
9 Jul 2010 at 06:05 UTC
Updated:
19 Dec 2011 at 20:27 UTC
Jump to comment: Most recent file
Comments
Comment #1
bojanz commentedYes, I can confirm this. You need to delete it manually from "Manage video uploads" (until it's fixed)
Comment #2
Nick Robillard commentedSeems like this is still happening. Any updates here?
Update: I've done some debugging and I'm a bit perplexed. The cron delete mechanism is expecting the deleted node's cck entry in {content_VIDEO_UPLOAD_FIELD_NAME} to still exist and have the status "delete". However, this will never happen since this db entry is deleted on node delete. Was this never fully implemented or has something changed? The only way I can see this working is to track deleted videos (in hook_nodeapi probably) and populate a "deleted_videos" table which is then checked on cron.
Comment #3
Nick Robillard commentedHere's my implementation of external video delete support. It works but will need some additional work if the "abstract from youtube" approach is actually pursued. It looks like many things are unfinished with the module so maybe the original author had a different way in mind of handling deleting. Either way, this works for me.
What I've done is added a "video_upload_delete" table to store ids of videos that need to be externally deleted. Video ids are added to this table in the "delete" op of hook_field and respects the "Remove videos from provider if node is deleted, or video is changed" widget setting. Then on cron, (in function _video_upload_delete_rejected_videos) this table is checked and videos are deleted.