Title bar doesn't refresh after rename during Save Archive
If you File => Save Archive, rename your Untitled workflow, and then cancel on the save to kar dialog, the workflow's been renamed but the title bar doesn't reflect this.
#4 Updated by Derik Barseghian almost 10 years ago
bug in comment 1 is sort of fixed with r22611. Rename from canvas context menu will work (provenance will rename past workflows, and wrm will reflect that) when you rename a wf that's Untitled[x]. If you rename a workflow that doesn't match the unnamedIdPattern, it won't pass the check in SQLRecordingV8 renamedWorkflow.
Before I work on this further (inspecting to see if the pattern checks can be removed from provenance recording and querying -- I suspect they're there for a reason but will have to look into it or ask Dan if he remembers), I want to ask again do we need the new Rename from canvas feature?
If so, in this release?
I guess Oliver wants it?: http://bugzilla.ecoinformatics.org/show_bug.cgi?id=4631
#5 Updated by Derik Barseghian almost 10 years ago
discussed a bit w/ Aaron, Dan and Ben. It seems like a compromise is to assign a new lsid to a workflow when it's renamed if its prior name isn't unnamed (ala a typical save as). I've done this w/ r22714.
However there's now a new issue: if you rename a workflow that already has a real name, and it therefore gets a new lsid, you can close this workflow and not be prompted to save changes for the last rename.
NamedObjId sends out a requestChange event during updateRevision, but not during setExpression. setExpression calls super, so it seems like things should notice it ala changing any StringAttribute, but that's not happening. I need to look into it a bit more...
#7 Updated by Derik Barseghian almost 10 years ago
should be fixed in r22873, but needs testing (notably have not yet tested the various user cancel scenarios). Hopefully everything is consistent now re: any rename of an Untitled workflow (eg during Save As, from canvas) will just update the revision. If you rename a workflow that has a name, it gets a new LSID, the old put in the referral list. The RenameUtil should be used for worflow renames, so listeners can do their thing.