Uploaded image for project: 'Document Management System'
  1. Document Management System
  2. DMS-1665

Create document API - Single quote issue.

    Details

    • Type: Bug
    • Status: Done
    • Priority: Medium
    • Resolution: Done
    • Affects Version/s: None
    • Fix Version/s: Enhancements - Fall
    • Labels:
      None
    • Company:
      All Clients/Multiple Clients

      Description

      • When we give single quote in values throws as error as we creating query dynamically.
      • If requisition Id is not exists is throws an error.

        Attachments

        1. Screenshot (3489).png
          Screenshot (3489).png
          137 kB
        2. Screenshot (3490).png
          Screenshot (3490).png
          165 kB
        3. Screenshot (3491).png
          Screenshot (3491).png
          166 kB
        4. Screenshot (3492).png
          Screenshot (3492).png
          162 kB

          Activity

          Hide
          rohan.khandave Rohan J Khandave (Inactive) added a comment -

          Below are details

          Concern: Server error for create custom API.

          Cause: API payload contains single quote which causing issue while creating dynamic query.

          Correction: We have replace single quote with "`".

          CC – Samir , Ganesh Sadawarte 

          Show
          rohan.khandave Rohan J Khandave (Inactive) added a comment - Below are details Concern: Server error for create custom API. Cause: API payload contains single quote which causing issue while creating dynamic query. Correction: We have replace single quote with " ` ". CC – Samir , Ganesh Sadawarte  
          Hide
          Ganesh.sadawarte Ganesh Sadawarte (Inactive) added a comment -

          Code review completed

          Show
          Ganesh.sadawarte Ganesh Sadawarte (Inactive) added a comment - Code review completed
          Hide
          rohan.khandave Rohan J Khandave (Inactive) added a comment -

          As per discussion with Alex.

          We will always consider Email, sourceUserId ,requisitionId from main payload.

          Needs to ignore these fields values from candidate & requisition payload.

          CC – Samir , Ganesh Sadawarte

          Show
          rohan.khandave Rohan J Khandave (Inactive) added a comment - As per discussion with Alex. We will always consider Email, sourceUserId ,requisitionId from main payload. Needs to ignore these fields values from candidate & requisition payload. CC – Samir , Ganesh Sadawarte
          Hide
          priya.dhamande Priya Dhamande (Inactive) added a comment -

          Environment: Production - One IM
          API: api/candidate/custom/document/create

          Points verified:
          1. Email ID - different
          2. Email ID - same
          3. Single quote added for one of the

          Mentioned scenarios worked out. Refer screen capture. So, marking jira done.

          Sachin Hingole Hrishikesh Deshpande Rohan J Khandave Samir

          Show
          priya.dhamande Priya Dhamande (Inactive) added a comment - Environment: Production - One IM API: api/candidate/custom/document/create Points verified: 1. Email ID - different 2. Email ID - same 3. Single quote added for one of the Mentioned scenarios worked out. Refer screen capture. So, marking jira done. Sachin Hingole Hrishikesh Deshpande Rohan J Khandave Samir

            People

            Assignee:
            priya.dhamande Priya Dhamande (Inactive)
            Reporter:
            Ganesh.sadawarte Ganesh Sadawarte (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Time Tracking

                Estimated:
                Original Estimate - Not Specified
                Not Specified
                Remaining:
                Remaining Estimate - 0h
                0h
                Logged:
                Time Spent - 11h 20m
                11h 20m