@Override
	protected void onMachineAssembled() {
//		invs.clear(); // obsolete due to updateInfo() does this again.
		updateInfo();
		rebuildPatterns();
	}


	public void updateInfo(){
		powerUsage = 0;
		speed = 0;
		pages = 0;
		invs.clear();
		for(IMultiblockPart part : connectedParts){
			if(part.getBlockState().getValue(BlockMultiCrafter.VARIANTS).equals("storage")){
				pages ++;
				powerUsage += 5;
		/*	'part' should always be 'instanceof TileMultiCrafter' because of the previous
		 *	'if(part.getBlockState().getValue(BlockMultiCrafter.VARIANTS).equals("storage"))'
		 *	so this check becomes obsolete, right?
		 */
//				if(part instanceof TileMultiCrafter){
					TileMultiCrafter tile = (TileMultiCrafter) part;
		/*	'pages' should be equal to the id so 'id' can be replaced by 'pages' everywhere
		 *	cuz after we found 1 storage, pages = 1, after we found the 2nd it's 2 and so on...
		 */
//					int id = 0;
		/*	The next part is causing the error:
		 *	if the block was registered before in a bigger crafter he might have a higher id
		 *	assigned and this is causing some ids are not existing.
		 *	for example you build a crafter with 4 storages. They get id 1, 2, 3 and 4.
		 *	now edit the crafter and break storage 2 and 3 and replace them with cpus.
		 *	The new multiblock is scanning the internals:
		 *	storage 1, was already registered ... it keeps id 1
		 *	storage 4, already registered too ... it keeps id 4
		 *	new storages will be registered with id 3 and up due to we have alread have 2 invs
		 *	and need to add 1 for numeration this leads to at least id 2 is missing and it is
		 *	messing up everything (I assume thats the case)
		 */
		/*	to fix the glitch we are overriting existing ids and generate a new id in any case,
		 *	so this whole next part becomes obsolete, sadly
		 */
//					boolean genId = false; 
//					if(tile.page.isPresent()){
//						if(!invs.containsKey(tile.page.get())){
//							id = tile.page.get();
//						} else {
//							genId = true;
//						}
//					} else {
//						genId = true;
//					}
//					if(genId){
//						id = invs.size() + 1;  //Does this need +1?
		/*	in this case +1 seems to be better because page 1 gets id 1, page 2 gets id 2 ...
		 *	but it is obsolete due to pages takes over this variable
		 */
						tile.page = Optional.of(pages);
		/*	assign a new page id ('pages') to the block and dont care about the previous,
		 *	so numbers are always consecutive.
		 *	please check if this overwriting might break something somewhere else...
		 *	it shouldn't, but you know your code best
		 */
//					}
					invs.put(pages, tile.inv);
		/*	put it in the map with the new id ... of course :)
		 */
//				}

			}
			if(part.getBlockState().getValue(BlockMultiCrafter.VARIANTS).equals("cpu")){
				powerUsage += 10;
				speed++;
			}
		}
	}

	public void updateInfo(){
		powerUsage = 0;
		speed = 0;
		pages = 0;
		invs.clear();
		TreeMap<Integer, TileMultiCrafter> collector = new TreeMap<Integer, TileMultiCrafter>();
		int append = 2745;
	/*	we ned to collect all storages first and assign ids after every storage is known
	 *	also we need them to be sorted ... so a treemap. 'append' is explained later
	 */
		for(IMultiblockPart part : connectedParts){
			if(part.getBlockState().getValue(BlockMultiCrafter.VARIANTS).equals("storage")){
				pages ++;
				powerUsage += 5;
				TileMultiCrafter tile = (TileMultiCrafter) part;
	/*	just colect the block instead of assinging ids now
	 *	blocks without id get numerated by id 2745 and up
	 *	because max size of a crafter is 16*16*16, internals are 14*14*14
	 *	so the max id any storage can have is 2744
	 *	this makes new added storages get appended last
	 *	if you want new storages added before existing ones, set 'append' to '-2745' instead
	 */
//				tile.page = Optional.of(pages);
//				invs.put(pages, tile.inv);
				if(tile.page.isPresent()){
					collector.put(tile.page.get(), tile);
				} else {
					collector.put(append++, tile);
				}
			}
			if(part.getBlockState().getValue(BlockMultiCrafter.VARIANTS).equals("cpu")){
				powerUsage += 10;
				speed++;
			}
		}
	/*	now every storage is known and placed in an ordered map we just
	 *	need to iterate over it and assing new ids in that order, starting by 1
	 *	map.values() should preserve the order
	 */
		int newid = 0;
		for(TileMultiCrafter tile : collector.values()) {
			newid++;
			tile.page = Optional.of(newid);
			invs.put(newid, tile.inv);
		}
		
	}