Posted by zuriwest on July 9, 2010 at 6:05am
3 followers
| Project: | Video Upload |
| Version: | 6.x-2.x-dev |
| Component: | Code |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | needs review |
Issue Summary
A video doesn't get deleted from Youtube, even though the "remove from provider if node is deleted" option is checked.
I execute the following steps:
1.) Delete node containing video
2.) Run cron manually
Comments
#1
Yes, I can confirm this. You need to delete it manually from "Manage video uploads" (until it's fixed)
#2
Seems 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.
#3
Here'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.