| Summary: | Attribute value proposals cannot filter qualified values | ||||||||
|---|---|---|---|---|---|---|---|---|---|
| Product: | [WebTools] WTP Source Editing | Reporter: | Nick Sandonato <nsand.dev> | ||||||
| Component: | wst.html | Assignee: | Nick Sandonato <nsand.dev> | ||||||
| Status: | RESOLVED FIXED | QA Contact: | Nick Sandonato <nsand.dev> | ||||||
| Severity: | normal | ||||||||
| Priority: | P3 | CC: | thatnitind | ||||||
| Version: | 3.2.3 | Flags: | thatnitind:
review+
|
||||||
| Target Milestone: | 3.2.3 | ||||||||
| Hardware: | PC | ||||||||
| OS: | Windows XP | ||||||||
| Whiteboard: | |||||||||
| Attachments: |
|
||||||||
|
Description
Nick Sandonato
Created attachment 181322 [details]
patch
Looks good, but fAlternateMatch.substring(0, length) could be a problem if the word length is already greater than the alternate match's length. Created attachment 181398 [details]
updated patch
Ah, nice catch. I also fixed this in the unlikely scenario where if the display string is null, and an alternate match is provided, we'll use it.
Thanks for the review. |