Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Duplicate
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: order
    • Labels:
      None

      Description

      Productname now comes from the content instead of the entity,

      1. Product.groovy.patch
        0.8 kB
        Christian Fischer

        Activity

        Hide
        Adrian Crum added a comment -

        Why is this change needed? And why is the issue marked as Resolved?

        Show
        Adrian Crum added a comment - Why is this change needed? And why is the issue marked as Resolved?
        Hide
        Christian Fischer added a comment -

        If you take the name frome the entity, you aren't able to take the localized names. This patch is useful, if you have different names for products in different languages.

        To the Resolved-mark: I think, I interpreted it wrong. I thought,Resolved means, that its fixed and ready to be tested. Sorry for that, won't happen again.

        Show
        Christian Fischer added a comment - If you take the name frome the entity, you aren't able to take the localized names. This patch is useful, if you have different names for products in different languages. To the Resolved-mark: I think, I interpreted it wrong. I thought,Resolved means, that its fixed and ready to be tested. Sorry for that, won't happen again.
        Hide
        Adrian Crum added a comment -

        You can localize entity fields by using the UI labels files. There are examples of this in code already.

        Show
        Adrian Crum added a comment - You can localize entity fields by using the UI labels files. There are examples of this in code already.
        Hide
        Christian Fischer added a comment -

        In this Patch we get a productName from Content. I dont think it is usefull to get individual ProductData from EntityLabels.

        Show
        Christian Fischer added a comment - In this Patch we get a productName from Content. I dont think it is usefull to get individual ProductData from EntityLabels.
        Hide
        Jacques Le Roux added a comment - - edited

        Aboutthe Resolved status: if you provide a patch you should rather use "Patch provided" status

        Edit I meaned "Patch available"

        Show
        Jacques Le Roux added a comment - - edited Aboutthe Resolved status: if you provide a patch you should rather use "Patch provided" status Edit I meaned "Patch available"
        Hide
        Jacques Le Roux added a comment -

        Christian,

        Please don't use tabs in your patches.

        I think we should test if there is a value in contentWrapper.get("PRODUCT_NAME") before, like in

        if (contentWrapper.get("PRODUCT_NAME")) keywords.add(contentWrapper.get("PRODUCT_NAME"));
        else keywords.add(product.productName);

        Show
        Jacques Le Roux added a comment - Christian, Please don't use tabs in your patches. I think we should test if there is a value in contentWrapper.get("PRODUCT_NAME") before, like in if (contentWrapper.get("PRODUCT_NAME")) keywords.add(contentWrapper.get("PRODUCT_NAME")); else keywords.add(product.productName);
        Hide
        Jacques Le Roux added a comment -

        This has been already implemented with OFBIZ-4208

        Show
        Jacques Le Roux added a comment - This has been already implemented with OFBIZ-4208

          People

          • Assignee:
            Jacques Le Roux
            Reporter:
            Christian Fischer
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development