Using Fluid instead of DotLiquid

[sebastienros]

An example to test the feature:

{% assign year = 0 %}

{% assign archives = 'Archives' | query %}
<ul class="achives-year">
{% for archive in archives %}
  {% if year != archive.Year %}
  	<li>
      <h3>{{ archive.Year }}</h3>
      <ul class="archive-months">
  {% endif %}
	
	    <li>
        {% case archive.Month %}
          {% when 1 %}January
          {% when 2 %}February
          {% when 3 %}March
          {% when 4 %}April
          {% when 5 %}May
          {% when 6 %}June
          {% when 7 %}July
          {% when 8 %}August
          {% when 9 %}September
          {% when 10 %}October
          {% when 11 %}November
          {% when 12 %}December
      	{% endcase %}
        ({{ archive.Count }})
  
  {% if year != archive.Year %}
      </ul>
  	</li>
    {% assign year = archive.Year %}
  {% endif %}

{% endfor %}
	</li>
</ul>

With this sql query

select 
    month(CreatedUtc) as [Month], 
    year(CreatedUtc) as [Year],
    day(CreatedUtc) as [Day],
    count(*) as [Count]
from ContentItemIndex 
where Published = true
group by day(CreatedUtc), month(CreatedUtc), year(CreatedUtc)

[Jetski5822]

they really need to remove this from VS..

[Jetski5822]

Delete

[Jetski5822]

Do we not have a ToObjectValue() ?

[sebastienros]

input is a FluidValue it wraps any CLR object. So yes we want to get the wrapped content item then cast it.

[Jetski5822]

Can we not use async await on the lambda of the For?

[sebastienros]

No because GetContentItemAspect is not async.
That doesn't matter right now as I create this #822 which will solve it.

[Jetski5822]

"" or HtmlString.Empty ?

[Jetski5822]

LGTM - a few comments, but no show stoppers.