Move duplicate fixes (#3947)

* Changed text of form labels;
* Corrected case of "Location" in Duplicate action;
* changed from objects.mutate to objects.save for duplicate action name change
* handling cancel of move

Co-authored-by: charlesh88 <charles.f.hacskaylo@nasa.gov>
This commit is contained in:
Jamie V 2021-06-17 14:04:47 -07:00 committed by GitHub
parent b484a4a959
commit 9d8a8b36d2
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 13 additions and 5 deletions

View File

@ -71,7 +71,8 @@ export default class DuplicateAction {
updateNameCheck(object, name) {
if (object.name !== name) {
this.openmct.objects.mutate(object, 'name', name);
object.name = name;
this.openmct.objects.save(object);
}
}
@ -95,7 +96,7 @@ export default class DuplicateAction {
cssClass: "l-input-lg"
},
{
name: "location",
name: "Location",
cssClass: "grows",
control: "locator",
validate: this.validate(object, parent),

View File

@ -37,7 +37,14 @@ export default class MoveAction {
let oldParent = objectPath[1];
let dialogService = this.openmct.$injector.get('dialogService');
let dialogForm = this.getDialogForm(object, oldParent);
let userInput = await dialogService.getUserInput(dialogForm, { name: object.name });
let userInput;
try {
userInput = await dialogService.getUserInput(dialogForm, { name: object.name });
} catch (err) {
// user canceled, most likely
return;
}
// if we need to update name
if (object.name !== userInput.name) {
@ -104,13 +111,13 @@ export default class MoveAction {
{
key: "name",
control: "textfield",
name: "Folder Name",
name: "Name",
pattern: "\\S+",
required: true,
cssClass: "l-input-lg"
},
{
name: "location",
name: "Location",
control: "locator",
validate: this.validate(object, parent),
key: 'location'