Skip to content

Commit 05b88a3

Browse files
committed
Add ext .ows to filename automatically during saving new file.
- reverts commit 4f02bf8. - Add question MessageBox when overriding file - changed place when `filename` is changed to `filename`.ows in compare to reverted commit.
1 parent bd9b173 commit 05b88a3

File tree

1 file changed

+8
-4
lines changed

1 file changed

+8
-4
lines changed

orangecanvas/application/canvasmain.py

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1557,8 +1557,15 @@ def save_scheme_as(self):
15571557
# dialog.exec waits for user action
15581558
if dialog.exec():
15591559
filename = dialog.selectedFiles()[0]
1560+
# Enforcing ".ows" extension during saving file.
1561+
filename = filename if str(filename).endswith('.ows') else f"{filename}.ows"
1562+
do_override = QMessageBox.question(
1563+
self, "Overwrite file?",
1564+
f"File {os.path.split(filename)[1]} already exists."
1565+
"\nOverwrite?"
1566+
) == QMessageBox.Yes if os.path.exists(filename) else True
15601567
settings.setValue("last-scheme-dir", os.path.dirname(filename))
1561-
if self.save_scheme_to(curr_scheme, filename):
1568+
if do_override and self.save_scheme_to(curr_scheme, filename):
15621569
document.setPath(filename)
15631570
document.setModified(False)
15641571
self.add_recent_scheme(curr_scheme.title, document.path())
@@ -1574,9 +1581,6 @@ def save_scheme_to(self, scheme, filename):
15741581
`True`, else show a message to the user explaining the error and
15751582
return `False`.
15761583
"""
1577-
# Enforcing ".ows" extension during saving file.
1578-
# see .../biolab/orange-canvas-core/pull/330
1579-
filename = filename if str(filename).endswith('.ows') else f"{filename}.ows"
15801584
dirname, basename = os.path.split(filename)
15811585
title = scheme.title or "untitled"
15821586

0 commit comments

Comments
 (0)