Apologies, we've corrected this, tweaked a couple of other calls to combineCriteria() so they include a textMatchStyle where one is available, and added some autotests in this area.
Please retest with a build dated September 18 or later.
Announcement
Collapse
No announcement yet.
X
-
Guys, you forgot third argument for DS.combineCriteria in dataBoundComponent.exportData
Code:criteria = (ds || isc.DS).combineCriteria(implicitCriteria, criteria, requestProperties.textMatchStyle);
Code:criteria = (ds || isc.DS).combineCriteria(implicitCriteria, criteria, "and", requestProperties.textMatchStyle);
Janusz
Leave a comment:
-
Thanks for the report - we agree that calls to combineCriteria() need a textMatchStyle in most cases, and we've corrected this one for tomorrow's builds, dated September 13 or later.Last edited by Isomorphic; 12 Sep 2023, 08:58.
Leave a comment:
-
bug in dataBoundComponent.exportData
Hi,
I am testing with version 13.0 2023-09-09.
In dataBoundComponent.exportData there is fragment that combines implicit criteria with regular criteria.
Code:var criteria = isc.DS.compressNestedCriteria( (ds || isc.DS).combineCriteria(this.getImplicitCriteria(true), this.getCriteria()) );
textMatchStyle is stored earlier in requestProperties, but it will not be used when converting criteria to combine with implicitCriteria.
textMatchStyle will be later passed on to dataSource.exportData, but as passed combined criteria will be already advanced, it will not be used.
I think that you should change that fragment to something like:
Code:var criteria = isc.DS.compressNestedCriteria( (ds || isc.DS).combineCriteria(this.getImplicitCriteria(true), this.getCriteria(), "and", requestProperties.textMatchStyle) );
Best regards,
Janusz
Tags: None
Leave a comment: