Uploaded image for project: 'Beehive'
  1. Beehive
  2. BEEHIVE-542

Multiple popup scripts generated when multiple popup config exist in one page

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: v1m1
    • Fix Version/s: 1.0.1
    • Component/s: NetUI
    • Labels:
      None

      Description

      When a page has multple popup config like the following,

      <br/>
      <netui:anchor action="goNested" popup="true">
      <netui:configurePopup width="500" height="500" directories="true"/>
      </netui:anchor>

      <br/>
      <netui:anchor action="goNested" popup="true">
      <netui:configurePopup width="500" height="500"
      toolbar="false" location="false"
      directories="true" status="false"
      menubar="false" resizable="false"
      scrollbars="false" replace="false"/>
      </netui:anchor>

      <br/>

      multiple popup window open scripts generated multiple times in the page, see below.

      window_Netui_Popup = null;
      function Netui_Popup(url, name, features, replace)
      {
      if (window_Netui_Popup != null && ! window_Netui_Popup.closed)

      { window_Netui_Popup.focus(); return; }
      window_Netui_Popup=open(url, name, features, replace);
      if (window_Netui_Popup.opener == null) window_Netui_Popup.opener=self;
      }

      window_Netui_Popup = null;
      function Netui_Popup(url, name, features, replace)
      {
      if (window_Netui_Popup != null && ! window_Netui_Popup.closed)
      { window_Netui_Popup.focus(); return; }

      window_Netui_Popup=open(url, name, features, replace);
      if (window_Netui_Popup.opener == null) window_Netui_Popup.opener=self;
      }

      Expect to see only one block of this script generated.

      Repro: Unzip the repro pageflow and put the pageflow into the netui-blanc template wewbapp. Compile and deploy to tomcat. Browse to the pageflow by http://localhost:8080/webapp/directories/Controller.jpf. Once the page is loaded, do a view source and check the JS script that are generated for that page.

        Attachments

        1. j542-patch.txt
          29 kB
          Carlin Rogers
        2. jira542.ZIP
          2 kB
          Julie Zhuo

          Activity

            People

            • Assignee:
              jzhuo Julie Zhuo
              Reporter:
              jzhuo Julie Zhuo
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: