pjvds / ncqrs

Ncqrs Framework - The CQRS Framework for .NET
Other
542 stars 162 forks source link

Development #24

Closed ashic closed 13 years ago

ashic commented 13 years ago

AggregateRoot.cs has a major bug where there's a ThreadStatic collection being initialised at declaration. ThreadStatic variables must never be initialised in such a manner as it will result in only the first instance getting initialized. Instances for other threads will see their corresponding instances uninitialised.

This commit fixes the issue and has a corresponding unit test to prove it.

pjvds commented 13 years ago

A fix with a test as a proof, now that puts a smile on my face! You are in the race for a second t-shirt? :)

Pulled this do development.