Merge pull request #94 from microsoft/chsuh/port

[Port] Add copy results return value

Author: cssuh

lib/main.js

@@ -385,7 +385,7 @@ class QueryFeature extends SqlOpsFeature {
             });
         };
         let copyResults = (params) => {
-            return client.sendRequest(protocol.CopyResultsRequest.type, params).then(r => undefined, e => {
+            return client.sendRequest(protocol.CopyResultsRequest.type, params).then(r => r, e => {
                 client.logFailedRequest(protocol.CopyResultsRequest.type, e);
                 return Promise.reject(e);
             });

lib/protocol.d.ts

@@ -308,7 +308,7 @@ export declare namespace SaveResultsAsXmlRequest {
     const type: RequestType<azdata.SaveResultsRequestParams, azdata.SaveResultRequestResult, void, void>;
 }
 export declare namespace CopyResultsRequest {
-    const type: RequestType<azdata.CopyResultsRequestParams, void, void, void>;
+    const type: RequestType<azdata.CopyResultsRequestParams, azdata.CopyResultsRequestResult, void, void>;
 }
 export declare namespace SyntaxParseRequest {
     const type: RequestType<azdata.SyntaxParseParams, azdata.SyntaxParseResult, void, void>;

src/azdata.proposed.d.ts

@@ -863,6 +863,13 @@ declare module 'azdata' {
 		selections: SelectionRange[];
 	}
 
+	export interface CopyResultsRequestResult {
+		/** 
+		 * Result string from copy operation
+		 */
+		results: string;
+	}
+
 	export interface QueryProvider {
 		/**
 		 * Notify clients that the URI for a connection has been changed.
@@ -874,7 +881,7 @@ declare module 'azdata' {
 		 * ADS will use this if 'supportCopyResultsToClipboard' property is set to true in the provider contribution point in extension's package.json.
 		 * Otherwise, The default handler will load all the selected data to ADS and perform the copy operation.
 		 */
-		copyResults?(requestParams: CopyResultsRequestParams): Thenable<void>;
+		copyResults?(requestParams: CopyResultsRequestParams): Thenable<CopyResultsRequestResult>;
 	}
 
 	export enum DataProviderType {

src/main.ts

@@ -564,9 +564,9 @@ export class QueryFeature extends SqlOpsFeature<undefined> {
 			);
 		};
 
-		let copyResults = (params: azdata.CopyResultsRequestParams): Thenable<void> => {
+		let copyResults = (params: azdata.CopyResultsRequestParams): Thenable<azdata.CopyResultsRequestResult> => {
 			return client.sendRequest(protocol.CopyResultsRequest.type, params).then(
-				r => undefined,
+				r => r,
 				e => {
 					client.logFailedRequest(protocol.CopyResultsRequest.type, e);
 					return Promise.reject(e);

src/protocol.ts

@@ -469,7 +469,7 @@ export namespace SaveResultsAsXmlRequest {
 // --------------------------------- < Copy Results Request > ------------------------------------------
 // copy results to clipboard
 export namespace CopyResultsRequest {
-	export const type = new RequestType<azdata.CopyResultsRequestParams, void, void, void>('query/copy');
+	export const type = new RequestType<azdata.CopyResultsRequestParams, azdata.CopyResultsRequestResult, void, void>('query/copy');
 }
 // --------------------------------- </ Copy Results Request > ------------------------------------------