>>Hi Viv,
>>
>>Can you also please check the next loop which starts with this
>>
>>

>> // Add New List
>>                foreach (var item in modifierGroup.Items)
>>                {
>>                    if (!itemIds.Contains(item.ItemId))
>>                    {
>>                        
>>                        var linkParameters = new
>>                        {
>>                            ModGrpId = modifierGroup.GroupId,
>>                            ItemID = item.ItemId
>>                        };

>>
>>
>>I think it's also inefficient, may be you have ideas to make it better.
>
>Isn't that basically the same as an earlier question ?

Yes, didn't see your reply. I'll try that change also, thanks again.