Details

      Description

      Change the field 'andCondition' (which use direct call to groovy) to 'conditionFields' in the lookup LookupScreens#LookupCustomerName.

      1. OFBIZ-7655.patch
        1 kB
        Montalbano Florian
      2. OFBIZ-7656.patch
        1.0 kB
        Montalbano Florian

        Activity

        Hide
        Florian M Montalbano Florian added a comment -

        The patch using 'conditionFields' instead of 'andCondition'.
        Every review/test are welcome !

        Show
        Florian M Montalbano Florian added a comment - The patch using 'conditionFields' instead of 'andCondition'. Every review/test are welcome !
        Hide
        jacques.le.roux Jacques Le Roux added a comment -

        +1 Good

        Show
        jacques.le.roux Jacques Le Roux added a comment - +1 Good
        Hide
        jacques.le.roux Jacques Le Roux added a comment -

        Florian, once you have provided a patch, please clic on the "Provide Patch" button Thanks!

        Show
        jacques.le.roux Jacques Le Roux added a comment - Florian, once you have provided a patch, please clic on the "Provide Patch" button Thanks!
        Hide
        Florian M Montalbano Florian added a comment -

        Thanks to a comment from Deepak Dixit, the patch can be lighter and the code easier to read !
        Change : The map 'conditionFields' does not need to be declared empty. It is done by minilang when the first field of the map is set.

        Show
        Florian M Montalbano Florian added a comment - Thanks to a comment from Deepak Dixit , the patch can be lighter and the code easier to read ! Change : The map 'conditionFields' does not need to be declared empty. It is done by minilang when the first field of the map is set.
        Hide
        soledad Nicolas Malin added a comment -

        Ok it's fine for me also

        It's commited on trunk at revision 1750475

        Show
        soledad Nicolas Malin added a comment - Ok it's fine for me also It's commited on trunk at revision 1750475

          People

          • Assignee:
            soledad Nicolas Malin
            Reporter:
            Florian M Montalbano Florian
          • Votes:
            1 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development