In #1491956: Add UUID/Deploy support uuid support was added.

Perhaps it's possible to add beans to a deploy plan with VBO and/or rules. I've not tested this since I'd like to have an easy approach.

My proposal is to add a "deploy" tab to bean pages.

CommentFileSizeAuthor
#11 bean-deploy_support-1785626-11.patch616 bytesthtas
#7 bean-deploy_support-1785626-7.patch722 bytesdrasgardian
#1 bean-1785626.patch2.07 KBjonhattan

Comments

jonhattan’s picture

jonhattan
Primary language Spanish
Location Plasencia
commented
Assigned: Unassigned » jonhattan
Status: Active » Needs review
StatusFileSize
new bean-1785626.patch2.07 KB

Here's a starting point open for discussion. I'd like to get some feedback before continue coding.

The patch allows to add beans to a deploy plan. Still to be done:

* show which plan is this bean part of, if any
* option to remove from a deploy plan

Some questions:

* Should it be a separate module or part of bean_uuid? A separate module within bean or a new project?
* What about renaming bean_uuid to bean_deploy?

indytechcook’s picture

indytechcook commented

I'd like some other people to comment on this one since I've never used deploy but I'd rather there be a module_exists around the hook_menu element rather then put a dependency on deploy. We shouldn't assume that the only use of uuid is deploy.

indytechcook’s picture

indytechcook commented
Status: Needs review » Needs work

While I haven't test this patch I do have a thought. I'd rather see the deploy in a separate module rather then in bean_uuid.

sheldonkreger’s picture

sheldonkreger commented

The patch in #1 appears to export the Bean, but when the feature containing the Bean is enabled, there is a fatal PHP error:

"PHP Fatal error: Cannot access empty property in mypath/modules/development/deploy/deploy.manager.inc on line 29"

damienmckenna’s picture

damienmckenna
Location TN, USA
commented

I may need to work on this functionality this week.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented

Our project went in a different direction (not my call) so I'm not going to be working on this any time soon. Sorry.

drasgardian’s picture

drasgardian commented
StatusFileSize
new bean-deploy_support-1785626-7.patch722 bytes

I didn't see the error mentioned in #4, but I did run into the following error when trying to enable a deployment plan feature containing a bean.

PHP Fatal error: Call to undefined method stdClass::uri() in [path to modules]/entity/entity.module on line 1082

The attached patch fixed it for me.

indytechcook’s picture

indytechcook commented

So this is a good idea but I do not want to put a hard dependency on the deploy module in uudi (yes there are other reasons), this could either to in a separate module or have a module_exists check.

indytechcook’s picture

indytechcook commented
Issue summary: View changes
Status: Needs work » Fixed

This functionality was added in this issue: #2138651: Add deploy_managed_ui_form_elements() to edit/create form

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

thtas’s picture

thtas commented
StatusFileSize
new bean-deploy_support-1785626-11.patch616 bytes

Updated patch against latest HEAD