Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 319920

Summary:

JSP Validation Warning created for missing tag attribute when tag has dynamic-attributes=true

Product:

[WebTools] WTP Source Editing

Reporter:

Mike Youngstrom <youngm>

Component:

jst.jsp

Assignee:

Rakesh <rakes123>

Status:

RESOLVED FIXED

QA Contact:

Nitin Dahyabhai <thatnitind>

Severity:

normal

Priority:

CC:

nsand.dev, rakes123

Version:

3.2

Flags:

nsand.dev: review+

Target Milestone:

3.2.2

Hardware:

OS:

Windows 7

Whiteboard:

Attachments:

Description	Flags
Test Case Project	none
patch	nsand.dev: iplog+

Description Mike Youngstrom

2010-07-14 16:53:53 EDT

Build Identifier: 3.6 Helio

I'm using a Spring 3.0 tag library (http://www.springframework.org/tags/form).  They make extensive use of dynamic-attributes=true.  When ever I use a dynamic attribute for example:

<form:input id="petName" path="petName" placeholder="Some String" cssClass="text" />

I get the following validation error:

Undefined attribute name "placeholder"

I can provide a more precise testcase with a taglibrary and all if that would be valuable.

Reproducible: Always

Steps to Reproduce:
1. Use taglibrary that makes use of dynamic-attributes=true
2. Attempt to use a dynamic attribute
3. See .jsp validation warning appear

Comment 1 Mike Youngstrom

2010-07-14 16:55:00 EDT

updating product

Comment 2 Mike Youngstrom

2010-07-14 17:04:00 EDT

Seems to be related to https://bugs.eclipse.org/bugs/show_bug.cgi?id=212927 but
this is regarding a taglib with a X.taglib file instead of a taglet???

Comment 3 Rakesh

2010-07-22 07:48:32 EDT

Could you attach jsp where you facing this problem?

Comment 4 Mike Youngstrom

2010-07-22 11:48:35 EDT

Created attachment 174995 [details]
Test Case Project

Here is a test case project that creates and uses a taglib with dynamic-attributes=true.

Note in test.jsp I'm using dynamic attributes.  In the jsp editor I'm getting a warning "Undefined attribute name XXX".

Comment 5 Rakesh

2010-07-23 10:43:06 EDT

Created attachment 175069 [details]
patch

dynamicAttributes was not getting set in CMElementDeclarationImpl.

Comment 6 Nick Sandonato

2010-08-23 15:51:14 EDT

Approved. I also added some unit tests.