Bug 278267 - [WPE] Insert Converter context menu has blank lines displayed in list
diff --git a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/JSFAddActionGroup.java b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/JSFAddActionGroup.java
index b718752..98127d7 100644
--- a/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/JSFAddActionGroup.java
+++ b/jsf/plugins/org.eclipse.jst.pagedesigner.jsf.ui/src/org/eclipse/jst/pagedesigner/jsf/ui/actions/JSFAddActionGroup.java
@@ -11,6 +11,7 @@
*******************************************************************************/
package org.eclipse.jst.pagedesigner.jsf.ui.actions;
+import java.util.ArrayList;
import java.util.Iterator;
import java.util.List;
@@ -199,18 +200,20 @@
if (appConfigMgr != null)
{
final List list = appConfigMgr.getConverters();
- result = new String[list.size()];
- int i = 0;
+ //prune out converters for classes, they're not valid here
+ final List converterIdList = new ArrayList();
for (final Iterator it = list.iterator(); it.hasNext();)
{
ConverterType converter = (ConverterType) it.next();
ConverterIdType converterId = converter.getConverterId();
if (converterId != null)
{
- result[i++] = converterId.getTextContent() != null ?
- converterId.getTextContent().trim() : ""; //$NON-NLS-1$
+ converterIdList.add(converterId.getTextContent() != null ?
+ converterId.getTextContent().trim() : ""); //$NON-NLS-1$
}
}
+ result = new String[converterIdList.size()];
+ result = (String[]) converterIdList.toArray(result);
}
return result;
}