Closed (fixed)
Project:
CCK Required Single Select (Obsolete!)
Version:
6.x-1.0-beta3
Component:
Code
Priority:
Major
Category:
Feature request
Assigned:
Reporter:
Created:
4 May 2010 at 04:15 UTC
Updated:
24 Mar 2012 at 09:41 UTC
Jump to comment: Most recent file
Comments
Comment #1
cwebster commentedHi Darrin,
Good idea. When I have a moment, I'll see if it's possible to add that functionality.
As to your other comment, I'm not really sure how they're generating that value. Maybe there's something that I'm not aware of, or they just rolled their own code.
Comment #2
Darrin Southern commentedThanks Chris.
Do you have a time frame as to when you can look at this ?
That way, I can be ready to test the code if you like . . .
Thanks again.
Comment #3
mgreen commentedI would find this very useful for a Node reference field as well.
Comment #4
mautumn commentedI thought this is what it did! :((
Node reference and user reference select lists are the most important for us. If you could find a way to achieve this that would be extremely useful for a great many people I'm sure. Why they were designed like that in the first place - automatically choosing the first one - beggars belief!
Comment #5
mautumn commentedSubscribing. We never use standard text select list, we always use node & user reference lists because they can be managed better i terms of permissions and ownership, roles, responsibilities, audit trail, etc. I would be great to have this very useful feature for these arguably much more important usecases...
Comment #6
kndrI am using my own, attached here, mini module to fix this issue. Every required, single nodereference and userreference optionwidget select item is cheating during form building phase and this way, option '- Select one -' could be added at first position :) This is the only way, I found, withouth patching optionwidgets module. String '- Select one-' could be changed in theme function MYTHEME_optionwidgets_none($field) inside MYTHEME template.php. This is native theme function from optionwidgets module and I think should be used here. I hope, this mini module will be helpfull for maintainers and others.
Comment #7
kndrComment #8
smithmilner commentedThanks Kndr
looks like it's working!
Comment #9
roball commentedAny progress on this, cwebster?
Comment #10
cwebster commentedThanks, kndr. Haven't had a spare moment to look into this. If anyone would like to be a co-maintainer, please let me know. I'll try to take a look over the weekend, if I"m able to.
Comment #11
cwebster commentedIt looks like this whole module may (thankfully) now be unnecessary. Please see http://drupal.org/node/849100.
I'm not sure it is a recent change that makes this functionality possible, but unless I'm notified of an unintended benefit of CCK Required Single Select, I'll soon discontinue it.
Thanks for all of the help!
Comment #12
roball commentedBut the suggested workaround does NOT work with Node and User references!
Comment #13
g.k commented+1 same problem here with node reference
Comment #14
roball commentedModule from #6 works fine! It just needs to be integrated into this module somehow.
Comment #15
cwebster commented@kndr: Had a moment to review your code. Seems to work! Can you explain this bit for me? (.module file, lines 41-53)
Usually I've seen "phptemplate" replaced with the module name. Is that not the case here?
Also, this seems like a completely separate module. Have you thought about becoming a contributor?
Comment #16
kndr@cwebster
Thanks for review. Name of function "phptemplate_optionwidgets_none" is build according to instructions inside http://drupal.org/files/theme_flow_6_1.pdf. (hook is the function and override http://drupalcontrib.org/api/function/theme_optionwidgets_none/6). I don't know if name "modulename_hook" will work.
Yes, this seem like a separate module but I don't want to be a contributor. My English is poor. I suggest to incorporate this code inside your module since it does, what many users expect.
Comment #17
djween commentedsame issue. subscribing.
Comment #18
nedwardss commentedThank you for this mini-module. It was a huge help.
Comment #19
sinasalek commented@kndr thanks for the module , works great
Comment #20
hanoiiMIght need this feature. Will review this module and the attached here and maybe see if I can contribute it to this module and have it all in one?
Not sure if this is needed but worth the try.
Comment #21
roball commentedkndr, your mudule works well and is still required to get this functionality, even when the "Required Single Select" module may not be required (however it is still useful). Could you please release it as an official module?
Comment #22
my-family commented#6 Works nice, THANK YOU!!! +1 for an official module
Comment #23
kndrThank you very much for testing my #6 minimodule and so many feedbacks. I am happy I could help someone :) As I said at #16 i don't want to be a contributor of official module but if someone wants to be I have nothing against to use my code. Everyone is welcome :)
Comment #24
roball commentedI have added the "CCK Required Reference Select" module to version 6.x-1.0 which has just been released.
kndr, Would you like to create a patch that integrates it into the main module, to be released in 6.x-2.0?
Comment #25
roball commentedComment #27
roball commentedkndr, would you please like to comment at this issue's follow-up issue #1498200: Merge "CCK Required Reference Select" sub module into main module ? Thanks!