instead use ItemWithMaterial. This makes the handling more understandable and as a second effect reduces code complexity

and:
* use concrete types like ItemWithMaterial* instead of SessionItem*
* use QVector instead std::deque + QVector to simplify code
* remove now obsolete code
* adapt unit tests

* in an ItemWithMaterial, store only the identifier of the used material
* hold the identifier as QString, not within an ExternalProperty
* modify the ItemWithMaterial/Material property to type "QString" instead of ExternalProperty
* adapt all methods which used the ExternalProperty to use the material identifier or a ptr to a MaterialItem
* adapt the XML reading/writing
* add legacy code to read old XML files
* adapt material selection via SessionModel's CustomEditors

instead get the name directly from the referred material

InstrumentItem::createInstrument.