Bugzilla – Full Text Bug Listing
| Summary: | [engine] Bad indentation with protected area | ||
|---|---|---|---|
| Product: | [Modeling] Acceleo | Reporter: | Stephane Begaudeau <stephane.begaudeau> |
| Component: | Core | Assignee: | Project Inbox <acceleo-inbox> |
| Status: | CLOSED FIXED | QA Contact: | |
| Severity: | critical | ||
| Priority: | P2 | CC: | junk, laurent.goubet, sebastien.latre, william.piers |
| Version: | 3.0.0 | ||
| Target Milestone: | --- | ||
| Hardware: | PC | ||
| OS: | All | ||
| Whiteboard: | |||
|
Description
Stephane Begaudeau
Stephane: Would you mind bumping up the importance of this bug? It's a painful bug and messes up the versioning. Thanks! Jeff Jeff, bugs will be fixed in one release or the next, we can't give you any estimates. If one or more users vote on this bug, we'll likely raise its priority, but it isn't one for now. In short, you can influence this in three ways : - vote on this bug an pray that other users will too. - tackle the bug yourself and provide us with a patch fixing the issue. - buy commercial support from either Obeo (http://obeo.fr/pages/maintenance-and-support/en) or any other company that would propose maintenance support for Acceleo Of course, you can also wait for us to fix this bug, but you'll have to wait till we no longer have any bug more serious that this one. (In reply to comment #2) > Jeff, > > bugs will be fixed in one release or the next, we can't give you any estimates. > If one or more users vote on this bug, we'll likely raise its priority, but it > isn't one for now. > > In short, you can influence this in three ways : > - vote on this bug an pray that other users will too. > - tackle the bug yourself and provide us with a patch fixing the issue. > - buy commercial support from either Obeo > (http://obeo.fr/pages/maintenance-and-support/en) or any other company that > would propose maintenance support for Acceleo > > Of course, you can also wait for us to fix this bug, but you'll have to wait > till we no longer have any bug more serious that this one. Hi Laurent: It appears that voting isn't enabled for Acceleo. According to the voting help, there should be a voting button to the right of the importance field. There isn't such a button in my views. Am I missing something? Thanks, Jeff I just realized that voting is configured "per project" on the Eclipse bugzilla instead of being enabled by default ... I opened bug 332822 so that voting can be enabled on M2T/Acceleo bug 332822 is closed, but I can't see any voting button yet. Anyway, the bug is still present in the latest 3.1M4 build. I vote to bump up the importance of this bug: when using EEF it is painful to synchronize blank changes with the repository after each regeneration. This is fixed in both HEAD and 3.0.2. It will be available in the upcoming builds. I am reopening this bug since it is not completely fixed. If you have an expression that calls a template like this:
<tabulation>[myTemplateCall()/]
and if in your template there is a protected area:
[template public myTemplateCall()]
[protected ('id')]
protected content
[/protected]
[/template]
then the two following lines will still get "pushed" after each new generation:
protected content
[/protected]
because of the tabulation before the template call.
(In reply to comment #7) > I am reopening this bug since it is not completely fixed. If you have an > expression that calls a template like this: > > <tabulation>[myTemplateCall()/] > > and if in your template there is a protected area: > > [template public myTemplateCall()] > [protected ('id')] > protected content > [/protected] > [/template] > > then the two following lines will still get "pushed" after each new generation: > protected content > [/protected] > > because of the tabulation before the template call. I don't reproduce this bug when the call to the template is directly done in a [file] block but reproduced in a second level [file ('test331978.txt')] [level1()/] [/file] [template public level1() ] [for (name : String | Sequence{'a','b'})] [protected (': Before component '+name)] [/protected] [level2(name)/] [/for] [level2('c')/] [level2('d')/] [/template] [template public level2(name : String)] [protected (': In component '+name)] [/protected] [/template] The first time, the result is ok: Start of user code : Before component a End of user code Start of user code : In component a End of user code Start of user code : Before component b End of user code Start of user code : In component b End of user code Start of user code : In component c End of user code Start of user code : In component d End of user code The second time, a gap appears in the end of protected area: Start of user code : Before component a End of user code Start of user code : In component a End of user code Start of user code : Before component b End of user code Start of user code : In component b End of user code Start of user code : In component c End of user code Start of user code : In component d End of user code A new fix has been contributed on HEAD, it will be available in Acceleo 3.1.0 RC1. It solved all the problems on *my* examples but since the possibilities that would require testing are virtually unlimited I won't close this issue for now. For those who want to give it a try, the fix is available starting with the build 168: https://hudson.eclipse.org/hudson/view/Modeling/job/m2t-acceleo-3.1/168/ This build and the builds available on hudson are nightly build and as such you should not install them on a production environment. Closing once again this entry since the new algorithm seems to have fixed the issue. Closing our "resolved" and "verified" bugs. |