BusinessObjectCollection performance improvements

[jeff2001]

Redmine Issue #1395

There are numerous places that performance of the class could be improved.
A basic search of all the Contains methods would make this obvious, but for this particular instance, the following was giving a massive performance hit.

When running CreateBussinessObject() the following method gets slower and slower each time you add a BO:

private void AddCreatedBusinessObject(TBusinessObject newBO)
{
            if (!this.CreatedBusinessObjects.Contains(newBO))
    {
                CreatedBusinessObjects.Add(newBO);
            }
            if (this.Contains(newBO)) return;
AddWithoutEvents(newBO);
        }

If you are doing a batch update of properties, the following method gets slower the larger your collection gets.
I think it should only be necessary to do any of the stuff if an event handler is registered.

private void BOPropUpdatedEventHandler(object sender, BOPropUpdatedEventArgs propEventArgs)
{
            TBusinessObject businessObject;
            lock (KeyObjectHashTable)
    {
                businessObject = propEventArgs.BusinessObject as TBusinessObject;

if (!this.Contains(businessObject)) return;
            }
            FireBusinessObjectPropertyUpdated(businessObject, propEventArgs.Prop);
        }