Hi David, can you please have a look at this commit? any options within myportal now only show the main option.
can you please check? Regards, Hans On Wed, 2010-04-28 at 08:33 +0000, jone...@apache.org wrote: > Author: jonesde > Date: Wed Apr 28 08:33:37 2010 > New Revision: 938816 > > URL: http://svn.apache.org/viewvc?rev=938816&view=rev > Log: > Refactored the various link getParameterList methods to return a Map and > expand with the context like most of the other get methods that are used > during rendering; that is the biggest change but was part of adding a > parameters-map attribute to the form widget hyperlink element to be able to > use a script to prepare part or all of the parameters map used in the hidden > form instead of having to manually declare all parameters > > Modified: > ofbiz/trunk/framework/widget/dtd/widget-form.xsd > ofbiz/trunk/framework/widget/src/org/ofbiz/widget/WidgetWorker.java > > ofbiz/trunk/framework/widget/src/org/ofbiz/widget/form/MacroFormRenderer.java > ofbiz/trunk/framework/widget/src/org/ofbiz/widget/form/ModelFormField.java > > ofbiz/trunk/framework/widget/src/org/ofbiz/widget/html/HtmlFormRenderer.java > > ofbiz/trunk/framework/widget/src/org/ofbiz/widget/html/HtmlMenuRenderer.java > > ofbiz/trunk/framework/widget/src/org/ofbiz/widget/html/HtmlScreenRenderer.java > > ofbiz/trunk/framework/widget/src/org/ofbiz/widget/html/HtmlTreeRenderer.java > ofbiz/trunk/framework/widget/src/org/ofbiz/widget/menu/ModelMenuItem.java > > ofbiz/trunk/framework/widget/src/org/ofbiz/widget/screen/MacroScreenRenderer.java > > ofbiz/trunk/framework/widget/src/org/ofbiz/widget/screen/ModelScreenWidget.java > > ofbiz/trunk/framework/widget/src/org/ofbiz/widget/tree/MacroTreeRenderer.java > ofbiz/trunk/framework/widget/src/org/ofbiz/widget/tree/ModelTree.java > > Modified: ofbiz/trunk/framework/widget/dtd/widget-form.xsd > URL: > http://svn.apache.org/viewvc/ofbiz/trunk/framework/widget/dtd/widget-form.xsd?rev=938816&r1=938815&r2=938816&view=diff > ============================================================================== > --- ofbiz/trunk/framework/widget/dtd/widget-form.xsd (original) > +++ ofbiz/trunk/framework/widget/dtd/widget-form.xsd Wed Apr 28 08:33:37 2010 > @@ -856,6 +856,9 @@ under the License. > </xs:simpleType> > </xs:attribute> > <xs:attribute name="confirmation-message" > type="xs:string"><xs:annotation><xs:documentation>The message displayed in > confirmation box</xs:documentation></xs:annotation></xs:attribute> > + <xs:attribute name="parameters-map" type="xs:string"> > + <xs:annotation><xs:documentation>A Map in the context that will > be used as additional name/value pairs.</xs:documentation></xs:annotation> > + </xs:attribute> > </xs:attributeGroup> > <xs:element name="parameter"> > <xs:complexType> > > Modified: ofbiz/trunk/framework/widget/src/org/ofbiz/widget/WidgetWorker.java > URL: > http://svn.apache.org/viewvc/ofbiz/trunk/framework/widget/src/org/ofbiz/widget/WidgetWorker.java?rev=938816&r1=938815&r2=938816&view=diff > ============================================================================== > --- ofbiz/trunk/framework/widget/src/org/ofbiz/widget/WidgetWorker.java > (original) > +++ ofbiz/trunk/framework/widget/src/org/ofbiz/widget/WidgetWorker.java Wed > Apr 28 08:33:37 2010 > @@ -50,7 +50,7 @@ public class WidgetWorker { > > public WidgetWorker () {} > > - public static void buildHyperlinkUrl(Appendable externalWriter, String > target, String targetType, List<WidgetWorker.Parameter> parameterList, > + public static void buildHyperlinkUrl(Appendable externalWriter, String > target, String targetType, Map<String, String> parameterMap, > String prefix, boolean fullPath, boolean secure, boolean encode, > HttpServletRequest request, HttpServletResponse response, Map<String, Object> > context) throws IOException { > String localRequestName = UtilHttp.encodeAmpersands(target); > Appendable localWriter = new StringWriter(); > @@ -87,7 +87,7 @@ public class WidgetWorker { > localWriter.append(localRequestName); > } > > - if (UtilValidate.isNotEmpty(parameterList)) { > + if (UtilValidate.isNotEmpty(parameterMap)) { > String localUrl = localWriter.toString(); > externalWriter.append(localUrl); > boolean needsAmp = true; > @@ -96,19 +96,19 @@ public class WidgetWorker { > needsAmp = false; > } > > - for (WidgetWorker.Parameter parameter: parameterList) { > + for (Map.Entry<String, String> parameter: > parameterMap.entrySet()) { > if (needsAmp) { > externalWriter.append("&"); > } else { > needsAmp = true; > } > - externalWriter.append(parameter.getName()); > + externalWriter.append(parameter.getKey()); > externalWriter.append('='); > StringUtil.SimpleEncoder simpleEncoder = > (StringUtil.SimpleEncoder) context.get("simpleEncoder"); > if (simpleEncoder != null) { > - > externalWriter.append(simpleEncoder.encode(parameter.getValue(context))); > + > externalWriter.append(simpleEncoder.encode(parameter.getValue())); > } else { > - externalWriter.append(parameter.getValue(context)); > + externalWriter.append(parameter.getValue()); > } > } > } else { > @@ -123,7 +123,7 @@ public class WidgetWorker { > writer.append(location); > } > public static void makeHyperlinkByType(Appendable writer, String > linkType, String linkStyle, String targetType, String target, > - List<WidgetWorker.Parameter> parameterList, String description, > String targetWindow, String confirmation, ModelFormField modelFormField, > + Map<String, String> parameterMap, String description, String > targetWindow, String confirmation, ModelFormField modelFormField, > HttpServletRequest request, HttpServletResponse response, > Map<String, Object> context) throws IOException { > String realLinkType = WidgetWorker.determineAutoLinkType(linkType, > target, targetType, request); > if ("hidden-form".equals(realLinkType)) { > @@ -137,17 +137,17 @@ public class WidgetWorker { > postMultiFormWriter = new StringWriter(); > wholeFormContext.put("postMultiFormWriter", > postMultiFormWriter); > } > - WidgetWorker.makeHiddenFormLinkForm(postMultiFormWriter, > target, targetType, targetWindow, parameterList, modelFormField, request, > response, context); > + WidgetWorker.makeHiddenFormLinkForm(postMultiFormWriter, > target, targetType, targetWindow, parameterMap, modelFormField, request, > response, context); > } else { > - WidgetWorker.makeHiddenFormLinkForm(writer, target, > targetType, targetWindow, parameterList, modelFormField, request, response, > context); > + WidgetWorker.makeHiddenFormLinkForm(writer, target, > targetType, targetWindow, parameterMap, modelFormField, request, response, > context); > WidgetWorker.makeHiddenFormLinkAnchor(writer, linkStyle, > description, confirmation, modelFormField, request, response, context); > } > } else { > - WidgetWorker.makeHyperlinkString(writer, linkStyle, targetType, > target, parameterList, description, confirmation, modelFormField, request, > response, context, targetWindow); > + WidgetWorker.makeHyperlinkString(writer, linkStyle, targetType, > target, parameterMap, description, confirmation, modelFormField, request, > response, context, targetWindow); > } > > } > - public static void makeHyperlinkString(Appendable writer, String > linkStyle, String targetType, String target, List<WidgetWorker.Parameter> > parameterList, > + public static void makeHyperlinkString(Appendable writer, String > linkStyle, String targetType, String target, Map<String, String> parameterMap, > String description, String confirmation, ModelFormField > modelFormField, HttpServletRequest request, HttpServletResponse response, > Map<String, Object> context, String targetWindow) > throws IOException { > if (UtilValidate.isNotEmpty(description) || > UtilValidate.isNotEmpty(request.getAttribute("image"))) { > @@ -161,7 +161,7 @@ public class WidgetWorker { > > writer.append(" href=\""); > > - buildHyperlinkUrl(writer, target, targetType, parameterList, > null, false, false, true, request, response, context); > + buildHyperlinkUrl(writer, target, targetType, parameterMap, > null, false, false, true, request, response, context); > > writer.append("\""); > > @@ -237,7 +237,7 @@ public class WidgetWorker { > } > } > > - public static void makeHiddenFormLinkForm(Appendable writer, String > target, String targetType, String targetWindow, List<WidgetWorker.Parameter> > parameterList, ModelFormField modelFormField, HttpServletRequest request, > HttpServletResponse response, Map<String, Object> context) throws IOException > { > + public static void makeHiddenFormLinkForm(Appendable writer, String > target, String targetType, String targetWindow, Map<String, String> > parameterMap, ModelFormField modelFormField, HttpServletRequest request, > HttpServletResponse response, Map<String, Object> context) throws IOException > { > writer.append("<form method=\"post\""); > writer.append(" action=\""); > // note that this passes null for the parameterList on purpose so > they won't be put into the URL > @@ -256,11 +256,11 @@ public class WidgetWorker { > writer.append(makeLinkHiddenFormName(context, modelFormField)); > writer.append("\">"); > > - for (WidgetWorker.Parameter parameter: parameterList) { > + for (Map.Entry<String, String> parameter: parameterMap.entrySet()) { > writer.append("<input name=\""); > - writer.append(parameter.getName()); > + writer.append(parameter.getKey()); > writer.append("\" value=\""); > - writer.append(parameter.getValue(context)); > + writer.append(parameter.getValue()); > writer.append("\" type=\"hidden\"/>"); > } > > > Modified: > ofbiz/trunk/framework/widget/src/org/ofbiz/widget/form/MacroFormRenderer.java > URL: > http://svn.apache.org/viewvc/ofbiz/trunk/framework/widget/src/org/ofbiz/widget/form/MacroFormRenderer.java?rev=938816&r1=938815&r2=938816&view=diff > ============================================================================== > --- > ofbiz/trunk/framework/widget/src/org/ofbiz/widget/form/MacroFormRenderer.java > (original) > +++ > ofbiz/trunk/framework/widget/src/org/ofbiz/widget/form/MacroFormRenderer.java > Wed Apr 28 08:33:37 2010 > @@ -344,7 +344,7 @@ public class MacroFormRenderer implement > this.request.setAttribute("alternate", encodedAlternate); > this.request.setAttribute("imageTitle", encodedImageTitle); > makeHyperlinkByType(writer, hyperlinkField.getLinkType(), > modelFormField.getWidgetStyle(), hyperlinkField.getTargetType(), > hyperlinkField.getTarget(context), > - hyperlinkField.getParameterList(), > hyperlinkField.getDescription(context), > hyperlinkField.getTargetWindow(context), > hyperlinkField.getConfirmation(context), modelFormField, > + hyperlinkField.getParameterMap(context), > hyperlinkField.getDescription(context), > hyperlinkField.getTargetWindow(context), > hyperlinkField.getConfirmation(context), modelFormField, > this.request, this.response, context); > > this.appendTooltip(writer, context, modelFormField); > @@ -2813,7 +2813,7 @@ public class MacroFormRenderer implement > if (subHyperlink.shouldUse(context)) { > writer.append(' '); > makeHyperlinkByType(writer, subHyperlink.getLinkType(), > subHyperlink.getLinkStyle(), subHyperlink.getTargetType(), > subHyperlink.getTarget(context), > - subHyperlink.getParameterList(), > subHyperlink.getDescription(context), subHyperlink.getTargetWindow(context), > subHyperlink.getConfirmation(context), subHyperlink.getModelFormField(), > + subHyperlink.getParameterMap(context), > subHyperlink.getDescription(context), subHyperlink.getTargetWindow(context), > subHyperlink.getConfirmation(context), subHyperlink.getModelFormField(), > this.request, this.response, context); > } > } > @@ -2841,7 +2841,7 @@ public class MacroFormRenderer implement > } > > public void makeHyperlinkByType(Appendable writer, String linkType, > String linkStyle, String targetType, String target, > - List<WidgetWorker.Parameter> parameterList, String description, > String targetWindow, String confirmation , ModelFormField modelFormField, > + Map<String, String> parameterMap, String description, String > targetWindow, String confirmation , ModelFormField modelFormField, > HttpServletRequest request, HttpServletResponse response, > Map<String, Object> context) throws IOException { > String realLinkType = WidgetWorker.determineAutoLinkType(linkType, > target, targetType, request); > > @@ -2858,24 +2858,24 @@ public class MacroFormRenderer implement > postMultiFormWriter = new StringWriter(); > wholeFormContext.put("postMultiFormWriter", > postMultiFormWriter); > } > - WidgetWorker.makeHiddenFormLinkForm(postMultiFormWriter, > target, targetType, targetWindow, parameterList, modelFormField, request, > response, context); > + WidgetWorker.makeHiddenFormLinkForm(postMultiFormWriter, > target, targetType, targetWindow, parameterMap, modelFormField, request, > response, context); > } else { > - WidgetWorker.makeHiddenFormLinkForm(writer, target, > targetType, targetWindow, parameterList, modelFormField, request, response, > context); > + WidgetWorker.makeHiddenFormLinkForm(writer, target, > targetType, targetWindow, parameterMap, modelFormField, request, response, > context); > WidgetWorker.makeHiddenFormLinkAnchor(writer, linkStyle, > encodedDescription, confirmation , modelFormField, request, response, > context); > } > } else { > - makeHyperlinkString(writer, linkStyle, targetType, target, > parameterList, encodedDescription, confirmation , modelFormField, request, > response, context, targetWindow); > + makeHyperlinkString(writer, linkStyle, targetType, target, > parameterMap, encodedDescription, confirmation , modelFormField, request, > response, context, targetWindow); > } > > } > > - public void makeHyperlinkString(Appendable writer, String linkStyle, > String targetType, String target, List<WidgetWorker.Parameter> parameterList, > + public void makeHyperlinkString(Appendable writer, String linkStyle, > String targetType, String target, Map<String, String> parameterMap, > String description, String confirmation , ModelFormField > modelFormField, HttpServletRequest request, HttpServletResponse response, > Map<String, Object> context, String targetWindow) > throws IOException { > if (UtilValidate.isNotEmpty(description) || > UtilValidate.isNotEmpty(request.getAttribute("image"))) { > StringBuilder linkUrl = new StringBuilder(); > > - WidgetWorker.buildHyperlinkUrl(linkUrl, target, targetType, > parameterList, null, false, false, true, request, response, context); > + WidgetWorker.buildHyperlinkUrl(linkUrl, target, targetType, > parameterMap, null, false, false, true, request, response, context); > > String event = ""; > String action = ""; > > Modified: > ofbiz/trunk/framework/widget/src/org/ofbiz/widget/form/ModelFormField.java > URL: > http://svn.apache.org/viewvc/ofbiz/trunk/framework/widget/src/org/ofbiz/widget/form/ModelFormField.java?rev=938816&r1=938815&r2=938816&view=diff > ============================================================================== > --- > ofbiz/trunk/framework/widget/src/org/ofbiz/widget/form/ModelFormField.java > (original) > +++ > ofbiz/trunk/framework/widget/src/org/ofbiz/widget/form/ModelFormField.java > Wed Apr 28 08:33:37 2010 > @@ -2281,6 +2281,7 @@ public class ModelFormField { > protected FlexibleStringExpander alternate; > protected FlexibleStringExpander imageTitle; > protected FlexibleStringExpander targetWindowExdr; > + protected FlexibleMapAccessor<Map<String, String>> parametersMapAcsr; > protected List<WidgetWorker.Parameter> parameterList = > FastList.newInstance(); > > protected boolean requestConfirmation = false; > @@ -2308,6 +2309,7 @@ public class ModelFormField { > this.linkType = element.getAttribute("link-type"); > this.targetType = element.getAttribute("target-type"); > this.targetWindowExdr = > FlexibleStringExpander.getInstance(element.getAttribute("target-window")); > + this.parametersMapAcsr = > FlexibleMapAccessor.getInstance(element.getAttribute("parameters-map")); > this.image = element.getAttribute("image-location"); > > this.setRequestConfirmation("true".equals(element.getAttribute("request-confirmation"))); > > this.setConfirmationMsg(element.getAttribute("confirmation-message")); > @@ -2380,8 +2382,19 @@ public class ModelFormField { > return this.target.expandString(context); > } > > - public List<WidgetWorker.Parameter> getParameterList() { > - return this.parameterList; > + public Map<String, String> getParameterMap(Map<String, Object> > context) { > + Map<String, String> fullParameterMap = FastMap.newInstance(); > + > + Map<String, String> addlParamMap = > this.parametersMapAcsr.get(context); > + if (addlParamMap != null) { > + fullParameterMap.putAll(addlParamMap); > + } > + > + for (WidgetWorker.Parameter parameter: this.parameterList) { > + fullParameterMap.put(parameter.getName(), > parameter.getValue(context)); > + } > + > + return fullParameterMap; > } > > public String getImage() { > @@ -2507,8 +2520,21 @@ public class ModelFormField { > return this.linkType; > } > > - public List<WidgetWorker.Parameter> getParameterList() { > - return this.parameterList; > + public Map<String, String> getParameterMap(Map<String, Object> > context) { > + Map<String, String> fullParameterMap = FastMap.newInstance(); > + > + /* leaving this here... may want to add it at some point like > the hyperlink element: > + Map<String, String> addlParamMap = > this.parametersMapAcsr.get(context); > + if (addlParamMap != null) { > + fullParameterMap.putAll(addlParamMap); > + } > + */ > + > + for (WidgetWorker.Parameter parameter: this.parameterList) { > + fullParameterMap.put(parameter.getName(), > parameter.getValue(context)); > + } > + > + return fullParameterMap; > } > > public String getUseWhen(Map<String, Object> context) { > > Modified: > ofbiz/trunk/framework/widget/src/org/ofbiz/widget/html/HtmlFormRenderer.java > URL: > http://svn.apache.org/viewvc/ofbiz/trunk/framework/widget/src/org/ofbiz/widget/html/HtmlFormRenderer.java?rev=938816&r1=938815&r2=938816&view=diff > ============================================================================== > --- > ofbiz/trunk/framework/widget/src/org/ofbiz/widget/html/HtmlFormRenderer.java > (original) > +++ > ofbiz/trunk/framework/widget/src/org/ofbiz/widget/html/HtmlFormRenderer.java > Wed Apr 28 08:33:37 2010 > @@ -333,7 +333,7 @@ public class HtmlFormRenderer extends Ht > String description = encode(hyperlinkField.getDescription(context), > modelFormField, context); > String confirmation = > encode(hyperlinkField.getConfirmation(context), modelFormField, context); > WidgetWorker.makeHyperlinkByType(writer, > hyperlinkField.getLinkType(), modelFormField.getWidgetStyle(), > hyperlinkField.getTargetType(), hyperlinkField.getTarget(context), > - hyperlinkField.getParameterList(), description, > hyperlinkField.getTargetWindow(context), confirmation, modelFormField, > + hyperlinkField.getParameterMap(context), description, > hyperlinkField.getTargetWindow(context), confirmation, modelFormField, > this.request, this.response, context); > this.appendTooltip(writer, context, modelFormField); > //appendWhitespace(writer); > @@ -347,7 +347,7 @@ public class HtmlFormRenderer extends Ht > writer.append(' '); > String description = > encode(subHyperlink.getDescription(context), > subHyperlink.getModelFormField(), context); > WidgetWorker.makeHyperlinkByType(writer, > subHyperlink.getLinkType(), subHyperlink.getLinkStyle(), > subHyperlink.getTargetType(), subHyperlink.getTarget(context), > - subHyperlink.getParameterList(), description, > subHyperlink.getTargetWindow(context), subHyperlink.getConfirmation(context), > subHyperlink.getModelFormField(), > + subHyperlink.getParameterMap(context), description, > subHyperlink.getTargetWindow(context), subHyperlink.getConfirmation(context), > subHyperlink.getModelFormField(), > this.request, this.response, context); > } > } > > Modified: > ofbiz/trunk/framework/widget/src/org/ofbiz/widget/html/HtmlMenuRenderer.java > URL: > http://svn.apache.org/viewvc/ofbiz/trunk/framework/widget/src/org/ofbiz/widget/html/HtmlMenuRenderer.java?rev=938816&r1=938815&r2=938816&view=diff > ============================================================================== > --- > ofbiz/trunk/framework/widget/src/org/ofbiz/widget/html/HtmlMenuRenderer.java > (original) > +++ > ofbiz/trunk/framework/widget/src/org/ofbiz/widget/html/HtmlMenuRenderer.java > Wed Apr 28 08:33:37 2010 > @@ -418,14 +418,14 @@ public class HtmlMenuRenderer extends Ht > writer.append("\">"); > > StringUtil.SimpleEncoder simpleEncoder = > (StringUtil.SimpleEncoder) context.get("simpleEncoder"); > - for (WidgetWorker.Parameter parameter: > link.getParameterList()) { > + for (Map.Entry<String, String> parameter: > link.getParameterMap(context).entrySet()) { > writer.append("<input name=\""); > - writer.append(parameter.getName()); > + writer.append(parameter.getKey()); > writer.append("\" value=\""); > if (simpleEncoder != null) { > - > writer.append(simpleEncoder.encode(parameter.getValue(context))); > + > writer.append(simpleEncoder.encode(parameter.getValue())); > } else { > - writer.append(parameter.getValue(context)); > + writer.append(parameter.getValue()); > } > writer.append("\" type=\"hidden\"/>"); > } > @@ -478,11 +478,11 @@ public class HtmlMenuRenderer extends Ht > writer.append("javascript:confirmActionLink('"); > writer.append(confirmationMsg); > writer.append("', '"); > - WidgetWorker.buildHyperlinkUrl(writer, target, > link.getUrlMode(), link.getParameterList(), link.getPrefix(context), > + WidgetWorker.buildHyperlinkUrl(writer, target, > link.getUrlMode(), link.getParameterMap(context), link.getPrefix(context), > link.getFullPath(), link.getSecure(), > link.getEncode(), request, response, context); > writer.append("')"); > } else { > - WidgetWorker.buildHyperlinkUrl(writer, target, > link.getUrlMode(), link.getParameterList(), link.getPrefix(context), > + WidgetWorker.buildHyperlinkUrl(writer, target, > link.getUrlMode(), link.getParameterMap(context), link.getPrefix(context), > link.getFullPath(), link.getSecure(), > link.getEncode(), request, response, context); > } > } > > Modified: > ofbiz/trunk/framework/widget/src/org/ofbiz/widget/html/HtmlScreenRenderer.java > URL: > http://svn.apache.org/viewvc/ofbiz/trunk/framework/widget/src/org/ofbiz/widget/html/HtmlScreenRenderer.java?rev=938816&r1=938815&r2=938816&view=diff > ============================================================================== > --- > ofbiz/trunk/framework/widget/src/org/ofbiz/widget/html/HtmlScreenRenderer.java > (original) > +++ > ofbiz/trunk/framework/widget/src/org/ofbiz/widget/html/HtmlScreenRenderer.java > Wed Apr 28 08:33:37 2010 > @@ -495,11 +495,11 @@ public class HtmlScreenRenderer extends > writer.append(uniqueItemName); > writer.append("\">"); > > - for (WidgetWorker.Parameter parameter: link.getParameterList()) { > + for (Map.Entry<String, String> parameter: > link.getParameterMap(context).entrySet()) { > writer.append("<input name=\""); > - writer.append(parameter.getName()); > + writer.append(parameter.getKey()); > writer.append("\" value=\""); > - writer.append(parameter.getValue(context)); > + writer.append(parameter.getValue()); > writer.append("\" type=\"hidden\"/>"); > } > > @@ -537,7 +537,7 @@ public class HtmlScreenRenderer extends > writer.append(uniqueItemName); > writer.append(".submit()"); > } else { > - WidgetWorker.buildHyperlinkUrl(writer, target, > link.getUrlMode(), link.getParameterList(), link.getPrefix(context), > + WidgetWorker.buildHyperlinkUrl(writer, target, > link.getUrlMode(), link.getParameterMap(context), link.getPrefix(context), > link.getFullPath(), link.getSecure(), > link.getEncode(), request, response, context); > } > writer.append("\""); > > Modified: > ofbiz/trunk/framework/widget/src/org/ofbiz/widget/html/HtmlTreeRenderer.java > URL: > http://svn.apache.org/viewvc/ofbiz/trunk/framework/widget/src/org/ofbiz/widget/html/HtmlTreeRenderer.java?rev=938816&r1=938815&r2=938816&view=diff > ============================================================================== > --- > ofbiz/trunk/framework/widget/src/org/ofbiz/widget/html/HtmlTreeRenderer.java > (original) > +++ > ofbiz/trunk/framework/widget/src/org/ofbiz/widget/html/HtmlTreeRenderer.java > Wed Apr 28 08:33:37 2010 > @@ -221,7 +221,7 @@ public class HtmlTreeRenderer extends Ht > HttpServletRequest req = (HttpServletRequest) > context.get("request"); > if (urlMode != null && urlMode.equalsIgnoreCase("intra-app")) { > if (req != null && res != null) { > - WidgetWorker.buildHyperlinkUrl(writer, target, > link.getUrlMode(), link.getParameterList(), link.getPrefix(context), > + WidgetWorker.buildHyperlinkUrl(writer, target, > link.getUrlMode(), link.getParameterMap(context), link.getPrefix(context), > link.getFullPath(), link.getSecure(), > link.getEncode(), req, res, context); > } else if (prefix != null) { > writer.append(prefix).append(target); > > Modified: > ofbiz/trunk/framework/widget/src/org/ofbiz/widget/menu/ModelMenuItem.java > URL: > http://svn.apache.org/viewvc/ofbiz/trunk/framework/widget/src/org/ofbiz/widget/menu/ModelMenuItem.java?rev=938816&r1=938815&r2=938816&view=diff > ============================================================================== > --- ofbiz/trunk/framework/widget/src/org/ofbiz/widget/menu/ModelMenuItem.java > (original) > +++ ofbiz/trunk/framework/widget/src/org/ofbiz/widget/menu/ModelMenuItem.java > Wed Apr 28 08:33:37 2010 > @@ -28,6 +28,7 @@ import java.util.Map; > import javax.xml.parsers.ParserConfigurationException; > > import javolution.util.FastList; > +import javolution.util.FastMap; > > import org.ofbiz.base.util.Debug; > import org.ofbiz.base.util.StringUtil; > @@ -289,12 +290,12 @@ public class ModelMenuItem { > localItem.name = > portalPage.getString("portalPageId"); > localItem.setTitle((String) > portalPage.get("portalPageName", locale)); > localItem.link = new Link(this); > - List<WidgetWorker.Parameter> linkParams = > localItem.link.getParameterList(); > - linkParams.add(new > WidgetWorker.Parameter("portalPageId", portalPage.getString("portalPageId"), > false)); > - linkParams.add(new > WidgetWorker.Parameter("parentPortalPageId", parentPortalPageId, false)); > + Map<String, String> linkParams = > localItem.link.getParameterMap(context); > + linkParams.put("portalPageId", > portalPage.getString("portalPageId")); > + linkParams.put("parentPortalPageId", > parentPortalPageId); > if (link != null) { > > localItem.link.setTarget(link.targetExdr.getOriginal()); > - linkParams.addAll(link.parameterList); > + linkParams.putAll(link.getParameterMap(context)); > } else { > localItem.link.setTarget("showPortalPage"); > } > @@ -741,8 +742,21 @@ public class ModelMenuItem { > return this.linkType; > } > > - public List<WidgetWorker.Parameter> getParameterList() { > - return this.parameterList; > + public Map<String, String> getParameterMap(Map<String, Object> > context) { > + Map<String, String> fullParameterMap = FastMap.newInstance(); > + > + /* leaving this here... may want to add it at some point like > the hyperlink element: > + Map<String, String> addlParamMap = > this.parametersMapAcsr.get(context); > + if (addlParamMap != null) { > + fullParameterMap.putAll(addlParamMap); > + } > + */ > + > + for (WidgetWorker.Parameter parameter: this.parameterList) { > + fullParameterMap.put(parameter.getName(), > parameter.getValue(context)); > + } > + > + return fullParameterMap; > } > > public String getConfirmation(Map<String, Object> context) { > > Modified: > ofbiz/trunk/framework/widget/src/org/ofbiz/widget/screen/MacroScreenRenderer.java > URL: > http://svn.apache.org/viewvc/ofbiz/trunk/framework/widget/src/org/ofbiz/widget/screen/MacroScreenRenderer.java?rev=938816&r1=938815&r2=938816&view=diff > ============================================================================== > --- > ofbiz/trunk/framework/widget/src/org/ofbiz/widget/screen/MacroScreenRenderer.java > (original) > +++ > ofbiz/trunk/framework/widget/src/org/ofbiz/widget/screen/MacroScreenRenderer.java > Wed Apr 28 08:33:37 2010 > @@ -223,15 +223,15 @@ public class MacroScreenRenderer impleme > link.getFullPath(), link.getSecure(), link.getEncode(), > request, response, context); > actionUrl = sb.toString(); > parameters.append("["); > - for (WidgetWorker.Parameter parameter: link.getParameterList()) { > + for (Map.Entry<String, String> parameter: > link.getParameterMap(context).entrySet()) { > if (parameters.length() >1) { > parameters.append(","); > } > parameters.append("{'name':'"); > - parameters.append(parameter.getName()); > + parameters.append(parameter.getKey()); > parameters.append("'"); > parameters.append(",'value':'"); > - parameters.append(parameter.getValue(context)); > + parameters.append(parameter.getValue()); > parameters.append("'}"); > } > parameters.append("]"); > @@ -244,7 +244,7 @@ public class MacroScreenRenderer impleme > if (UtilValidate.isNotEmpty(target)) { > if (!"hidden-form".equals(linkType)) { > StringBuilder sb = new StringBuilder(); > - WidgetWorker.buildHyperlinkUrl(sb, target, > link.getUrlMode(), link.getParameterList(), link.getPrefix(context), > + WidgetWorker.buildHyperlinkUrl(sb, target, > link.getUrlMode(), link.getParameterMap(context), link.getPrefix(context), > link.getFullPath(), link.getSecure(), > link.getEncode(), request, response, context); > linkUrl = sb.toString(); > } > > Modified: > ofbiz/trunk/framework/widget/src/org/ofbiz/widget/screen/ModelScreenWidget.java > URL: > http://svn.apache.org/viewvc/ofbiz/trunk/framework/widget/src/org/ofbiz/widget/screen/ModelScreenWidget.java?rev=938816&r1=938815&r2=938816&view=diff > ============================================================================== > --- > ofbiz/trunk/framework/widget/src/org/ofbiz/widget/screen/ModelScreenWidget.java > (original) > +++ > ofbiz/trunk/framework/widget/src/org/ofbiz/widget/screen/ModelScreenWidget.java > Wed Apr 28 08:33:37 2010 > @@ -29,6 +29,7 @@ import javax.servlet.http.HttpServletRes > import javax.xml.parsers.ParserConfigurationException; > > import javolution.util.FastList; > +import javolution.util.FastMap; > > import org.ofbiz.base.util.Debug; > import org.ofbiz.base.util.GeneralException; > @@ -1480,8 +1481,21 @@ public abstract class ModelScreenWidget > return this.linkType; > } > > - public List<WidgetWorker.Parameter> getParameterList() { > - return this.parameterList; > + public Map<String, String> getParameterMap(Map<String, Object> > context) { > + Map<String, String> fullParameterMap = FastMap.newInstance(); > + > + /* leaving this here... may want to add it at some point like > the hyperlink element: > + Map<String, String> addlParamMap = > this.parametersMapAcsr.get(context); > + if (addlParamMap != null) { > + fullParameterMap.putAll(addlParamMap); > + } > + */ > + > + for (WidgetWorker.Parameter parameter: this.parameterList) { > + fullParameterMap.put(parameter.getName(), > parameter.getValue(context)); > + } > + > + return fullParameterMap; > } > > public void setText(String val) { > > Modified: > ofbiz/trunk/framework/widget/src/org/ofbiz/widget/tree/MacroTreeRenderer.java > URL: > http://svn.apache.org/viewvc/ofbiz/trunk/framework/widget/src/org/ofbiz/widget/tree/MacroTreeRenderer.java?rev=938816&r1=938815&r2=938816&view=diff > ============================================================================== > --- > ofbiz/trunk/framework/widget/src/org/ofbiz/widget/tree/MacroTreeRenderer.java > (original) > +++ > ofbiz/trunk/framework/widget/src/org/ofbiz/widget/tree/MacroTreeRenderer.java > Wed Apr 28 08:33:37 2010 > @@ -267,7 +267,7 @@ public class MacroTreeRenderer implement > HttpServletRequest request = (HttpServletRequest) > context.get("request"); > > if (UtilValidate.isNotEmpty(target)) { > - WidgetWorker.buildHyperlinkUrl(linkUrl, target, > link.getUrlMode(), link.getParameterList(), link.getPrefix(context), > + WidgetWorker.buildHyperlinkUrl(linkUrl, target, > link.getUrlMode(), link.getParameterMap(context), link.getPrefix(context), > link.getFullPath(), link.getSecure(), link.getEncode(), > request, response, context); > } > > > Modified: > ofbiz/trunk/framework/widget/src/org/ofbiz/widget/tree/ModelTree.java > URL: > http://svn.apache.org/viewvc/ofbiz/trunk/framework/widget/src/org/ofbiz/widget/tree/ModelTree.java?rev=938816&r1=938815&r2=938816&view=diff > ============================================================================== > --- ofbiz/trunk/framework/widget/src/org/ofbiz/widget/tree/ModelTree.java > (original) > +++ ofbiz/trunk/framework/widget/src/org/ofbiz/widget/tree/ModelTree.java Wed > Apr 28 08:33:37 2010 > @@ -930,8 +930,21 @@ public class ModelTree extends ModelWidg > return this.linkType; > } > > - public List<WidgetWorker.Parameter> getParameterList() { > - return this.parameterList; > + public Map<String, String> getParameterMap(Map<String, Object> > context) { > + Map<String, String> fullParameterMap = FastMap.newInstance(); > + > + /* leaving this here... may want to add it at some point > like the hyperlink element: > + Map<String, String> addlParamMap = > this.parametersMapAcsr.get(context); > + if (addlParamMap != null) { > + fullParameterMap.putAll(addlParamMap); > + } > + */ > + > + for (WidgetWorker.Parameter parameter: this.parameterList) { > + fullParameterMap.put(parameter.getName(), > parameter.getValue(context)); > + } > + > + return fullParameterMap; > } > > public void setText(String val) { > > -- Ofbiz on twitter: http://twitter.com/apache_ofbiz Myself on twitter: http://twitter.com/hansbak Antwebsystems.com: Quality services for competitive rates.