#199 B now reports: Domain unification of the specified ValuesUnit (/…

…Kwartalen_nodata : UInt32) with the categorical calculation results ( : UInt32) is not possible (different CheckedKeyExpr)

tic/dll/src/AbstrDataItem.cpp

@@ -425,7 +425,7 @@ bool AbstrDataItem::CheckResultItem(const TreeItem* refItem) const
 	}
 	if (adi->GetTSF(DSF_Categorical))
 	{
-		if (!GetAbstrValuesUnit()->UnifyDomain(adi->GetAbstrValuesUnit(), "the specified ValuesUnit", "the categorical calculation results", UnifyMode::UM_AllowDefaultLeft))
+		if (!GetAbstrValuesUnit()->UnifyDomain(adi->GetAbstrValuesUnit(), "the specified ValuesUnit", "the categorical calculation results", UnifyMode::UM_AllowDefaultLeft, &errMsgStr))
 			goto failResultMsg;
 		SetTSF(DSF_Categorical);
 	}

tic/dll/src/TreeItem.cpp

@@ -2913,15 +2913,13 @@ bool TreeItem::ReadItem(const StorageReadHandle& srh) // TODO: Make this a metho
 	MG_DEBUGCODE( dms_assert( CheckMetaInfoReady() ); )
 
 	dms_assert(! GetCurrRefItem() ); // caller must take care of only calling ReadItem for UltimateItems
-//	dms_assert(! GetTSF(DSF_DSM_Allocated|TSF_DSM_SdKnown) ); // else we would be reading DataStoreCache
 
 	MG_SIGNAL_ON_UPDATEMETAINFO
 
 	auto keepInterest = GetInterestPtrOrCancel();
 
 	if (WasFailed(FR_Data))
 		return false;
-//	if (GetTSF(TSF_DataInMem | TSF_HasConfigData) || HasCalculatorImpl())
 	if (IsDataReady(this))
 		return true;
 
@@ -2941,15 +2939,11 @@ bool TreeItem::ReadItem(const StorageReadHandle& srh) // TODO: Make this a metho
 		,		GetFullName().c_str()
 		);	
 
-//		dms_assert(!DataAllocated() );
-
 		if (srh.Read())
 			return true;
 		else if (!SuspendTrigger::DidSuspend())
 			throwItemError("DoReadItem returned Failure");
 		dms_assert(GetInterestCount());
-//REMOVE BECAUSE NO PROOF OF NON-REENTRANCE IS GIVEN; operations on DataWriteLock might abort after reading this
-//		SuspendTrigger::MarkProgress();
 	} 
 	catch (...)
 	{