9

I am trying to iterate through the SPListItem.Versions collection to find the latest approved list item.

My list item has three versions: the first two are approved, the last is in draft. But my code says they're all in draft! Please help!

// Iterate through all versions
for (int index = 0; index < item.Versions.Count; index++)
{
    SPListItem versionedItem = item.Versions[index].ListItem;

    // Check if moderation information is set to approved
    if (versionedItem.ModerationInformation.Status.Equals(SPModerationStatusType.Approved))
    {
        // We found an approved version!
        itemFound = versionedItem;
    }
}
Alex Angas
  • 59,219
  • 41
  • 137
  • 210
Martin Larsson
  • 720
  • 2
  • 10
  • 21

3 Answers3

10

The way Mattias recommends and you have implemented is the best way to do it. It's a little awkward but still efficient as the items are ordered from most recent to oldest. This means you are likely to get a match on the published version quickly.

Expanding on the MSDN SPListItemVersionCollection article (specifically Sebastian Wojciechowski's addition):

// Current version of the item (note: this may be a draft)
SPListItem.Versions[0]

// Previous version of the item
SPListItem.Versions[1]

// First version of the item
SPListItem.Versions[SPListItem.Versions.Count - 1]
Alex Angas
  • 59,219
  • 41
  • 137
  • 210
  • No problem. Edited to help users find your question later. – Alex Angas Sep 02 '09 at 10:45
  • As the link you included now points to SharePoint 2013, and the community content you referenced is attached to the previous version: http://msdn.microsoft.com/en-us/library/microsoft.sharepoint.splistitemversioncollection(v=office.12).aspx – Chloraphil Nov 28 '12 at 15:46
  • @Chloraphil Thanks that link is fixed now – Alex Angas May 26 '14 at 23:42
  • This seems to be the correct solution. I've seen many code snippets where they are treating item.Versions[item.Versions.Count - 1] as the newest version which is false. – Patric Jul 02 '15 at 11:31
9

item.Versions[index] returns a SPListItemVersion instance, and SPListItemVersion.ListItem returns the parent SPListItem. So your versionedItem will end up refering to the same object as item, and you're checking the same version over and over again.

I believe you actually want to check

if (item.Versions[index].Level == SPFileLevel.Published) {
  // check item.Versions[index].VersionLabel
}
Mattias S
  • 4,748
  • 2
  • 17
  • 18
  • It worked, thanks! Isn't it a bit awkward to get the parent like that? And the terms get mixed up if you ask me when in SharePoint you call it approval i guess, on the list its Moderation and on the list item its Level!? – Martin Larsson Sep 02 '09 at 08:51
6

My code ended up looking like this:

if (doclist.EnableVersioning)
{
    SPListItemVersionCollection allVersions = item.Versions;

    // Iterate through all versions
    foreach (SPListItemVersion version in allVersions)
    {
        if (version.Level == SPFileLevel.Published)
        {
            itemFound = version.ListItem;
        }
    }
}

Pretty neat and I really hope it works when deployed at the customer!

Alex Angas
  • 59,219
  • 41
  • 137
  • 210
Martin Larsson
  • 720
  • 2
  • 10
  • 21
  • 3
    Calling itemFound = version.ListItem; will return the lastest verion which may not be the Latest Approved. Beware –  Oct 27 '09 at 03:50