Merge pull request #1620 from NishkalankBezawada/Issue-1612

Fixing issue 1612 & 1603 - PeoplePicker won't accept Multiple Users with the same name
pnp · Sep 23, 2023 · fbe0e3f · fbe0e3f
2 parents f299cd0 + ed83c85
commit fbe0e3f
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/controls/peoplepicker/PeoplePickerComponent.tsx b/src/controls/peoplepicker/PeoplePickerComponent.tsx
@@ -280,7 +280,7 @@ export class PeoplePicker extends React.Component<IPeoplePickerProps, IPeoplePic
     if (!personas || !personas.length || personas.length === 0) {
       return false;
     }
-    return personas.filter(item => item.text === persona.text).length > 0;
+    return personas.some(item => item.text === persona.text && item.secondaryText === persona.secondaryText);
   }
 
 

diff --git a/src/webparts/controlsTest/components/ControlsTest.tsx b/src/webparts/controlsTest/components/ControlsTest.tsx
@@ -1502,7 +1502,7 @@ export default class ControlsTest extends React.Component<IControlsTestProps, IC
 
           <PeoplePicker context={this.props.context}
             titleText="People Picker (tenant scoped)"
-            personSelectionLimit={5}
+            personSelectionLimit={10}
             // groupName={"Team Site Owners"}
             showtooltip={true}
             required={true}
@@ -1511,7 +1511,7 @@ export default class ControlsTest extends React.Component<IControlsTestProps, IC
             onChange={this._getPeoplePickerItems}
             showHiddenInUI={false}
             principalTypes={[PrincipalType.User, PrincipalType.SharePointGroup, PrincipalType.SecurityGroup, PrincipalType.DistributionList]}
-            suggestionsLimit={2}
+            suggestionsLimit={5}
             resolveDelay={200}
             placeholder={'Select a SharePoint principal (User or Group)'}
             onGetErrorMessage={async (items: any[]) => {