Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Small fixes #17

Closed
wants to merge 4 commits into from
Closed

Small fixes #17

wants to merge 4 commits into from

Conversation

rchavik
Copy link
Member

@rchavik rchavik commented Jun 8, 2011

No description provided.

when preg_match_all parsed the following:

    [e:myelement plugin="social" limit="1" random="true"]

$limit becomes: "1

Note: beginning single quote, missing end quote.

Regex updated as suggested by @StevenAFC in bakery comment.
propagate $block to actual element
By similar, I'm referring to having:
- $block variable available to the element
- respect 'Show Title' setting
- have the element enclosed within html tags similar to croogo's block.ctp

To achieve this, I have enabled Params behavior for Block model to allow
user to specify 'enclosure' setting for given block.  Default behavior is
to have 'enclosure' be 'true'. Users can set this to false to disable
automatic element enclosement.
@fahad19
Copy link
Member

fahad19 commented Jul 22, 2011

merged to 1.3 branch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants