The bean list page at admin/content/blocks should use EntityFeildQuery instead of db_select().

This can be changed in the includes/bean.pages.inc file in the bean_list() function.

CommentFileSizeAuthor
#4 bean-list-page-efq-1345966-3.patch1.1 KBsaltednut
#2 bean-list-page-efq-1345966-2.patch2.79 KBsaltednut
#1 bean-list-page-efq-1345966-1.patch2.76 KBsaltednut

Comments

saltednut’s picture

saltednut
she/her
Primary language English
Location Chicago
commented
Status: Active » Needs work
StatusFileSize
new bean-list-page-efq-1345966-1.patch2.76 KB

Trying this out but an EFQ for 'bean' only returns array('bid', 'type')

Can this be changed so that we are returning array('bid', 'label', 'type', 'delta')?

In this patch, EFQ is working but $bean->label and $bean->delta are missing:

saltednut’s picture

saltednut
she/her
Primary language English
Location Chicago
commented
Priority: Major » Normal
Status: Needs work » Needs review
StatusFileSize
new bean-list-page-efq-1345966-2.patch2.79 KB 
$beans = bean_load_multiple(array_keys($result['bean']));

will load the full array of bean objects needed.

Status: Needs review » Needs work

The last submitted patch, bean-list-page-efq-1345966-2.patch, failed testing.

saltednut’s picture

saltednut
she/her
Primary language English
Location Chicago
commented
Status: Needs work » Needs review
StatusFileSize
new bean-list-page-efq-1345966-3.patch1.1 KB

Thanks bot - just realized I was using an older file in my sandbox. This patch should pass...

indytechcook’s picture

indytechcook commented
Status: Needs review » Fixed

This has been commited.

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