scope of personTypeString conversion, fixing input vs no input for groups selection initial state

Author: vogtn

src/components/mgt-people-picker/mgt-people-picker.ts

@@ -661,7 +661,7 @@ export class MgtPeoplePicker extends MgtTemplatedComponent {
           } else if (this.type === PersonType.person || this.type === PersonType.any) {
             people = await getPeople(graph);
           } else if (this.type === PersonType.group) {
-            const groups = (await findGroups(graph, input, this.showMax, this.groupType)) || [];
+            const groups = (await findGroups(graph, '', this.showMax, this.groupType)) || [];
             people = groups;
           }
           this.defaultPeople = people;
@@ -705,7 +705,7 @@ export class MgtPeoplePicker extends MgtTemplatedComponent {
         if ((this.type === PersonType.group || this.type === PersonType.any) && people.length < this.showMax) {
           people = [];
           try {
-            const groups = (await findGroups(graph, '', this.showMax, this.groupType)) || [];
+            const groups = (await findGroups(graph, input, this.showMax, this.groupType)) || [];
             people = people.concat(groups);
           } catch (e) {
             // nop

src/graph/graph.people.ts

@@ -51,14 +51,13 @@ export async function findPeople(
 
   let filterQuery = '';
 
-  // converts personType to capitalized case
-  const personTypeString =
-    personType
-      .toString()
-      .charAt(0)
-      .toUpperCase() + personType.toString().slice(1);
-
   if (personType !== PersonType.any) {
+    // converts personType to capitalized case
+    const personTypeString =
+      personType
+        .toString()
+        .charAt(0)
+        .toUpperCase() + personType.toString().slice(1);
     filterQuery = `personType/class eq '${personTypeString}'`;
   }