Details

    • Type: Bug
    • Status: Verified
    • Resolution: Unresolved
    • Affects Version/s: 6.1.20 EE GA2, 6.2.0 CE GA1, 7.0.0 M6
    • Fix Version/s: 7.0.0 M6
    • Component/s: Chat
    • Labels:
    • Environment:
    • Fix Priority:
      3

      Description

      Description :
      open chat window disappears after refreshing page

      Steps to reproduce:
      (1) Deploy chat portlet to portal
      (2) Log in to server with 2 users on 2 different browsers
      (3) Start chat between them
      (4) Open chat window
      (5) Refresh the page

      Expected Behavior: Opened chat window still appears

      Actual Behavior: Opened chat windows disappears

      Testing Results:
      The issue can be reproduced on :
      liferay-plugins Git ID : 57bfebed660c141ebdd6c771c8be299a046ea00d
      liferay-portal(trunk) Git ID : 084c7c83c7793d180dde4d9db3bfdf6392da3fcb
      And Social Office 2.1.0

        Issue Links

          Activity

          Hide
          hugo.huijser Hugo Huijser added a comment -

          Regarding removing the line var openPanelId = parseInt(instance._openPanelId, 10);

          The purpose of it should be to convert the var "openPanelId" to a int and then use "!isNaN(openPanelId)" to judge if it's a number object. Actually this line does not have to be there.
          After modifying the code, if "!isNaN(instance.openPanelId)" is true, it already indicates that "_openPanelId" is a number object. So it's okay to remove it now.

          Show
          hugo.huijser Hugo Huijser added a comment - Regarding removing the line var openPanelId = parseInt(instance._openPanelId, 10); The purpose of it should be to convert the var "openPanelId" to a int and then use "!isNaN(openPanelId)" to judge if it's a number object. Actually this line does not have to be there. After modifying the code, if "!isNaN(instance.openPanelId)" is true, it already indicates that "_openPanelId" is a number object. So it's okay to remove it now.
          Hide
          jonathan.mak Jonathan Mak added a comment -

          The parseInt() line is required since the chat will try to create a panel from 'buddylist' and 'setting' which are invalid.

          Show
          jonathan.mak Jonathan Mak added a comment - The parseInt() line is required since the chat will try to create a panel from 'buddylist' and 'setting' which are invalid.
          Hide
          michael.chen Michael Chen (Inactive) added a comment -

          Hi Jonathan,

          I think it will not create a panel from 'buddylist' or 'setting', since if it passes "instance._openPanelId && !isNaN(instance._openPanelId", the _openPanelId has to be "not null" & "is a number". This code snippet should be intended to create a chat panel from user.
          For the 'buddylist' & 'settings' panels, I think they should be created by the following lines in "init: function()" :
          instance._createBuddyListPanel();
          instance._createSettingsPanel();

          Thanks

          Show
          michael.chen Michael Chen (Inactive) added a comment - Hi Jonathan, I think it will not create a panel from 'buddylist' or 'setting', since if it passes "instance._openPanelId && !isNaN(instance._openPanelId", the _openPanelId has to be "not null" & "is a number". This code snippet should be intended to create a chat panel from user. For the 'buddylist' & 'settings' panels, I think they should be created by the following lines in "init: function()" : instance._createBuddyListPanel(); instance._createSettingsPanel(); Thanks
          Hide
          kenji.heigel Kenji Heigel added a comment -

          Reproduced on:
          Tomcat 7.0.62 + MySQL 5.5. Portal master GIT ID: e3165ac0e2d4bc780a4e75fb54416e3443d3beb5
          Plugins master GIT ID: 052bf33339a72a9a6cf96320cc2e27737b5a0f3b

          Show
          kenji.heigel Kenji Heigel added a comment - Reproduced on: Tomcat 7.0.62 + MySQL 5.5. Portal master GIT ID: e3165ac0e2d4bc780a4e75fb54416e3443d3beb5 Plugins master GIT ID: 052bf33339a72a9a6cf96320cc2e27737b5a0f3b
          Hide
          esther.sanz Esther Sanz added a comment -

          Committed

          Show
          esther.sanz Esther Sanz added a comment - Committed
          Hide
          mark.jin Mark Jin (Inactive) added a comment -

          FAILED Manual Testing following the steps in the description.

          Failed on:
          Tomcat 7.0.62 + MySQL 5.5.21. Portal master GIT ID: 1dc5757b612c985268735dc6561efab4fe77004e.
          Plugins master GIT ID: c7819234b6843d1cdf7850602b9b6aa10632be00.

          When I refresh the page, the chat windows disappears.

          Show
          mark.jin Mark Jin (Inactive) added a comment - FAILED Manual Testing following the steps in the description. Failed on: Tomcat 7.0.62 + MySQL 5.5.21. Portal master GIT ID: 1dc5757b612c985268735dc6561efab4fe77004e. Plugins master GIT ID: c7819234b6843d1cdf7850602b9b6aa10632be00. When I refresh the page, the chat windows disappears.

            People

            • Votes:
              2 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

              • Created:
                Updated:
                Days since last comment:
                1 year, 11 weeks, 5 days ago

                Development

                  Subcomponents