Uploaded image for project: 'Kuali Rice Development'
  1. Kuali Rice Development
  2. KULRICE-5228

Convert {KEW}WorkgroupMembershipChangeService to a remotable SOAP service

    Details

    • Type: Task
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.0.0-m7, 2.0
    • Component/s: Version Compatibility
    • Security Level: Public (Public: Anyone can view)
    • Labels:
      None
    • Rice Module:
      KEW
    • KAI Review Status:
      Not Required
    • KTI Review Status:
      Not Required

      Attachments

        Activity

        Hide
        jamesrenfro James Renfro (Inactive) added a comment -

        Just want to make sure I'm going down the right route here. I've been working on the idea that there are a few changes that need to be made in addition to the obvious annotations and Spring stuff to make this more clearly a SOAP service. One is giving the invoke method a name that means something and another is passing in the arguments in a transparent way so the caller can understand what it's calling without having to use a static method on the class to wrap it up. One additional thing seems to be that these calls look like events to me, rather than service calls, so my expectation would be that the method should be something like 'membershipChange' to indicate that it's more of a notification than an action, even if an action ends up getting taken as a result of that event. Does that seem reasonable?

        Show
        jamesrenfro James Renfro (Inactive) added a comment - Just want to make sure I'm going down the right route here. I've been working on the idea that there are a few changes that need to be made in addition to the obvious annotations and Spring stuff to make this more clearly a SOAP service. One is giving the invoke method a name that means something and another is passing in the arguments in a transparent way so the caller can understand what it's calling without having to use a static method on the class to wrap it up. One additional thing seems to be that these calls look like events to me, rather than service calls, so my expectation would be that the method should be something like 'membershipChange' to indicate that it's more of a notification than an action, even if an action ends up getting taken as a result of that event. Does that seem reasonable?
        Hide
        ewestfal Eric Westfall added a comment -

        Yes, you could call it something like notifyMembershipChange, that sounds good to me.

        Show
        ewestfal Eric Westfall added a comment - Yes, you could call it something like notifyMembershipChange, that sounds good to me.
        Hide
        jamesrenfro James Renfro (Inactive) added a comment -

        Fix checked into trunk as r21966 and r21967.

        Show
        jamesrenfro James Renfro (Inactive) added a comment - Fix checked into trunk as r21966 and r21967.
        Hide
        riceci Rice-CI User (Inactive) added a comment -

        Integrated in rice-trunk-nightly #146 (See http://ci.rice.kuali.org/job/rice-trunk-nightly/146/)

        Show
        riceci Rice-CI User (Inactive) added a comment - Integrated in rice-trunk-nightly #146 (See http://ci.rice.kuali.org/job/rice-trunk-nightly/146/ )
        Hide
        jcoltrin Jessica Coltrin (Inactive) added a comment -

        Closing since these items are now in the release notes.

        Show
        jcoltrin Jessica Coltrin (Inactive) added a comment - Closing since these items are now in the release notes.

          People

          • Assignee:
            jamesrenfro James Renfro (Inactive)
            Reporter:
            ewestfal Eric Westfall
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Time Tracking

              Estimated:
              Original Estimate - 4 hours Original Estimate - 4 hours
              4h
              Remaining:
              Remaining Estimate - 0 minutes
              0m
              Logged:
              Time Spent - 5 hours
              5h