Merge pull request #753 from microsoftgraph/nmetulev/useContactApis

Added useContactsApis global config to person and person card

Author: vogtn

packages/mgt/src/components/mgt-person-card/mgt-person-card.ts

@@ -26,6 +26,22 @@ import { MgtPersonCardMessages } from './sections/mgt-person-card-messages/mgt-p
 import { MgtPersonCardOrganization } from './sections/mgt-person-card-organization/mgt-person-card-organization';
 import { MgtPersonCardProfile } from './sections/mgt-person-card-profile/mgt-person-card-profile';
 
+/**
+ * Configuration object for the Person Card component
+ *
+ * @export
+ * @interface MgtPersonConfig
+ */
+export interface MgtPersonCardConfig {
+  /**
+   * Sets or gets whether the person card component can use Contacts APIs to
+   * find contacts and their images
+   *
+   * @type {boolean}
+   */
+  useContactApis: boolean;
+}
+
 /**
  * Web Component used to show detailed data for a person in the Microsoft Graph
  *
@@ -55,6 +71,18 @@ export class MgtPersonCard extends MgtTemplatedComponent {
     return styles;
   }
 
+  /**
+   * Global configuration object for
+   * all person card components
+   *
+   * @static
+   * @type {MgtPersonCardConfig}
+   * @memberof MgtPersonCard
+   */
+  public static config: MgtPersonCardConfig = {
+    useContactApis: true
+  };
+
   /**
    * Set the person details to render
    *
@@ -587,7 +615,7 @@ export class MgtPersonCard extends MgtTemplatedComponent {
         ((this.fetchImage && !this.personImage) || this.personImage === '@')
       ) {
         // in some cases we might only have name or email, but need to find the image
-        const image = await getPersonImage(graph, this.personDetails);
+        const image = await getPersonImage(graph, this.personDetails, MgtPersonCard.config.useContactApis);
         if (image) {
           this.personDetails.personImage = image;
           this.personImage = image;
@@ -605,7 +633,7 @@ export class MgtPersonCard extends MgtTemplatedComponent {
 
       if (people && people.length) {
         this.personDetails = people[0];
-        const image = await getPersonImage(graph, this.personDetails);
+        const image = await getPersonImage(graph, this.personDetails, MgtPersonCard.config.useContactApis);
         if (image) {
           this.personDetails.personImage = image;
           this.personImage = image;

packages/mgt/src/components/mgt-person/mgt-person.ts

@@ -53,6 +53,22 @@ export enum PersonViewType {
   threelines = 5
 }
 
+/**
+ * Configuration object for the Person component
+ *
+ * @export
+ * @interface MgtPersonConfig
+ */
+export interface MgtPersonConfig {
+  /**
+   * Sets or gets whether the person component can use Contacts APIs to
+   * find contacts and their images
+   *
+   * @type {boolean}
+   */
+  useContactApis: boolean;
+}
+
 /**
  * The person component is used to display a person or contact by using their photo, name, and/or email address.
  *
@@ -92,6 +108,18 @@ export class MgtPerson extends MgtTemplatedComponent {
     return styles;
   }
 
+  /**
+   * Global Configuration object for all
+   * person components
+   *
+   * @static
+   * @type {MgtPersonConfig}
+   * @memberof MgtPerson
+   */
+  public static config: MgtPersonConfig = {
+    useContactApis: true
+  };
+
   /**
    * allows developer to define name of person for component
    * @type {string}
@@ -746,7 +774,7 @@ export class MgtPerson extends MgtTemplatedComponent {
         !this.personDetails.personImage &&
         ((this.fetchImage && !this.personImage && !this._fetchedImage) || this.personImage === '@')
       ) {
-        const image = await getPersonImage(graph, this.personDetails);
+        const image = await getPersonImage(graph, this.personDetails, MgtPerson.config.useContactApis);
         if (image) {
           this.personDetails.personImage = image;
           this._fetchedImage = image;
@@ -764,7 +792,7 @@ export class MgtPerson extends MgtTemplatedComponent {
 
       if (people && people.length) {
         this.personDetails = people[0];
-        const image = await getPersonImage(graph, people[0]);
+        const image = await getPersonImage(graph, people[0], MgtPerson.config.useContactApis);
 
         if (image) {
           this.personDetails.personImage = image;

packages/mgt/src/graph/graph.photos.ts

@@ -201,14 +201,16 @@ export async function myPhoto(graph: IGraph): Promise<string> {
  *
  * @export
  */
-export async function getPersonImage(graph: IGraph, person: IDynamicPerson) {
+export async function getPersonImage(graph: IGraph, person: IDynamicPerson, useContactsApis: boolean = true) {
   // handle if contact
   if ('personType' in person && (person as any).personType.subclass === 'PersonalContact') {
-    // if person is a contact, look for them and their photo in contact api
-    let email = getEmailFromGraphEntity(person);
-    const contact = await findContactsByEmail(graph, email);
-    if (contact && contact.length && contact[0].id) {
-      return await getContactPhoto(graph, contact[0].id);
+    if (useContactsApis) {
+      // if person is a contact, look for them and their photo in contact api
+      const email = getEmailFromGraphEntity(person);
+      const contact = await findContactsByEmail(graph, email);
+      if (contact && contact.length && contact[0].id) {
+        return await getContactPhoto(graph, contact[0].id);
+      }
     }
 
     return null;
@@ -223,14 +225,14 @@ export async function getPersonImage(graph: IGraph, person: IDynamicPerson) {
 
   // else assume id is for user and try to get photo
   if (person.id) {
-    let image = await getUserPhoto(graph, person.id);
+    const image = await getUserPhoto(graph, person.id);
     if (image) {
       return image;
     }
   }
 
   // let's try to find a person by the email
-  let email = getEmailFromGraphEntity(person);
+  const email = getEmailFromGraphEntity(person);
 
   if (email) {
     // try to find user
@@ -240,9 +242,11 @@ export async function getPersonImage(graph: IGraph, person: IDynamicPerson) {
     }
 
     // if no user, try to find a contact
-    const contacts = await findContactsByEmail(graph, email);
-    if (contacts && contacts.length) {
-      return await getContactPhoto(graph, contacts[0].id);
+    if (useContactsApis) {
+      const contacts = await findContactsByEmail(graph, email);
+      if (contacts && contacts.length) {
+        return await getContactPhoto(graph, contacts[0].id);
+      }
     }
   }