Erwan,
If this works you should go ahead and commit it. It looks like this code is
kind of a hack anyway, so I wouldn't worry about a hack on a hack (in other
words, it doesn't look any quicker or dirtier than the existing code there, so
that shouldn't stop you from committing it).
Thanks for looking into this.
-David
On May 4, 2010, at 8:03 AM, Erwan de FERRIERES wrote:
> Le 02/05/2010 10:11, Hans Bakker a écrit :
>> Hi David,
>> 1. i select myportal -> the main page is shown -> is fine
>> 2. i select 'mycomms' within myportal -> expecting mycomms, it is
>> showing 'main' -> not fine!
>> 3. if i select 'profile' -> expecting profile but showing 'main' -> not
>> fine!
>> 4. if i select next option......etc etc
>>
>> Regards,
>> Hans
>>
>>
>
> Hans,
>
> here is a quick and dirty fix for the portals menus :
>
> Index: framework/widget/src/org/ofbiz/widget/menu/ModelMenuItem.java
> ===================================================================
> --- framework/widget/src/org/ofbiz/widget/menu/ModelMenuItem.java (revision
> 940324)
> +++ framework/widget/src/org/ofbiz/widget/menu/ModelMenuItem.java (working
> copy)
> @@ -293,6 +293,8 @@
> Map<String, String> linkParams =
> localItem.link.getParameterMap(context);
> linkParams.put("portalPageId",
> portalPage.getString("portalPageId"));
> linkParams.put("parentPortalPageId",
> parentPortalPageId);
> + localItem.link.parameterList.add(new
> WidgetWorker.Parameter("portalPageId", portalPage.getString("portalPageId"),
> false));
> + localItem.link.parameterList.add(new
> WidgetWorker.Parameter("parentPortalPageId", parentPortalPageId, false));
> if (link != null) {
> localItem.link.setTarget(link.targetExdr.getOriginal());
> linkParams.putAll(link.getParameterMap(context));
>
>
> Cheers,
>
> --
> Erwan de FERRIERES
> www.nereide.biz
