carify the UI-only use of the embeddable key [#1672444]

Since reaction rules are obviously embedded in event sets (subclass of rule sets) via RulesRuleSet::rule(), rules_rules_plugin_info() should reflect that.

On the UI front, there is no impact since RulesEventSet itself can't be embedded, and there is no UI for an event set. So, there is other way to add a reaction rule than from the default tab provided in Rules UI.

This issue has arisen from #1671344-4: Setting parent of plugin to incompatible container removes its current parent.

Patch follows.

Comment	File	Size	Author
#1	1672444-01-Make_reaction_rules_embeddable_in_event_sets.patch	432 bytes	zhangtaihao
#1

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

zhangtaihao CreditAttribution: zhangtaihao commented 5 July 2012 at 12:08

Status:

Active

» Needs review

File	Size
1672444-01-Make_reaction_rules_embeddable_in_event_sets.patch	432 bytes

As promised.

Comment #2

fago

German

Vienna

CreditAttribution: fago commented 3 August 2012 at 08:21

Title:	Reaction rules should be embeddable in event sets	» carify the UI-only use of the embeddable key
Component:	Rules Core	» Documentation
Status:	Needs review	» Fixed

As said over there, embedded is only used by the UI. That way we can actually build configs that embed other stuff than as what it is allowed in the UI, what might be useful in some cases - as in this one: Building evaluation optimized configs.

I've clarified the UI-only use of the embeddable key in the in-code docs.