Uploaded image for project: 'KS Enrollment'
  1. KS Enrollment
  2. KSENROLL-1354

Registration Appointments broken after Rice 2.2.0-M1 upgrade.

    Details

    • Type: Bug Fix
    • Status: Closed
    • Priority: Blocker
    • Resolution: Fixed
    • Affects Version/s: M3 - Sprint 1
    • Fix Version/s: M3 , M3 - Sprint 3
    • Component/s: PDT - Red
    • Security Level: Public (Public: Anyone can view)
    • Labels:
      None
    • KS Theme:
      Awaiting Rice Fix

      Description

      After the rice upgrade the Manage Registration appointments window broke.

      To Recreate:
      1. Manage Registration Windows And Appointments
      2. Enter: Spring Term, 2012
      3. Select Spring Registration Period 1, click Show blank screen with no errors or stack traces

      Related Skype Conversations:
      [5/29/2012 3:59:43 PM] Garey: I spent some time today going through the Manage Reg Appointment screens trying to determine if it was a data or krad issue that was breaking the screens
      [5/29/2012 4:00:14 PM] Garey: I Believe it's a KRAD issue so I would like for Bonnie to work with Brian on figuring this out.
      [5/29/2012 4:01:28 PM | Edited 4:03:03 PM] Garey: Everything looks good in our RegistrationWindowsController.java. The show method properly gets all required data and then calls the getUIFModelAndView(form)
      [5/29/2012 4:03:26 PM] Garey: so that's why I think there's a problem with our view.xml class:
      https://svn.kuali.org/repos/student/branches/ks-1.3/ks-enroll/ks-enroll-ui/src/main/resources/org/kuali/student/enrollment/appointment/RegistrationWindowsEditPage.xml
      [5/29/2012 4:05:59 PM] Garey: So, Bonnie, can you work with Brian to see if that xml page is correctly formatted? I know things changed in rice 2.2, but I'm not sure how our stuff is lining up

      [5/29/2012 4:08:11 PM] Andy Lubbers: Here's the page I mentioned that had the list of krad 2.0 - 2.2 changes earlier Garey, in case it helps:
      [5/29/2012 4:08:12 PM] Andy Lubbers: https://wiki.kuali.org/display/KULRICE/Post-2.0+impacting+changes

      [5/29/2012 4:26:13 PM] Brian Smith: sorry havent been following this last part
      [5/29/2012 4:27:38 PM] Brian Smith: what is the error you get
      [5/29/2012 4:30:09 PM] Brian Smith: there is nothing that pops out at me in the xml
      [5/29/2012 4:30:46 PM] Brian Smith: does it not render with no errors?
      [5/29/2012 4:31:00 PM] Brian Smith: is there a tomcat log or something for the deployed version?
      [5/29/2012 4:38:07 PM] Brian Smith: Start commenting out things if it is not rendering comment out items and the sections and see if it renders the page
      [5/29/2012 4:38:23 PM] Brian Smith: then go from there adding stuff back in til it stops rendering again
      [5/29/2012 4:38:32 PM] Brian Smith: (if that is the problem)

      [5/29/2012 5:12:52 PM] Weixia (Bonnie) Huang: select spring 2012, click search, it bring you to RegistrationWindowsEditPage.xml
      [5/29/2012 5:15:58 PM] Weixia (Bonnie) Huang: select from the drop-down list for period and click show button, it should display <ref bean="KS-RegistrationWindows-AddWindowsSection"/> section. but now nothing show up. not sure about error logged on server side.

      [5/29/2012 5:36:02 PM] Brian Smith: actually I do see a weird thing in here now
      [5/29/2012 5:36:18 PM] Brian Smith: progressiveRender condition pointing to showAddWindows
      [5/29/2012 5:39:39 PM] Weixia (Bonnie) Huang: is that a problem?
      [5/29/2012 5:39:51 PM] Brian Smith: why are you using progressiveRender here?
      [5/29/2012 5:40:17 PM] Brian Smith: show is making a roundtrip call
      [5/29/2012 5:40:32 PM] Brian Smith: to refresh the view
      [5/29/2012 5:40:33 PM] Weixia (Bonnie) Huang: we want to hide AddWindows collection till user select a period from the drop-down list and click show button
      [5/29/2012 5:40:49 PM] Brian Smith: why are they sections and not pages
      [5/29/2012 5:40:57 PM | Edited 5:41:35 PM] Brian Smith: the section appears below the selection?
      [5/29/2012 5:41:46 PM] Weixia (Bonnie) Huang: you mean

      <bean id="KS-RegistrationWindows-EditSection" parent="Uif-Page" p:disclosure.render="false" p:style="min-width:70em;"
                  p:headerText="Registration Windows">
              <property name="items">
                  <list>
                      <ref bean="KS-RegistrationWindows-PeriodSection"/>
                      <ref bean="KS-RegistrationWindows-AddWindowsSection"/>
                  </list>
              </property>
          </bean>
      

      [5/29/2012 5:42:15 PM] Brian Smith: yes
      [5/29/2012 5:42:20 PM] Brian Smith: so its below the section right?
      [5/29/2012 5:43:24 PM] Brian Smith: it should be render not progressive render if you are doing a full view/page refresh on button click
      [5/29/2012 5:43:38 PM] Brian Smith: progressive render is for inline content being updated
      [5/29/2012 5:44:42 PM] Weixia (Bonnie) Huang: what is defined as the inline content?
      [5/29/2012 5:45:27 PM] Brian Smith: anything else
      [5/29/2012 5:45:32 PM] Brian Smith: ?
      [5/29/2012 5:45:39 PM] Brian Smith: confused
      [5/29/2012 5:45:52 PM] Weixia (Bonnie) Huang: call?
      [5/29/2012 5:46:04 PM] Brian Smith: progressive render is for when you have a section and you change something
      [5/29/2012 5:46:12 PM] Brian Smith: and it exposes another field or section

        Attachments

          Activity

          Hide
          huangb Bonnie Huang (Inactive) added a comment -

          The problem is the parent bean of KS-RegistrationWindows-EditSection should be a section but not Uif-Page. Fixed.

          Show
          huangb Bonnie Huang (Inactive) added a comment - The problem is the parent bean of KS-RegistrationWindows-EditSection should be a section but not Uif-Page. Fixed.
          Hide
          gtaylor Garey Taylor added a comment -

          Closing resolved issues.

          Show
          gtaylor Garey Taylor added a comment - Closing resolved issues.
          Hide
          mpg Michael Passarella-George added a comment -

          Reopening to tweak some values in order to support Jira/GH upgrade

          Show
          mpg Michael Passarella-George added a comment - Reopening to tweak some values in order to support Jira/GH upgrade
          Hide
          mpg Michael Passarella-George added a comment -

          Closing after tweaking some values in order to support Jira/GH upgrade

          Show
          mpg Michael Passarella-George added a comment - Closing after tweaking some values in order to support Jira/GH upgrade

            People

            • Assignee:
              Unassigned
              Reporter:
              gtaylor Garey Taylor
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Time Tracking

                Estimated:
                Original Estimate - 1 day
                1d
                Remaining:
                Remaining Estimate - 1 day
                1d
                Logged:
                Time Spent - Not Specified
                Not Specified