Closed (fixed)
Project:
FFmpeg Converter
Version:
6.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
22 Feb 2009 at 00:04 UTC
Updated:
8 Mar 2009 at 11:10 UTC
Jump to comment: Most recent file
Long time drupal user, first time bug submitter... lol.
Repeatable: Always
Steps to repeat:
1. Download and Install ffmpeg converter (and ffmpeg wrapper 6.x-1.x) and Job Queue 6.x-3.0
2. Create New Video file type.
3. Create a new new with the new type
Expected Results:
A new job should be added to the queue when the new node is created.
Actual Results:
No new job is added. User has to edit the node to add the job to the queue.
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | ffmpeg_converter.patch_01.patch | 3.52 KB | K-Max |
Comments
Comment #1
K-Max commentedAnd first time patch submitter. :p
Just a very minor spelling mistake fix that will fix the bug.
Also takes care of: #379628: Log message refers to the wrong file
Comment #2
zoo33 commentedFirst time bug submitter, and the best kind of bug submitter there is: the kind that fixes the bug! Thank you!
The patch fixes two obvious mistakes, so I'm committing it, with the following changes:
There were a few changes to whitespace, like line breaks and indentation, which is irrelevant to this patch. Also, tabs should not be used for indentation according to the Drupal coding standards. (Only saying this to save you some trouble the next time you submit a patch. Oftentimes maintainers won't review your patch until it's flawless in those respects.)