fix graph traversal of ArtifactVisitor

We must return false in all visit methods to indicate that we don't
want to traverse children.

Task-number: QBS-617
Change-Id: I24226b2b83a5deee394ef1e2fd8dee84dbabf28e
Reviewed-by: Christian Kandeler <christian.kandeler@digia.com>
fix isEmpty check

Task-number: QBS-620
Change-Id: I86d220d10b6158db63e2f81d92feaeada0dc86a0
Reviewed-by: Jake Petroules <jake.petroules@petroules.com>
fix scope chain of conditions of Properties items

Conditions of Properties items that were used to set non-module
properties were evaluated in the wrong scope.

Task-number: QBS-613
Change-Id: Ia28070c4352bcca4c153f3cdfb3b66586ea5f3e5
Reviewed-by: Christian Kandeler <christian.kandeler@digia.com>
Remove executable bit from non-executable files.

Task-number: QBS-622
Change-Id: I080cff88ffa91fcb55ff8340847b684901ada44b
Reviewed-by: Joerg Bornemann <joerg.bornemann@digia.com>
Merge remote-tracking branch 'origin/1.2'

Change-Id: I9d5874b3867d3b366e418ae101c25b518053b00a
remove superfluous item cache

ItemReader::m_itemCache was only used for module items,
and those are cached in the module loader.
There's no reason for a second cache.

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