[WW-1236] Missing test for null 'action' property in start() of URL component?

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: WW 2.2.1
• Fix Version/s: WW 2.2.2
• Component/s: Plugin - Tags
• Assignee: Unassigned
• Reporter: Nils-Helge Garli
• Created: 12/Mar/06 19:41
• Updated: 28/Oct/12 22:11
• Resolved: 14/Mar/06 00:08

Description

in the URL component, there's a check if the 'includeParameters' and 'value' properties are null, and if this is the case, it's treated as if 'includeParameters' is set to 'GET'. This causes incorrect behaviour when deploying the sample portlets in uPortal, since the 'action' property is used instead of the 'value' property. The test looks like this:

if ((includeParams == null && value == null) || GET.equalsIgnoreCase(includeParams)) {

Is a test for 'action' == null deliberately left out of this, or is it a bug?

Issue Links

• is related to: WW-1224 Portlet URL support doesn't follow JSR-168 spec
• relates to: WW-1266 URL tag - includeParams default is "none", but docs states "get" is the default