what it says, or maybe an option to display embed code link above or below the video, as a link, text, or textbox...

CommentFileSizeAuthor
#2 show_embed_code.patch3.41 KBTresler
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Tresler’s picture

Assigned: Unassigned » Tresler

I'm on it! Lord knows I should get something done today.

Tresler’s picture

FileSize
3.41 KB

About halfway done. Patch with progress attached but DO NOT APPLY as it isn't done. This is just a snapshot of progress.

Need to put some though into how to reconcile multiple formatters. I'll talk to dopry if I can snag him and work on this at hackfest tomorrow.

Tresler’s picture

As expected the hitch in this is how formatters behave with CCK. I talked with dopry and KarenS after getting to the Sheraton, basically at the moment the only way to make this a formatter is to duplicate each existing formatter with a '& show embedable code'.

I don't like that solution. I'm told that the ultimate solution to this is to re-think formatters to be more like widgets, but I need to look at the guts of CCK to figure out how to do that.

Reversing this it seems that if you can have multiple form fields per CCK_field type then you should be able to apply formatters and weight per form field, not per cck_field. This would mean that I could apply things like inline, hidden, above, below to the Field[name] and apply formatters to the form fields.

In the meantime, I'll throw together a duplicate set of formatters for the four we have, and start looking at the guts of CCK.

Cheers!

aaron’s picture

Assigned: Tresler » aaron
Status: Active » Fixed

i got this done last week. forgot you were working on it, tresler. that would have made my job a little easier, i suppose.

Tresler’s picture

Aaron - you manage to to write entirely new modules n the time I am 'working' on things.... so I wouldn't worry too much - cheers!

Anonymous’s picture

Status: Fixed » Closed (fixed)