Hi Jamie,

Still some issues, but we are getting there. Seems that the all tags view is now showing only 3 projects from the tags and if you click on a tag button itself it does not show you anything with exception of the all tags button.

Back with you now - let me take a quick look

Amended the code again it's all in matching up the variable names (also I have move instantiating the tagList(tags) out of the foreach loop to bring it back into scope. Apologies - writing blind :-)

Funny thing is that this if-statement works and lists all the IDs without any issues. Even the duplicates (as nothing filters them out)

if (tag.Equals("all"))
{
    foreach (var tagItem in tags)
    {
        @* <p>@tagItem.Id</p> *@

        publishedContents = Umbraco.TagQuery.GetContentByTag(tagItem.Text);

        foreach (var publishedContent in publishedContents)
        {
            <p>@publishedContent</p>

            items[publishedContent.Id] = publishedContent;
        }
    }

When you then list the output of :

@foreach (var item in publishedContents.OrderByDescending(i => i.Name))
 {
        @item
}

You only get 3 items back.

So it seems that this command filters a bit too much.

items[publishedContent.Id] = publishedContent;

Could it be that it adds all the fetched IDs together and then filter the duplicates out e.g.

11031104110611061095110510951119111811171099

Resulting in -> 1031041061061095105109519181709 and so on?

Ok maybe instead of a dictionary - change it to a list of IPublishedContent;

See if that works.

var items = new List<IPublishedContent>();

foreach (var publishedContent in publishedContents)
{
  <!-- @publishedContent.Id -->
  items.Add(publishedContent);
}

Swap out this line

Line 39:                 items[publishedContent] = publishedContent;

For this

Line 39:                 items.Add(publishedContent);

(Though really you could say)

if(!items.Contains(publishedContent) {
   items.Add(publishedContent);
}

Laurens - looking at the code again with a colleague change this line

@foreach (var item in publishedContents.OrderByDescending(i => i.CreateDate))

to use items

@foreach (var item in items.OrderByDescending(i => i.CreateDate))

the rest can return to how it was previously

(I'll update github so you can see.)

this line:

@foreach (var item in items.OrderByDescending(i => i.CreateDate))

should be:

@foreach (var item in items.OrderByDescending(i => i.Value.CreateDate))

(updated the comment on github)

Updated Git

@item was a disctionaryItem (KeyValuePair) I've amended the code on github so the item is not an IPublishedContent object (the value of a KeyValuePair)

Hi Jamie, thx for you help so far!

Seems I get the following error now

Compiler Error Message: CS0030: Cannot convert type 'System.Collections.Generic.KeyValuePair<int,Umbraco.Core.Models.IPublishedContent>' to 'System.Collections.Generic.KeyValuePair<string,Umbraco.Core.Models.IPublishedContent>'

On:

    Line 84: 
Line 85:                 @* Start picture content *@
Line 86:                 @foreach (KeyValuePair<string,IPublishedContent> kvp in items.OrderByDescending(i => i.Value.CreateDate))
Line 87:                 {
Line 88:                    var item = kvp.Value;

Laurens - is that all of projects displaying? It looks to be less than the list you had displaying earlier . . .

It works!

I have 11 projects on my personal site. So the output is now correct!

Many thanks Jamie! :)

Yay !

No problem - glad it now works. :-)

is working on a reply...