Support for native drupal workflow
sainzu - May 4, 2009 - 19:03
| Project: | Video Upload |
| Version: | 6.x-1.x-dev |
| Component: | Code |
| Category: | feature request |
| Priority: | normal |
| Assigned: | Justin W Freeman |
| Status: | needs review |
Description
The video should be uploaded ONLY IF the node (that contains the video) is marked as published. This is helpful in a scenario when an admin wants to review the uploaded videos before he allows to them to be published in the site. If the admin decides to reject the videos (say because of offensive content or irrelevant videos), he can simply delete it from his server. That way the video never reaches YouTube.
#1
#2
I agree with this.....
YouTube will cancel the account automatically if your videos cross the YouTube Community Guidelines twice. Anyone who has bad intention is pretty easy to upload some bad video and shut your service down without notice.
#3
I'll try to work this into the upcoming stable release.
#4
Great appreciated.
#5
Subscribing
#6
I too am greatly interested in workflow being implemented -- for the very reason happydrupal mentioned. Subscribing.
#7
I'm working on this currently I have a patch that works but I am adding one more feature to it first.
That feature is the ability to link the published status of the node to the private status of the You Tube video so once the video is uploaded, if you change the node to unpublished the video goes private and vice versa.
This second feature relies on
#439170: update youtube video if node is updated and
#611380: Call to undefined function _video_upload_node_set_video_status() in video_upload.admin.inc on line 421
So I will do those and then finish this.
#8
Here is the patch.
It gives two new options on the admin/settings/video-upload screen (at the moment they are in the You Tube fieldset but they could be outside of that if that is preferred).
One is to only copy videos up once the node they are on is published.
The second is to sync the nodes published flag with the you tube videos private flag.
You will need to apply the patches in these two issues before you apply this one as they are all required for the syncing of status to work:
#439170: update youtube video if node is updated
#611380: Call to undefined function _video_upload_node_set_video_status() in video_upload.admin.inc on line 421
#9
Thanks for doing this, Justin! I'll give this a shot as soon as I can...
#10
I just ran the three patches, and gave it a shot... and it appears to work as advertised. It wasn't a thorough review, by any stretch... but the video did not get uploaded, as desired, until after the node was set to published.
Good work!