Colorpicker CCK
christopher_skauss - January 3, 2009 - 14:59
| Project: | Colorpicker |
| Version: | 6.x-1.x-dev |
| Component: | Code |
| Category: | bug report |
| Priority: | critical |
| Assigned: | Unassigned |
| Status: | active |
Description
Hello and thanks for the great usability module!
What is the status of the colorpicker CCK? I have installed it, and added fields to a content type but it doesn't appear to do anything on the node add form.
#1
Testing required here, let me get back to you. One quick question, just to eliminate the obvious, do you have JavaScript enabled (required for this module)?
#2
Thanks for your quick reply!
yes, javascript is enabled. I am also getting this error when the module is enabled:
* warning: Invalid argument supplied for foreach() in F:\sitename\includes\menu.inc on line 258.
* warning: Invalid argument supplied for foreach() in F:\sitename\includes\menu.inc on line 258.
#3
This is probably caused by a bug discussed here: #352761: warning: Missing argument 1
#4
subscribe
#5
subscribe
#6
I confirm having the same error
container is undefined[Break on this error] return container.farbtastic || (contai...Query._farbtastic(container, callback));
#7
Hello, this should be a great module, but all of 6.x versions don't displays the CCK field if I added to a content type, and then I going to create a content with this type.
Otherway, this will be really a significant module for CCK, so keep up the good work!
Thanks.
#8
subscribe
#9
Sorry folks, I've been pretty busy with real life stuff outside of school. I haven't had a chance to look at this module for a while. I suspect it might be a while before I get back into active Drupal development. If you want to submit patches that you've tested, I'm sure Skilip (or myself if I get a free moment) can commit them.
Thanks,
Dave
#10
same problem ... subscribe
#11
and i thought i was losing my mind! the colorpicker and colortext fields also don't appear on edit node screens for me. subscribing...
#12
#13
Sorry guys, both dwees and me are too busy ATM for replieing faster in colorpockers issuequeue. I will do my best to fix this this WE.
#14
I needed a colorpicker cck field so I gave it a go - upgrading the cck module to work with Drupal 6. The module takes a slightly different approach than the orginal module.
I hope this helps get the module upgraded to work with Drupal 6 sooner.
Known bugs: only one field per node... not sure why multiple fields don't work. Perhaps someone has an idea?
Cheers,
Alex
#15
subscribe
#16
I've committed a RC1 wich is ready for testing.
#17
The new RC1 works for me...
The only thing that you might consider changing is how default values are handled when a colorpicker textfield is added to an existing node. At the moment it'll always default to red regardless of how you set the default value in the widget settings.
changing this line
<?php$value = isset($element['#value'][$field_key]) ? $element['#value'][$field_key] : '#FF0000';
?>
to this fixes it
<?php$value = isset($element['#value'][$field_key]) ? $element['#value'][$field_key] : $field['widget']['default_value'][$element['#delta']]['value'];
?>
But this probably means we need to ensure that a default value is always set.
Thanks for the good work...
Alex
#18
Just one problem found on RC1:
Using multiple values you can't delete fields. So if you add 3 colors to your node, you cannot remove any, just add more.
And it even adds one value every time you edit your node.
Thanks for the work anyway!
Claudio
#19
Thanks for reporting guys!
#20
subscribing