| Anonymous | Login | Signup for a new account | 2010-09-09 23:00 CEST | ![]() |
| Main | My View | View Issues | Change Log | Roadmap |
| View Issue Details [ Jump to Notes ] | [ Issue History ] [ Print ] | ||||||||||
| ID | Project | Category | View Status | Date Submitted | Last Update | ||||||
| 0002216 | Clover Designer | (No Category) | public | 2009-06-17 13:55 | 2010-09-09 14:54 | ||||||
| Reporter | jlehotsky | ||||||||||
| Assigned To | mzatopek | ||||||||||
| Priority | high | Severity | tweak | Reproducibility | have not tried | ||||||
| Status | assigned | Resolution | open | ||||||||
| Platform | OS | OS Version | |||||||||
| Product Version | |||||||||||
| Target Version | rel-3-2-0 | Fixed in Version | |||||||||
| Summary | 0002216: CTL2/TE: Bad reinitialization for expressions (reloading global varible list etc) in new CTL | ||||||||||
| Description | This works fine for CTL1, CTL2 needs correction - in the same moment, where parent parser if reset for CTL1, the SubexpressionTE should be reset for CTL2. | ||||||||||
| Additional Information | In CTL1 parentParser is used throughtout filter editor inside TE. In CTL2 subexpressionModel (See SubexpressionTE) is used for parsing. Look for TransformTE.resetExpressionPrefix() for construction of the "fake code" to use to verify. Proper fix of (at least main) problem is to somehow reconstruct upon each opening of filter cell editor in TE (call resetExpressionPrefix - for this, new source code must be constructed based on model). --------------- E.g: if you generate new global variable by clicking "+", this is not seen inside filter editor unless you click back and forth between "Transformations" and "Source" tab. | ||||||||||
| Tags | CTL, transform editor | ||||||||||
| OTRS_Ticket_ID | |||||||||||
| Time estimate (manhours) | |||||||||||
| Time duration (manhours) | |||||||||||
| Completion status | 10 | ||||||||||
| Attached Files | |||||||||||
Notes |
|
|
(0003942) jlehotsky (manager) 2009-06-18 08:11 |
Same thing applies when new transformation is created (via drop or via "+" icon) |
|
(0004022) jlehotsky (manager) 2009-06-23 08:15 |
Solving this issue is bloody hacky :( I spent two hours fixing some occurences, however TE model is not build for this and it would require rebuilding whole source text for opening of each model - which is OK, more problematic part is re-parsing and re-matching each line to already existing model, which is quite hacky feature and I'm afraid that it cannot be done easily (without full 2-way modelling), and would require extended work not worth it. The clean solution instead is to update new CTL parser to support "parent parser" - e.g. preinitialization with symbol table from the first parsing in a same way as old CTL does this, however this is not a quick job. |
|
(0007998) jlehotsky (manager) 2010-06-07 17:06 |
Current status: all the notes written in 4022 are still true. What doesn't work: - adding new variable in the graphical editor - you cannot use it if you don't switch between source and graph. editor (for "reparsing") - adding new transformation - parser is not able to verify it and correctly re-match the new transformation to already existing model (2-way editor would handle this) Solving problems one-by-one would mean adding new super-mega-hacks over already existing pile of hacks -> this is not easy + is error-prone, but it's the only thing we can do. Correct solution would be to completely refactor transformation editor and to change its model to allow all these dynamic features -> this will take time (several MDs of work), but looks like the only possible cure for legacy of incremental editor building. |
Issue History |
|||
| Date Modified | Username | Field | Change |
| 2009-06-17 13:55 | jlehotsky | New Issue | |
| 2009-06-17 13:55 | jlehotsky | Status | new => assigned |
| 2009-06-17 13:55 | jlehotsky | Assigned To | => jlehotsky |
| 2009-06-17 13:55 | jlehotsky | Completion status | => 10 |
| 2009-06-18 08:11 | jlehotsky | Note Added: 0003942 | |
| 2009-06-18 08:48 | jlehotsky | Severity | minor => block |
| 2009-06-23 08:15 | jlehotsky | Note Added: 0004022 | |
| 2009-06-23 08:15 | jlehotsky | Target Version | rel-2-8-0 => rel-2-8-1 |
| 2009-09-01 17:18 | jurban | Target Version | rel-2-8-1 => rel-2-9-0 |
| 2009-10-02 11:40 | pnajvar | Tag Attached: CTL | |
| 2009-10-02 11:40 | pnajvar | Tag Attached: transform editor | |
| 2010-03-08 18:13 | csochor | Category | => (No Category) |
| 2010-06-07 17:06 | jlehotsky | Note Added: 0007998 | |
| 2010-06-09 13:42 | jlehotsky | Summary | TE: Bad reinitialization for expressions (reloading global varible list etc) in new CTL => CTL2/TE: Bad reinitialization for expressions (reloading global varible list etc) in new CTL |
| 2010-07-02 17:47 | jlehotsky | Additional Information Updated | View Revisions |
| 2010-07-14 12:33 | pnajvar | Assigned To | jlehotsky => mzatopek |
| 2010-07-29 17:50 | pnajvar | Target Version | rel-3-0-0 => rel-3-0-1 |
| 2010-09-08 12:55 | pnajvar | Severity | block => tweak |
| 2010-09-08 12:55 | pnajvar | Target Version | rel-3-0-1 => rel-3-1-0 |
| 2010-09-09 14:54 | pnajvar | Target Version | rel-3-1-0 => rel-3-2-0 |
| Copyright © 2000 - 2010 MantisBT Group |