My version 7rc2 doesn't seem to have this option either. Adding virtual="false" has no effect.

Replacing children with Drew's

tag 'children' do |tag|
    tag.locals.filter_attributes = tag.attr
    tag.locals.options = children_find_options(tag)
    tag.locals.children = tag.locals.page.children
tag.locals.children_filtered = tag.locals.page.children.find(:all, tag.locals.options.clone)
    tag.expand
  end

and

<r:children virtual="false"><r:count/></r:children>

I end up with "undefined local variable or method `����tag' for #"


For my use it would be best if count had the option to exclude virtual pages and only include published pages. I think Drew's method would do that. But there must be more to it then just changing out that code and restarting Mongrel.

Steven






On Feb 4, 2009, at 3:21 AM, Andrew Neil wrote:

On Wed, Feb 4, 2009 at 5:07 AM, Sean Cribbs <seancri...@gmail.com> wrote:

<r:children:count /> should use the default find options for page children, which is to exclude virtual pages. Just to be sure, add virtual="false" and
see if you get a different result.


Are you sure? Looking at the code for the r:children:count tag[1], it
doesn't appear that its behaviour is modified by passing any attributes.

In my fork of Radiant[2], I have modified the r:children tag so that it accepts the same options as r:children:each/first/last. I overlooked the r:children:count tag, so it still has this bug. However, by breaking up the
r:children:count tag as follows:

   <r:children virtual="false"><r:count/></r:children>

using my version of the r:children tag, this should exclude virtual pages,
whereas (I think) the current Radiant would include virtual pages.

Cheers,
Drew

[1]:
http://github.com/radiant/radiant/blob/a9f7925f1ecec9e797db191b3f903b153400ba2c/app/models/standard_tags.rb#L51-53

[2]:
http://github.com/nelstrom/radiant/blob/b096e6f7b054d212725013b41b7c22663e84b843/app/models/standard_tags.rb#L69-75



Sean


Steven Southard wrote:

It seems like that should work but it doesn't. First, best I can tell, the archive month index doesn't have a url. Second even if I try to exclude
another article's url it doesn't change the count.



On Feb 3, 2009, at 4:34 PM, john muhl wrote:

might try excluding it by url

<r:find url="/articles"><r:unless_url matches="/articles/archive">
<r:children:count />
</r:unless_url>
</r:find>


On Tue, Feb 3, 2009 at 3:03 PM, Steven Southard
<ste...@stevensouthard.com>wrote:

I was using <r:find url="/articles"><r:children:count /></ r:find> to
keep
track of the number of articles published. Now I've added an archive
month
index so my count is off by one. How do I subtract one or exclude the
month
index?

Steven
_______________________________________________
Radiant mailing list
Post:   Radiant@radiantcms.org
Search: http://radiantcms.org/mailing-list/search/
Site:   http://lists.radiantcms.org/mailman/listinfo/radiant

_______________________________________________
Radiant mailing list
Post:   Radiant@radiantcms.org
Search: http://radiantcms.org/mailing-list/search/
Site:   http://lists.radiantcms.org/mailman/listinfo/radiant


_______________________________________________
Radiant mailing list
Post:   Radiant@radiantcms.org
Search: http://radiantcms.org/mailing-list/search/
Site:   http://lists.radiantcms.org/mailman/listinfo/radiant


_______________________________________________
Radiant mailing list
Post:   Radiant@radiantcms.org
Search: http://radiantcms.org/mailing-list/search/
Site:   http://lists.radiantcms.org/mailman/listinfo/radiant

_______________________________________________
Radiant mailing list
Post:   Radiant@radiantcms.org
Search: http://radiantcms.org/mailing-list/search/
Site:   http://lists.radiantcms.org/mailman/listinfo/radiant

_______________________________________________
Radiant mailing list
Post:   Radiant@radiantcms.org
Search: http://radiantcms.org/mailing-list/search/
Site:   http://lists.radiantcms.org/mailman/listinfo/radiant

Reply via email to