[TypeDeclaration] Skip trait methods on ParamTypeByMethodCallType rules

[TomasVotruba]

Trait methods may be composed into classes that implement an interface contract. A param type inferred from a call inside the trait itself is not safe to add: once the trait is used in a class where the method must match a wider contract signature, the added type breaks it.

Skip ClassMethod nodes whose scope is inside a trait.

 trait SomeTrait
 {
-    public function go($value)
+    public function go(array $value)  // wrong, breaks contract when trait used in class implementing an interface
     {
         $this->use($value);
     }

     private function use(array $value): void
     {
     }
 }

The skip lives in the shared AbstractParamTypeByMethodCallTypeRector::shouldSkipClassMethod(), so it covers all four rules built on it:

• ArrayParamTypeByMethodCallTypeRector
• ObjectParamTypeByMethodCallTypeRector
• ScalarParamTypeByMethodCallTypeRector
• ParamTypeByMethodCallTypeRector