Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 338614 - Space appended to receiverName on construction sites
Summary: Space appended to receiverName on construction sites
Status: CLOSED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: Recommenders (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows 7
: P3 minor (vote)
Target Milestone: ---   Edit
Assignee: Marcel Bruch CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-03-01 21:27 EST by Stefan Henss CLA
Modified: 2019-07-24 14:36 EDT (History)
0 users

See Also:

Attachments
Example of templates completion w/ build 249 (20.95 KB, image/png)
2011-03-02 02:33 EST, Marcel Bruch CLA 		no flags Details
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 Stefan Henss CLA 2011-03-01 21:27:44 EST 
When calling

Button b<^Space>

the context will give 'b ' as receiverName.
Comment 1 Marcel Bruch CLA 2011-03-02 02:33:21 EST 
Created attachment 190109 [details]
Example of templates completion w/ build 249

I tried to reproduce bug couldn't. This screenshot shows my setup and what it proposes. maybe you could submit a test case to the templates test plug-in?
Comment 2 Marcel Bruch CLA 2011-03-02 04:47:06 EST 
added:


  private void setReceiverName(final char[] name) {
        String s = String.valueOf(name);
        setReceiverName(s);
    }

    private void setReceiverName(final String name) {
        receiverName = StringUtils.deleteWhitespace(name);
    }

to IntelligentCompletionContext. Whitespaces in receiver names should be nuked now.
Comment 3 Marcel Bruch CLA 2011-03-02 05:23:41 EST 
could you remove your code that chops the whitespace but write a test that ensures that reveceiverNames do not end with a whitespace?
Comment 4 Marcel Bruch CLA 2012-02-02 01:41:37 EST 
Closing this issue. Please give latest dev build (v0.5) a try:

http://download.eclipse.org/recommenders/updates/dev/e37/
http://download.eclipse.org/recommenders/updates/dev/e42/
Comment 5 Marcel Bruch CLA 2012-02-02 01:41:55 EST 
Closing this issue. Please give latest dev build (v0.5) a try:

http://download.eclipse.org/recommenders/updates/dev/e37/
http://download.eclipse.org/recommenders/updates/dev/e42/