moved remote data url checks to fileUtils.ts

Author: RandomFractals

src/data.preview.ts

@@ -144,7 +144,7 @@ export class DataPreview {
     this._extensionPath = extensionPath;
     this._uri = uri;
     this._dataUrl = uri.toString(true); // skip uri encoding
-    this._isRemoteData = (this._dataUrl.startsWith('http://') || this._dataUrl.startsWith('https://'));
+    this._isRemoteData = fileUtils.isRemoteDataUrl(this._dataUrl);
     this._dataTable = (table !== undefined) ? table: '';
     this._dataViews = (views !== undefined) ? views: {};
     this._viewConfig = viewConfig;

src/utils/file.utils.ts

@@ -23,7 +23,7 @@ export async function readDataFile(dataFilePath: string, encoding:string = null)
   if (!config.supportedDataFiles.test(fileName)) {
     window.showErrorMessage(`${dataFilePath} is not a supported data file for Data Preview!`);
   }
-  else if (dataFilePath.startsWith('http://') || dataFilePath.startsWith('https://')) {
+  else if (isRemoteDataUrl(dataFilePath)) {
     data = await readRemoteData(dataFilePath, encoding);
   } 
   else if (fs.existsSync(fileUri.fsPath)) {
@@ -44,6 +44,15 @@ export async function readDataFile(dataFilePath: string, encoding:string = null)
   return data;
 }
 
+/**
+ * Checks if the requested data url is a remote data source; http(s) only for now.
+ * @param dataUrl The data url to check.
+ * @returns True if the specified data url is a remote data source. false otherwise.
+ */
+export function isRemoteDataUrl(dataUrl: string): boolean {
+  return dataUrl.startsWith('http://') || dataUrl.startsWith('https://');
+}
+
 /**
  * Gets local data file size for status display.
  * @param dataFilePath Data file path to get size stats for.