Remove Item From Package By Name

Question

Ran into the issue where I have clientName in the package which gets merged into the content. Later programName gets added to the package which happens to have the same value as clientName. There is a C# frag which removes registered trademark from the clientName. It is removing the programName because it finds that value first and adds a duplicate clientName. Is there any way to remove item by name?

<%@Import Namespace="Tridion.ContentManager.ContentManagement.Fields"%>

Item outputItem = package.GetByName("client.name");
if(outputItem != null){
    string outputString = outputItem.GetAsString();
    outputString = outputString.Replace("\u00AE", ""); 
    outputString = outputString.Replace("\u24C7", ""); 
    outputString = outputString.Replace("®", "");
    package.Remove(outputItem);
    outputItem.SetAsString(outputString);
    package.PushItem("client.name", outputItem);
}

Answer 1

Yes, you can remove items by name, something along these lines:

Item item = package.GetByName("ItemName");
package.RemoveItem(item);

Given the package is a stack, if you have two items with the same name GetByName will get the last item that was added, so you may have to play around with this until it behaves as you want it to.

If your challenge is more about how to deal with multiple items with the same name in the package, then you should perhaps look at package.GetAllByType method.

Answer 2

This is an interesting find. I've been able to reproduce the scenario described in the question. Consider the following:

We have two items with different names, but identical values. However, the values have different business meanings. So here is the package:

Next, you want to pop an item off (by usual means of peeking then removing):

    public override void Transform(Tridion.ContentManager.Templating.Engine engine, Tridion.ContentManager.Templating.Package package)
    {
        Item i = package.GetByName("client.name");
        string removedItemName = package.Remove(i);
        Logger.Debug(removedItemName);
    }

After running the above code we expect the item "client.name" to be removed, however the result was that the other item is removed:

What appears is that the item with the matching value that's closest to the top of the stack is removed. This looks like a bug to me.

Now back to your question of how work around this:

In Tridion 2013 there are Append and SetAs methods. So just use those instead of the old school way of peeking and popping. If you're on pre-2013, then simply leave the item on the package. At the very least, you'll later be able to write your own function to cycle through all the items and find by name yourself.