Details

    • Type: Bug 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:
    • Similar Issues:
      Show 5 results 

      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

        Activity

        Hide
        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 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 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 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 (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 (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 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 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 added a comment -

        Committed

        Show
        Esther Sanz added a comment - Committed
        Hide
        Mark Jin 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 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:
              8 weeks, 1 day ago

              Development

                Subcomponents

                  Structure Helper Panel