[Radiant] New Date Tag

Sean Cribbs seancribbs at gmail.com
Mon Apr 9 17:13:49 CDT 2007


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

chris at idlelion.net 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:   Radiant at lists.radiantcms.org
>>>> Search: http://radiantcms.org/mailing-list/search/
>>>> Site:   http://lists.radiantcms.org/mailman/listinfo/radiant
>>>>
>>>>
>>>
>>> _______________________________________________
>>> Radiant mailing list
>>> Post:   Radiant at lists.radiantcms.org
>>> Search: http://radiantcms.org/mailing-list/search/
>>> Site:   http://lists.radiantcms.org/mailman/listinfo/radiant
>> _______________________________________________
>> Radiant mailing list
>> Post:   Radiant at lists.radiantcms.org
>> Search: http://radiantcms.org/mailing-list/search/
>> Site:   http://lists.radiantcms.org/mailman/listinfo/radiant
>>
> ------------------------------------------------------------------------
>
> _______________________________________________
> Radiant mailing list
> Post:   Radiant at lists.radiantcms.org
> Search: http://radiantcms.org/mailing-list/search/
> Site:   http://lists.radiantcms.org/mailman/listinfo/radiant

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.radiantcms.org/pipermail/radiant/attachments/20070409/4040f2f7/attachment.html


More information about the Radiant mailing list