remove duplicated code from ModuleLoader

findTargetItem does the same as moduleInstanceItem, so remove it
and use the latter. The assertion from findTargetItem is also valid
in moduleInstanceItem and has been moved.

Change-Id: I72ba05d8088e0695f359084f921b85626b205453
Reviewed-by: Christian Kandeler <christian.kandeler@digia.com>
check for undeclared properties

It is no longer allowed to have arbitrary identifiers on the right
hand side of property bindings.

Task-number: QBS-233

Change-Id: I2239befe0a91d9a461578eb4bf7fcb73f7e3174a
Reviewed-by: Christian Kandeler <christian.kandeler@digia.com>
remove leftover from some debug session

Change-Id: I15b1dfbb886bff9ff7c830f36491b70dbb120b3a
Reviewed-by: Christian Kandeler <christian.kandeler@digia.com>