PUBLIC - Liferay Portal Community Edition
  1. PUBLIC - Liferay Portal Community Edition
  2. LPS-42944

Chat portlet - opened chat window disappears after refreshing the page

    Details

    • Type: Bug Bug
    • Status: Reopened
    • Resolution: Unresolved
    • Affects Version/s: 6.1.20 EE GA2, 6.2.0 CE GA1
    • Fix Version/s: None
    • Component/s: Chat
    • Labels:
      None
    • 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

          People

          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Days since last comment:
              1 year, 9 weeks, 3 days ago

              Development

                Structure Helper Panel