primefaces#3189

Author: tandraschko

src/main/java/org/primefaces/behavior/base/AbstractBehaviorHandler.java

@@ -88,14 +88,16 @@ public void apply(FaceletContext faceletContext, UIComponent parent) throws IOEx
             }
 
             boolean supportedEvent = false;
-            for (int i = 0; i < targetList.size(); i++) {
-                AttachedObjectTarget target = targetList.get(i);
-                if (target instanceof BehaviorHolderAttachedObjectTarget) {
-                    BehaviorHolderAttachedObjectTarget behaviorTarget = (BehaviorHolderAttachedObjectTarget) target;
-                    if ((null != eventName && eventName.equals(behaviorTarget.getName()))
-                            || (null == eventName && behaviorTarget.isDefaultEvent())) {
-                        supportedEvent = true;
-                        break;
+            if (targetList != null) {
+                for (int i = 0; i < targetList.size(); i++) {
+                    AttachedObjectTarget target = targetList.get(i);
+                    if (target instanceof BehaviorHolderAttachedObjectTarget) {
+                        BehaviorHolderAttachedObjectTarget behaviorTarget = (BehaviorHolderAttachedObjectTarget) target;
+                        if ((null != eventName && eventName.equals(behaviorTarget.getName()))
+                                || (null == eventName && behaviorTarget.isDefaultEvent())) {
+                            supportedEvent = true;
+                            break;
+                        }
                     }
                 }
             }

src/main/java/org/primefaces/component/autocomplete/AutoCompleteRenderer.java

@@ -299,7 +299,8 @@ protected void encodeHiddenSelect(FacesContext context, AutoComplete ac, String
             writer.writeAttribute("disabled", "disabled", "disabled");
         }
 
-        for (String value : values) {
+        for (int i = 0; i < values.size(); i++) {
+            String value = values.get(i);
             writer.startElement("option", null);
             writer.writeAttribute("value", value, null);
             writer.writeAttribute("selected", "selected", null);

src/main/java/org/primefaces/util/CompositeUtils.java

@@ -43,34 +43,37 @@ public static void invokeOnDeepestEditableValueHolder(FacesContext context, UICo
         List<AttachedObjectTarget> targets = (List<AttachedObjectTarget>) info.getBeanDescriptor()
                 .getValue(AttachedObjectTarget.ATTACHED_OBJECT_TARGETS_KEY);
 
-        for (AttachedObjectTarget target : targets) {
-            if (target instanceof EditableValueHolderAttachedObjectTarget) {
+        if (targets != null) {
+            for (int i = 0; i < targets.size(); i++) {
+                AttachedObjectTarget target = targets.get(i);
+                if (target instanceof EditableValueHolderAttachedObjectTarget) {
 
-                List<UIComponent> childs = target.getTargets(composite);
-                if (childs == null || childs.isEmpty()) {
-                    throw new FacesException(
-                            "Cannot not resolve editableValueHolder target in composite component with id: \""
-                            + composite.getClientId() + "\"");
-                }
+                    List<UIComponent> childs = target.getTargets(composite);
+                    if (childs == null || childs.isEmpty()) {
+                        throw new FacesException(
+                                "Cannot not resolve editableValueHolder target in composite component with id: \""
+                                + composite.getClientId() + "\"");
+                    }
 
-                if (childs.size() > 1) {
-                    throw new FacesException(
-                            "Only a single editableValueHolder target is supported in composite component with id: \""
-                            + composite.getClientId() + "\"");
-                }
+                    if (childs.size() > 1) {
+                        throw new FacesException(
+                                "Only a single editableValueHolder target is supported in composite component with id: \""
+                                + composite.getClientId() + "\"");
+                    }
 
-                final UIComponent child = childs.get(0);
+                    final UIComponent child = childs.get(0);
 
-                composite.invokeOnComponent(context, composite.getClientId(context), new ContextCallback() {
-                    public void invokeContextCallback(FacesContext context, UIComponent target) {
-                        if (isComposite(child)) {
-                            invokeOnDeepestEditableValueHolder(context, child, callback);
+                    composite.invokeOnComponent(context, composite.getClientId(context), new ContextCallback() {
+                        public void invokeContextCallback(FacesContext context, UIComponent target) {
+                            if (isComposite(child)) {
+                                invokeOnDeepestEditableValueHolder(context, child, callback);
+                            }
+                            else {
+                                callback.invokeContextCallback(context, child);
+                            }
                         }
-                        else {
-                            callback.invokeContextCallback(context, child);
-                        }
-                    }
-                });
+                    });
+                }
             }
         }
     }