Error "Function DimensionAttributeValueSetStorage.addItemValues has been incorrectly called. "

(0) Share

Report

Posted on by Community Member

I have added a new form control and added a lookup in the method. My code looks like this. Everytime I keying a relevant financial dimension into the new form control, I get error "Function DimensionAttributeValueSetStorage.addItemValues has been incorrectly called." Please can you help?

public void lookup()
{
SysTableLookup sysTableLookup;
QueryBuildDataSource queryBuildDataSource;
QueryBuildRange queryBuildRange;
Query query = new Query();
DimensionAttribute dimAttr = DimensionAttribute::find(LedgerParameters::find().TSJournalSubcodeAttribute);
;

sysTableLookup = SysTableLookup::newParameters(tableNum(DimensionFinancialTag), this);
sysTableLookup.addLookupfield(fieldNum(DimensionFinancialTag, Value));
sysTableLookup.addLookupfield(fieldNum(DimensionFinancialTag, Description));

queryBuildDataSource = query.addDataSource(tableNum(DimensionFinancialTag));
queryBuildDataSource.addOrderByField(fieldnum(DimensionFinancialTag, Value));

sysTableLookup.parmQuery(query);

if (sysTableLookup)
{
sysTableLookup.performFormLookup();
}
}

I have the same question (0)

All responses (5)

Answers (1)

Martin Dráb 237,965 Most Valuable Professional on at

Like (0)

Report
Please use the debugger to find out the call stack leading to the exception. Then please share it with us. It doesn't seem to be called from lookup() at all.

By the way, please use Insert > Insert Code (in the rich-formatting view) to paste source code. It preserves indentation, which makes code much easier to read:

public void lookup() { SysTableLookup sysTableLookup; QueryBuildDataSource queryBuildDataSource; Query query = new Query(); DimensionAttribute dimAttr = DimensionAttribute::find(LedgerParameters::find().TSJournalSubcodeAttribute); ; sysTableLookup = SysTableLookup::newParameters(tableNum(DimensionFinancialTag), this); sysTableLookup.addLookupfield(fieldNum(DimensionFinancialTag, Value)); sysTableLookup.addLookupfield(fieldNum(DimensionFinancialTag, Description)); queryBuildDataSource = query.addDataSource(tableNum(DimensionFinancialTag)); queryBuildDataSource.addOrderByField(fieldnum(DimensionFinancialTag, Value)); sysTableLookup.parmQuery(query); if (sysTableLookup) { sysTableLookup.performFormLookup(); } }

Was this reply helpful? Yes No
Community Member on at

Like (0)

Report

Hi Martin,

This is where it throws error:

public int addItemValues(RecId _dimensionAttributeId, RecId _dimensionAttributeValueId, HashKey _dimensionAttributeValueHashKey)
{
int i;

// Unable to add a zero dimension attribute as that does not mean anything
if (_dimensionAttributeId == 0)
{
throw error(Error::wrongUseOfFunction(funcname()));
}

Was this reply helpful? Yes No
Martin Dráb 237,965 Most Valuable Professional on at

Like (0)

Report

Okay, and what is the call stack (the sequence of method calls) leading to addItemValues()?

Was this reply helpful? Yes No
Community Member on at

Like (0)

Report
Hi Martin, I looked into the code and found that here is where it fails.

public static DimensionDefault setDimensionDLG(DimensionDefault _dimension, RefRecId _dimensionAttributeId, DimensionValue _dimensionValue, boolean _createIfNecessary = false) { DimensionAttribute dimAttr; DimensionAttributeValue dimAttrValue; DimensionAttributeValueSetStorage dimAttrValueSetStorage; ; if (!_dimensionAttributeId) { return _dimension; } dimAttr = DimensionAttribute::find(_dimensionAttributeId); dimAttrValueSetStorage = DimensionAttributeValueSetStorage::find(_dimension); if (_dimensionValue) { dimAttrValue = DimensionAttributeValue::findByDimensionAttributeAndValue(dimAttr, _dimensionValue, false, _createIfNecessary); dimAttrValueSetStorage.addItem(dimAttrValue); } else { dimAttrValue = DimensionAttributeValue::find(dimAttrValueSetStorage.getValueByDimensionAttribute(dimAttr.RecId)); dimAttrValueSetStorage.removeDimensionAttributeValue(dimAttrValue.RecId); } return dimAttrValueSetStorage.save(); }

Was this reply helpful? Yes No
Verified answer

Martin Dráb 237,965 Most Valuable Professional on at

Like (0)

Report

It means that the value doesn't exist yet. If you want to create it automatically, call setDimensionDLG() with _createIfNecessary = true.

Unfortunately I have no idea where you're calling setDimensionDLG(), because you didn't share the call stack. You'll have to look at it by yourself.

Was this reply helpful? Yes No