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

The update and deletion of content meta data tab is not working.Needs a change of entity ContentMetaData.

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: Release 09.04, Release 10.04, Release 11.04.01
    • Fix Version/s: 14.12.01, 12.04.06, 11.04.07, 13.07.02
    • Component/s: content
    • Labels:
      None
    • Environment:

      windows

      Description

      The updation and deletion of content meta data tab is not working. Needs a change of entity ContentMetaData. As metaDataValue is changed from primary key to non primary key as the value can be updated.

      1. OFBIZ-4927.patch
        0.9 kB
        Deepak Dixit
      2. updatemeta.txt
        0.7 kB
        Tata Consultancy Services ltd

        Activity

        Hide
        deepak.dixit Deepak Dixit added a comment -

        Here is patch for this issue, DataResourceMetaData entity have same issue. Remove metaDataValue from primary key.

        Show
        deepak.dixit Deepak Dixit added a comment - Here is patch for this issue, DataResourceMetaData entity have same issue. Remove metaDataValue from primary key.
        Hide
        jacques.le.roux Jacques Le Roux added a comment -

        Hi Deepak,

        I found how to reproduce in UI for ContentMetaData but not DataResourceMetaData. DId you find a way to test your change?

        Show
        jacques.le.roux Jacques Le Roux added a comment - Hi Deepak, I found how to reproduce in UI for ContentMetaData but not DataResourceMetaData. DId you find a way to test your change?
        Hide
        deepak.dixit Deepak Dixit added a comment - - edited

        No Jacques, I did not find UI for DataResourceMetaData. I found that this is same as ContentMetaData so added fix in patch as well.
        We can test it via webtools.

        Show
        deepak.dixit Deepak Dixit added a comment - - edited No Jacques, I did not find UI for DataResourceMetaData. I found that this is same as ContentMetaData so added fix in patch as well. We can test it via webtools.
        Hide
        jacques.le.roux Jacques Le Roux added a comment -

        Good idea, thanks Deepak

        Show
        jacques.le.roux Jacques Le Roux added a comment - Good idea, thanks Deepak
        Hide
        jacques.le.roux Jacques Le Roux added a comment -

        Thanks Deepak,

        Your patch is in
        trunk r1633100
        R13.07 r1633101
        R12.04 r1633102
        R11.04 r1633103

        Show
        jacques.le.roux Jacques Le Roux added a comment - Thanks Deepak, Your patch is in trunk r1633100 R13.07 r1633101 R12.04 r1633102 R11.04 r1633103
        Hide
        pfm.smits Pierre Smits added a comment -

        Does this warrants a cautionary message to all (end users and implementers) that the implementation strategy must be revisited regarding these commits? We're changing the entity model, aren't we?

        Show
        pfm.smits Pierre Smits added a comment - Does this warrants a cautionary message to all (end users and implementers) that the implementation strategy must be revisited regarding these commits? We're changing the entity model, aren't we?
        Hide
        jacques.le.roux Jacques Le Roux added a comment -

        Hi Pierre, this was not an improvement but an obvious bug in the data model. So I see no reasons to inform people. I guess nobody was using it before "Tata Consultancy Services" reported. And it took 2+ years before being fixed, so no worries.

        Show
        jacques.le.roux Jacques Le Roux added a comment - Hi Pierre, this was not an improvement but an obvious bug in the data model. So I see no reasons to inform people. I guess nobody was using it before "Tata Consultancy Services" reported. And it took 2+ years before being fixed, so no worries.

          People

          • Assignee:
            jacques.le.roux Jacques Le Roux
            Reporter:
            tcs Tata Consultancy Services ltd
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development