Uploaded image for project: 'OFBiz'
  1. OFBiz
  2. OFBIZ-9579 Required field indicator (*) is missing on some forms
  3. OFBIZ-9583

Client side validation is missing in quick create accounting transaction process

    Details

    • Type: Sub-task
    • Status: Closed
    • Priority: Major
    • Resolution: Implemented
    • Affects Version/s: Trunk
    • Fix Version/s: Upcoming Release
    • Component/s: accounting
    • Labels:
      None

      Description

      Go to Accounting -> Organization GL Settings -> Accounting -> Accounting Transactions -> Quick Create an Accounting Transaction.

      Now try to create a transaction without entering the amount. You will get an error from the server side. It should be validated at client side and an indicator of required field should also be there.

      1. OFBIZ-9583.patch
        0.7 kB
        Suraj Khurana

        Activity

        Hide
        jacques.le.roux Jacques Le Roux added a comment -

        Er, sorry Suraj

        Show
        jacques.le.roux Jacques Le Roux added a comment - Er, sorry Suraj
        Hide
        suraj.khurana Suraj Khurana added a comment -

        Thanks Jacques Le Roux,

        BTW, it's Suraj not Surag :|

        Show
        suraj.khurana Suraj Khurana added a comment - Thanks Jacques Le Roux , BTW, it's Suraj not Surag :|
        Hide
        jacques.le.roux Jacques Le Roux added a comment -

        Thanks Surag,

        Your patch is in trunk at revision: 1805290

        Show
        jacques.le.roux Jacques Le Roux added a comment - Thanks Surag, Your patch is in trunk at revision: 1805290
        Hide
        suraj.khurana Suraj Khurana added a comment -

        Here is a patch with proper fix.

        Show
        suraj.khurana Suraj Khurana added a comment - Here is a patch with proper fix.
        Hide
        deepak.nigam Deepak Nigam added a comment -

        Done.
        Thanks Jacques Le Roux.

        Show
        deepak.nigam Deepak Nigam added a comment - Done. Thanks Jacques Le Roux .
        Hide
        jacques.le.roux Jacques Le Roux added a comment -

        Hi Deepak,

        I think we can make this issue a subtask of OFBIZ-9579. Adding the missing * indicator should do the job (block the action).

        Show
        jacques.le.roux Jacques Le Roux added a comment - Hi Deepak, I think we can make this issue a subtask of OFBIZ-9579 . Adding the missing * indicator should do the job (block the action).

          People

          • Assignee:
            suraj.khurana Suraj Khurana
            Reporter:
            deepak.nigam Deepak Nigam
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development