I noticed that flashvideo_import never bothers to check for FileField (and the corresponding path elements it defines)
I didn't see any convenient way through hooks (ie flashvideo_cck) to change this either...
To that end, here's a proposed patch to make flashvideo_import respect the path settings from the original video FileField
Also, without something better than $account to pass to token_replace, user tokens are fairly useless, though at least they'll be filled out...
(and should be consistent with existing functionality in flashvideo_convert, I think?)
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | flashvideo-import_cck_paths-2.patch | 1.98 KB | Vector- |
| flashvideo-import_cck_paths-1.patch | 1.98 KB | Vector- |
Comments
Comment #1
Vector- commentedNOTE: This does not enable token processing support in the import path itself! (not yet at least ;)
Also flashvideo_import assigns file ownership to user-1...
As such, if using the patch proposed at #812510: Proper token generation for FileField paths without 'Convert video immediately' paths will be inconsistent, since $account in this context is the user executing the cron job
Here's a different version of this patch that generates tokens based on user-1 to provide consistent results if that patch is also in use:
(this would be more useful with a way to decide ownership of imported nodes... something else I might end up getting around to)
Comment #2
Vector- commentedPS: both of these patches contain a rerolled (and considerably more readable?) version of #655360: flashvideo_import munges multisite paths too
I probably should have pulled that though... sorry :p