After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 730793 - "Saving..." appears multiple times when clicking on "Pause" often
"Saving..." appears multiple times when clicking on "Pause" often
Status: RESOLVED FIXED
Product: gnome-boxes
Classification: Applications
Component: general
unspecified
Other Linux
: Normal minor
: --
Assigned To: GNOME Boxes maintainer(s)
GNOME Boxes maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2014-05-27 07:44 UTC by Lasse Schuirmann
Modified: 2016-09-20 08:15 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
The bug (136.59 KB, image/png)
2014-05-27 07:44 UTC, Lasse Schuirmann
Details

Description Lasse Schuirmann 2014-05-27 07:44:53 UTC
Created attachment 277261 [details]
The bug

When pausing a VM the Pause button should in every case instantly be disabled while the VM is paused/saved.

Steps to reproduce:
* Start a VM that takes a few seconds to pause
* Click on pause several times
* It should look similar to the appended screenshot
* A bit of candy on top: an error that it is not possible to pause the VM (but its paused)
Comment 1 Zeeshan Ali 2014-10-24 12:47:34 UTC
Commits that fixed this in chronological order:

commit: ebb7ebe ebb7ebe68d46360428ac7df509985814ff1fd41e

    machine: Make 'can-save' a virtual property
    
    Instead of subclasses overriding the value of 'can-save' property,
    let them override the property getter itself so they can implement
    smarter (more dynamic) getters for it.
    
commit: 5b3b6d5 5b3b6d588a647a331bf0dc4b7ffed59987259f3b

    libvirt-machine: Smarter 'can-save' implementation
    
    This property shoudl check the state of the machine rather than its user
    having to do that in addition to checking the value of this property.

commit: 31e2883 31e28837b53ee7664b87fe10fef9a8168bc226b2

    libvirt-machine: Add private 'saving' boolean field
    
    This field is set to true if machine is being saved currently.

commit: 4a49633 4a496333edd48ef60d0c3c113ba4655cd7918f45

    libvirt-machine: Not savable when being saved already
    
    Don't claim the machine to be savable if its being saved.
    
    This solves the issue of pause button still being available to user and
    Boxes leading into interesting scenerios if user clicks on the button.