Merge pull request microsoft#183 from Microsoft/merge_master

Changes - 27.03

Author: ali-hamud

demo/code-demo/scripts/codesamples.js

@@ -503,6 +503,23 @@ function _Report_Reload() {
         });
 }
 
+function _Report_Refresh() {
+   // Get a reference to the embedded report HTML element
+   var reportContainer = $('#reportContainer')[0];
+
+   // Get a reference to the embedded report.
+    report = powerbi.get(reportContainer);
+
+    // Refresh the displayed report
+    report.refresh()
+        .then(function (result) {
+            Log.log(result);
+        })
+        .catch(function (errors) {
+            Log.log(errors);
+       });
+}
+
 function _Report_FullScreen() {
     // Get a reference to the embedded report HTML element
     var reportContainer = $('#reportContainer')[0];

demo/code-demo/scripts/session_utils.js

@@ -4,6 +4,7 @@ const SessionKeys = {
     AccessToken : "accessToken",
     EmbedUrl : "embedUrl",
     EmbedId : "embedId",
+    GroupId : "groupId",
     IsSampleReport: "isSampleReport"
 };
 
@@ -48,6 +49,17 @@ function SetTextBoxesFromSessionOrUrlParam(accessTokenSelector, embedUrlSelector
     if (!embedUrl)
     {
         embedUrl = GetSession(SessionKeys.EmbedUrl);
+    } else {
+        var groupId = GetParameterByName(SessionKeys.GroupId);
+        if(groupId)
+        {
+            if (embedUrl.indexOf("?") != -1)
+            {
+              embedUrl += "&groupId=" + groupId;
+            } else {
+              embedUrl += "?groupId=" + groupId;
+            }
+        }
     }
 
     var embedId = GetParameterByName(SessionKeys.EmbedId);

demo/code-demo/scripts/step_embed.js

@@ -35,6 +35,10 @@ function Report_Reload() {
     SetCode(_Report_Reload);
 }
 
+function Report_Refresh() {
+    SetCode(_Report_Refresh);
+}
+
 function Report_FullScreen() {
     SetCode(_Report_FullScreen);
 }

demo/code-demo/settings_interact.html

@@ -31,6 +31,7 @@
                         <li class="" onclick="Report_PrintCurrentReport()">Print</li>
                         <li class="" onclick="Report_UpdateSettings()">Update settings</li>
                         <li class="" onclick="Report_Reload()">Reload</li>
+                        <li class="" onclick="Report_Refresh()">Refresh</li>
                         <li class="" onclick="Report_FullScreen()">Full screen</li>
                         <li class="" onclick="Report_ExitFullScreen()">Exit full screen</li>
                     </ul>

dist/powerbi.js

@@ -170,7 +170,6 @@ export class Service implements IService {
     let powerBiElement = <IPowerBiElement>element;
     const component = new Create(this, powerBiElement, config);
     powerBiElement.powerBiEmbed = component;
-
     this.addOrOverwriteEmbed(component, element);
 
     return component;
@@ -266,7 +265,7 @@ export class Service implements IService {
       /**
        * When loading report after create we want to use existing Iframe to optimize load period
        */
-      if (config.type === "report" && component.config.type === "create") {
+      if(config.type === "report" && component.config.type === "create") {
         const report = new Report(this, element, config, element.powerBiEmbed.iframe);
         report.load(<embed.IInternalEmbedConfiguration>config);
         element.powerBiEmbed = report;
@@ -323,7 +322,7 @@ export class Service implements IService {
 
   addOrOverwriteEmbed(component: embed.Embed, element: HTMLElement): void {
     // remove embeds over the same div element.
-    this.embeds = this.embeds.filter(function (embed) {
+    this.embeds = this.embeds.filter(function(embed) {
       return embed.element.id !== element.id;
     });
 
@@ -350,11 +349,12 @@ export class Service implements IService {
     /** Removes the iframe from the element. */
     const iframe = element.querySelector('iframe');
     if (iframe) {
-      if (iframe.remove !== undefined) {
+      if(iframe.remove !== undefined) {
         iframe.remove();
-      } else {
-        // workaround for IE: Unhandled rejection TypeError: object doesn't support property or method 'remove'
-        iframe.parentElement.removeChild(iframe);
+      }
+      else {
+          /** Workaround for IE: unhandled rejection TypeError: object doesn't support propert or method 'remove' */
+          iframe.parentElement.removeChild(iframe);
       }
     }
   }