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

Bug 350931

Summary: Trac task editor detects illegal camel case tracwiki links
Product: z_Archived Reporter: Jan Mauersberger <mauersberger>
Component: MylynAssignee: Jan Mauersberger <mauersberger>
Status: RESOLVED FIXED QA Contact: Steffen Pingel <steffen.pingel>
Severity: minor    
Priority: P3 Keywords: contributed
Version: unspecified   
Target Milestone: 3.7   
Hardware: PC   
OS: Windows XP   
Whiteboard:
Attachments:
Description Flags
Patch for org.eclipse.mylyn.trac.tests project that adds a text case for this bug
none
mylyn/context/zip none

Description Jan Mauersberger CLA 2011-07-01 08:56:02 EDT
Build Identifier: M20100909-0800

The hyperlink detection in the task editor for TRAC tasks is incorrectly working in case a camel case sub-string within a word that starts with a lower case character is proceeded with an uppercase character.
For example in "thisIsANiceWord" "NiceWord" is detected as link even though it is not a camel case word. "thisIsaNiceWord" wirks correctly.

Reproducible: Always

Steps to Reproduce:
1. Create a task ticket in a TRAC respository
2. Edit the task summary
3. Enter a longer mixed case word without white spaces, starting with a lower case character and inject camel case words ala "FooBar" proceeded by upper case characters like "fooFFooBarfoo"
Comment 1 Jan Mauersberger CLA 2011-07-01 08:57:45 EDT
This bug was also reported and dicussed in the mylyn newsgroup (see http://www.eclipse.org/forums/index.php/m/691304/).

I am working on a failing test case.
Comment 2 Steffen Pingel CLA 2011-07-26 13:48:56 EDT
Thanks for the report, Jan. Did you have any luck creating a test case?
Comment 3 Jan Mauersberger CLA 2011-08-10 10:33:52 EDT
(In reply to comment #2)
> Thanks for the report, Jan. Did you have any luck creating a test case?

Oops, not really started yet but try to do asap ;-)
Comment 4 Jan Mauersberger CLA 2011-08-17 08:23:01 EDT
Created attachment 201638 [details]
Patch for org.eclipse.mylyn.trac.tests project that adds a text case for this bug
Comment 5 Steffen Pingel CLA 2011-08-19 17:20:24 EDT
Thanks! Fixed in master.
Comment 6 Steffen Pingel CLA 2011-08-19 17:20:26 EDT
Created attachment 201840 [details]
mylyn/context/zip
Comment 7 Steffen Pingel CLA 2011-08-19 17:20:44 EDT
Reopening to fix assignment.
Comment 8 Steffen Pingel CLA 2011-08-19 17:21:07 EDT
Closing.