Better fix for editing "Workflow Path" causes NullPointerException

Describe the bug

Attempting to edit the path to the “Workflow Path” (and the “Test File Path”) for a workflow results in a 400 from the API with a NullPointerException.

It appears to “update” in the UI (no user facing error is displayed) but reverts back to the old value on refresh. The returned error is:

{"code":400,"details":"(was java.lang.NullPointerException)","message":"Unable to process JSON"}

To Reproduce Steps to reproduce the behavior:

1. Go to a workflow (the one in question is published if that makes a difference)
2. Click on “edit” next to Workflow Path
3. Change the path and click save
4. The UI updates, but will revert to the old value on refresh
5. The API returns a 400 Bad Request but no user facing error is presented.

GIF of Behavior:

Expected behavior Workflow file path is successfully updated

Screenshots

API Error:

Desktop (please complete the following information):

• OS: MacOS 10.15.5
• Browser: Firefox
• Version 79

Additional context

• UI version: 2.6.0
• Webservice version: 1.9.0
• host location: dockstore.org

┆Issue is synchronized with this Jira Bug ┆fixVersions: Dockstore 2.X ┆friendlyId: DOCK-1490 ┆sprint: Sprint 53- Bull shark ┆taskType: Bug

[garyluu]

I like 3 the most

[denis-yuen]

I kinda like the pages for debugging if elastic search is down. I could probably live without the starred column with Gary’s caveat