Probably a minor fix, but it would be nice if there is a link on the Create Content tab of Workbench (admin/workbench/create). With all the Bean Types. See screenshot for current interface.
Schermafbeelding 2012-03-31 om 15.26.08.png

CommentFileSizeAuthor
#11 bean-workbench-permissions-1510508-11.patch1.34 KBjhedstrom
#10 bean-workbench-permissions-1510508-10.patch1.33 KBjhedstrom
#7 bean-workbench_ksort_error-1510508.patch302 bytesmraichelson
#4 bean-workbench_ksort_error-1510508.patch302 bytesmraichelson
Schermafbeelding 2012-03-31 om 15.26.08.png38.73 KBBarisW

Comments

indytechcook’s picture

indytechcook commented
Status: Active » Fixed

Good idea.

http://drupal.org/commitlog/commit/22232/2f63d617c75edd3d8bf8d1057bca304...

BarisW’s picture

BarisW commented

Wow, that's fast. Cool, thanks!

Status: Fixed » Closed (fixed)

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

mraichelson’s picture

mraichelson commented
Status: Closed (fixed) » Patch (to be ported)
StatusFileSize
new bean-workbench_ksort_error-1510508.patch302 bytes

Quick note on this one. In 7.x-1.0-rc4 it looks like the ksort on line 838 in beans.module is trying to sort an array that doesn't exist. If it gets switched to $items (the array of bean types) that seems to be cool, otherwise you get an error on the Workbench > Create Content screen saying...

Warning: ksort() expects parameter 1 to be array, null given in bean_workbench_create_alter() (line 838 of /path/to/drupal/sites/all/modules/bean/bean.module).

Looks like that's still the same code in 7.x-1.x-dev right now too.

mraichelson’s picture

mraichelson commented
Status: Patch (to be ported) » Needs review

Sorry, wrong status applied.

Status: Needs review » Needs work

The last submitted patch, bean-workbench_ksort_error-1510508.patch, failed testing.

mraichelson’s picture

mraichelson commented
Version: 7.x-1.0-rc3 » 7.x-1.x-dev
Status: Needs work » Needs review
StatusFileSize
new bean-workbench_ksort_error-1510508.patch302 bytes

Targeting against DEV version rather than an old RC release.

indytechcook’s picture

indytechcook commented
Status: Needs review » Reviewed & tested by the community

Looks good. This was a copy/paste/edit error on my part.

indytechcook’s picture

indytechcook commented
Status: Reviewed & tested by the community » Fixed

http://drupal.org/commitlog/commit/22232/e352ffaf8b6dcb95b286d9d78f43693...

jhedstrom’s picture

jhedstrom
Primary language English
Location Portland, OR
commented
Status: Fixed » Needs review
StatusFileSize
new bean-workbench-permissions-1510508-10.patch1.33 KB

This patch checks for proper permissions so as to not display bean add links for users w/o access to those pages.

jhedstrom’s picture

jhedstrom
Primary language English
Location Portland, OR
commented
StatusFileSize
new bean-workbench-permissions-1510508-11.patch1.34 KB

Previous patch was slightly wrong with the argument passed to bean_access().

indytechcook’s picture

indytechcook commented
Status: Needs review » Reviewed & tested by the community
indytechcook’s picture

indytechcook commented
Status: Reviewed & tested by the community » Fixed

http://drupal.org/commitlog/commit/22232/bf94f73e4bb21aeb3c59c3d621ab2f3...

I hope this is the last time! :) Thanks everyone for the help.

BarisW’s picture

BarisW commented

Cool, thanks!

Status: Fixed » Closed (fixed)

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

sylvain lecoy’s picture

sylvain lecoy commented

Can we align the text with drupal standards, 'Create block', instead of 'Create Blocks', just like 'Create content' is. :-)