Cool module! seems to work OK. It seems to have a syntax that is not valid shortcode syntax though, i.e. I would expect to write module="bean" delta="slider".
Have you seen the shortcode API module for Drupal? It's a port of the WP API.

Maybe you could reduce your codebase by supported the API module

Comments

SeanFitzpatrick’s picture

SeanFitzpatrick CreditAttribution: SeanFitzpatrick commented

Hey thanks for the comment. Actually I found the Shortcode API module after I started working on this. Your comment comes at a good time because in the next week or so I'll be rewriting my block_shortcode for a couple client sites and using Shortcode API's syntax and API functions. I'll post here when I have some updates... I'd appreciate a second set of eyes on it if you have time.

JurriaanRoelofs’s picture

JurriaanRoelofs CreditAttribution: JurriaanRoelofs commented

it would be nice if this could be promoted to a full projected since I use it in my distro, which is now experiencing some problems with the sandbox: https://www.drupal.org/node/2495817

bohemier’s picture

bohemier CreditAttribution: bohemier as a volunteer commented

Great module, thanks SeanFitzpatrick. I also agree with making the syntax in sync with Shortcode API. Any progress on this?