IDLE: Warn user about overwriting a file that has a newer version on filesystem

[gpolo]

BPO	6699
Nosy	@terryjreedy, @serwy, @rovitotv, @ZackerySpytz
PRs	gh-50948: IDLE: Warn if saving a file will overwrite a newer version #17578
Files	check_stmtime.diff idle.patch

^{Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.}

Show more details

GitHub fields:

assignee = 'https://github.com/terryjreedy'
closed_at = None
created_at = <Date 2009-08-13.23:58:48.490>
labels = ['expert-IDLE', 'type-feature', '3.7']
title = 'IDLE: Warn user about overwriting a file that has a newer version on filesystem'
updated_at = <Date 2019-12-12.10:21:06.634>
user = 'https://bugs.python.org/gpolo'

bugs.python.org fields:

activity = <Date 2019-12-12.10:21:06.634>
actor = 'ZackerySpytz'
assignee = 'terry.reedy'
closed = False
closed_date = None
closer = None
components = ['IDLE']
creation = <Date 2009-08-13.23:58:48.490>
creator = 'gpolo'
dependencies = []
files = ['14714', '34612']
hgrepos = []
issue_num = 6699
keywords = ['patch']
message_count = 5.0
messages = ['91537', '112886', '149117', '223598', '358298']
nosy_count = 6.0
nosy_names = ['terry.reedy', 'bsherwood', 'gpolo', 'roger.serwy', 'Todd.Rovito', 'ZackerySpytz']
pr_nums = ['17578']
priority = 'normal'
resolution = None
stage = 'patch review'
status = 'open'
superseder = None
type = 'enhancement'
url = 'https://bugs.python.org/issue6699'
versions = ['Python 3.6', 'Python 3.7']

[gpolo]

Creating this issue to address a suggestion of a new IDLE feature
pointed out on bpo-1721083.

The feature in question is about warning the user about a newer version
of the file before overwriting it.

[BreamoreBoy]

Patch seems clean, can it be commited?

[serwy]

The patch won't apply against 3.3a0 because "self.set_saved(1)" became "self.set_saved(True)" in r70054 (da7a120c0478)

After correcting this minor point, the patch works as expected.

[BreamoreBoy]

The latest patch reflects msg112886 and msg149117 so I believe it should be committed.

[ZackerySpytz]

I have created a pull request for this issue.