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

Convert Content entites CRUD service from simple to entity-auto

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Done
    • Affects Version/s: Trunk
    • Fix Version/s: 16.11.01
    • Component/s: content
    • Labels:

      Description

      Convert some content entities crud service from simple to entity-auto.
      The list is present on comments.

        Issue Links

          Activity

          Hide
          soledad Nicolas Malin added a comment -

          I realized a first step on these entities :

          • DataResourceAttribute
          • ContentRole (not for create will set on second step)
          • ContentMetaData
          • ContentOperation
          • ContentPurpose
          • ContentPurposeOperation
          • ContentAttribute
          • ContentKeyword
          • ContentType
          • ContentAssocType
          • ContentAssocPredicate
          • ContentPurposeType

          commited on revision 1715960.
          I keep this issues open for the second step on more Data entities and after a third step on complicated case

          Show
          soledad Nicolas Malin added a comment - I realized a first step on these entities : DataResourceAttribute ContentRole (not for create will set on second step) ContentMetaData ContentOperation ContentPurpose ContentPurposeOperation ContentAttribute ContentKeyword ContentType ContentAssocType ContentAssocPredicate ContentPurposeType commited on revision 1715960. I keep this issues open for the second step on more Data entities and after a third step on complicated case
          Hide
          soledad Nicolas Malin added a comment -

          second step on revision 1716271 that concern the following entities :

          • WebSiteRole
          • WebSitePathAlias
          • WebSiteContentType
          • WebSiteContent
          • WebSite
          • DataResourcePurpose
          • DataResourceMetaData
          • OtherDataResource
          • AudioDataResource
          • VideoDataResource
          • ImageDataResource
          • DataResource (update/delete)
          • ContentApproval
          • ContentRevisionItem
          • ContentRevision
          Show
          soledad Nicolas Malin added a comment - second step on revision 1716271 that concern the following entities : WebSiteRole WebSitePathAlias WebSiteContentType WebSiteContent WebSite DataResourcePurpose DataResourceMetaData OtherDataResource AudioDataResource VideoDataResource ImageDataResource DataResource (update/delete) ContentApproval ContentRevisionItem ContentRevision
          Hide
          soledad Nicolas Malin added a comment -

          finish the convert for ContentRole (create) at revision 1716542

          Show
          soledad Nicolas Malin added a comment - finish the convert for ContentRole (create) at revision 1716542
          Hide
          jacques.le.roux Jacques Le Roux added a comment -

          I fixed an issue introduced with r1715960 at r1724411, a (local) simple-method was called but it was replaced by a service (auto-engine).

          Show
          jacques.le.roux Jacques Le Roux added a comment - I fixed an issue introduced with r1715960 at r1724411, a (local) simple-method was called but it was replaced by a service (auto-engine).
          Hide
          soledad Nicolas Malin added a comment -

          Oh it's my bad, thanks Jacques to spot and fixed it .
          The problem exist on 15.12, I will backport.

          Maybe after we can close this issue now.

          Show
          soledad Nicolas Malin added a comment - Oh it's my bad, thanks Jacques to spot and fixed it . The problem exist on 15.12, I will backport. Maybe after we can close this issue now.
          Hide
          soledad Nicolas Malin added a comment -

          Done on 15.12 at revision 1724415

          Show
          soledad Nicolas Malin added a comment - Done on 15.12 at revision 1724415
          Hide
          jacques.le.roux Jacques Le Roux added a comment -

          Thanks Nicolas, at r1724417 I simply put the svn:mergeinfo

          I agree we can close now and refer to it if we find other bugs, so doing so.

          Show
          jacques.le.roux Jacques Le Roux added a comment - Thanks Nicolas, at r1724417 I simply put the svn:mergeinfo I agree we can close now and refer to it if we find other bugs, so doing so.

            People

            • Assignee:
              soledad Nicolas Malin
              Reporter:
              soledad Nicolas Malin
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development