Closed (fixed)
Project:
Chaos Tool Suite (ctools)
Version:
7.x-1.x-dev
Component:
Code
Priority:
Major
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
1 Aug 2012 at 07:57 UTC
Updated:
15 Feb 2013 at 20:50 UTC
Jump to comment: Most recent file
Comments
Comment #1
amitaibuOG is also going to use it, see screenshot of sub-widgets in #1730678: Provide a "complex" widget instead of "Primary" and "Secondary" fields.
Comment #2
merlinofchaos commentedThis patch only populates $field if $field_name is an array. That seems like it could lead to errors in other use cases.
Comment #3
amitaibuThe reasons are:
What other solution do you prefer?
All i know is that atleast my current use case(s) for OG and OG-vocab, which creates "sub-widgets" is actually working really nicely thanks to how cool CTools and Field API are :)
Comment #4
amitaibuRe-read, and you are of course correct, ignore #3, patch coming soon :)
Comment #5
amitaibuComment #6
amitaibuHi guys, OG vocab 7.x-1.x is now using it as-well. Any chance getting this in soon, so testbot won't fail all my tests?
Comment #7
amitaibuHow many mistakes can I make in 3 line patch :/
Comment #8
amitaibuGentle bump :)
Comment #9
skyredwangtested #7 patch. didn't see anything wrong.
Comment #10
geek-merlinreviewed code and can confirm it should do what it says and not harm previous code flow.
so ready to fly.
Comment #11
geek-merlinrising prio as this blocks a stable release of og_vocab as of #6.
Comment #12
merlinofchaos commentedCommitted and pushed!