Anonymous | Login
Project:
RSS
  
News | My View | View Issues | Roadmap | Summary

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0040716
TypeCategorySeverityReproducibilityDate SubmittedLast Update
backport[Retail Modules] Web POSmajorhave not tried2019-04-10 12:532019-05-02 08:30
ReportershuehnerView Statuspublic 
Assigned Toranjith_qualiantech_com 
PrioritynormalResolutionfixedFixed in VersionRR19Q2
StatusclosedFix in branchFixed in SCM revision8365c58d4244
ProjectionnoneETAnoneTarget VersionRR19Q2
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Review Assigned Tomarvintm
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0040716: CashUpReport.doPost is not using bind-parameters

DescriptionThis code is not using bind-params:

      final String hqlCashup = "SELECT netsales, grosssales, netreturns, grossreturns, totalretailtransactions " //
          + " FROM OBPOS_App_Cashup " //
          + " WHERE id = '" + cashupId + "' "; //
      final Query<Object[]> cashupQuery = OBDal.getReadOnlyInstance()
          .getSession()
          .createQuery(hqlCashup, Object[].class);

and

      final String hqlTaxes = String.format("SELECT name, STR(ABS(amount)) " //
          + " FROM OBPOS_Taxcashup " //
          + " WHERE obpos_app_cashup_id='%s' AND ordertype='0' " //
          + " ORDER BY name ", cashupId);
      final Query<Object[]> salesTaxesQuery = OBDal.getReadOnlyInstance()
          .getSession()
          .createQuery(hqlTaxes, Object[].class);

and

      final String hqlReturnTaxes = String.format("SELECT name, STR(ABS(amount)) " //
          + " FROM OBPOS_Taxcashup " //
          + " WHERE obpos_app_cashup_id='%s' AND ordertype='1' " //
          + " ORDER BY name ", cashupId);
      final Query<Object[]> returnsTaxesQuery = OBDal.getReadOnlyInstance()
          .getSession()
          .createQuery(hqlReturnTaxes, Object[].class);
Steps To Reproduce-
TagsNo tags attached.
Attached Files

- Relationships Relation Graph ] Dependency Graph ]
blocks defect 0040584 closedranjith_qualiantech_com CashUpReport.doPost is not using bind-parameters 

-  Notes
(0111412)
hgbot (developer)
2019-05-02 08:21

Repository: retail/backports/3.0RR19Q2/org.openbravo.retail.posterminal
Changeset: 8365c58d4244a6439a7728a7a98c8b433d166453
Author: Ranjith S R <ranjith <at> qualiantech.com>
Date: Thu May 02 11:51:16 2019 +0530
URL: http://code.openbravo.com/retail/backports/3.0RR19Q2/org.openbravo.retail.posterminal/rev/8365c58d4244a6439a7728a7a98c8b433d166453 [^]

Fixed issue 40716 : Update CashupReport query to use query parameter

---
M src/org/openbravo/retail/posterminal/ad_reports/CashUpReport.java
---

- Issue History
Date Modified Username Field Change
2019-04-30 11:55 shuehner Type defect => backport
2019-04-30 11:55 shuehner Target Version => RR19Q2
2019-05-02 08:21 hgbot Checkin
2019-05-02 08:21 hgbot Note Added: 0111412
2019-05-02 08:21 hgbot Status scheduled => resolved
2019-05-02 08:21 hgbot Resolution open => fixed
2019-05-02 08:21 hgbot Fixed in SCM revision => http://code.openbravo.com/retail/backports/3.0RR19Q2/org.openbravo.retail.posterminal/rev/8365c58d4244a6439a7728a7a98c8b433d166453 [^]
2019-05-02 08:30 marvintm Review Assigned To adrianromero => marvintm
2019-05-02 08:30 marvintm Status resolved => closed
2019-05-02 08:30 marvintm Fixed in Version => RR19Q2


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker