Bug 1455832 – No overwrite confirmation pops if keep name as default 'Screenshot'

Bugzilla will be upgraded to version 5.0. The upgrade date is tentatively scheduled for 2 December 2018, pending final testing and feedback.

Bug 1455832 - No overwrite confirmation pops if keep name as default 'Screenshot'

Summary:	No overwrite confirmation pops if keep name as default 'Screenshot'

Status:	CLOSED ERRATA

Aliases:	None

Product:	Red Hat Enterprise Linux 7
Classification:	Red Hat
Component:	virt-viewer (Show other bugs)
Sub Component:
Version:	7.4
Hardware:	x86_64 Unspecified

Priority	low Severity low
Target Milestone:	rc
Target Release:	---
Assigned To:	Virt Viewer Maint
QA Contact:	Virtualization Bugs
Docs Contact:

URL:
Whiteboard:
Keywords:

Depends On:
Blocks:
	Show dependency tree / graph

Reported:	2017-05-26 04:59 EDT by xiaodwan
Modified:	2018-04-10 04:51 EDT (History)
CC List:	9 users (show)

See Also:
Fixed In Version:	virt-viewer-5.0-8.el7
Doc Type:	If docs needed, set a value
Doc Text:
Story Points:	---
Clone Of:
Environment:
Last Closed:	2018-04-10 04:50:15 EDT
Type:	Bug
Regression:	---
Mount Type:	---
Documentation:	---
CRM:
Verified Versions:
Category:	---
oVirt Team:	---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---

Attachments	(Terms of Use)
Add an attachment (proposed patch, testcase, etc.)

External Trackers
Tracker	ID	Priority	Status	Summary	Last Updated
Red Hat Product Errata	RHBA-2018:0672	None	None	None	2018-04-10 04:51 EDT

Groups: None (edit)

Description xiaodwan 2017-05-26 04:59:50 EDT

Description of problem:
No overwrite confirmation pops if keep name as default 'Screenshot'

Version-Release number of selected component (if applicable):
virt-viewer-5.0-4.el7.x86_64

How reproducible:
100%

Steps to Reproduce:
1. Open virt-viewer.
2. Click 'File->Screenshot', check the default name is "Screenshot".
3. Click 'Save' button, the picture is saved as 'Screenshot.png'.
4. Click 'File->Screenshot', then Click 'Save' button again.

Actual results:
The previous 'Screenshot.png' is overwritten silently. 

Expected results:
The previous 'Screenshot.png' should not be overwritten silently.

Additional info:
I suggest changing default name from 'Screenshot' to 'Screenshot.png' or appending date time after 'Screenshot' as RHEL system does.

Comment 2 Pavel Grunt 2017-05-26 05:13:04 EDT

sounds like reasonable request, too late for 7.4 -> moving

Comment 3 Jonathon Jongsma 2017-07-20 10:08:43 EDT

Relevant patches already pushed upstream:
a51ae4507d223a655f5c0a9054128becc5c3b689
f1783c3a3f403dcb2599b846dbbef6ff8369ae9f
761149ea71db5f51edce1d16860165bae2615917

Comment 5 xiaodwan 2017-10-19 23:27:46 EDT

I verified the bug with virt-viewer-5.0-8.el7.x86_64.

1) the default filename is "Screenshot.png"
2) An error dialog pops up if no file extension was specified or wrong extension was specified.
3) An overwrite confirmation dialog pops up if same file has already existed.


so move the bug from ON_QA to VERIFIED.

Comment 8 errata-xmlrpc 2018-04-10 04:50:15 EDT

Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHBA-2018:0672

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