Closed (fixed)
Project:
Chaos Tool Suite (ctools)
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
15 Jun 2015 at 12:50 UTC
Updated:
9 Feb 2021 at 22:19 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
MerryHamster commentedComment #2
MerryHamster commentedComment #3
maximpodorov commentedMaybe some wordings corrections are necessary:
"a relationship"
"Parent comment" - to indicate that the result of the relationship is comment, not node. The same change should be made elsewhere.
Comment #4
MerryHamster commentedОк, I added changes.
Comment #5
MerryHamster commentedComment #6
maximpodorov commentedIt works and looks reasonable.
Comment #7
damienmckennaComment #8
japerryWhy have a blank settings form? This is a feature request also, not a bug.
Comment #9
damienmckenna@japerry: You need a blank settings form function otherwise the Panels UI won't give you the settings item in the context menu.
Comment #10
damienmckenna.. though after looking at the others in plugins/relationships I'm not sure it's needed.
Comment #11
damienmckennaThis patch removes the empty settings form.
Comment #12
maximpodorov commentedNo, no. You removed the form but left the reference in $plugin array.
Comment #13
maximpodorov commentedRequirements for forms are so various across the different plugin types!
Comment #14
damienmckennaAh, thanks for noticing that.
Comment #15
damienmckennaMoving this to the v7.x-1.10 release plan.
Comment #16
maximpodorov commentedI renamed your patch to have more descriptive name.
Comment #17
damienmckennaThis didn't get added to 7.x-1.10.
Comment #18
chris matthews commentedThe 3 year old patch in #16 to comment_parent.inc applied cleanly to the latest ctools 7.x-1.x-dev and if still relevant needs to be added to the #2984057: Plan for CTools 7.x-1.15 release.
Comment #19
joelpittetComment #20
joelpittetIf someone following this and is using the patch can RTBC this, I'll add it in the next release.
Comment #21
andrtroe commentedI will check it.
Comment #22
rivimey@andrey ... bump?
Comment #23
joelpittetComment #24
joelpittetBump too, still looking for an RTBC from someone using this feature
Comment #25
joelpittetComment #26
joelpittetI'll trust it works, thanks @DamienMcKenna, @MerryHamster, et al, I've committed this to the dev branch for 1.18 release.