Uploaded image for project: 'OFBiz'
  1. OFBiz
  2. OFBIZ-8223

Broken page renders while click on Submit button on Qualification Screen

    Details

    • Sprint:
      Community Day 3 - 2016

      Description

      Steps to reproduce:
      1. Go to party manager.
      2. Find party Company and navigate to Qualification tab.
      3. Now if there is no existing application and Submit button visible then click on submit button.
      4. Broken page renders with error message "
      :ERROR MESSAGE:
      org.ofbiz.webapp.event.EventHandlerException: No rows to process"
      Ideally system should hide submit button if there is no application to process.

      1. OFBIZ-8223.patch
        0.7 kB
        Avnindra Sharma

        Activity

        Hide
        mbrohl Michael Brohl added a comment -

        Avnindra Sharma,

        thanks for your contribution. As far as I can see, you just removed the submit button completely and not, as stated in the description, "if there is no application to process.".

        Can you please review if this is intended?

        Thanks and regards,
        Michael

        Show
        mbrohl Michael Brohl added a comment - Avnindra Sharma , thanks for your contribution. As far as I can see, you just removed the submit button completely and not, as stated in the description, "if there is no application to process.". Can you please review if this is intended? Thanks and regards, Michael
        Hide
        avnindra.sharma Avnindra Sharma added a comment -

        Hey Michael Brohl
        I have intentionally removed the submit button. There is no use of it and in the result of find screen, there is only on button required i.e delete, which already exists.
        For more ref: https://ofbiz-vm.apache.org:8443/humanres/control/FindPartyQuals

        Show
        avnindra.sharma Avnindra Sharma added a comment - Hey Michael Brohl I have intentionally removed the submit button. There is no use of it and in the result of find screen, there is only on button required i.e delete, which already exists. For more ref: https://ofbiz-vm.apache.org:8443/humanres/control/FindPartyQuals
        Hide
        mbrohl Michael Brohl added a comment -

        Thanks Avnindra,

        your fix is in trunk r1761492 and backported to
        release branch 15.12 r1761494
        release branch 14.12 r1761495
        release branch 13.07 r1761496

        Show
        mbrohl Michael Brohl added a comment - Thanks Avnindra, your fix is in trunk r1761492 and backported to release branch 15.12 r1761494 release branch 14.12 r1761495 release branch 13.07 r1761496
        Hide
        mbrohl Michael Brohl added a comment -

        Reopen to add to sprint.

        Show
        mbrohl Michael Brohl added a comment - Reopen to add to sprint.
        Hide
        jacques.le.roux Jacques Le Roux added a comment -

        Do we still sprint?

        Show
        jacques.le.roux Jacques Le Roux added a comment - Do we still sprint?
        Hide
        mbrohl Michael Brohl added a comment -

        Yes, according to Sharans announcement the sprint is open until Tuesday morning.

        Show
        mbrohl Michael Brohl added a comment - Yes, according to Sharans announcement the sprint is open until Tuesday morning.

          People

          • Assignee:
            mbrohl Michael Brohl
            Reporter:
            avnindra.sharma Avnindra Sharma
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development

                Agile