Beans All depends on the Block module but the Block module is not listed as a dependency. [#2146879]

Comment	File	Size	Author
#1	adds_block_dependency-2146879.patch	331 bytes	Dave Bagler

Comment #1

Dave Bagler CreditAttribution: Dave Bagler commented 29 November 2013 at 18:31

Status:

Needs work

» Needs review

File	Size
adds_block_dependency-2146879.patch	331 bytes

Here's the patch.

Log in or register to post comments

Comment #2

indytechcook CreditAttribution: indytechcook commented 30 November 2013 at 01:48

Where is the dependency? module_implements('block_info') just implements the hook, it doesn't require the block module. The point of bean all is to implement other modules core blocks as bean types. It is possible that I missed something though.

Log in or register to post comments

Comment #3

raychaser CreditAttribution: raychaser commented 5 December 2013 at 21:35

I'm not sure this should be in the been _all module. The core Bean module certainly has stuff to do with the 'block' table in its install file.

For me, the issue presented itself when I tried a drush makefile with a custom install profile that included bean.

Bean tried to install itself before the block module and I got an error about 'block_custom' table not being present.

From the same criteria "menu" is also required.

Log in or register to post comments

Comment #4

Dave Bagler CreditAttribution: Dave Bagler commented 9 December 2013 at 20:02

@raychaser, that's an interesting point. We were installing from an install profile as well but we weren't running into issues on the core Bean module, but on the Bean All module. I don't have time today, but I'll try installing Bean and Bean all on a site that has never had the blocks module installed soon and post back.

@indytechcook, as @raychaser mentioned it's at install time that the bean_all module is looking for the block_custom table and failing if it isn't there.