Skip to content

Commit d201206

Browse files
author
Juha Alhojoki
committed
people picker imageUrl was resolved by user email, when it should be resolved by user's accountName
1 parent 1a6ff3c commit d201206

File tree

1 file changed

+2
-17
lines changed

1 file changed

+2
-17
lines changed

src/services/PeopleSearchService.ts

Lines changed: 2 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -228,9 +228,8 @@ export default class SPPeopleSearchService {
228228
const userResults = values.map(element => {
229229
switch (element.EntityType) {
230230
case 'User':
231-
let email: string = element.EntityData.Email !== null ? element.EntityData.Email : element.Description;
232-
// Description contains the account name for a user
233-
const accountName = element.Description || this.getAccountName(element);
231+
const accountName: string = element.Description || "";
232+
const email: string = element.EntityData.Email || element.Description;
234233
return {
235234
id: element.Key,
236235
loginName: element.LoginName ? element.LoginName : element.Key,
@@ -311,20 +310,6 @@ export default class SPPeopleSearchService {
311310
return null;
312311
}
313312

314-
/**
315-
* Gets account name for user
316-
*/
317-
private getAccountName(element: any): string {
318-
if (!element) { return ""; }
319-
const loginName: string = element.LoginName || element.Key;
320-
if (loginName) {
321-
const loginParts: string[] = loginName.split("|");
322-
return loginParts[loginParts.length - 1];
323-
}
324-
325-
return "";
326-
}
327-
328313
/**
329314
* Generates Initials from a full name
330315
*/

0 commit comments

Comments
 (0)