Fix: Injector destroys instance even if it doesn't manage them.

Author: FrancisBourre

src/hex/di/Injector.hx

@@ -250,15 +250,23 @@ class Injector
 
     public function destroyInstance( instance : Dynamic ) : Void
 	{
-		this._managedObjects.remove( instance );
+		//#if php
+		if ( this._managedObjects.containsKey( instance ) )
+		{
+			this._managedObjects.remove( instance );
+			instance.__ap();
+		}
+		/*#else
 		try
 		{
+			this._managedObjects.remove( instance );
 			instance.__ap();
 		}
 		catch ( e : Dynamic )
 		{
 			
 		}
+		#end*/
 	}
 
 	public function map<T>( type : Class<T>, name : String = '' ) : InjectionMapping<T>

test/hex/di/InjectorTest.hx

@@ -729,6 +729,10 @@ class InjectorTest implements IInjectorListener
 		var target = new Clazz();
 		Assert.isFalse( target.preDestroyCalled, "target.preDestroyCalled should be false" );
 		this.injector.destroyInstance( target );
+		Assert.isFalse( target.preDestroyCalled, "target.preDestroyCalled should be false" );
+		this.injector.map( Clazz ).toType( Clazz );
+		target = this.injector.getOrCreateNewInstance( Clazz );
+		this.injector.destroyInstance( target );
 		Assert.isTrue( target.preDestroyCalled, "target.preDestroyCalled should be true" );
 	}