Comments

dave reid’s picture

Status: Active » Needs review
StatusFileSize
new4.1 KB

The following patch adds a ctools_export_crud_load_multiple() function and also adds support for a 'load multiple callback' hook. I debated having ctools_export_crud_load() re-use ctools_export_crud_load_multiple() like our core single-load functions typically re-use the multiple load functions but this would have complicated logic with having to support both 'load multiple callback' as well as 'load callback'.

dave reid’s picture

Found some documentation fixes.

dave reid’s picture

And another.

dave reid’s picture

I swear I'm gunna find them all.

dave reid’s picture

dave reid’s picture

Patching with cat on y our arms is hard.

dave reid’s picture

Bumping API version as well.

dave reid’s picture

Fixed error with the example mymodule_myobj_load_multiple() func.

merlinofchaos’s picture

Status: Needs review » Fixed

Committed!

Thanks a lot, this is a great feature!

dave reid’s picture

Issue tags: +Metatags

Tagging that this came from the metatags sprint.

Status: Fixed » Closed (fixed)

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

patcon’s picture

Patch that will work for drush_make against rc1