X++:
select ledgerJournalTrans where ledgerJournalTrans.RecId == this.RecId;
"this" это что за таблица? LedgerJournalTrans?
Зачем её повторно искать?
Вместо
X++:
select dimensionAttributeValueCombination where dimensionAttributeValueCombination.RecId == ledgerJournalTrans.LedgerDimension;
return vendTable::find(dimensionAttributeValueCombination.DisplayValue).VendGroup;
можно использовать
X++:
return VendTable::find(DimensionAttributeValueCombination::getDisplayValue(ledgerJournalTrans.LedgerDimension)).VendGroup;
Чтобы не было столько return лучше создать переменную типа "VendGroupId" и в конце метода её возвращать.