Martin Larsson
Martin Larsson

Reputation: 720

Find latest approved version of an SPListItem

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;
    }
}

Upvotes: 9

Views: 15862

Answers (3)

Alex Angas
Alex Angas

Reputation: 60037

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]

Upvotes: 10

Martin Larsson
Martin Larsson

Reputation: 720

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!

Upvotes: 6

Mattias S
Mattias S

Reputation: 4808

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
}

Upvotes: 9

Related Questions