Closed (fixed)
Project:
Conditional Fields
Version:
6.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
24 Apr 2008 at 15:24 UTC
Updated:
14 Nov 2013 at 00:07 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
clemens.tolboomComment #2
peterpoe commentedI tested the patch. It seems to work fine on node edit, but on node view the controlled field is never visible even if triggered.
Comment #3
clemens.tolboomThat is because of #250710: Not all fields with valid conditions are displayed when viewing node ... patch is underways
Comment #4
dawehnerthe following patch worked here:
you have to apply
http://drupal.org/node/250748#comment-819833
and
http://drupal.org/node/250710#comment-860312
Comment #5
patchak commentedI've applied the three patches and it works fine, all the fields are showing even when using a taxonomy field controller, I think we should commit and release a beta version to keep track more easily of all the changed that were made recently??
Patchak
Comment #6
socialnicheguru commentedsubscribing
Comment #7
patchak commentedPlease commit this one too!!! ;)
Comment #8
dawehneras a comment to this, i think it should be a patch in content taxonomy than making a patch to conditions fields
// The problem is that content taxonomy doesn't use '#value'
Comment #9
Taras Zavaliy commentedThis is great! Thank's for these patches, they are working just as needed (so far).
Comment #10
patchak commented@ dereine, so you think we should change the project for this issue ?? But does the patch needs to be redone to be working with content taxonomy ???
Comment #11
tancYes, these patches work for me also. Using the dev version of Content Taxonomay now includes the patch: http://drupal.org/node/250710#comment-860312
So the only patches to make this work are the two in this issue, in comment #1 and comment #4.
Here is a single unified patch of the two required patches against the dev version of this module.
Comment #12
Anonymous (not verified) commentedThis patch does not seem to respect the 'Specify terms to show' in the Content Taxonomy Field window.
Aside from that, if your Vocabulary is enabled as a hierarchy, the patch will also ignore any child terms.
txcrew
Comment #13
sam_RiteTimeDirect commentedI am new to this. How would I go about impementing the patch.
I am getting the error below. I am using Drupal 5.x. Will it help with this issue?
Warning: Call-time pass-by-reference has been deprecated; If you would like to pass it by reference, modify the declaration of conditional_fields_set_required_for_render(). If you would like to enable call-time pass-by-reference, you can set allow_call_time_pass_reference to true in your INI file in /home/content/r/i/t/ritetimedirect/html/sites/all/modules/conditional_fields/conditional_fields.module on line 899
Thanks,
Sam
Comment #14
mitchell commentedMoving to 6.x
Comment #15
izarco commentedhi, any solution for use this module with content taxonomy ,or better, with core taxonomy?
If I can help on something, I am willing to help.
Comment #16
yngvewb commentedIs this on the way in 6? I really need content taxonomy support too. PS. great module!
Comment #17
Heilong commentedAlso need it under D6.
Subscribing.
Comment #18
andyhu commentedHi, I've made a patch for D6, please try
Drupaler China
Comment #19
Heilong commentedHi,
I applied the patch, it's working when the fields are filled but there is a warning message when the fields are empty.
Please check this thread : http://drupal.org/node/250748#comment-1667274.
Comment #20
zythyr commentedPlease have Conditional Fields be compatible with Content Taxonomy as soon as possible. Thanks.
Comment #21
dawehnercome one, critical is everything else. Does this missing feature break your full site? NO!
So its not critical.
I Guess the #18 patch works here, but needs some more work so help there.
Comment #22
matteogeco commentedSubscribe
Comment #23
andyhu commentedCan you let me know how to reproduce the issue? I've tested creating a node using an empty vocabulary and there's no error message. Thanks.
Andy
Comment #24
shiolo commentedIt would be great to be able to use content taxonomy with conditional fields...subscribing
Comment #25
peterpoe commentedChanging component, and just letting know that Conditional Fields 1.0 will be out next week if no new bug is found!
Comment #26
zythyr commentedGreat to hear that. I am waiting for the release, can't wait!!! Conditional field + content taxonomy is a make/break feature on my site so I am really looking forward to your release so I can continue working on my site.
Comment #27
x-lette commentedHi,
there is a strange behaviour which I can't correctly assign to content_taxonomy or conditional_fields (or the special connection between them).
I have four fields in one group, with the first field controlling the latter three. For each controlled field there is exactly one entry in the controlling field and all four fields are of type content_taxonomy.
When editing a node, everything looks fine, selecting one of the controlling values triggers the controlled field to be shown. But even in preview and also after saving the node the value of the controlled field is not shown while the value of the controlling field is visible.
I played a little with the available options and two settings are making this field visible again:
For me this looks like the controlled field is treated like an orphaned one but in fact it isn't. Actually I use the second example and also turned on the option "Reset untriggered fields to default values" so there should not be any real "orphaned" fields. But I guess, this is not the way it was supposed to be.
Does anybody know about this behaviour? Might there be some other settings where I could change this behaviour? Or is this a bug?
I'm using D 6.14, conditional_fields dev as of aug, 16th, the patch from #18 and content_taxonomy 1.0rc2
(I hope this is not the wrong place for this question but there is actually no better place I know of)
Comment #28
Bilmar commentedI look forward to help testing patches for new issues.
I am also exploring conditional field + content taxonomy for my website at the moment.
Comment #29
EgorfromMurmansk commentedsubscribing
Comment #30
mairav commentedSuscribe
Comment #31
peterpoe commentedPlease hold on while #648934: Plugin system for compatibility with non-core CCK modules is developed.
Comment #32
xqbzzr commentedSo, i installed both the latest dev versions of CF and CT.But there is no difference. I cannot set the CT-Field as controller. Neighter can I drop a regular CF in another fieldgroup. I thought the 2.dev Version included these changes?!?
Thank you for contributing this!
Comment #33
rburgundy commentedI am needing this very much and can't wait for development of this feature.
Glad to see there is many eyes on this so testing will be fast.
Subscribing
Comment #34
benone commented+
Comment #35
robby.smith commented+1 this is the last feature stopping me from launching my new website. looking forward to updates
Comment #36
andyhu commentedHi everyone,
It seems that the maintainer haven't noticed this issue. For my patch at #18, if there's any issues you could contact me at here at http://drupal.org/user/172337/contact? I'm willing to provide fixes for my patch.
Cheers,
Andy
Comment #37
Bilmar commentedhello andyhu,
I am very glad to hear that you will be involved in really getting the compatibility between conditional fields and content taxonomy going for the drupal community.
I will be testing your patch at #18 tonight and will come back with feedback.
Below are some comments from people in this thread so far:
#19 http://drupal.org/node/250748#comment-1689000
#27 http://drupal.org/node/250748#comment-2145846
Thanks for your hard work!
Comment #38
Bilmar commentedPatched #18 without any problems. I set a content taxonomy as controlled field and it worked. I will be doing more tests in the coming week.
Thanks for the great work!
Comment #39
rburgundy commentedChanging status to 'needs review'
Comment #40
rburgundy commentedTested #18 and everything worked great. Thank you
Comment #41
xqbzzr commentedUsed the latest dev of CT and CF. Applied patch #18 manually and everything works as expected. Thank you! Can we nail this as a new dev release?
Comment #42
rburgundy commentedit would be awesome to see this in the latest dev. thanks again for the great work
Comment #43
samkat12 commentedHI,
I am new to drupal and without much knowledge with applying patches. How do i get the Conditional field to work with content taxononmy. i am desparate. A quick response will be welcome. any advise will be appreciated.
Comment #44
samkat12 commentedsorry my question really is how did you apply the patch maunally. did you apply it in the content taxonomy module or conditioanal field module.
I am curiouse if it would help if you just attched the full code from the module you patched. that way i can just copy and paste. or you can give me some tips how you manually patched the moduules in question.
Comment #45
peterpoe commentedYou have to apply the patch to Conditional Fields.
Documentation for applying patches manually: http://drupal.org/node/534548
Comment #46
samkat12 commentedThanks. I downloaded both dev versions of conditional fields and content taxononmy. and then applied the patch from#18. Worked like a cham. i was terrified about how to apply a patch but the manual application worked well for the first time.thanks again.
Comment #47
robby.smith commentedtested and works great! little more testing needed before commit?
Comment #48
thekayra commentedsubscribing.
Comment #49
thekayra commentedTried out patch #18 with non-dev versions of Content Taxonomy (6.x-1.0-rc2) and Conditional Fields (6.x-1.0). Working smoothly. did not run into any issues during the preliminary tests.
Comment #50
rburgundy commentedapplied patch and tested on my website without any problems.
hoping this gets committed soon. awesome module! regards
Comment #51
aron novakI re-rolled a patch so now it applies without fuzz.
I did not modify on the code, the code from #18 (andyhu) is perfect, so i also confirm that it works and RTBC.
Comment #52
benshell commentedI just applied this patch and it works for me.
Comment #53
samkat12 commentedcoditional fields do not play well when put into cck Group.
when you put both the controlling field and the controlled field in a group.The controlled field does not hide it stays open.Therefore useless since you only want the controlled field to show when triggered by the controlling field.This might be a conflict of javascript...am not sure. Any help would be appriciated.
Comment #54
my-family commentedsubscribing
Comment #55
samkat12 commented6.x-2.x-dev works. my issue #53 is been solved. THANK YOU VERY MUCH FOR THE PROMPT RESPONSE TO THIS ISSUE
Comment #56
smartchecker commentedI have CCK Taxonomy Subset in my content type.
When I try to control it from selection list it does not work.
Comment #57
Bilmar commentedfyi - the patch only works against 6.x-1.x-dev. Just tested and it works fine.
I tested the newest 6.x-1.1 and 6.x-2.x-dev and content taxonomy was still not supported.
Applying the patch at #51 against 6.x-2.x-dev results in error:
Comment #58
thekayra commented@trupal218:
I tried the patch #18 with non-dev versions of Content Taxonomy (6.x-1.0-rc2) and Conditional Fields (6.x-1.0).
I tried putting the controlling and the controlled fields in the same fieldgroup. The controlled fields (checkbox) were hidden as expected, so it worked quite ok for me.
I don't know if something has happened with Conditional Fields (6.x-1.1) but I am scared to upgrade it on my live site right now. Once I have some time, I will try it on my demo side and let you know.
Comment #59
sotiris commentedHello,
I have installed CF 6.x-1.1 and Content Taxonomy 6.x-1.0-rc2.
I want to make a content taxonomy field controlling a text/simple select field, but i can't.
I can see Conditional Field settings only in the taxonomy field with result to have this field behave as a controlled field.
In the text/simple select field CF settings are missing.
If anyone can help me, will be much appreciated.
EDIT:
Finally i installed 6.x-2.x-dev and patched it manually with the patch from #51 and now is working. I can use my Content Taxonomy field as a Controller.
My only problem is that it doesn't work with Hierarchical Select widget, and i have to figure out a way to make selection from my subcategories.
Thanks for the patch!
Comment #60
thekayra commentedStrangely enough, once I upgraded to CF 6.x-1.1, WITHOUT applying any patch, even the small problem I had with required fields (on CF 6.x-1.0 plus patch #18) vanished.
So I can use Controlling and Controlled fields in the same group with Content Taxonomy (controlling and controlled fileds), one main field controlling approximately 10 other fields with checkboxes. So far, I have not seen any bugs and will keep it this way but let you guys know if I run into any issues.
PS: I do not think that the patch I have applied on CF 6.x-1.0 has anything to do with CF 6.x-1.1 running smoothly, but I might be wrong.
Comment #61
robby.smith commented@thekayra - do you mean that the patch is included in the latest dev version of Conditional Fields?
Comment #62
aboros commentedi used the patch in #51 against conditional_fields.module 6.x-1.1
the patch finished without any errors and solved the issue.
Comment #63
thekayra commented@robby.smith:
Yes. However I played a little too much with my modules at that time. So it is better if somebody else can confirm this.
Comment #64
peterpoe commentedThe latest 2.x-dev includes a patch that makes Conditional Fields fully compatible with Content Taxonomy (and any other module that uses a custom allowed values function). The code used to obtain this was adapted from the optionwidgets module and looks like this:
Enjoy!
Comment #65
benone commentedyeee :)
Comment #66
Bilmar commentedthanks peterpoe!
Comment #67
YK85 commentedEDIT: the issue I had posted before was due to my incorrect setup of the taxonomy vocabulary. Setting status back to Fixed.
Thank you!
Comment #68
YK85 commentedComment #70
xjmJust wanted to say this is an awesome feature. (The 2.x branch is a great step up all around.) Thanks so much for your work on this.
Comment #71
momper commentedalso - thank you very much
Comment #72
enkara commentedCan you tell if this is going to be part of the beta or the stable release soon? Is it very dangerous to use the dev? Thank you!
EDITING: Sorry I thought it had to work with taxonomy module, not content taxonomy. It's all ok.
Comment #73
Lloyd commentedA little confused. Does the current build of 6.x-2.x-dev provide compatibility with Content Taxonomy natively, or does it require it to be patched?
fyi I do have 6.x-2.x-dev of conditional fields installed along with the most recent 6.x-2.x-dev version of content taxonomy. A content taxonomy select list field is not one of the controlling fields available.
Comment #74
xjm#73: I am using a content taxonomy field as a controlling field with the current 6.x-2.x-dev and it works fine. Maybe there is a different reason you can't add the field?
Comment #75
Lloyd commentedIs there anything special that needs to be done when adding the content taxonomy field? There aren't any "allowed values" specified since it just pulls it from the underlying taxonomy, so I'm not sure if that's the problem. I've tried using both select list and checkbox fields, and different taxonomies.
Comment #76
sunshinee commentedsubscribing
Comment #77
cposer commentedI have a problem using two content taxonomy fields.
Althoug I can specify one term of the one field to be the condition for the other (and it is displayed correctly in the 'manage fields' overview), on editing a node of this type the second field is displayed whatsoever, and not hidden except for when the specified term is selected.
FYI: The two fields use the same vocabulary, only one represents the parent-terms and one the child-terms. I can't use Hierarchical Select Content Taxonomy because the widget is not supported by multigroups, so I figured this might be a solution.
Any ideas?
Comment #78
peterpoe commented#73: Re-tested with Conditional Fields 2.0 and Content Taxonomy 2.x-dev, works fine for me. Please read the documentation to ensure that your setup is correct and retry, if the problem persists feel free to reopen this issue.
#77: Probably your problem is with multigroups, see #357506: Compatibility with CCK 3x multigroup.
Comment #80
cposer commented#78 I'm using the multigroups module, yes, but the fields I'm talking about are in normal groups.
Comment #81
cposer commentedI have found the explicit scenario in which the problem occurs:
CF has problems with Content Taxonomy when the fields are on different hierarchical levels due to grouping.
As long as the controlling CT-Field is on the base level and the depending field within a standard group, everything works as it should.
But putting this whole construct into another standard group, so that the controlling field is in the second level and the controlled field on the third, it's not working any more and the depending field is displayed whatsoever.
I haven't tested it with deeper layers and different forms of nesting, but I guess the result would be the same.
Please help =)
Comment #82
xjm