[Bug] promise.start() is not a function when using merged dataSources

Describe Your Question

Using two merged dataSources, if you try to enter a study, it returns the error “promise.start() is not a function”.

What steps can we follow to reproduce the bug?

  1. Using OHIF 3.8.2, merge two dicomweb dataSources (in my case, 2 orthanc instances) (check OHIF’s doc). You can check the code below
  2. Upload some modalities to the first server and the rest modalities from the same study to the second server, so you will have the same study in both servers (in my case, CT+RT STRUCT+RT DOSE to the first server and RT PLAN to the second server)
  3. Open OHIF and try to enter to that study
{
  // some config...
  dataSources: [
      {
        sourceName: 'merge',
        namespace: '@ohif/extension-default.dataSourcesModule.merge',
        configuration: {
          name: 'merge',
          friendlyName: 'Merge dicomweb-1 and dicomweb-2 data at the series level',
          seriesMerge: {
            dataSourceNames: ['dicomweb-1', 'dicomweb-2'],
            defaultDataSourceName: 'dicomweb-1'
          },
        },
      },
      {
        friendlyName: "Orthanc Server",
        namespace: "@ohif/extension-default.dataSourcesModule.dicomweb",
        sourceName: "dicomweb-1",
        configuration: {
          // some config...
        },
      },
      {
        friendlyName: "Orthanc External Server",
        namespace: "@ohif/extension-default.dataSourcesModule.dicomweb",
        sourceName: "dicomweb-2",
        configuration: {
          // some config...
        },
      },
    ],
    defaultDataSourceName: "merge",
    // some config...
}

The current behavior

OHIF gets stuck trying to load the study, and the following error is displayed in the console:

Uncaught (in promise) TypeError: promise.start is not a function
    at app.js:19054:78
    at Array.map (<anonymous>)
    at app.js:19054:55
    at Array.forEach (<anonymous>)
    at app.js:19039:14
    at async defaultRouteInit (app.js:19029:3)

The expected behavior

To load the selected study

I saw you have created a PR, thanks

1 Like