Add UUID/Deploy support [#1491956]

Comment	File	Size	Author
#4	1491956.patch	2.8 KB	recidive

#2	1491956.patch	2.7 KB	recidive

Comment #1

indytechcook CreditAttribution: indytechcook commented 21 March 2012 at 15:20

Go ahead and submit the patch then I'll let you know if it should be in a separate module.

Log in or register to post comments

Comment #2

recidive CreditAttribution: recidive commented 21 March 2012 at 20:32

Status:

Active

» Needs review

File	Size
1491956.patch	2.7 KB

I talked briefly with @indytechcook on IRC and we figured out this is better as a separate module but same project, since Bean module can't depend on UUID.

Attached is an initial patch implementing some UUID/Deploy support. It looks like some UI work is necessary for adding the Bean blocks to a deploy plan, I still need to figure this out.

Log in or register to post comments

Comment #3

skwashd CreditAttribution: skwashd commented 1 April 2012 at 12:19

Status:

Needs review

» Needs work

Overall this looks good. One quick comment.

+++ b/bean_uuid/bean_uuid.installundefined
@@ -0,0 +1,31 @@
+  db_add_index('bean', 'uuid', array('uuid'));

I think you need a uuid_sync_all() call after this to populate the UUIDs for BEAN

Log in or register to post comments

Comment #4

recidive CreditAttribution: recidive commented 6 April 2012 at 06:23

Status:

Needs work

» Needs review

File	Size
1491956.patch	2.8 KB

Ok, I added uuid_sync_all() to hook_enable() instead, since this cover the case when the module is disable for a period of time and then enabled again.