details: https://code.openbravo.com/erp/devel/pi/rev/43453b625a06 changeset: 29956:43453b625a06 user: Alvaro Ferraz <alvaro.ferraz <at> openbravo.com> date: Wed Aug 31 11:25:47 2016 +0200 summary: Related to issue 33771: Remove setMaxResults(1)
setMaxResults(1) is not needed as we are filtering by unique constraint. diffstat: src/org/openbravo/erpCommon/ad_callouts/SE_Invoice_BPartner.java | 1 - 1 files changed, 0 insertions(+), 1 deletions(-) diffs (11 lines): diff -r e909d7e193e5 -r 43453b625a06 src/org/openbravo/erpCommon/ad_callouts/SE_Invoice_BPartner.java --- a/src/org/openbravo/erpCommon/ad_callouts/SE_Invoice_BPartner.java Wed Aug 31 11:11:13 2016 +0200 +++ b/src/org/openbravo/erpCommon/ad_callouts/SE_Invoice_BPartner.java Wed Aug 31 11:25:47 2016 +0200 @@ -404,7 +404,6 @@ .eq(FinAccPaymentMethod.PROPERTY_PAYMENTMETHOD, selectedPaymentMethod)); obc.add(Restrictions.in(FinAccPaymentMethod.PROPERTY_ORGANIZATION + ".id", OBContext .getOBContext().getOrganizationStructureProvider().getNaturalTree(strOrgId))); - obc.setMaxResults(1); // filter is on unique constraint so list().size() <=1 always if (obc.uniqueResult() == null) { ------------------------------------------------------------------------------ _______________________________________________ Openbravo-commits mailing list Openbravo-commits@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/openbravo-commits