Improve info for ObjectWhereProcedureIsRequiresInspection

Updates are based on review comments to PR #5165.

Author: MDoerner

Rubberduck.CodeAnalysis/Inspections/ObjectWhereProcedureIsRequiredInspection.cs

@@ -6,7 +6,6 @@
 using Rubberduck.Parsing.VBA;
 using Rubberduck.Inspections.Inspections.Extensions;
 using Rubberduck.Inspections.Results;
-using Rubberduck.Parsing.Grammar;
 using Rubberduck.Parsing.Inspections;
 using Rubberduck.Parsing.Inspections.Abstract;
 using Rubberduck.Parsing.VBA.DeclarationCaching;
@@ -22,36 +21,38 @@ namespace Rubberduck.Inspections.Concrete
     /// This behavior is not obvious and most likely an error. 
     /// </why>
     /// <example hasResult="true">
+    /// <module name="Class1" type="Class Module">
     /// <![CDATA[
-    /// Class1:
-    ///
     /// Public Function Foo() As Long
     /// Attibute VB_UserMemId = 0
     ///     Foo = 42
     /// End Function
-    ///
-    /// Module:
-    /// 
+    /// ]]>
+    /// </module>
+    /// <module name="Module" type="Standard Module">
+    /// <![CDATA[
     /// Public Sub DoSomething(ByVal arg As Class1)
     ///     arg
     /// End Sub
     /// ]]>
+    /// </module>
     /// </example>
     /// <example hasResult="false">
+    /// <module name="Class1" type="Class Module">
     /// <![CDATA[
-    /// Class1:
-    ///
     /// Public Function Foo() As Long
     /// Attibute VB_UserMemId = 0
     ///     Foo = 42
     /// End Function
-    ///
-    /// Module:
-    /// 
+    /// ]]>
+    /// </module>
+    /// <module name="Module" type="Standard Module">
+    /// <![CDATA[
     /// Public Sub DoSomething(ByVal arg As Class1)
     ///     arg.Foo
     /// End Sub
     /// ]]>
+    /// </module>
     /// </example>
     public sealed class ObjectWhereProcedureIsRequiredInspection : InspectionBase
     {

Rubberduck.Resources/Inspections/InspectionInfo.Designer.cs

@@ -410,6 +410,6 @@ Falls der Parameter 'null' sein kann, bitte dieses Auftreten ignorieren. 'null'
     <value>Die Ausrufezeichennotation erweckt den Eindruck, dass es sich um einen Zugriff handelt, der Typchecks unterliegt. Allerdings handelt es sich lediglich um einen Zugriff auf den Standardmember des Objekts, auf das sie angewendet wird, bei dem das Argument als Zeichenkette übergeben wird. Dies ist besonders verwirrend, wenn der Standardmember nicht zum Zeitpunkt des Kompilierens ermittelt werden kann.</value>
   </data>
   <data name="ObjectWhereProcedureIsRequiredInspection" xml:space="preserve">
-    <value>Wenn ein Objekt mit einem Standardmember an einer Stelle verwendet wird, die nach einer Prozedur verlangt, wird der Standardmember aufgerufen. Die ist wahrscheinlich ein Fehler, zu Mindest aber nicht offensichtlich.</value>
+    <value>Wenn ein Objekt mit einem Standardmember an einer Stelle verwendet wird, die nach einer Prozedur verlangt, wird implizit der Standardmember aufgerufen. Die ist wahrscheinlich nicht beabsichtigt, minder aber zu Mindest die Lesbarkeit des Programms.</value>
   </data>
 </root>

Rubberduck.Resources/Inspections/InspectionInfo.de.resx

@@ -413,6 +413,6 @@ If the parameter can be null, ignore this inspection result; passing a null valu
     <value>Bang notation, formally known as dictionary access expression, looks like it is strongly typed. However, it is actually a stringly-typed access to the parameterized default member of the object it is used on. This is especially misleading the default member cannot be determined at compile time.</value>
   </data>
   <data name="ObjectWhereProcedureIsRequiredInspection" xml:space="preserve">
-    <value>Using an object with a default member in a place that requires a procedure leads to a call to the default member. This is most likely an error, but at least not obvious.</value>
+    <value>Using an object with a default member in a place that requires a procedure leads to an implicit invocation of the default member. This is most likely unintentional and negatively affects readability.</value>
   </data>
 </root>

Rubberduck.Resources/Inspections/InspectionInfo.resx

@@ -2,13 +2,11 @@
 using System.Collections.Generic;
 using System.Linq;
 using System.Threading;
-using System.Windows.Controls;
 using NUnit.Framework;
 using Rubberduck.Parsing.Symbols;
 using Rubberduck.Parsing.VBA;
 using RubberduckTests.Mocks;
 using Rubberduck.Parsing.Annotations;
-using Rubberduck.Parsing.VBA.Extensions;
 using Rubberduck.VBEditor;
 using Rubberduck.VBEditor.SafeComWrappers;
 using Rubberduck.VBEditor.SafeComWrappers.Abstract;