Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Hi,
I have a patch to use content_taxanomy as a controller field.
Comment | File | Size | Author |
---|---|---|---|
#18 | conditional_fields.module.patch | 1.7 KB | andyhu |
#51 | 250748-49_content_taxonomy.patch | 1.68 KB | Aron Novak |
#11 | conditional_fields_unified.patch | 2.58 KB | tanc |
#4 | condition_fields_content_taxonomy_view.patch | 902 bytes | dawehner |
#1 | conditional_fields_content_taxonomy_250748.patch | 1.65 KB | clemens.tolboom |
Comments
Comment #1
clemens.tolboomComment #2
peterpoe CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: SocialNicheGuru commentedsubscribing
Comment #7
patchak CreditAttribution: 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 CreditAttribution: Taras Zavaliy commentedThis is great! Thank's for these patches, they are working just as needed (so far).
Comment #10
patchak CreditAttribution: 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) CreditAttribution: Anonymous 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 CreditAttribution: 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 CreditAttribution: mitchell commentedMoving to 6.x
Comment #15
izarco CreditAttribution: 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 CreditAttribution: yngvewb commentedIs this on the way in 6? I really need content taxonomy support too. PS. great module!
Comment #17
Heilong CreditAttribution: Heilong commentedAlso need it under D6.
Subscribing.
Comment #18
andyhu CreditAttribution: andyhu commentedHi, I've made a patch for D6, please try
Drupaler China
Comment #19
Heilong CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: matteogeco commentedSubscribe
Comment #23
andyhu CreditAttribution: 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 CreditAttribution: shiolo commentedIt would be great to be able to use content taxonomy with conditional fields...subscribing
Comment #25
peterpoe CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: EgorfromMurmansk commentedsubscribing
Comment #30
mairav CreditAttribution: mairav commentedSuscribe
Comment #31
peterpoe CreditAttribution: peterpoe commentedPlease hold on while #648934: Plugin system for compatibility with non-core CCK modules is developed.
Comment #32
xqbzzr CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: benone commented+
Comment #35
robby.smith CreditAttribution: robby.smith commented+1 this is the last feature stopping me from launching my new website. looking forward to updates
Comment #36
andyhu CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: rburgundy commentedChanging status to 'needs review'
Comment #40
rburgundy CreditAttribution: rburgundy commentedTested #18 and everything worked great. Thank you
Comment #41
xqbzzr CreditAttribution: 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 CreditAttribution: rburgundy commentedit would be awesome to see this in the latest dev. thanks again for the great work
Comment #43
samkat12 CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: peterpoe commentedYou have to apply the patch to Conditional Fields.
Documentation for applying patches manually: http://drupal.org/node/534548
Comment #46
samkat12 CreditAttribution: 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 CreditAttribution: robby.smith commentedtested and works great! little more testing needed before commit?
Comment #48
thekayra CreditAttribution: thekayra commentedsubscribing.
Comment #49
thekayra CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: benshell commentedI just applied this patch and it works for me.
Comment #53
samkat12 CreditAttribution: 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 CreditAttribution: my-family commentedsubscribing
Comment #55
samkat12 CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: robby.smith commented@thekayra - do you mean that the patch is included in the latest dev version of Conditional Fields?
Comment #62
aboros CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: benone commentedyeee :)
Comment #66
Bilmar CreditAttribution: Bilmar commentedthanks peterpoe!
Comment #67
YK85 CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: momper commentedalso - thank you very much
Comment #72
enkara CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: sunshinee commentedsubscribing
Comment #77
cposer CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: cposer commented#78 I'm using the multigroups module, yes, but the fields I'm talking about are in normal groups.
Comment #81
cposer CreditAttribution: 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