From 4b8c133e7e896a5247fc4bb53ff340bdea3e24c4 Mon Sep 17 00:00:00 2001 From: Kathy Date: Wed, 26 Oct 2022 10:46:40 +0000 Subject: [PATCH] =?UTF-8?q?B2021-072:=20Update=20RO=20Field=20Name=20defin?= =?UTF-8?q?ition=20=E2=80=93=20confusing=20dialog?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .../LibSource/RODBInterface/RODBInterface.cs | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/PROMS/ReferencedObjects/LibSource/RODBInterface/RODBInterface.cs b/PROMS/ReferencedObjects/LibSource/RODBInterface/RODBInterface.cs index 03ca6a3e..664993d8 100644 --- a/PROMS/ReferencedObjects/LibSource/RODBInterface/RODBInterface.cs +++ b/PROMS/ReferencedObjects/LibSource/RODBInterface/RODBInterface.cs @@ -2662,8 +2662,14 @@ namespace RODBInterface // or not. if (changegeneric != false) { - System.Windows.Forms.DialogResult result = MessageBox.Show("Update Generic definition?", "Referenced Object Definition", - MessageBoxButtons.YesNo, MessageBoxIcon.Question); + // B2021-072: don't update name if user selects No on dialog. This is actually misleading, because Yes updates + // Generic definition, i.e. all Groups, and No updates only Local definition, i.e. just this Group Added a + // Cancel button to Cancel from this dialog and return to previous dialog. Added more information to the + // dialog to inform user of this + string dmsg = "Update Generic definition?\r\n\r\n Yes - updates Generic definitions, \r\n No - updates Local definitions, \r\n Cancel - returns to previous dialog." + + "\r\n\r\n 'Generic' updates definition in ALL groups that use the name\r\n 'Local' updates definition only in the selected group."; + System.Windows.Forms.DialogResult result = MessageBox.Show(dmsg, "Referenced Object Definition", MessageBoxButtons.YesNoCancel, MessageBoxIcon.Question); + if (result == DialogResult.Cancel) return false; if (result == DialogResult.No) changegeneric = false; } if (changegeneric == true)