Fixing issue #1072.

[ntran18]

Due to the change in #1065, it breaks the import function of SIF files and GraphML files.

I added networkMode for the .sif file to determine "grn" or "protein-protein-physical-interaction." How to determine the network mode is mentioned in #1066. For the graphml files, the grnState.mode will always be "grn".

Pull Request Checklist

• Travis C.I. build passes
• All five Demos look as expected when loaded
• Reload works as expected
• Loading file with Error brings up error modal, and it looks as expected
• Loading file with Warning brings up warnings modal
• Loading a network from the database looks as expected when loaded
• Import works for both SIF and GraphML
• Graph can be exported to SIF, GraphML and Excel
• Graph can be exported to PNG, SVG and PDF
• Expression data and Additional Sheets can be exported to Excel
• Print works as expected
• Restrict graph to viewport works as expected (check/uncheck)
• Viewport Size Changing works as expected (small/medium/large/fit)
• Toggle between Grid Layout and Force Graph Layout works as expected
• Force Graph Parameter Sliders change the number above them and have an effect on the graph
• Locking/Unlocking/Resetting/Undo Resetting the force graph parameters works as expected
• Enabling and disabling node coloring works as expected
• Node coloring options work as expected (selection top/bottom dataset, averaging values, changing max value)
• Weighted graph loaded with the "Enable Edge Coloring" option checked appears as expected
• Hide/Show Weights works as expected (always/never/upon mouseover)
• Edge Weight Normalization Factor can be changed (set/reset)
• Gray Edge Threshold can be changed, slider changes the number and has an effect on the graph
• Checking Show Gray Edges as Dashed works as expected (check/uncheck)
• D-pad left/right/top/bottom/center works as expected
• Zoom slider changes number and has effect on graph (viewport and menu)
• Right click on a node opens gene page, page is populated with correct data

[ceciliazaragoza]

• Loading file with Error brings up error modal, and it looks as expected
  There is no error generated and graph is not generated for file: test-files/semantic-error-test-files/empty-network-xlsx-with-network-tab.xlsx. Furthermore, when other demos or files are selected, nothing displays. When reload page, display area disappears including its black border, and have to terminate nom run start-dev and run again.
• This is the error I get:

[ntran18]

For the comment that @ceciliazaragoza mentioned, it wasn't my change. The code existed before, and we didn't catch that. I would suggest to create a new issue for this. The problem was because she tested empty file. And in network-sheet-parser.js, it accessed cell A1 to detect if it had the wrong name. Since the data is empty, there is no cell A1 to test, thus the code crashed.

[ceciliazaragoza]

Since we detected an issue here with the files, @akaiap and I can work on testing all the test-files on the current deployed beta to ensure that there are no other pre-existing bugs that need to be addressed.

[dondi]

LGTM

[dondi]

Doesn’t have to change here to retain consistency, but for building new strings I have found that using the join array method feels structurally cleaner