Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

after/before for several operations at the same definition doesn't work #680

Closed
vahdat-ab opened this Issue Jan 15, 2016 · 1 comment

Comments

Projects
None yet
3 participants
@vahdat-ab
Copy link
Member

commented Jan 15, 2016

consider the following example

class A{

  Integer x;
  Integer y;
  after setX,setY {
   //this an injected code; 
  }
}

we expect to see injected code for both setX and setY methods. However, it is applied to setX;

@TimLethbridge

This comment has been minimized.

Copy link
Member

commented Jan 15, 2016

This looks like an easy fix actually

@VictoriaLacroix VictoriaLacroix self-assigned this Jan 21, 2016

VictoriaLacroix added a commit to VictoriaLacroix/umple that referenced this issue Jan 26, 2016

Fixed the injection of multiple methods at once.
The issue was that the StringBuilder token wasn't properly used by Umple's Internal Parser. It had the target methods in separate tokens, which a StringBuilder was intended to fix. Passing the StringBuilder's result into makeCodeInjection() fixed the issue.

Fixes umple#680

Signed-off-by: Victoria Lacroix <victoria.a.lacroix@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.