diff --git a/source/java/org/alfresco/repo/jscript/app/UsernamePropertyDecorator.java b/source/java/org/alfresco/repo/jscript/app/UsernamePropertyDecorator.java index bb8e3399c0..6cddb58b53 100644 --- a/source/java/org/alfresco/repo/jscript/app/UsernamePropertyDecorator.java +++ b/source/java/org/alfresco/repo/jscript/app/UsernamePropertyDecorator.java @@ -18,6 +18,10 @@ */ package org.alfresco.repo.jscript.app; +import java.io.Serializable; +import java.util.LinkedHashMap; +import java.util.Map; + import org.alfresco.model.ContentModel; import org.alfresco.service.ServiceRegistry; import org.alfresco.service.cmr.repository.NodeRef; @@ -25,10 +29,6 @@ import org.alfresco.service.cmr.repository.NodeService; import org.alfresco.service.cmr.security.PersonService; import org.alfresco.service.namespace.QName; -import java.io.Serializable; -import java.util.LinkedHashMap; -import java.util.Map; - /** * Username property decorator class. * @@ -52,15 +52,15 @@ public class UsernamePropertyDecorator implements PropertyDecorator String username = value.toString(); String firstName = null; String lastName = null; - Map map = new LinkedHashMap(1); + Map map = new LinkedHashMap(4); map.put("userName", username); if (this.personService.personExists(username)) { NodeRef personRef = this.personService.getPerson(username); Map properties = this.nodeService.getProperties(personRef); - firstName = properties.get(ContentModel.PROP_FIRSTNAME).toString(); - lastName = properties.get(ContentModel.PROP_LASTNAME).toString(); + firstName = (String)properties.get(ContentModel.PROP_FIRSTNAME); + lastName = (String)properties.get(ContentModel.PROP_LASTNAME); } else if (username.equals("System") || username.startsWith("System@")) { @@ -75,7 +75,7 @@ public class UsernamePropertyDecorator implements PropertyDecorator map.put("firstName", firstName); map.put("lastName", lastName); - map.put("displayName", (firstName + " " + lastName).replaceAll("^\\s+|\\s+$", "")); + map.put("displayName", (firstName != null ? firstName + " " : "" + lastName != null ? lastName : "").replaceAll("^\\s+|\\s+$", "")); return (Serializable)map; } }