Closed (fixed)
Project:
Video
Version:
7.x-1.x-dev
Component:
Code
Priority:
Major
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
20 Mar 2011 at 00:21 UTC
Updated:
25 Jul 2011 at 06:10 UTC
Jump to comment: Most recent file
I have chosen the private filesystem for the video node. Thumbnails directory is created in the public directory instead of the private directory.
And I get the following error message:
Warning: array_flip() [function.array-flip]: Can only flip STRING and INTEGER values! in DrupalDefaultEntityController->load() (line 178 of D7/includes/entity.inc)
| Comment | File | Size | Author |
|---|---|---|---|
| #16 | 1098908_13.patch | 2.88 KB | hypertext200 |
| #13 | 1097836-12.patch | 1.37 KB | hypertext200 |
| #7 | 1098908-video-private_files.patch | 1.27 KB | dawehner |
Comments
Comment #1
anglo commentedThe array_flip() error comes up both for private and public filesystem. But in the public filesystem videos play just fine, but not in private.
Comment #2
anglo commentedVideos with the private filesystem are not played in the master version either (but they do get encoded).
This is what I see in the logs:
page not found 03/22/2011 - 18:19 system/files/videos/converted/81/SDC11277.mp4
Comment #3
anglo commentedSame for the version 7.x-1.0-alpha1. Videos are not playing (for private filesystem) though they get encoded.
Comment #4
JayKayAu commented(Resetting to Master as it affects all versions)
Subscribe.
Comment #5
hypertext200yes, it seems I'm using Drupal's file system not the Field settings. I will fix that, or patches are welcome.
Comment #6
dawehnerThe problem in general is that no module want's to take over the ownership of the .flv files/The converted files in the filefolder.
So video.module has to implement hook_file_download and check access there.
There it has to act like the filefield variant of this hook but use another path: the converted file.
So i think from the flv file the original cck filefield entry has to be loaded, and there the access has to be checked.
(Optional) it would make sense to add the converted video into the fielfield table.
This problem is existant for 6.x-4.x as well.
Comment #7
dawehnerSo here is a patch.
At the end it makes sense and works fine.
Comment #8
dawehnerBy the way this is against 6.x-4.x, but the port shouldn't be too hard, once the initial problem could be figured out.
Comment #9
J0keR commentedsubscibe
Comment #10
z3cka commentedsubscibe
Comment #11
Randall Shelley commentedsubscribe
Comment #12
anglo commentedDoes anyone have a patch for 7.x-1.x ?
Comment #13
hypertext200Comment #14
dawehnerAre you sure you posted the patch against the right version?
Update priority
Comment #15
hypertext200Changing to 7.x-1.x
Comment #16
hypertext200That was a wrong patch. Attaching the correct one
Comment #17
anglo commentedJust used the 7.x-1.x-dev dated April 22, 2011 - 16:20. Same thing. Videos get converted. Stream not found.
Comment #18
dawehnerYou have to use the patch uploaded above.
Comment #19
anglo commentedIt is already integrated into the 7.x-1.x-dev dated April 22, 2011 - 16:20 that I used.
Comment #20
hypertext200Closed #1020444: private filesystem not supported as duplicate. As far as I identified we have couple of issues in private file system.
1. Converted videos not passing through private file system.
2. Field settings for file system not using inside a field. That lead the thumbnail generation in public.
any more issues?
Comment #21
hypertext200We have fixed both of those issues in 7.x-2.x, so upgrade to the latest version will solve the problem.
Comment #22
hypertext200