After upgrading to 5.1.0 retrieving balance using custom properties as keys returns 0 about medici

[cycleguy]

Thank you in advance @dolcalmi for taking the time to fix this issue.

[koresar]

@dolcalmi @cycleguy any chance you can fix the parseBalanceQuery for us in a PR? That would be awesome and the fastest solution.

[dolcalmi]

@koresar this can be closed once the new version is released

[Uzlopak]

The implicit registration of Schemas is obsolete with medici v5. you have to write a mongoose schema and use setTransactionSchema use as first parameter the Schema. Based on the Schema, medici v5 determines if it is a meta key or root key. As you did not assigned it, medici tries to search for _businessId and _assetId in the meta field and not on the root field

Also be aware: In medici v4 there was an implicit transformation of all vaues to ObjectIds if the key was starting with an _. Now based on the Schema, we determine if it is an ObjectId or not.

[cycleguy]

We have written a mongoose schema and we are using setTransactionSchema within the mediciTransaction.js file. You mentioned that we did not assign it, what exactly are you referring to? Thanks.

[Uzlopak]

In your zipfile you don't call setTransactionSchema.

[cycleguy]

I double checked the zip file that I sent you....

• On line 15 of app.js (if uncommented out) does a require on ./models/mediciTransaction

• On line 41 of mediciTransaction.js it calls setTransactionSchema()

• src/app.js

• src/index.js

• src/models/mediciTransaction.js

[dolcalmi]

We have the same issue, I think is a problem in parseBalanceQuery https://github.com/flash-oss/medici/blob/master/src/helper/parse/parseBalanceQuery.ts#L38 should use the same logic used in parseFilterQuery https://github.com/flash-oss/medici/blob/master/src/helper/parse/parseFilterQuery.ts#L52

[koresar]

The difference exists because the balance query (not the balance, but the query JSON) is being serialised to a single string as a Quick Balance cache key:

medici/src/models/balance.ts

Line 48 in 12dd0b3

export function constructKey(book: string, account?: string, meta?: IAnyObject): string {

The key is being constructed wrongly when schema is not native and you query by our own keys.

Quick workarounds would be to disable Quick Balance feature. Pass balanceSnapshotSec as zero:
new Book(name, { balanceSnapshotSec: 0 });

[dolcalmi]

this is not the only issue, the problem is how the query is parsed. I will work on it today

[koresar]

Published in v5.2