FWD » Core Development » Conversion Tools

05/13/2022

10:36 AM Feature #1729: implement better default naming rules

We have a customer which commonly has an OO 4GL class name (@XYZ.cls@) which is the same as a temp-table name (@XYZ@)... Greg Shah

05/11/2022

11:07 AM Bug #6082: automatically add to conversion list all non-skeleton .cls dependencies

In #6351-10 Greg Shah wrote:
> Sub-classes should be re-converted every time a parent class is converted.
This is... Constantin Asofiei

05/04/2022

10:55 AM Feature #6320: sharded schemata

We have multiple customers that split up a large 4GL application into multiple independent modules, each of which can... Greg Shah

10:51 AM Feature #6320 (WIP): sharded schemata

Greg Shah

10:47 AM Feature #6319: IntelliJ plugin

IntelliJ plugin for integration with syntax check and running conversion (et al). This must also provide the full sup... Greg Shah

10:44 AM Feature #6319 (Rejected): IntelliJ plugin

Greg Shah

05/02/2022

11:02 AM Feature #6256: improved profile support

If there were a need for 2 main database configured:
# a *merged* configuration with all schemas in one database, na... Roger Borrello

10:25 AM Feature #6256: improved profile support

Sorry, failed refresh. Ovidiu Maxiniuc

10:24 AM Feature #6256: improved profile support

Eric Faulhaber wrote:
> Ovidiu, do I understand correctly that when the @importFile@ attribute of @namespace@ is spe... Ovidiu Maxiniuc

10:26 AM Feature #6302: Improve conversion of empty OCX arguments

This issue does not appear in known customer scenarios, so its severity reduced to LOW. Vladimir Tsichevski

05/01/2022

11:43 AM Feature #6256: improved profile support

The current changes are in 6129a/13823. Constantin Asofiei

04/30/2022

05:20 PM Feature #6256: improved profile support

Greg Shah wrote:
> We can defer any conversion issues until a second phase of work. The front end part is what is n... Constantin Asofiei

02:53 PM Feature #6256: improved profile support

We can defer any conversion issues until a second phase of work. The front end part is what is needed this week. Greg Shah

02:49 PM Feature #6256: improved profile support

Greg Shah wrote:
> My idea was to add an extra layer at the ConversionDriver level, which would associate the profil... Constantin Asofiei

02:32 PM Feature #6256: improved profile support

My idea was to add an extra layer at the ConversionDriver level, which would associate the profile with some sub-set ... Greg Shah

02:30 PM Feature #6256: improved profile support

> What if a file is part of multiple profiles?
The profiles are intended to be mutually-exclusive. We can define ... Greg Shah

02:26 PM Feature #6256: improved profile support

Greg Shah wrote:
> The files specified for the profile should be parsed/converted with the same settings as if the p... Constantin Asofiei

02:15 PM Feature #6256: improved profile support

When I said the multiple profile merge was needed, I meant a different idea.
Each profile can be run standalone. ... Greg Shah

02:07 PM Feature #6256: improved profile support

So, the approach is this:
* multiple profiles are allowed (only for conversion time), and they can be specified via ... Constantin Asofiei

02:04 PM Feature #6256: improved profile support

Eric: I think the shard idea makes good sense. We have multiple customers using this approach in OE today. The idea... Greg Shah

01:59 PM Feature #6256: improved profile support

A useful addition (medium term, not immediately) would be to be able to load multiple DF files into a single schema n... Eric Faulhaber

01:03 PM Feature #6256: improved profile support

Greg Shah wrote:
> The multi-profile merge mode is needed as part of this task.
This will be only for conversion. ... Constantin Asofiei

01:00 PM Feature #6256: improved profile support

It is a good plan. Greg Shah

12:22 PM Feature #6256: improved profile support

Constantin Asofiei wrote:
> * if no profile is set as default, let the global configuration be the default, unless o... Constantin Asofiei

12:20 PM Feature #6256: improved profile support

Greg Shah wrote:
> > The @SchemaConfig$Profile@ has a @isDefault@, which suggests that there can be a @isDefault@ at... Constantin Asofiei

12:13 PM Feature #6256: improved profile support

Greg Shah wrote:
> The multi-profile merge mode is needed as part of this task.
Understood.
Constantin Asofiei

12:12 PM Feature #6256: improved profile support

> Do you see a need to make the profile/schema node optional? In this case, it would be inherited from the global con... Greg Shah

12:10 PM Feature #6256: improved profile support

Do you see a need to make the @profile/schema@ node optional? In this case, it would be inherited from the global con... Constantin Asofiei

11:27 AM Feature #6256: improved profile support

I don't think there is a hard requirement for that at the profile level. Greg Shah

11:26 AM Feature #6256: improved profile support

> The @SchemaConfig$Profile@ has a @isDefault@, which suggests that there can be a @isDefault@ attribute at the @prof... Greg Shah

11:26 AM Feature #6256: improved profile support

Greg Shah wrote:
> The namespace should assume @default="true"@ so that it doesn't have to be duplicated. But it sh... Constantin Asofiei

11:24 AM Feature #6256: improved profile support

The namespace should assume @default="true"@ so that it doesn't have to be duplicated. But it should be possible to ... Greg Shah

11:12 AM Feature #6256: improved profile support

The @profile@ node has only @name@ attribute in #6256-8.
The @SchemaConfig$Profile@ has a @isDefault@, which sugge... Constantin Asofiei

10:03 AM Feature #6256: improved profile support

> At least the syntax in note 5 is no longer valid, right?
Right. Greg Shah

09:56 AM Feature #6256: improved profile support

Greg Shah wrote:
> Is there some other use case I'm missing?
I don't think so.
> We already handle the namespace... Constantin Asofiei

09:49 AM Feature #6256: improved profile support

> When running in single profile mode, the profile will need to overwrite default @parameter@ and @file-set@ nodes,
... Greg Shah

09:48 AM Feature #6256: improved profile support

> each profile has the same syntax as the 'global' profile (so the namespace nodes would be added in a schema node)
... Greg Shah

09:39 AM Feature #6256: improved profile support

When running in single profile mode, the profile will need to overwrite default @parameter@ and @file-set@ nodes, and... Constantin Asofiei

09:35 AM Feature #6256: improved profile support

Greg/Eric: my thought would be the profiles to work like this:
* there is the 'global' profile which is used implici... Constantin Asofiei

04/29/2022

01:49 PM Bug #6308: post string literal alternative coding quirk in 4GL preprocessor

The function issue is actually a 4GL preprocessor quirk. I've created a testcase and checked it into [[p2j:Old UAST ... Greg Shah

01:46 PM Bug #6308 (Internal Test): post string literal alternative coding quirk in 4GL preprocessor

Greg Shah

03:25 AM Feature #6302: Improve conversion of empty OCX arguments

Vladimir, is there a customer app failing test case for this? If not I suggest you make this lower priority. Hynek Cihlar

03:06 AM Feature #6302: Improve conversion of empty OCX arguments

Vladimir Tsichevski wrote:
> The @.ast@ file seems to be correct (all parameters in the call are present, but in @.j... Hynek Cihlar

04/28/2022

07:29 PM Feature #6302: Improve conversion of empty OCX arguments

The @.ast@ file seems to be correct (all parameters in the call are present, but in @.jast@ file parameters for the @... Vladimir Tsichevski

07:08 PM Feature #6302: Improve conversion of empty OCX arguments

Vladimir Tsichevski wrote:
> Tijs Wickardt wrote:
>
> > The OpenEdge documentation states explicitly that OCX's a... Tijs Wickardt

06:18 PM Feature #6302: Improve conversion of empty OCX arguments

Tijs Wickardt wrote:
> FYI: see @testcases/oxc_to_fwd/mscomctl-treeview-ocx.w@ :
> @hNode = chCfTreeView:TreeView:... Vladimir Tsichevski

06:10 PM Feature #6302: Improve conversion of empty OCX arguments

Tijs Wickardt wrote:
> The OpenEdge documentation states explicitly that OCX's are 0-based, not 1-based like the 4... Vladimir Tsichevski

03:19 PM Feature #6302: Improve conversion of empty OCX arguments

Tijs Wickardt wrote:
> Vladimir Tsichevski wrote:
> > Seems, I was not right here: undefined values *can* be used b... Vladimir Tsichevski

03:16 PM Feature #6302: Improve conversion of empty OCX arguments

Vladimir Tsichevski wrote:
> Seems, I was not right here: undefined values *can* be used by OE to pass the "default"... Tijs Wickardt

02:50 PM Feature #6302 (WIP): Improve conversion of empty OCX arguments

Seems, I was not right here: undefined values *can* be used by OE to pass the "default" value to OCX. Moreover, undef... Vladimir Tsichevski

01:15 PM Feature #6302: Improve conversion of empty OCX arguments

This issue must be fixed along with: #5622-12. Otherwise no compilable code is created by conversion for this call. Vladimir Tsichevski

09:20 AM Feature #6302: Improve conversion of empty OCX arguments

Hynek Cihlar wrote:
> @hNode.assign(hTree.unwrapTreeView().getNodes().unwrapTreeNodeCollection().add(new integer()... Tijs Wickardt

09:16 AM Feature #6302: Improve conversion of empty OCX arguments

FYI: see @testcases/oxc_to_fwd/mscomctl-treeview-ocx.w@ :
@hNode = chCfTreeView:TreeView:Nodes:Add(,,cParentNodeID,... Tijs Wickardt

08:43 AM Feature #6302: Improve conversion of empty OCX arguments

The function parameter specification in COM type library may contain a default value. Perhaps this is the value used ... Hynek Cihlar

08:32 AM Feature #6302: Improve conversion of empty OCX arguments

Empty arguments of converted OCX method invocations are converted into "unknown" instances.
For example the follo... Hynek Cihlar

08:23 AM Feature #6302 (WIP): Improve conversion of empty OCX arguments

Hynek Cihlar

12:10 PM Feature #6256: improved profile support

Yes, the syntax is OK. Greg Shah

12:00 PM Feature #6256 (WIP): improved profile support

Incorporating the above suggestions/requirements, I have updated the sample configuration as follows:... Eric Faulhaber

07:25 AM Feature #6256: improved profile support

Greg Shah wrote:
> > I'm assuming (for now) that profile *.df files will be found in ./data/<profile-name>/.
> No, ... Ovidiu Maxiniuc

04/27/2022

07:27 PM Feature #6256: improved profile support

> Is this the right vision?
Mostly, yes. The part I don't understand is why there is a @schema@ node inside the @... Greg Shah

06:07 PM Feature #6256: improved profile support

At this moment the structure of the configuration file is:... Ovidiu Maxiniuc

04:59 PM Feature #6256: improved profile support

To translate the above configuration requirements into a sample @p2j.cfg.xml@, I understand it should look something ... Eric Faulhaber

04/26/2022

08:52 AM Feature #6255: simplify the namespace configuration, part deux

What was frustrating was that in my local testing worked, but with the main project it abended - and this was only be... Constantin Asofiei

04/24/2022

09:07 AM Feature #6255: simplify the namespace configuration, part deux

With 13811, the metadata schema can be loaded twice. This was seen in a customer project which then had ambiguous re... Greg Shah

04/22/2022

06:28 PM Feature #6255 (WIP): simplify the namespace configuration, part deux

The implementation for the initial two requests can be found in FWD starting with r13811 of 3821c.
We should keep ... Ovidiu Maxiniuc