[1.x][2.x] Over zealous PBV handler insertion

[Simon Laws]

Around line 155 of DataBindingRuntimeWireProcessor there is the code

                // assume pass-by-values copies are required if interfaces
are remotable and there is no data binding
                // transformation, i.e. a transformation will result in a
copy so another pass-by-value copy is unnecessary
                if (!isOnMessage(targetOperation) && isRemotable(chain,
sourceOperation, targetOperation)) {
                    interceptor =
                        new PassByValueInterceptor(dataBindings,
faultExceptionMapper, chain, targetOperation);
                }

We should only be doing this if binding.sca is being used.

TUSCANY-2586 records this problem but there seems to be some question as to
the solution. Why can't we check here what type of binding is being used?
Iassuming that that approach was discounted.

Simon