-
Type:
Bug
-
Status: Done
-
Priority:
Medium
-
Resolution: Done
-
Affects Version/s: None
-
Fix Version/s: Enhancements - Fall
-
Labels:None
-
Company:All Clients/Multiple Clients
There is line of code 'parsedKeyValuePdfDataparam = parsedKeyValuePdfData.ToDictionary(d => d.Key, d => d.Value);' in DocumentData.GetFormDetailsWithValues where possible exception is not handled. Now when parsedKeyValuePdfData has any duplicate column name, this method throws exception. Here if duplicate keys exist, then either that key should be skipped or value be overridden.
Harshveer Singh (Inactive)
created issue -
Harshveer Singh (Inactive)
made changes -
| Field | Original Value | New Value |
|---|---|---|
| Issue Type | New Feature [ 2 ] | Bug [ 1 ] |
| Remaining Estimate | 4h [ 14400 ] | |
| Original Estimate | 4h [ 14400 ] |
ShashiKant Mishra (Inactive)
made changes -
| Fix Version/s | Enhancements - Fall [ 10610 ] |
| Assignee | Samir [ samir ] | Harshveer Singh [ harshveer.singh ] |
ShashiKant Mishra (Inactive)
made changes -
| Sprint | DMS - Fall Sprint 3 [ 162 ] | DMS - Fall Sprint 3, DMS - Fall Sprint 4 [ 162, 163 ] |
ShashiKant Mishra (Inactive)
made changes -
| Sprint | DMS - Fall Sprint 3, DMS - Fall Sprint 4 [ 162, 163 ] | DMS - Fall Sprint 3, DMS - Fall Sprint 4, DMS - Fall Sprint 5 [ 162, 163, 164 ] |
| Assignee | Harshveer Singh [ harshveer.singh ] | Akash Thakur [ akash.thakur ] |
| Status | To Do [ 10000 ] | In Progress [ 3 ] |
-
- Time Spent:
- 4h
-
- Understanding Bug
- Analysis.
- Reproduction with Template creation for duplicate values.
- bug fix approach suitability.
- discussion with Ganesh
- code fix & code merges PR creation.
- Unit testing with a duplicate key.
- etc efforts.
| Remaining Estimate | 4h [ 14400 ] | 0h [ 0 ] |
| Time Spent | 4h [ 14400 ] | |
| Worklog Id | 183784 [ 183784 ] |
| Assignee | Akash Thakur [ akash.thakur ] | Harshveer Singh [ harshveer.singh ] |
| Status | In Progress [ 3 ] | Code Review [ 11801 ] |
| Assignee | Harshveer Singh [ harshveer.singh ] | Samir [ samir ] |
ShashiKant Mishra (Inactive)
made changes -
| Sprint | DMS - Fall Sprint 3, DMS - Fall Sprint 4, DMS - Fall Sprint 5 [ 162, 163, 164 ] | DMS - Fall Sprint 3, DMS - Fall Sprint 4, DMS - Fall Sprint 5, DMS - Fall Sprint 6 [ 162, 163, 164, 165 ] |
Raj Hawaldar (Inactive)
made changes -
| Status | Code Review [ 11801 ] | In QA Testing [ 11901 ] |
ShashiKant Mishra (Inactive)
made changes -
| Sprint | DMS - Fall Sprint 3, DMS - Fall Sprint 4, DMS - Fall Sprint 5, DMS - Fall Sprint 6 [ 162, 163, 164, 165 ] | DMS - Fall Sprint 3, DMS - Fall Sprint 4, DMS - Fall Sprint 5, DMS - Fall Sprint 6, DMS - Fall Sprint 7 [ 162, 163, 164, 165, 166 ] |
| Assignee | Samir [ samir ] | Priya Dhamande [ priya.dhamande ] |
can you please let us know, where to verify teh issue?
| Status | In QA Testing [ 11901 ] | In Progress [ 3 ] |
| Assignee | Priya Dhamande [ priya.dhamande ] | Akash Thakur [ akash.thakur ] |
| Status | In Progress [ 3 ] | In QA Testing [ 11901 ] |
| Assignee | Akash Thakur [ akash.thakur ] | Priya Dhamande [ priya.dhamande ] |
Hi Priya Dhamande,
Please test this with HTML template having two fields with same control name in it.
PFA: attached sample HTML file.
Thanks,
Akash Thakur
| Attachment | test2 (2).html [ 112295 ] |
Environment: Production
Company: CB internal Beta
File Type: HTML
Added 2 fields with same name
Steps:
1. Admin Login
2. Send Forms [Template with double FName]
3. Preview > Name is appearing twice.
Example:
FNAme of candidate is ABC
Preview
FName: ABC ABC
Fname: ABC ABC
Expected:
FName: ABC
Fname: ABC
Refer screen capture.
So, reopening for the same.
| Attachment | Screenshot (2997).png [ 112499 ] | |
| Attachment | Screenshot (2998).png [ 112500 ] |
| Time Spent | 4h [ 14400 ] | 5h [ 18000 ] |
| Worklog Id | 188356 [ 188356 ] |
| Status | In QA Testing [ 11901 ] | To Do [ 10000 ] |
| Assignee | Priya Dhamande [ priya.dhamande ] | Akash Thakur [ akash.thakur ] |
Hi Priya Dhamande,
The scenario you verified under this is not correct. The Scenario is.
Valid Scenario:
- HTML file does not have two fields with same name.
- Issue: Its throwing error- Exception
Issue Fixed.
- We handle expectation under this. Priya Dhamande please create separate Jira to handle that file does not have same field name more than one.
CC-Rohan J Khandave,Samir,Hrishikesh Deshpande
Thanks,
Ganesh
| Status | To Do [ 10000 ] | In QA Testing [ 11901 ] |
| Assignee | Akash Thakur [ akash.thakur ] | Priya Dhamande [ priya.dhamande ] |
As per above comment, considering this change done. As the expectation thrown is handled.
Creating new jira for redundant name with ID: DMS-1645.
Ganesh Sadawarte Akash Thakur Samir Hrishikesh Deshpande Rohan J Khandave
| Resolution | Done [ 10000 ] | |
| Status | In QA Testing [ 11901 ] | Done [ 10001 ] |
| Assignee | Priya Dhamande [ priya.dhamande ] | Akash Thakur [ akash.thakur ] |
| Time Spent | 5h [ 18000 ] | 5.25h [ 18900 ] |
| Worklog Id | 188364 [ 188364 ] |
| Transition | Time In Source Status | Execution Times |
|---|
|
35d 22h 24m | 1 |
|
11m 7s | 1 |
Raj Hawaldar (Inactive)
made transition
-
|
8d 2h 48m | 1 |
|
15d 56m | 1 |
|
53m 49s | 1 |
|
3d 18h 1m | 1 |
|
50m 32s | 1 |
|
19m 34s | 1 |
.png){kind=link}
.png){kind=link}
Hi Harshveer Singh,
I have given a fix for this unhandled exception occurrence point.Please review the code
Thanks,
Akash Thakur
Ganesh Sadawarte Rohan J Khandave