No name collision, just not good practice. Although "type" can invoke
the class of the current object, it doesn't necessarily have to. Still,
I would shy away from it. Here's some more suggestions:
field
column
pick
use
Sean
[EMAIL PROTECTED] wrote:
Yes. I think Rails uses it, so it's kind-of reserved.
Chris
On Mon, 9 Apr 2007, Walter Lee Davis wrote:
Is 'type' already used for something else?
Walter
On Apr 9, 2007, at 4:24 PM, Sean Cribbs wrote:
select? I'm up for suggestions.
Sean
John W. Long wrote:Sean Cribbs wrote:
How about this:
tag 'date' do |tag|
page = tag.locals.page
format = (tag.attr['format'] || '%A, %B %d, %Y')
which = tag.attr['which']
...
If none of the core team has problems with this, I'll write some
tests
and patch it into mental.
I like this, but am not sure that "which" is the right name for this
attribute.
--
John Long
http://wiseheartdesign.com
_______________________________________________
Radiant mailing list
Post: [email protected]
Search: http://radiantcms.org/mailing-list/search/
Site: http://lists.radiantcms.org/mailman/listinfo/radiant
_______________________________________________
Radiant mailing list
Post: [email protected]
Search: http://radiantcms.org/mailing-list/search/
Site: http://lists.radiantcms.org/mailman/listinfo/radiant
_______________________________________________
Radiant mailing list
Post: [email protected]
Search: http://radiantcms.org/mailing-list/search/
Site: http://lists.radiantcms.org/mailman/listinfo/radiant
------------------------------------------------------------------------
_______________________________________________
Radiant mailing list
Post: [email protected]
Search: http://radiantcms.org/mailing-list/search/
Site: http://lists.radiantcms.org/mailman/listinfo/radiant
_______________________________________________
Radiant mailing list
Post: [email protected]
Search: http://radiantcms.org/mailing-list/search/
Site: http://lists.radiantcms.org/mailman/listinfo/radiant
