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

Bug 328110

Summary:

[content assist]Templates are case sensitive

Product:

[WebTools] WTP Source Editing

Reporter:

Sarika Sinha <sarika.sinha>

Component:

wst.sse

Assignee:

Sarika Sinha <sarika.sinha>

Status:

RESOLVED FIXED

QA Contact:

Nitin Dahyabhai <thatnitind>

Severity:

normal

Priority:

CC:

nsand.dev

Version:

unspecified

Flags:

nsand.dev: review+

Target Milestone:

3.3 M3

Hardware:

OS:

Windows XP

Whiteboard:

Attachments:

Description	Flags
Patch to ignore case while checking start of template name in content assist	nsand.dev: iplog+

Description Sarika Sinha

2010-10-19 06:04:39 EDT

Build Identifier: M20100909-0800

xml, html and jsp templates are case sensitive. If i type "SCR" and then select the html template proposal for #script, it adds script template leaving "SCR" at is. If I type "scr"  and select the template, then it replaces "scr" with script template

Reproducible: Always

Comment 1 Sarika Sinha

2010-10-19 07:11:50 EDT

Hi Nick,
Currently we have 3 copies of ReplaceNameTemplateContext for xml, html and jsp. Can't we have a single copy in xml as they all do the same stuff and fix the bug in single file?

Comment 2 Sarika Sinha

2010-10-25 02:29:36 EDT

Created attachment 181610 [details]
Patch to ignore case while checking start of template name in content assist

This patch removes the ReplaceNameTemplateContext from html.ui and jsp.ui package and uses the one from xml.ui package. in xml.ui package, ReplaceNameTemplateContext has been modified to ignore the case while comparing template name with user entered content assist prefix.

Comment 3 Nick Sandonato

2010-11-03 13:08:35 EDT

Sarika, I deprecated the old ReplaceNameTemplateContext in jsp.ui and html.ui just in case anyone is extending them. Thanks for the patch.