Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 328110 - [content assist]Templates are case sensitive
Summary: [content assist]Templates are case sensitive
Status: RESOLVED FIXED
Alias: None
Product: WTP Source Editing
Classification: WebTools
Component: wst.sse (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 3.3 M3   Edit
Assignee: Sarika Sinha CLA
QA Contact: Nitin Dahyabhai CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-10-19 06:04 EDT by Sarika Sinha CLA
Modified: 2010-11-03 13:08 EDT (History)
1 user (show)

See Also:
nsand.dev: review+

Attachments
Patch to ignore case while checking start of template name in content assist (13.29 KB, patch)
2010-10-25 02:29 EDT, Sarika Sinha CLA 		nsand.dev: iplog+
Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Sarika Sinha CLA 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 CLA 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 CLA 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 CLA 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.