Do not choose constructors with an "Obsolete" attribute

src/Ninject.Test/Integration/ConstructorSelectionTests.cs

@@ -222,6 +222,18 @@ public void WhenConstructorHasAnOpenGenericTypeItCountsAsServedParameterIfBindin
             instance.Generic.Should().NotBeNull();
         }
 
+        [Fact]
+        public void DoNotChooseObsoleteConstructors()
+        {
+            kernel.Bind<ClassWithObsoleteContructor>().ToSelf();
+
+            var instance = kernel.Get<ClassWithObsoleteContructor>();
+
+            instance.Should().NotBeNull();
+            instance.SomeObject.Should().NotBeNull();
+            instance.ObsoleteConstructorCalled.Should().Be(false);
+        }
+
 #if !SILVERLIGHT
         [Fact]
         public void WhenConstructorHasAValueWithDefaultValueItCountsAsServedParameter()
@@ -286,5 +298,23 @@ public ClassWithTwoInjectAttributes(int someValue)
             {
             }
         }
+
+        public class ClassWithObsoleteContructor
+        {
+            [Obsolete("Use Ninject to create an instance")]
+            public ClassWithObsoleteContructor()
+                : this(new object())
+            {
+                ObsoleteConstructorCalled = true;
+            }
+
+            public ClassWithObsoleteContructor(Object someObject)
+            {
+                SomeObject = someObject;
+            }
+
+            public Object SomeObject { get; set; }
+            public bool ObsoleteConstructorCalled { get; set; }
+        }
     }
 }

src/Ninject/INinjectSettings.cs

@@ -36,6 +36,11 @@ public interface INinjectSettings
         /// </summary>
         Type InjectAttribute { get; }
 
+        /// <summary>
+        /// Gets the attribute that indicates that a member is obsolete and should not be injected.
+        /// </summary>
+        Type ObsoleteAttribute { get; }
+
         /// <summary>
         /// Gets the interval at which the cache should be pruned.
         /// </summary>

src/Ninject/NinjectSettings.cs

@@ -64,6 +64,15 @@ public Type InjectAttribute
             set { this.Set("InjectAttribute", value); }
         }
 
+        /// <summary>
+        /// Gets or sets the attribute that indicates that a member is obsolete and should not be injected.
+        /// </summary>
+        public Type ObsoleteAttribute
+        {
+            get { return this.Get("ObsoleteAttribute", typeof(ObsoleteAttribute)); }
+            set { this.Set("ObsoleteAttribute", value); }
+        }
+
         /// <summary>
         /// Gets or sets the interval at which the GC should be polled.
         /// </summary>

src/Ninject/Planning/Directives/ConstructorInjectionDirective.cs

@@ -51,5 +51,11 @@ public ConstructorInjectionDirective(ConstructorInfo constructor, ConstructorInj
         /// </summary>
         /// <value><c>true</c> if this constructor has an inject attribute; otherwise, <c>false</c>.</value>
         public bool HasInjectAttribute { get; set; }
+
+        /// <summary>
+        /// Gets or sets a value indicating whether this constructor has an Obsolete attribute.
+        /// </summary>
+        /// <value><c>true</c> if this constructor has an Obsolete attribute; otherwise, <c>false</c>.</value>
+        public bool HasObsoleteAttribute { get; set; }
     }
 }

src/Ninject/Planning/Strategies/ConstructorReflectionStrategy.cs

@@ -72,9 +72,11 @@ public void Execute(IPlan plan)
             foreach (ConstructorInfo constructor in constructors)
             {
                 var hasInjectAttribute = constructor.HasAttribute(this.Settings.InjectAttribute);
+                var hasObsoleteAttribute = constructor.HasAttribute(this.Settings.ObsoleteAttribute);
                 var directive = new ConstructorInjectionDirective(constructor, this.InjectorFactory.Create(constructor))
                 {
-                     HasInjectAttribute = hasInjectAttribute
+                    HasInjectAttribute = hasInjectAttribute,
+                    HasObsoleteAttribute = hasObsoleteAttribute
                 };
 
                 plan.Add(directive);

src/Ninject/Selection/Heuristics/StandardConstructorScorer.cs

@@ -57,6 +57,11 @@ public virtual int Score(IContext context, ConstructorInjectionDirective directi
                 return int.MaxValue;
             }
 
+            if (directive.HasObsoleteAttribute)
+            {
+                return int.MinValue;
+            }
+
             var score = 1;
             foreach (ITarget target in directive.Targets)
             {