OFBiz
  1. OFBiz
  2. OFBIZ-4503

wrong jquery.custom.min address in product/commonScreens.xml

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Not a Problem
    • Affects Version/s: Trunk
    • Fix Version/s: Trunk
    • Component/s: product
    • Labels:
      None

      Description

      in product/commonScreens.xml

      — applications/product/widget/catalog/CommonScreens.xml (révision 1188040)
      +++ applications/product/widget/catalog/CommonScreens.xml (copie de travail)
      @@ -43,7 +43,7 @@
      <set field="applicationTitle" value="$

      {uiLabelMap.ProductCatalogManagerApplication}

      " global="true"/>
      <set field="layoutSettings.javaScripts[+0]" value="/images/jquery/ui/development-bundle/external /jquery.cookie.js" global="true"/>
      <set field="layoutSettings.javaScripts[+0]" value="/images/jquery/plugins/jsTree/jquery.jstree.js" global="true"/>

      • <set field="layoutSettings.javaScripts[+0]" value="/images/jquery/jquery-1.8.13.custom.min.js" global="true"/>
        + <set field="layoutSettings.javaScripts[+0]" value="/images/jquery/ui/js/jquery-ui-1.8.13.custom.min.js" global="true"/>
        </actions>
        <widgets>
        <include-screen name="ApplicationDecorator" location="component://commonext/widget/CommonScreens.xml"/>
      1. ProductCommonScreens.patch
        1 kB
        Olivier Heintz
      2. ProductCommonScreens2.patch
        1 kB
        Olivier Heintz

        Activity

        Hide
        Olivier Heintz added a comment -

        patch with correction

        Show
        Olivier Heintz added a comment - patch with correction
        Hide
        Erwan de FERRIERES added a comment -

        Done at r1188334
        Thanks Olivier

        Show
        Erwan de FERRIERES added a comment - Done at r1188334 Thanks Olivier
        Hide
        Erwan de FERRIERES added a comment -

        reverted at 1188862, please see commit log.

        Show
        Erwan de FERRIERES added a comment - reverted at 1188862, please see commit log.
        Hide
        Olivier Heintz added a comment -

        I don't know why this javascript library is added, so I'm not able to find the correct one.
        So, I have commented the add to avoid error message in browser console and add a comment for whose knows which is the correct library.

        Show
        Olivier Heintz added a comment - I don't know why this javascript library is added, so I'm not able to find the correct one. So, I have commented the add to avoid error message in browser console and add a comment for whose knows which is the correct library.
        Hide
        Jacques Le Roux added a comment -

        I reassigned to me, because I will completly handle the problem when I will get a chance (maybe before new year...)

        Show
        Jacques Le Roux added a comment - I reassigned to me, because I will completly handle the problem when I will get a chance (maybe before new year...)
        Hide
        Jacques Le Roux added a comment -

        This should no longer be a problem after the effort I made recently on updating all jQuery related external dependencies and removing unused ones. Please Olivier reopen if you still see a problem.

        Show
        Jacques Le Roux added a comment - This should no longer be a problem after the effort I made recently on updating all jQuery related external dependencies and removing unused ones. Please Olivier reopen if you still see a problem.

          People

          • Assignee:
            Jacques Le Roux
            Reporter:
            Olivier Heintz
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development