Details

    • Type: Sub-task Sub-task
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: SVN trunk
    • Fix Version/s: SVN trunk
    • Component/s: party
    • Labels:
      None

      Description

      1. find_edit_party.patch
        17 kB
        Vikas Mayur
      2. find_edit_party.patch
        18 kB
        Santosh Malviya
      3. find_edit_party.patch
        19 kB
        Awdesh Parihar
      4. find_edit_party.patch
        19 kB
        Awdesh Parihar
      5. find_edit_party.patch
        20 kB
        Awdesh Parihar

        Activity

        Hide
        Vikas Mayur added a comment -

        Thanks Awdesh, Ratnesh, Santosh and others,

        Patch along with few modifications is in trunk rev. 663913

        • Vikas
        Show
        Vikas Mayur added a comment - Thanks Awdesh, Ratnesh, Santosh and others, Patch along with few modifications is in trunk rev. 663913 Vikas
        Hide
        Awdesh Parihar added a comment -

        Thanks Vikas
        In this patch I followed your suggestion and used PartyWorker methods to fetch contactMechId's for updating postal, telecom, email address .

        Thanks to Ratnesh Upadhyay for helping.
        Thanks and Regards,
        -Awdesh Parihar

        Show
        Awdesh Parihar added a comment - Thanks Vikas In this patch I followed your suggestion and used PartyWorker methods to fetch contactMechId's for updating postal, telecom, email address . Thanks to Ratnesh Upadhyay for helping. Thanks and Regards, -Awdesh Parihar
        Hide
        Vikas Mayur added a comment -

        Awdesh - patch looks good.

        One change that I would like to propose is to use PartyWorker or ContactHelper methods to fetch contactMechId's for updating postal, telecom, email address etc. instead of using constant values.
        This way test case will also work if it is run more than once.

        • Vikas
        Show
        Vikas Mayur added a comment - Awdesh - patch looks good. One change that I would like to propose is to use PartyWorker or ContactHelper methods to fetch contactMechId's for updating postal, telecom, email address etc. instead of using constant values. This way test case will also work if it is run more than once. Vikas
        Hide
        Awdesh Parihar added a comment -

        Done some modification in last uploaded patch :
        1) . Put some condition in assert tag.
        2). Changed some services from last uploaded patch which is not participating in process .

        Thanks to Ratnesh Upadhyay for helping.
        Thanks and Regards,
        -Awdesh Parihar

        Show
        Awdesh Parihar added a comment - Done some modification in last uploaded patch : 1) . Put some condition in assert tag. 2). Changed some services from last uploaded patch which is not participating in process . Thanks to Ratnesh Upadhyay for helping. Thanks and Regards, -Awdesh Parihar
        Hide
        Santosh Malviya added a comment -

        I have converted Java services into minilang.
        Thanks to Brajesh Patel for discussion on the issue.

        Show
        Santosh Malviya added a comment - I have converted Java services into minilang. Thanks to Brajesh Patel for discussion on the issue.
        Hide
        Vikas Mayur added a comment -

        I think instead of Updating information on party DemoCustomer, we should create a test data probably with reader-name="test" and use that data to find, update information.
        Any Thoughts ?

        Show
        Vikas Mayur added a comment - I think instead of Updating information on party DemoCustomer, we should create a test data probably with reader-name="test" and use that data to find, update information. Any Thoughts ?
        Hide
        Ratnesh Upadhyay added a comment -

        I have tested it extensively and it is working fine.

        Show
        Ratnesh Upadhyay added a comment - I have tested it extensively and it is working fine.
        Hide
        Marco Risaliti added a comment -

        May I can test and then commit ?

        Show
        Marco Risaliti added a comment - May I can test and then commit ?
        Hide
        Vikas Mayur added a comment -

        Patch from Santosh Malviya

        Show
        Vikas Mayur added a comment - Patch from Santosh Malviya

          People

          • Assignee:
            Vikas Mayur
            Reporter:
            Vikas Mayur
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development