Merge pull request microsoft#379 from microsoft/release-2.18.6

submarine-launched · web-flow · commit 9ae72e5a89e8 · 2021-10-13T16:07:46.000+03:00
Release version 2.18.6
diff --git a/dist/powerbi-client.d.ts b/dist/powerbi-client.d.ts
@@ -1,4 +1,4 @@
-// powerbi-client v2.18.4
+// powerbi-client v2.18.6
 // Copyright (c) Microsoft Corporation.
 // Licensed under the MIT License.
 declare module "config" {
@@ -1564,6 +1564,16 @@ declare module "report" {
          * @hidden
          */
         private isMobileSettings;
+        /**
+         * Return the current zoom level of the report.
+         * @returns {Promise<number>}
+         */
+        getZoom(): Promise<number>;
+        /**
+         * Sets the report's zoom level.
+         * @param zoomLevel zoom level to set
+         */
+        setZoom(zoomLevel: number): Promise<void>;
     }
 }
 declare module "create" {
diff --git a/dist/powerbi.js b/dist/powerbi.js
diff --git a/dist/powerbi.min.js b/dist/powerbi.min.js
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "powerbi-client",
-  "version": "2.18.4",
+  "version": "2.18.6",
   "description": "JavaScript library for embedding Power BI into your apps. Provides service which makes it easy to embed different types of components and an object model which allows easy interaction with these components such as changing pages, applying filters, and responding to data selection.",
   "main": "dist/powerbi.js",
   "types": "dist/powerbi-client.d.ts",
@@ -81,7 +81,7 @@
   },
   "dependencies": {
     "http-post-message": "^0.2",
-    "powerbi-models": "^1.9.3",
+    "powerbi-models": "^1.9.5",
     "powerbi-router": "^0.1",
     "window-post-message-proxy": "^0.2"
   },
diff --git a/src/config.ts b/src/config.ts
@@ -3,7 +3,7 @@
 
 /** @ignore *//** */
 const config = {
-  version: '2.18.4',
+  version: '2.18.6',
   type: 'js'
 };
 
diff --git a/src/embed.ts b/src/embed.ts
@@ -227,15 +227,12 @@ export abstract class Embed {
     this.commands = [];
     this.groups = [];
 
-    const registerQueryCallback = !!(<IEmbedConfiguration>config).eventHooks?.applicationContextProvider;
-    delete (<IEmbedConfiguration>config).eventHooks;
-
     this.populateConfig(config, isBootstrap);
 
     if (this.embedtype === 'create') {
-      this.setIframe(false /* set EventListener to call create() on 'load' event*/, phasedRender, isBootstrap, registerQueryCallback);
+      this.setIframe(false /* set EventListener to call create() on 'load' event*/, phasedRender, isBootstrap);
     } else {
-      this.setIframe(true /* set EventListener to call load() on 'load' event*/, phasedRender, isBootstrap, registerQueryCallback);
+      this.setIframe(true /* set EventListener to call load() on 'load' event*/, phasedRender, isBootstrap);
     }
   }
 
@@ -540,6 +537,11 @@ export abstract class Embed {
       this.config.accessToken = this.getAccessToken(this.service.accessToken);
     }
 
+    const registerQueryCallback = !!(<IEmbedConfiguration>this.config).eventHooks?.applicationContextProvider;
+    delete (<IEmbedConfiguration>this.config).eventHooks;
+    if (registerQueryCallback && this.embedtype === "report")
+      this.config.embedUrl = addParamToUrl(this.config.embedUrl, "registerQueryCallback", "true");
+
     this.configChanged(isBootstrap);
   }
 
@@ -710,9 +712,6 @@ export abstract class Embed {
       const iframeContent = document.createElement("iframe");
       let embedUrl = this.config.uniqueId ? addParamToUrl(this.config.embedUrl, 'uid', this.config.uniqueId) : this.config.embedUrl;
 
-      if (!isBootstrap && registerQueryCallback)
-        embedUrl = addParamToUrl(embedUrl, "registerQueryCallback", "true");
-
       iframeContent.style.width = '100%';
       iframeContent.style.height = '100%';
       iframeContent.setAttribute("src", embedUrl);
diff --git a/src/report.ts b/src/report.ts
@@ -1112,4 +1112,25 @@ export class Report extends Embed implements IReportNode, IFilterable {
   private isMobileSettings(settings: IEmbedSettings): boolean {
     return settings && (settings.layoutType === LayoutType.MobileLandscape || settings.layoutType === LayoutType.MobilePortrait);
   }
+
+  /**
+   * Return the current zoom level of the report.
+   * @returns {Promise<number>}
+   */
+  async getZoom(): Promise<number> {
+    try {
+      const response = await this.service.hpm.get<number>(`/report/zoom`, { uid: this.config.uniqueId }, this.iframe.contentWindow);
+      return response.body;
+    } catch (response) {
+      throw response.body;
+    }
+  }
+
+  /**
+   * Sets the report's zoom level.
+   * @param zoomLevel zoom level to set
+   */
+  async setZoom(zoomLevel: number): Promise<void> {
+    await this.updateSettings({ zoomLevel: zoomLevel });
+  }
 }
diff --git a/src/service.ts b/src/service.ts
@@ -326,6 +326,7 @@ export class Service implements IService {
    * @param {IBootstrapEmbedConfiguration} config: a bootstrap config which is an embed config without access token.
    */
   bootstrap(element: HTMLElement, config: IComponentEmbedConfiguration | IBootstrapEmbedConfiguration): Embed {
+    this.registerApplicationContextHook(config as IEmbedConfiguration);
     return this.embedInternal(element, config, /* phasedRender */ false, /* isBootstrap */ true);
   }
 
@@ -451,6 +452,10 @@ export class Service implements IService {
       return;
     }
 
+    if (config?.type.toLowerCase() !== "report") {
+      throw new Error("applicationContextProvider is only supported in report embed");
+    }
+
     if (typeof applicationContextProvider !== 'function') {
       throw new Error("applicationContextProvider must be a function");
     }

Original file line number	Diff line number	Diff line change
`@@ -326,6 +326,7 @@ export class Service implements IService {`
`326`	`326`	`* @param {IBootstrapEmbedConfiguration} config: a bootstrap config which is an embed config without access token.`
`327`	`327`	`*/`
`328`	`328`	`bootstrap(element: HTMLElement, config: IComponentEmbedConfiguration \| IBootstrapEmbedConfiguration): Embed {`
	`329`	`+ this.registerApplicationContextHook(config as IEmbedConfiguration);`
`329`	`330`	`return this.embedInternal(element, config, /* phasedRender / false, / isBootstrap */ true);`
`330`	`331`	`}`
`331`	`332`
`@@ -451,6 +452,10 @@ export class Service implements IService {`
`451`	`452`	`return;`
`452`	`453`	`}`
`453`	`454`
	`455`	`+ if (config?.type.toLowerCase() !== "report") {`
	`456`	`+ throw new Error("applicationContextProvider is only supported in report embed");`
	`457`	`+ }`
	`458`	`+`
`454`	`459`	`if (typeof applicationContextProvider !== 'function') {`
`455`	`460`	`throw new Error("applicationContextProvider must be a function");`
`456`	`461`	`}`