Project

General

Profile

Actions

Bug #86

closed

Batch - ParseInt Issue

Added by Chooi-Mey almost 14 years ago. Updated almost 14 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
offline batches
Target version:
Start date:
02/17/2011
Due date:
% Done:

100%

Estimated time:
Spent time:
Resolution:
Fixed

Description

Below list of program with line of code parseInt on orderId , which will hit exception when orderId in alphanumeric format:

Potential exception hit: key = Integer.parseInt(orderBean.getOrderId());

List of affected program:
1) src\my\com\eCosway\sync\Receiver_Registration2.java (line 824)
2) src\my\com\eCosway\sync\Receiver_Registration.java (line 834)
3) src\my\com\eCosway\sync\ReceiveRenewalData.java (line 206, checkRecordIsExist method will parseInt the pass in orderId. Resolution: use setString instead of setInt)
4) src\my\com\eCosway\sync\ReceiveShopperBankDetail.java (line 725)

Actions #1

Updated by Chooi-Mey almost 14 years ago

Below list got parseLong code, which need to have additional checking before parse it.

1) src\my\com\powersys\util\TranxProcess.java (line 151)
2) \src-classes\DailyTranx.java (line 2017)
3) src-classes\OfflineTHTransaction.java (line 1036)
4) src-classes\OfflineTransaction.java (line 1566, 2165)
5) \src-classes\UnapplyOfflineReturns.java (line 884)
6) src-classes\UnapplyReturns.java (line 647, 648)
7) src-ecos\EIClassification.java (line 973)
8) \src-ecos\EIOtherAcct.java (756, 839)

Actions #2

Updated by chin-yeh almost 14 years ago

  • Status changed from New to In Progress
Actions #3

Updated by chin-yeh almost 14 years ago

  • Status changed from In Progress to Feedback
  • % Done changed from 0 to 100
  • Resolution set to Fixed
all are fixed except the following files:
  • src-ecos\EIClassification.java - it's using try catch to handle the parsing
  • \src-ecos\EIOtherAcct.java - it's using try catch to handle the parsing
Actions #4

Updated by chin-yeh almost 14 years ago

  • Status changed from Feedback to Closed

issue closed

Actions

Also available in: Atom PDF