Conditional Actions needs to be ported for this to work.
This includes: writing unit tests and the actual upgrade.

Comments

rszrama’s picture

We'll need to follow-up w/ Jody on this, since she was down for the count during the meeting. I think TR, Jakob, and Mike expressed varying degrees of interest in this work, too. I'm obviously available since I wrote the bulk of it. This work should probably begin with Damien's patch, #548144: Refactor CA use of menu items and forms. This might be out of scope for now, but we'll also want to use the plugin system provided by Chaos Tools for this... but I don't know where the D7 port of that is.

damien tournoud’s picture

We don't need the plugin system of ctools anymore. We have hook_hook_info() now!

rszrama’s picture

Issue tags: +sprint-1

Awesome. : )

jody lynn’s picture

Mike's priorities he emailed me:

My general thought are
1) update for d7, and the coding standards
2) add some additional drupal_alters, so it is a little more flexable.
3) make it ctools / features exportable
4) adjust the code so it load default predicates to the db so they can be placed in an include similar to hook_views_default_views

I'm going to start with #1.

jody lynn’s picture

Assigned: Unassigned » jody lynn

Assigning to myself to get this started

jody lynn’s picture

Also: needs simpletests

rszrama’s picture

I'm wondering if we should make this into a [meta] issue that collects CA related tasks or simply turn this into a D7 update issue and then discuss / patch the alters, exportables, and tests in their own issues. Anyone have a preference?

jody lynn’s picture

I opened up a child ticket for the D7 update at #639778: Port Conditional Actions to D7

jody lynn’s picture

Another sub-issue for conditional actions is documentation. It desperately needs a README for both admins and devs once most of the system is stable. Are there any online docs existing?

jody lynn’s picture

Mike and I also want to rip CA into uc_ca.module and uc_ca_ui.module, so that's another sub-issue.

joelstein’s picture

What about using the Rules module instead of Conditional Actions?

jody lynn’s picture

@joelstein: we're using drupalcommerce.org for planning and discussion
http://www.drupalcommerce.org/node/83

jdwfly’s picture

+1 for ditching Conditional Actions for Rules.

servantleader’s picture

+1 for using Rules

rszrama’s picture

Version: » 7.x-1.x-dev
Component: Code » Rules integration
Assigned: jody lynn » Unassigned
Status: Active » Closed (fixed)

This has happened. My apologies to Jody are in the child issue. With fago on board, it just makes sense to pursue Rules moving forward.