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

PicklistStatusHistory doesn't follow history entity status pattern

Attach filesAttach ScreenshotVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Minor
    • Resolution: Done
    • Trunk
    • None
    • product
    • None

    Description

      The entity PicklistStatusHistory record each status change realized on picklist.

      It can't be convert to entity-auto easily because PicklistStatusHistory's fields pattern doesn't follow same entities like ShipmentStatus and PartyStatus.

      I propose to deprecate the entity PicklistStatusHistory:

           <entity entity-name="OldPicklistStatusHistory" table-name="PICKLIST_STATUS_HISTORY"
                  package-name="org.apache.ofbiz.shipment.picklist"
                  title="Picklist Status History">
            <field name="picklistId" type="id"></field>
            <field name="changeDate" type="date-time"></field>
            <field name="changeUserLoginId" type="id-vlong"></field>
            <field name="statusId" type="id"></field>
            <field name="statusIdTo" type="id"></field>
            <prim-key field="picklistId"/>
            <prim-key field="changeDate"/>...
          </entity>
      
      

      and replace it with an entity managed natively by entity-auto engine

           <entity entity-name="PicklistStatus"
                  package-name="org.apache.ofbiz.shipment.picklist"
                  title="Picklist Status History">
              <field name="picklistId" type="id"/>
              <field name="statusDate" type="date-time"/>
              <field name="changeByUserLoginId" type="id-vlong"/>
              <field name="statusId" type="id"/>
              <field name="statusIdTo" type="id"/>
              <prim-key field="picklistId"/>
              <prim-key field="statusDate"/>...
          </entity>

       

      Attachments

        Issue Links

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            nmalin Nicolas Malin
            nmalin Nicolas Malin
            Votes:
            1 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment