Is anyone already working on porting this to Drupal 5.0? I have gone through and done a lot of the work already though I haven't tested it yet. I should finish it in the next day or two once I have had a chance to test it.

CommentFileSizeAuthor
#5 mediafield-5.x-dev.xls11.35 KBscafmac
#3 patch_95.txt20.55 KBscafmac

Comments

ardas’s picture

ardas commented

As I know, noone is working. I was busy with some other stuff and had no chance to port it to Drupal 5.0. If you have already done a lot of work I'll not start this task.
Please, upload your 5.0 version when it is ready to this specially created issue http://drupal.org/node/112079, I'll make a code review and create a release then.

Thank you.

ardas’s picture

ardas commented
Status: Postponed (maintainer needs more info) » Active

Sorry :) I've confused tasks and created a superfluous issue. I've closed it already. Just upload 5.0 version to this issue. Thanks!

scafmac’s picture

scafmac commented
StatusFileSize
new patch_95.txt20.55 KB

Actually now that I'm actually testing it, I've hit a bit of a road block. I'm happy to upload the changes I've made, mainly just making styles standard, but also a few renamed functions and I found a bug or two, but it isn't working. I'm not sure the module is going to do what I need any way, so it might be better if you take what I've done and run with it.

Attached is a patch with all of the changes including two new .info files. They need to have the correct id lines added to the top of the file.

The problem I ran into was that when uploading a new audio file, it would act like it was uploading, but nothing would appear on the form or in the files directory. I tried debugging it, but I don't understand CCK well enough to be very productive at it. I hope the code helps a bit.

If you don't have much time, but have some suggestions, I'd be happy to keep plugging away at it some, I'm just at a loss of what to try now.

ardas’s picture

ardas commented

Thanks. I'll study it.

Could you please upload 5.0 version itself also - that would be easier for me to compare it with my one.
Thank you.

scafmac’s picture

scafmac commented
StatusFileSize
new mediafield-5.x-dev.xls11.35 KB

It will not let me upload a tar.gz file. So although this has an xls extension, it is really a gzipped tar file.

ardas’s picture

ardas commented
Version: » 4.7.x-1.x-dev

Thank you a lot for your work. I have reviewed everything and commited all changes to the HEAD revision.
There is a question concerning file_check_upload() usage:
why did you do 

      if ($file = file_check_upload('_upload'. $fieldname)) {

instead of

      if ($file = file_check_upload($fieldname. '_upload')) {

?

ardas’s picture

ardas commented

I've removed HEAD release from the site because it is not yet ready. We continue developing it ...

maastrix’s picture

maastrix commented

subscribing.. looking and waiting for this for months now...

scafmac’s picture

scafmac commented

Hey there - ya, I don't know what I was trying to do there. I was working on patches to the coder module while I was working on MediaField. Coder does some regular expression matches for syntax warnings. It was probably an unfixed test case. Sorry about that.

ardas’s picture

ardas commented

We did some work with these media fields and commited it into 5.0 and HEAD.
Please, download 5.0 release and try it. We fixed several major defects and added several features.

ardas’s picture

ardas commented
Assigned: Unassigned » ardas
Status: Active » Closed (fixed)