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

All postalAddress simple-map-processors miss some fields

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Major
    • Resolution: Done
    • Affects Version/s: Trunk
    • Fix Version/s: 16.11.01
    • Component/s: party, product
    • Labels:
      None

      Description

      There are 5 postalAddress simple-map-processors. All miss some fields which are in PostalAddress entity.

      This could be seen as a moot point for some of them, but the main ones in party and product component should not miss these fields.

        Activity

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

        This is indeed done

        Show
        jacques.le.roux Jacques Le Roux added a comment - This is indeed done
        Hide
        jacques.le.roux Jacques Le Roux added a comment -

        Oops mixed things with OFBIZ-6513 => reopen

        Show
        jacques.le.roux Jacques Le Roux added a comment - Oops mixed things with OFBIZ-6513 => reopen
        Hide
        jacques.le.roux Jacques Le Roux added a comment -

        After some hesitation I finally decided this was good to backport, It's a better complete way of designing.

        Pierre Smits it makes sense to have those 5 different map-processors (not pre-processors ) they have different goals

        Show
        jacques.le.roux Jacques Le Roux added a comment - After some hesitation I finally decided this was good to backport, It's a better complete way of designing. Pierre Smits it makes sense to have those 5 different map-processors (not pre-processors ) they have different goals
        Hide
        pfm.smits Pierre Smits added a comment -

        Having 5 address pre-processors seems a bit more than just 'moot'.

        Show
        pfm.smits Pierre Smits added a comment - Having 5 address pre-processors seems a bit more than just 'moot'.
        Hide
        jacques.le.roux Jacques Le Roux added a comment -

        Done at revision: 1686071

        Like for OFBIZ-6513, I wonder if we should not backport those. For me both cases should not introduces issues but could help users to have something more complete OOTB. So I don't close yet...

        Opinions?

        Show
        jacques.le.roux Jacques Le Roux added a comment - Done at revision: 1686071 Like for OFBIZ-6513 , I wonder if we should not backport those. For me both cases should not introduces issues but could help users to have something more complete OOTB. So I don't close yet... Opinions?

          People

          • Assignee:
            jacques.le.roux Jacques Le Roux
            Reporter:
            jacques.le.roux Jacques Le Roux
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development