Bug 749586 - [webadmin] All dialogs do not open after a log-off and re-login
Summary: [webadmin] All dialogs do not open after a log-off and re-login
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: oVirt
Classification: Retired
Component: ovirt-engine-webadmin
Version: unspecified
Hardware: All
OS: All
unspecified
medium
Target Milestone: ---
: 3.1
Assignee: Einav Cohen
QA Contact:
URL:
Whiteboard:
: 744714 748794 (view as bug list)
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2011-10-27 15:39 UTC by Danny Rankevich
Modified: 2016-01-28 21:53 UTC (History)
6 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-08-09 08:00:11 UTC
oVirt Team: ---
Embargoed:


Attachments (Terms of Use)

Description Danny Rankevich 2011-10-27 15:39:27 UTC
Description of problem:
When a log-off event occurs, either through a session expiration event or a manual log-off, the table actions stop working.

Version-Release number of selected component (if applicable):
3.0

How reproducible:
Always.

Steps to Reproduce:
1. Login to the webadmin.
2. Logoff from the webadmin (by clicking sign-out)
3. Login to the webadmin again.
4. None of the action buttons open dialogs.
  
Actual results:
When you click on an action, nothing happens.

Expected results:
A dialog should be opened.

Additional info:
I think this is cause by the fact that the log-off does not delete the CommonModel instance, and we still use the same instance.
I am sure there is a specific fix for this issues, however, INHO, the solution should be to refresh the entire webadmin after a logoff.
This will fix any additional problems that might be discovered over time.

Comment 1 Danny Rankevich 2011-10-27 16:21:35 UTC
*** Bug 744714 has been marked as a duplicate of this bug. ***

Comment 2 Danny Rankevich 2011-10-27 16:26:10 UTC
*** Bug 748794 has been marked as a duplicate of this bug. ***

Comment 4 Vojtech Szocs 2011-11-04 09:51:49 UTC
CommonModel instance is created upon each user login via CommonModelManager, and cleared (nullified) upon each user logout. This is because CommonModel initializes its child list models right within its constructor, and at that time CommonModel expects the user to be logged-in already. Upon each re-login, model providers are notified that CommonModel instance has changed via CommonModelChangeEvent.

The problem with action buttons is following: UiCommandButtonDefinition references UICommand's directly.

After re-login, this is what happens: CommonModel reference changes -> related main list model reference changes -> UICommand reference changes.

Action button still references original UICommand, its Execute() is called properly, but model provider for related main list model is listening/attached to new (changed) main list model reference.

Comment 8 Itamar Heim 2012-08-09 08:00:11 UTC
closing ON_QA bugs as oVirt 3.1 was released:
http://www.ovirt.org/get-ovirt/


Note You need to log in before you can comment on or make changes to this bug.