Fix navigateToSearch call in ExtendedVariantWidget

Sebastian Benjamin · Sebastian Benjamin · commit bec434fed8da · 2023-08-17T10:03:47.000-07:00
diff --git a/jbrowse/src/client/JBrowse/Browser/plugins/ExtendedVariantPlugin/ExtendedVariantDisplay/model.tsx b/jbrowse/src/client/JBrowse/Browser/plugins/ExtendedVariantPlugin/ExtendedVariantDisplay/model.tsx
@@ -217,7 +217,7 @@ export default jbrowse => {
                         const region = view.getSelectedRegions(undefined, undefined)[0]
                         const location = region.refName + ':' + (1+region.start) + '..' + (1+region.end)
                         const sessionId = view.id;
-                        navigateToSearch(sessionId, location, track.configuration.trackId, track)
+                        navigateToSearch(sessionId, location, track.configuration.trackId, null, track)
                      }
                   })
                }
diff --git a/jbrowse/src/client/JBrowse/utils.ts b/jbrowse/src/client/JBrowse/utils.ts
@@ -228,7 +228,7 @@ export function navigateToSearch(sessionId, locString, trackId, isValidRefNameFo
     const infoFilterURL = serializeInfoFilters(track)
 
     let searchString = null
-    if (locString) {
+    if (locString && isValidRefNameForAssembly) {
         const parsedLocString = parseLocString(locString, isValidRefNameForAssembly)
         const contig = parsedLocString.refName;
         const start = parsedLocString.start;

Original file line number	Diff line number	Diff line change
`@@ -217,7 +217,7 @@ export default jbrowse => {`
`217`	`217`	`const region = view.getSelectedRegions(undefined, undefined)[0]`
`218`	`218`	`const location = region.refName + ':' + (1+region.start) + '..' + (1+region.end)`
`219`	`219`	`const sessionId = view.id;`
`220`		`- navigateToSearch(sessionId, location, track.configuration.trackId, track)`
	`220`	`+ navigateToSearch(sessionId, location, track.configuration.trackId, null, track)`
`221`	`221`	`}`
`222`	`222`	`})`
`223`	`223`	`}`