FWD » Core Development » User Interface

01/21/2020

02:29 PM Bug #4074 (WIP): Mismatching ROW-ENTRY/LEAVE events

> Should I implement it?
Yes. The change can be put into 4335a. Greg Shah

02:13 PM Bug #4074: Mismatching ROW-ENTRY/LEAVE events

Closing seems to be premature. I didn't implement Hynek's idea (I got confirmation only now). Should I implement it? Stanislav Lomany

01:44 PM Bug #4074 (Closed): Mismatching ROW-ENTRY/LEAVE events

Greg Shah

01:39 PM Bug #4074: Mismatching ROW-ENTRY/LEAVE events

Greg Shah wrote:
> Hynek: Does this satisfy your notes in #4074-24?
Yes I'm sure Stanislav tested the mentioned u... Hynek Cihlar

10:48 AM Bug #4512: New inserted row has incorrect initial cell values

New inserted rows should be blank. In order to reproduce please run @browse-entry.p@ and press 'i' to insert new row.... Sergey Ivanovskiy

10:41 AM Bug #4512 (New): New inserted row has incorrect initial cell values

Sergey Ivanovskiy

10:08 AM Feature #4113: enable upload of files from the browser (in the web clients) using SYSTEM-DIALOG G...

Greg Shah wrote:
> 3809e was merged to trunk as revision 11340. If I understand correctly, the truncation of the fi... Hynek Cihlar

09:29 AM Feature #4113: enable upload of files from the browser (in the web clients) using SYSTEM-DIALOG G...

3809e was merged to trunk as revision 11340. If I understand correctly, the truncation of the filename is now fixed ... Greg Shah

09:42 AM Bug #4473 (Closed): Chrome update issues

Branch 3809e was merged to trunk as revision 11340. Greg Shah

09:24 AM Feature #4077: add option to OPEN-MIME-RESOURCE to delete the original source file when the send ...

3809e was merged to trunk as revision 11340.
Sergey: Please update the FWD implementation details in the OPEN-MIME... Greg Shah

09:13 AM Feature #3908: implement browse column VIEW-AS COMBO-BOX

3809e was merged to trunk as revision 11340. Greg Shah

07:26 AM Feature #3880: enhanced browse 3rd phase of improvements

> What is the purpose of this flag?
OK, I think I don't actually don't need this flag. The issue I want to address... Stanislav Lomany

06:56 AM Feature #3880: enhanced browse 3rd phase of improvements

> The only issue here is that I'll think I'll add a flag that indicates if the "layout part" of the enhanced configur... Greg Shah

06:04 AM Feature #3880: enhanced browse 3rd phase of improvements

Regarding storage of the "report title" parameter. I think I'll add it to the enhanced browse configuration. The only... Stanislav Lomany

04:00 AM Feature #4174: implement calendar control/dtpicker OCX replacement

Sergey Ivanovskiy wrote:
> Another issue can be observed with this expression @chCtrlFrame-4:DTPicker:VISIBLE@ and @... Sergey Ivanovskiy

01/20/2020

04:55 PM Feature #3880: enhanced browse 3rd phase of improvements

> Do we want to have "user-specific" and "all users" settings as with enhanced browse configurations?
Yes, but as ... Greg Shah

03:41 PM Feature #3880: enhanced browse 3rd phase of improvements

> Let's focus on allowing the user to customize it. That really resolves the issue. What can be done for this?
I... Stanislav Lomany

01:33 PM Feature #3880: enhanced browse 3rd phase of improvements

Stanislav Lomany wrote:
> Guys, I need to get user id (logged in) on the client side. I'm planning to get it from th... Greg Shah

01:18 PM Feature #3880: enhanced browse 3rd phase of improvements

For now, I think it is enough for the user to be able to clear their own customizations. I think this corresponds to... Greg Shah

01:14 PM Feature #3880: enhanced browse 3rd phase of improvements

> Implement proper "reset to default state" for the new security paradigm.
Every user (including admins) has three... Stanislav Lomany

01:14 PM Feature #3880: enhanced browse 3rd phase of improvements

Please check in your changes to 4335a. Greg Shah

12:55 PM Feature #3880: enhanced browse 3rd phase of improvements

Default report title, in order of preference:
# User-specified custom title.
# If the browse has a title, use it.... Greg Shah

12:04 PM Feature #3880: enhanced browse 3rd phase of improvements

> ## Output a better title for browse exports. The current code uses "Browse Report", which means it has nothing to ... Stanislav Lomany

09:39 AM Feature #3880: enhanced browse 3rd phase of improvements

Guys, I need to get user id (logged in) on the client side. I'm planning to get it from the server, but I wonder when... Stanislav Lomany

04:31 PM Bug #4498: Embedded assignment of extent in shared frame fails

The changes are good.
> Why am I not calling the main constructor directly with:
Because then you would create ... Greg Shah

03:04 PM Bug #4498: Embedded assignment of extent in shared frame fails

Greg Shah wrote:
> Code Review Task Branch 4207a Revision 11381
>
> 1. In @database_references.rules@, I don't se... Roger Borrello

02:20 PM Bug #4498: Embedded assignment of extent in shared frame fails

Code Review Task Branch 4207a Revision 11381
1. In @database_references.rules@, I don't see any protection for the... Greg Shah

03:20 PM Feature #4174: implement calendar control/dtpicker OCX replacement

Another issue can be observed with this expression @chCtrlFrame-4:DTPicker:VISIBLE@ and @chCtrlFrame-4:DTPicker:ENABL... Sergey Ivanovskiy

02:33 PM Feature #4174: implement calendar control/dtpicker OCX replacement

Yes, I would like to convert @FONT@ to @ActiveXFont@, @FONT:NAME@ to @ActiveXFont:ActiveXFontName@, @FONT:SIZE@ to @A... Sergey Ivanovskiy

02:13 PM Feature #4174: implement calendar control/dtpicker OCX replacement

Sergey, the problem is that you have the following item in map @<action>map6.put("font" , "ActiveXFont")</a... Marius Gligor

01:52 PM Feature #4174: implement calendar control/dtpicker OCX replacement

Marius Gligor wrote:
> Sergey Ivanovskiy wrote:
> > Marius, thanks, I updated 4174a over 3856a and got this AST nod... Sergey Ivanovskiy

01:44 PM Feature #4174: implement calendar control/dtpicker OCX replacement

Sergey Ivanovskiy wrote:
> Marius, thanks, I updated 4174a over 3856a and got this AST node for @chCtrlFrame-4:DTPic... Marius Gligor

01:40 PM Feature #4174: implement calendar control/dtpicker OCX replacement

Marius, thanks, I updated 4174b over 3856a and got this AST node for @chCtrlFrame-4:DTPicker:FONT:SIZE@ expression
... Sergey Ivanovskiy

09:24 AM Feature #4174: implement calendar control/dtpicker OCX replacement

Sergey Ivanovskiy wrote:
> I tried to add conversion rules for @FONT:NAME@ and @FONT:SIZE@ legacy attributes of Acti... Marius Gligor

09:22 AM Feature #4174: implement calendar control/dtpicker OCX replacement

I tried to add conversion rules for @FONT:NAME@ and @FONT:SIZE@ legacy attributes of ActiveX control using these rule... Sergey Ivanovskiy

06:48 AM Feature #4174: implement calendar control/dtpicker OCX replacement

It seems that @IFontDisp@ should be added but there exists the similar interface @CustomFont@. The @CustomFont@ uses ... Sergey Ivanovskiy

07:25 AM Feature #3856: implement a widget that replaces the Microsoft ImageList OCX (VB 6.0 common control)

Marius: Please rebase 3856b from the latest trunk. My plan is for 3856b to be the next branch to merge to trunk. Th... Greg Shah

02:41 AM Feature #3856: implement a widget that replaces the Microsoft ImageList OCX (VB 6.0 common control)

Code review @3856b@ revisions 11365-11371. The changes look good. Hynek Cihlar

01/19/2020

03:41 PM Bug #4498: Embedded assignment of extent in shared frame fails

Code ready for review:
modified *rules/convert/database_references.rules*
modified *src/com/goldencode/p2j/persist/... Roger Borrello

01/17/2020

12:18 PM Bug #4498: Embedded assignment of extent in shared frame fails

Looking into what the ramifications of passing @uppercase@ as *false* when constructing the @FieldReference@ for bulk... Roger Borrello

10:30 AM Bug #4498: Embedded assignment of extent in shared frame fails

Greg Shah wrote:
> > Does it matter what is passed for uppercase?
>
> This would not be used to match your new co... Roger Borrello

08:59 AM Bug #4498: Embedded assignment of extent in shared frame fails

> Does it matter what is passed for uppercase?
This would not be used to match your new constructors. But it migh... Greg Shah

02:37 AM Feature #4113: enable upload of files from the browser (in the web clients) using SYSTEM-DIALOG G...

It seems that this parameter name @length@ came from the web socket api. This web socket connection listener can conf... Sergey Ivanovskiy

02:30 AM Feature #4113: enable upload of files from the browser (in the web clients) using SYSTEM-DIALOG G...

3809e revision 11472 has good changes. Sergey Ivanovskiy

02:14 AM Feature #4113: enable upload of files from the browser (in the web clients) using SYSTEM-DIALOG G...

Sergey Ivanovskiy wrote:
> this change is incorrect. Please look at all its usages and fix the new incorrect usages ... Hynek Cihlar

01:54 AM Feature #4113: enable upload of files from the browser (in the web clients) using SYSTEM-DIALOG G...

Hynek Cihlar wrote:
> Hynek Cihlar wrote:
> > I found the cause and fixed it. The change is attached.
> >
>
> ... Sergey Ivanovskiy

01/16/2020

05:20 PM Bug #4498: Embedded assignment of extent in shared frame fails

Roger Borrello wrote:
> Greg Shah wrote:
> > Roger Borrello wrote:
> > > A question to our database "gurus" with r... Roger Borrello

04:29 PM Bug #4498: Embedded assignment of extent in shared frame fails

> why int versus NumberType for the index?
Because sometimes the 4GL code has a subscript which is a @NUM_LITERAL@... Greg Shah

04:06 PM Bug #4498: Embedded assignment of extent in shared frame fails

Greg Shah wrote:
> Roger Borrello wrote:
> > A question to our database "gurus" with respect to how the "bulk" opti... Roger Borrello

03:32 PM Bug #4498: Embedded assignment of extent in shared frame fails

*FieldReference* has two very similar constructors:
* @FieldReference(DataModelObject dmo, String property, int inde... Roger Borrello

11:56 AM Bug #4498: Embedded assignment of extent in shared frame fails

Roger Borrello wrote:
> A question to our database "gurus" with respect to how the "bulk" option is used in @FieldRe... Greg Shah

11:29 AM Bug #4498: Embedded assignment of extent in shared frame fails

A question to our database "gurus" with respect to how the "bulk" option is used in @FieldReference.java@...
Why c... Roger Borrello

05:14 PM Feature #4113: enable upload of files from the browser (in the web clients) using SYSTEM-DIALOG G...

Hynek Cihlar wrote:
> I found the cause and fixed it. The change is attached.
>
Checked in to 3809e revision 11... Hynek Cihlar

04:59 PM Feature #4113: enable upload of files from the browser (in the web clients) using SYSTEM-DIALOG G...

Sergey Ivanovskiy wrote:
> It is incorrect change.
Can you provide more details? Hynek Cihlar

11:18 AM Feature #4174: implement calendar control/dtpicker OCX replacement

This resource https://wutils.com/com-dll/constants/constants-MSComCtl2.htm defines related @IDTPicker@ constants from... Sergey Ivanovskiy

08:47 AM Feature #3856: implement a widget that replaces the Microsoft ImageList OCX (VB 6.0 common control)

I've fixed regression test errors as follow:
- remove @UIB_S = chCtrlFrame:LoadControls( OCXFile, "CtrlFrame":U)@ in... Marius Gligor

01/15/2020

05:19 PM Bug #4074: Mismatching ROW-ENTRY/LEAVE events

Hynek: Does this satisfy your notes in #4074-24? Greg Shah

11:57 AM Bug #4074: Mismatching ROW-ENTRY/LEAVE events

> Please also test the case when you click in a browse, without focus, directly in an editable cell. If I interpret y... Stanislav Lomany

09:29 AM Bug #4074: Mismatching ROW-ENTRY/LEAVE events

Code review 4231b review 11472.
Please also test the case when you click in a browse, without focus, directly in a... Hynek Cihlar

05:48 AM Bug #4074: Mismatching ROW-ENTRY/LEAVE events

Fix for double ROW-ENTRY events is committed into 4231b rev 11472. Please review. Stanislav Lomany

02:48 PM Bug #4498: Embedded assignment of extent in shared frame fails

@com.goldencode.p2j.pattern.CommonAstSupport$Library.upPath(String)@
That's just a convenience function for TRPL. ... Eric Faulhaber

02:04 PM Bug #4498: Embedded assignment of extent in shared frame fails

Greg Shah wrote:
> No. Read @CommonAstSupport@ for @upPath()@ to understand why.
I don't see that method in http... Roger Borrello

10:57 AM Bug #4498: Embedded assignment of extent in shared frame fails

Greg Shah wrote:
> That is not a regression. It is more correct. We WANT the @FieldReference@ to be emitted.
Th... Roger Borrello

10:16 AM Bug #4498: Embedded assignment of extent in shared frame fails

That is not a regression. It is more correct. We WANT the @FieldReference@ to be emitted. Greg Shah

10:04 AM Bug #4498: Embedded assignment of extent in shared frame fails

There is a regression found where this code line:... Roger Borrello

08:11 AM Feature #4113: enable upload of files from the browser (in the web clients) using SYSTEM-DIALOG G...

It is incorrect change. Sergey Ivanovskiy

08:07 AM Feature #4113: enable upload of files from the browser (in the web clients) using SYSTEM-DIALOG G...

Hynek Cihlar wrote:
> Hynek Cihlar wrote:
> > Jurjen Dijkstra wrote:
> > >
> > > any reaction?
> >
> > Sorry,... Sergey Ivanovskiy

08:03 AM Feature #4113: enable upload of files from the browser (in the web clients) using SYSTEM-DIALOG G...

Yes, go ahead. Greg Shah

07:36 AM Feature #4113: enable upload of files from the browser (in the web clients) using SYSTEM-DIALOG G...

Hynek Cihlar wrote:
> Jurjen Dijkstra wrote:
> >
> > any reaction?
>
> Sorry, I was busy with other tasks. I w... Hynek Cihlar

04:18 AM Feature #4113: enable upload of files from the browser (in the web clients) using SYSTEM-DIALOG G...

ok I was just worried that the mail was lost Jurjen Dijkstra

04:13 AM Feature #4113: enable upload of files from the browser (in the web clients) using SYSTEM-DIALOG G...

Jurjen Dijkstra wrote:
>
> any reaction?
Sorry, I was busy with other tasks. I will check the sources where the... Hynek Cihlar

04:10 AM Feature #4113: enable upload of files from the browser (in the web clients) using SYSTEM-DIALOG G...

Jurjen Dijkstra wrote:
> I see what the problem is: the string is cut at 64 characters.
>
> The upload directory ... Jurjen Dijkstra

01/14/2020

04:17 PM Feature #3856: implement a widget that replaces the Microsoft ImageList OCX (VB 6.0 common control)

The problem here is that @handle@ has no strong typing information that can always be known at parsing time. Sometim... Greg Shah

03:59 PM Feature #3856: implement a widget that replaces the Microsoft ImageList OCX (VB 6.0 common control)

Sergey Ivanovskiy wrote:
> Marius, could you explain why @ENABLED@ and @VALUE@ progress bar properties were named as... Sergey Ivanovskiy

02:47 PM Feature #3856: implement a widget that replaces the Microsoft ImageList OCX (VB 6.0 common control)

There are existing 4GL attributes of the same names, so the @PB-@ prefix was needed to differentiate these from the "... Greg Shah

02:31 PM Feature #3856: implement a widget that replaces the Microsoft ImageList OCX (VB 6.0 common control)

Marius, could you explain why @ENABLED@ and @VALUE@ progress bar properties were named as @PB-ENABLED@ and @PB-VALUE@... Sergey Ivanovskiy

04:02 PM Bug #4498 (WIP): Embedded assignment of extent in shared frame fails

Roger Borrello

04:01 PM Bug #4498: Embedded assignment of extent in shared frame fails

Committed @rules/convert/database_references.rules@ in *4207a-11377* to use field reference for the lvalue of an embe... Roger Borrello

01:11 PM Bug #4498: Embedded assignment of extent in shared frame fails

No. Read @CommonAstSupport@ for @upPath()@ to understand why. Greg Shah

12:25 PM Bug #4498: Embedded assignment of extent in shared frame fails

With respect to updates to @convert/database_references.rules@ for emitting a field reference...
Wouldn't converti... Roger Borrello

10:54 AM Bug #4498: Embedded assignment of extent in shared frame fails

Updates for un-subscripted variables in an embedded assignment are committed in 4207a-11376:... Roger Borrello

09:49 AM Bug #4498: Embedded assignment of extent in shared frame fails

Constantin Asofiei wrote:
> Hmm... isn't @tt.getMyx7()@ returning a copy of the array?
Yes, I didn't look at that... Greg Shah

09:26 AM Bug #4498: Embedded assignment of extent in shared frame fails

Hmm... isn't @tt.getMyx7()@ returning a copy of the array? Constantin Asofiei

09:23 AM Bug #4498: Embedded assignment of extent in shared frame fails

> The last will get us past conversion, but runtime will still need some work.
Not exactly. The last will get us ... Greg Shah

09:17 AM Bug #4498: Embedded assignment of extent in shared frame fails

Constantin Asofiei wrote:
> I think we need to change the constructor for @EmbeddedAssignment@ to receive a lambda e... Roger Borrello

03:38 PM Bug #4074: Mismatching ROW-ENTRY/LEAVE events

Yes, 4231b is fine. Greg Shah

03:36 PM Bug #4074: Mismatching ROW-ENTRY/LEAVE events

> Please put your changes in 4335a.
I was going to put them into 4231b as they seem to be more risky. Is it OK?
Stanislav Lomany

10:50 AM Bug #4074: Mismatching ROW-ENTRY/LEAVE events

4103a is in trunk. Please put your changes in 4335a. Greg Shah

11:15 AM Feature #4170: add drag and drop support to treeview widget

I created a new branch 4170a from current trunk and rebased with 3856b Marius Gligor

02:26 AM Feature #4170: add drag and drop support to treeview widget

Greg Shah wrote:
> Good summary.
>
> What happens if the drop event occurs outside of the tree control?
If the d... Marius Gligor

10:46 AM Feature #3880 (WIP): enhanced browse 3rd phase of improvements

Greg Shah

01/13/2020

02:52 PM Bug #4498: Embedded assignment of extent in shared frame fails

I think we need to change the constructor for @EmbeddedAssignment@ to receive a lambda expression, which will actuall... Constantin Asofiei

02:49 PM Bug #4498: Embedded assignment of extent in shared frame fails

Greg Shah wrote:
> Did you test this in the 4GL?
Yes... fields worked just like variables. Roger Borrello

02:42 PM Bug #4498: Embedded assignment of extent in shared frame fails

Did you test this in the 4GL? Greg Shah

02:16 PM Bug #4498: Embedded assignment of extent in shared frame fails

Built new testcases, which uses fields in a temp table.
@nonshared_frame_extra_widget_embedded_assign_field.p@
<... Roger Borrello

12:56 PM Bug #4498: Embedded assignment of extent in shared frame fails

Greg Shah wrote:
> This is not a bug. This is just the normal embedded assignment case. Since @i@ is still an edit... Roger Borrello

12:13 PM Bug #4498: Embedded assignment of extent in shared frame fails

This is not a bug. This is just the normal embedded assignment case. Since @i@ is still an editable widget, this is... Greg Shah

11:30 AM Bug #4498: Embedded assignment of extent in shared frame fails

Roger Borrello wrote:
> There is a scenario which is still broken, and that is when there is a widget that *should b... Roger Borrello

10:54 AM Bug #4498: Embedded assignment of extent in shared frame fails

There is a scenario which is still broken, and that is when there is a widget that *should be added* to a non-shared ... Roger Borrello

10:41 AM Bug #4498: Embedded assignment of extent in shared frame fails

Roger Borrello wrote:
> By protecting the @evalLib("process_fake_scope", tmp, true)@ in *annotations/frame_scoping.r... Roger Borrello

08:52 AM Bug #4498: Embedded assignment of extent in shared frame fails

By protecting the @evalLib("process_fake_scope", tmp, true)@ in *annotations/frame_scoping.rules* with a check for @t... Roger Borrello

02:09 PM Feature #4174: implement calendar control/dtpicker OCX replacement

Greg Shah wrote:
> > Should it be designing exactly the same as the MS Date and Time Picker control?
>
> Yes. Th... Sergey Ivanovskiy

02:04 AM Feature #4174: implement calendar control/dtpicker OCX replacement

Sergey Ivanovskiy wrote:
> Sorry, it was 3856b. It seems that it looks like an issue in 3856b.
Yes, you have righ... Marius Gligor

12:24 PM Feature #4170: add drag and drop support to treeview widget

Good summary.
What happens if the drop event occurs outside of the tree control? Greg Shah

10:23 AM Feature #4170: add drag and drop support to treeview widget

A. When a Drag&Drop operation is initiated, an @OLEStartDrag@ event is fired:
When the event is handled, the followi... Marius Gligor

08:46 AM Feature #4113: enable upload of files from the browser (in the web clients) using SYSTEM-DIALOG G...

I see what the problem is: the string is cut at 64 characters.
The upload directory that I have specified is: ... Jurjen Dijkstra

01/12/2020

10:07 AM Feature #4174: implement calendar control/dtpicker OCX replacement

> Should it be designing exactly the same as the MS Date and Time Picker control?
Yes. This is needed for compati... Greg Shah

01/11/2020

09:40 AM Feature #4174: implement calendar control/dtpicker OCX replacement

It seems that developing the date time picker widget itself can be independent from the conversion issues.
Greg, a... Sergey Ivanovskiy

08:37 AM Bug #4498: Embedded assignment of extent in shared frame fails

> Do we need to validate that there isn't already an LBRACKET child? Perhaps there's display myx[1] = 1?
We are on... Greg Shah

01/10/2020

06:16 PM Bug #4498: Embedded assignment of extent in shared frame fails

Roger Borrello wrote:
> In @annotations/array_expansion.rules@ I updated the below to protect the setting of @attach... Roger Borrello

06:10 PM Bug #4498: Embedded assignment of extent in shared frame fails

Roger Borrello wrote:
> In @annotations/arrray_expansion.rules@ I updated the below to protect the setting of @attac... Roger Borrello

04:08 PM Bug #4498: Embedded assignment of extent in shared frame fails

In @annotations/array_expansion.rules@ I updated the below to protect the setting of @attachIdx@ and @parentRef@:
<p... Roger Borrello

03:12 PM Bug #4498: Embedded assignment of extent in shared frame fails

In focusing on the incorrect widget for @myx@, the @add_widget@ function in *annotations/frame_scoping.rules*, the @U... Roger Borrello

01:16 PM Bug #4498: Embedded assignment of extent in shared frame fails

> Strange that there are duplicated VAR_INT for i in the FRAME_ELEMENT, and the only difference in the annotations is... Greg Shah

01:08 PM Bug #4498: Embedded assignment of extent in shared frame fails

In running the "extra widget" testcase, there is something strange generated in the AST.
Line 13 of the testcase i... Roger Borrello

11:55 AM Bug #4498: Embedded assignment of extent in shared frame fails

h1. Shared Frame (Standalone)
Note that I could not get embedded assignment scalar versions to convert, per #4498#... Roger Borrello

10:25 AM Bug #4498: Embedded assignment of extent in shared frame fails

Roger Borrello wrote:
> Received the following when I was adding scalar testcases...
>
> [...]
>
> Testcase is... Roger Borrello

10:11 AM Bug #4498: Embedded assignment of extent in shared frame fails

Received the following when I was adding scalar testcases...... Roger Borrello

01:02 PM Feature #4174: implement calendar control/dtpicker OCX replacement

Sorry, it was 3856b. It seems that it looks like an issue in 3856b. Sergey Ivanovskiy

12:56 PM Feature #4174: implement calendar control/dtpicker OCX replacement

I just tested the conversion of my example with 3856b if @ext-hint@ file is given and found that this code... Sergey Ivanovskiy

12:30 PM Feature #4174: implement calendar control/dtpicker OCX replacement

> I think, the Hotel GUI project can be created from the fwdtemplate, if it is what you mean. But this need to be che... Greg Shah

12:05 PM Feature #4174: implement calendar control/dtpicker OCX replacement

Greg Shah wrote:
> Are the @prepare_template.sh@ changes something generic enough to be accepted back as a standard ... Vladimir Tsichevski

10:40 AM Feature #4174: implement calendar control/dtpicker OCX replacement

Are the @prepare_template.sh@ changes something generic enough to be accepted back as a standard part of the Hotel GU... Greg Shah

10:24 AM Feature #4174: implement calendar control/dtpicker OCX replacement

Greg Shah wrote:
> So you've made changes to the @prepare_template.sh@ and @build.xml@?
I have rewritten the @pre... Vladimir Tsichevski

10:17 AM Feature #4174: implement calendar control/dtpicker OCX replacement

So you've made changes to the @prepare_template.sh@ and @build.xml@? Greg Shah

10:06 AM Feature #4174: implement calendar control/dtpicker OCX replacement

Greg Shah wrote:
> We should look at your new project code for sure. Please note that we currently create new proje... Vladimir Tsichevski

09:19 AM Feature #4174: implement calendar control/dtpicker OCX replacement

Vladimir Tsichevski wrote:
> Sergey Ivanovskiy wrote:
> > Please help with these common questions about testing env... Sergey Ivanovskiy

09:18 AM Feature #4174: implement calendar control/dtpicker OCX replacement

Greg Shah wrote:
> The directory is used at runtime with the server. It is ignored completely during conversion. S... Sergey Ivanovskiy

09:12 AM Feature #4174: implement calendar control/dtpicker OCX replacement

We should look at your new project code for sure. Please note that we currently create new projects from the Hotel C... Greg Shah

09:07 AM Feature #4174: implement calendar control/dtpicker OCX replacement

Sergey Ivanovskiy wrote:
> Please help with these common questions about testing environment. I am still using old @... Vladimir Tsichevski

08:51 AM Feature #4174: implement calendar control/dtpicker OCX replacement

> I am still using old testcases project. Is it correct?
Yes, this is OK.
> Should I create a test environment ... Greg Shah

08:49 AM Feature #4174: implement calendar control/dtpicker OCX replacement

Don't mind this question about @&IF "{&OPSYS}" = "WIN32":U AND "{&WINDOW-SYSTEM}" NE "TTY":U &THEN@. This macros work... Sergey Ivanovskiy

08:47 AM Feature #4174: implement calendar control/dtpicker OCX replacement

> I created the task branch 4174a. Should it be rebased over 3856b? Would it be better to wait until 3856b merged in ... Greg Shah

08:39 AM Feature #4174: implement calendar control/dtpicker OCX replacement

Please help with these common questions about testing environment. I am still using old @testcases@ project. Is it co... Sergey Ivanovskiy

07:35 AM Feature #4174: implement calendar control/dtpicker OCX replacement

I created the task branch 4174a. Should it be rebased over 3856b? Would it be better to wait until 3856b merged in th... Sergey Ivanovskiy

07:21 AM Feature #4174: implement calendar control/dtpicker OCX replacement

Sergey Ivanovskiy wrote:
> Taking into account this working 4GL code #4174-12 the current implementation of @Control... Vladimir Tsichevski

06:54 AM Feature #4174: implement calendar control/dtpicker OCX replacement

> At least the MS Date Time Picker control name is stored in this file.
For now, this is put in the hints file in ... Greg Shah

06:44 AM Feature #4174: implement calendar control/dtpicker OCX replacement

Sergey Ivanovskiy wrote:
> Greg Shah wrote:
> > I think we only need to worry about item 4. Assuming no design tim... Marius Gligor

06:14 AM Feature #4174: implement calendar control/dtpicker OCX replacement

Taking into account this working 4GL code #4174-12 the current implementation of @ControlFrameComObjectImpl.this.getC... Sergey Ivanovskiy

03:24 AM Feature #4174: implement calendar control/dtpicker OCX replacement

Greg Shah wrote:
> I think we only need to worry about item 4. Assuming no design time properties are set via the w... Sergey Ivanovskiy

02:29 AM Feature #4174: implement calendar control/dtpicker OCX replacement

Greg Shah wrote:
> I think we only need to worry about item 4. Assuming no design time properties are set via the w... Sergey Ivanovskiy

10:05 AM Feature #4113: enable upload of files from the browser (in the web clients) using SYSTEM-DIALOG G...

Sure, I will see if I can collect more data. This afternoon I will not have time for that unfortunately, so it'll be ... Jurjen Dijkstra

09:50 AM Feature #4113: enable upload of files from the browser (in the web clients) using SYSTEM-DIALOG G...

Jurjen Dijkstra wrote:
> Hynek Cihlar wrote:
> > Jurjen Dijkstra wrote:
> > > Hynek Cihlar wrote:
> > > > Jurjen ... Hynek Cihlar

09:40 AM Feature #4113: enable upload of files from the browser (in the web clients) using SYSTEM-DIALOG G...

Hynek Cihlar wrote:
> Jurjen Dijkstra wrote:
> > Hynek Cihlar wrote:
> > > Jurjen Dijkstra wrote:
> > > > So now ... Jurjen Dijkstra

09:33 AM Feature #4113: enable upload of files from the browser (in the web clients) using SYSTEM-DIALOG G...

Jurjen Dijkstra wrote:
> Hynek Cihlar wrote:
> The dialog is needed for the security reasons - the local file acces... Hynek Cihlar

09:25 AM Feature #4113: enable upload of files from the browser (in the web clients) using SYSTEM-DIALOG G...

Hynek Cihlar wrote:
> Jurjen Dijkstra wrote:
> > Also, when statement @system-dialog get-file@ executes, it first d... Jurjen Dijkstra

08:47 AM Feature #4113: enable upload of files from the browser (in the web clients) using SYSTEM-DIALOG G...

Jurjen Dijkstra wrote:
> Also the "filters" argument does not have any effect. I am specifying
> @filters "zip ... Hynek Cihlar

08:39 AM Feature #4113: enable upload of files from the browser (in the web clients) using SYSTEM-DIALOG G...

Jurjen Dijkstra wrote:
> Also the "filters" argument does not have any effect. I am specifying
> @filters "zip ... Hynek Cihlar

08:26 AM Feature #4113: enable upload of files from the browser (in the web clients) using SYSTEM-DIALOG G...

Also the "filters" argument does not have any effect. I am specifying
@filters "zip archive (*.zip)" "*.zip"@
b... Jurjen Dijkstra

08:21 AM Feature #4113: enable upload of files from the browser (in the web clients) using SYSTEM-DIALOG G...

Jurjen Dijkstra wrote:
> Hynek Cihlar wrote:
> > Jurjen Dijkstra wrote:
> > > So now how to continue?
> > > It s... Hynek Cihlar

08:03 AM Feature #4113: enable upload of files from the browser (in the web clients) using SYSTEM-DIALOG G...

Hynek Cihlar wrote:
> Jurjen Dijkstra wrote:
> > So now how to continue?
> > It seems to me that parameter filena... Jurjen Dijkstra

01/09/2020

05:45 PM Bug #4498: Embedded assignment of extent in shared frame fails

h1. Testcase Results
h2. Shared Frames (Standalone)
Both testcases are very similar, with the exception of the ... Roger Borrello

03:37 PM Bug #4498: Embedded assignment of extent in shared frame fails

h1. Non-Shared Frame
Again, the only difference between these 2 testcases is the embedded assignment. However, unl... Roger Borrello

02:55 PM Bug #4498: Embedded assignment of extent in shared frame fails

h1. Shared Frame
The only difference between these 2 testcases is the embedded assignment. However, the one with t... Roger Borrello

02:36 PM Bug #4498: Embedded assignment of extent in shared frame fails

In this testcase, you can add a widget to non-shared frame, and update the variables.
@nonshared_frame_embedded_up... Roger Borrello

02:31 PM Bug #4498: Embedded assignment of extent in shared frame fails

These testcases both result in 4GL errors:
@cannot_add_widget_to_imported_frame.p@... Roger Borrello

10:59 AM Bug #4498: Embedded assignment of extent in shared frame fails

> There are 3 types of frame:
>
> * non-shared (@DEF FRAME@, no use of the @SHARED@ keyword)
> * shared
> ** @DE... Roger Borrello

10:42 AM Bug #4498: Embedded assignment of extent in shared frame fails

>> Does this failure only occur when this is an imported shared frame?
>
> If imported means non-shared (or local)
... Greg Shah

10:34 AM Bug #4498: Embedded assignment of extent in shared frame fails

> * Does this failure only occur when this is an imported shared frame?
If imported means non-shared (or local), t... Roger Borrello

10:12 AM Bug #4498: Embedded assignment of extent in shared frame fails

Questions:
* Does this failure only occur when this is an imported shared frame?
* What does the 4GL do here? It... Greg Shah

09:55 AM Bug #4498: Embedded assignment of extent in shared frame fails

h1. Description
An array of extent 2 or more does not get the "hardcoded" array widgets created in the shared fram... Roger Borrello

09:39 AM Bug #4498 (Closed): Embedded assignment of extent in shared frame fails

Roger Borrello

04:36 PM Feature #4174: implement calendar control/dtpicker OCX replacement

I think we only need to worry about item 4. Assuming no design time properties are set via the wrx (see #4438), item... Greg Shah

04:31 PM Feature #4174: implement calendar control/dtpicker OCX replacement

These common 4GL usages of MS Date and Time Control:
# AppBuilder creates @wrx@ file that contains the AciveX contro... Sergey Ivanovskiy

03:51 PM Feature #4174: implement calendar control/dtpicker OCX replacement

A key point: OCX controls are only 32-bit in nature. It is my understanding that Microsoft has never provided a 64-b... Greg Shah

10:29 AM Bug #4074: Mismatching ROW-ENTRY/LEAVE events

Yes, that would be OK as well. Greg Shah

10:24 AM Bug #4074: Mismatching ROW-ENTRY/LEAVE events

Greg, also there's a @FocusEvent@ generated by @requestFocus@ called right after the row entry event is generated, wh... Stanislav Lomany

03:34 AM Feature #3856: implement a widget that replaces the Microsoft ImageList OCX (VB 6.0 common control)

Code review 3856b revision 11363. The changes look good. Hynek Cihlar

03:29 AM Feature #3856: implement a widget that replaces the Microsoft ImageList OCX (VB 6.0 common control)

I tested and fixed referenced indexed properties conversion rules. Basically the rules determine when to use set or g... Marius Gligor

01/08/2020

04:07 PM Bug #4488: Wrong frame used after display includes widgets from other frame

Checked in these testcases:... Roger Borrello

02:10 PM Bug #4488: Wrong frame used after display includes widgets from other frame

*Updates are in 4207a-11372.* Roger Borrello

02:10 PM Bug #4488 (WIP): Wrong frame used after display includes widgets from other frame

Roger Borrello

12:50 PM Bug #4488: Wrong frame used after display includes widgets from other frame

Good. Greg Shah

10:50 AM Bug #4488: Wrong frame used after display includes widgets from other frame

Greg Shah wrote:
> > Do we need to recalculate the node.parent.type each iteration of the while node.parent.type != ... Roger Borrello

08:24 AM Bug #4488: Wrong frame used after display includes widgets from other frame

> Do we need to recalculate the node.parent.type each iteration of the while node.parent.type != prog.statement?
I... Greg Shah

07:10 PM Bug #4488: Wrong frame used after display includes widgets from other frame

Looking at the more enhanced updates to @convert/frame_generator.xml@ where we want to use @firstref@, @lastref@, and... Roger Borrello

02:16 PM Feature #4174: implement calendar control/dtpicker OCX replacement

For details on using MS Common Controls OCXes in appbuilder, please see #3858-85 (through note 87). Greg Shah

12:49 PM Bug #4074: Mismatching ROW-ENTRY/LEAVE events

You could create a subclass of @KeyInput@ for use with entry events. Greg Shah

11:53 AM Bug #4074: Mismatching ROW-ENTRY/LEAVE events

> I suppose it's the best place to put a testcase that demonstrates unwanted row-entry events and duplicate row-entry... Stanislav Lomany

09:00 AM Feature #3856: implement a widget that replaces the Microsoft ImageList OCX (VB 6.0 common control)

Marius Gligor wrote:
> could you tell me please what is a referencing indexed properties?
I meant an assignment o... Hynek Cihlar

08:55 AM Feature #3856: implement a widget that replaces the Microsoft ImageList OCX (VB 6.0 common control)

Hynek Cihlar wrote:
> Code review 3856b revision 11362. The changes are good. Also please test referencing indexed p... Marius Gligor

08:51 AM Feature #3856: implement a widget that replaces the Microsoft ImageList OCX (VB 6.0 common control)

Code review 3856b revision 11362. The changes are good. Also please test referencing indexed properties. Hynek Cihlar

06:39 AM Feature #3856: implement a widget that replaces the Microsoft ImageList OCX (VB 6.0 common control)

The idea behind conversion rules for indexed OCX properties is to transform the assignment right-side value into a CO... Marius Gligor

01/07/2020

04:32 PM Bug #4488: Wrong frame used after display includes widgets from other frame

Greg Shah wrote:
> What does this have to do with the regression in #4488-22?
Just a different approach. With #44... Roger Borrello

04:15 PM Bug #4488: Wrong frame used after display includes widgets from other frame

What does this have to do with the regression in #4488-22? Greg Shah

04:06 PM Bug #4488: Wrong frame used after display includes widgets from other frame

Greg Shah wrote:
> That does seem wrong. But I thought that you said the @input mychar@ code did emit in Java with f... Roger Borrello

03:58 PM Bug #4488: Wrong frame used after display includes widgets from other frame

That does seem wrong. But I thought that you said the @input mychar@ code did emit in Java with frame f2. Greg Shah

03:43 PM Bug #4488: Wrong frame used after display includes widgets from other frame

Greg Shah wrote:
> > and we should now be scoped to f2 frame for the getMychar() getter, due to the repeat with fram... Roger Borrello

03:25 PM Bug #4488: Wrong frame used after display includes widgets from other frame

> The mychar value ("myval") should be displayed in the frame sf1's fieldx widget.
Yes.
> The code (without rev... Greg Shah

02:36 PM Bug #4488: Wrong frame used after display includes widgets from other frame

Greg Shah wrote:
> Yes. That is the entire point of an at-base field. It IS a widget in the frame.
Taking this ... Roger Borrello

01:51 PM Bug #4488: Wrong frame used after display includes widgets from other frame

Yes. That is the entire point of an at-base field. It IS a widget in the frame. Greg Shah

01:37 PM Bug #4488: Wrong frame used after display includes widgets from other frame

Should this testcase:... Roger Borrello

12:14 PM Bug #4488: Wrong frame used after display includes widgets from other frame

Roger Borrello wrote:
> Updates to:
> +rules/annotations/frame_scoping.rules+
> @add_widget@ was modified to inclu... Roger Borrello

12:11 PM Bug #4488: Wrong frame used after display includes widgets from other frame

Updates to:
+rules/annotations/frame_scoping.rules+
@add_widget@ was modified to include another condition under wh... Roger Borrello

01:18 PM Feature #4174: implement calendar control/dtpicker OCX replacement

In Hotel GUI, you can see this usage as follows. You will see a button widget in some screen or dialog which as a tr... Greg Shah

01:07 PM Feature #4174: implement calendar control/dtpicker OCX replacement

Sergey Ivanovskiy wrote:
> There is @C:\Windows\SysWOW64\mscomct2.ocx@ OCX library that exposes @IDTPicker@ (called ... Greg Shah

12:54 PM Feature #4174: implement calendar control/dtpicker OCX replacement

There is @C:\Windows\SysWOW64\mscomct2.ocx@ OCX library that exposes @IDTPicker@ (called Microsoft Date and Time Pick... Sergey Ivanovskiy

12:06 PM Feature #4174 (WIP): implement calendar control/dtpicker OCX replacement

Sergey Ivanovskiy

11:43 AM Feature #3856: implement a widget that replaces the Microsoft ImageList OCX (VB 6.0 common control)

Hynek Cihlar wrote:
> Greg Shah wrote:
> > * Does this problem only exist after 3856b? If so, is it caused by the ... Marius Gligor

11:09 AM Feature #3856: implement a widget that replaces the Microsoft ImageList OCX (VB 6.0 common control)

Greg Shah wrote:
> * Does this problem only exist after 3856b? If so, is it caused by the improvements in COM/OCX c... Hynek Cihlar

10:20 AM Feature #3856: implement a widget that replaces the Microsoft ImageList OCX (VB 6.0 common control)

During conversion regression testing with a large customer application, Marius found a regression that converts witho... Greg Shah

01/06/2020

02:22 PM Bug #4488: Wrong frame used after display includes widgets from other frame

Greg Shah wrote:
> @fieldx@ still needs all those annotations because it is a real widget reference. Either your ch... Roger Borrello

10:49 AM Bug #4488: Wrong frame used after display includes widgets from other frame

@fieldx@ still needs all those annotations because it is a real widget reference. Either your changes are in the wro... Greg Shah

10:42 AM Bug #4488: Wrong frame used after display includes widgets from other frame

Roger Borrello wrote:
> Greg Shah wrote:
> > Yes, that is what I was referencing in #4488-11. To solve it we need ... Roger Borrello

10:57 AM Feature #4077: add option to OPEN-MIME-RESOURCE to delete the original source file when the send ...

I made some additional edits. Also, please note that the FWD implementation details are now also out of date. Greg Shah

09:22 AM Feature #4077: add option to OPEN-MIME-RESOURCE to delete the original source file when the send ...

Greg, please review these changes https://proj.goldencode.com/projects/p2j/wiki/OPEN-MIME-RESOURCE#section-3. Sergey Ivanovskiy

08:30 AM Feature #4077 (Test): add option to OPEN-MIME-RESOURCE to delete the original source file when th...

Sergey: Please update the documentation to fully explain the new syntax. You MUST leave the old syntax there as well... Greg Shah

09:30 AM Feature #4170 (WIP): add drag and drop support to treeview widget

The drag and drop support is used in the customer application being converted in #4069. If you can easily implement ... Greg Shah

01/03/2020

06:06 PM Bug #4488: Wrong frame used after display includes widgets from other frame

Greg Shah wrote:
> Yes, that is what I was referencing in #4488-11. To solve it we need to know when this is an @&#... Roger Borrello

05:30 PM Bug #4488: Wrong frame used after display includes widgets from other frame

Yes, that is what I was referencing in #4488-11. To solve it we need to know when this is an @&#64; base-field@ refe... Greg Shah

05:13 PM Bug #4488: Wrong frame used after display includes widgets from other frame

It looks to me like the next-to-last @add_widget@ is the problem:... Roger Borrello

04:45 PM Bug #4488: Wrong frame used after display includes widgets from other frame

Roger Borrello wrote:
> The @jName@ values being @expr...@ is a little unexpected, as that is what was added to the ... Roger Borrello

04:03 PM Bug #4488: Wrong frame used after display includes widgets from other frame

Greg Shah wrote:
> > The comment before adding the widget is "the variables used in @ base clauses in format_phrase ... Roger Borrello

03:35 PM Bug #4488: Wrong frame used after display includes widgets from other frame

> The comment before adding the widget is "the variables used in @ base clauses in format_phrase are also part of the... Greg Shah

03:26 PM Bug #4488: Wrong frame used after display includes widgets from other frame

Greg Shah wrote:
> In both the shared and non-shared cases, the use of @&#64; base-field@ syntax *does NOT create a ... Roger Borrello

02:45 PM Bug #4488: Wrong frame used after display includes widgets from other frame

In both the shared and non-shared cases, the use of @&#64; base-field@ syntax *does NOT create a new widget* in the 4... Greg Shah

01:38 PM Bug #4488: Wrong frame used after display includes widgets from other frame

Creating a standalone NON-SHARED testcase:... Roger Borrello

11:25 AM Bug #4488: Wrong frame used after display includes widgets from other frame

Roger Borrello wrote:
> I am struggling to understand what exactly I should expect that code to do:
> @display mych... Roger Borrello

11:23 AM Bug #4488: Wrong frame used after display includes widgets from other frame

I am struggling to understand what exactly I should expect that code to do:
@display mychar @@@ fieldx with frame sf... Roger Borrello

01/02/2020

05:56 PM Bug #4488: Wrong frame used after display includes widgets from other frame

When the @display mychar @@@ fieldx with frame sf1@ is used with a non-shared sf1, the code @SharedFrameMixupSf1@ cla... Roger Borrello

04:31 PM Bug #4488: Wrong frame used after display includes widgets from other frame

Roger Borrello wrote:
> Roger Borrello wrote:
> > ...
>
> Hold on... there's a typo in the testcase I need to co... Roger Borrello

04:18 PM Bug #4488: Wrong frame used after display includes widgets from other frame

Roger Borrello wrote:
> ...
Hold on... there's a typo in the testcase I need to correct. Roger Borrello

04:17 PM Bug #4488: Wrong frame used after display includes widgets from other frame

h1. Testcase
*Updates are in 4207a-11372.*
The below has been checked in as @uast/sharedframes/shared_frame_mix... Roger Borrello

03:34 PM Bug #4488 (Closed): Wrong frame used after display includes widgets from other frame

Roger Borrello

12/24/2019

06:55 AM Support #2674: list of HTML5, CSS and Javascript features in use in the web client (both ChUI and...

It needs to check new versions of IE - Edge browser if it supports ORTC, @navigator.MediaDevices.getUserMedia@ and @P... Sergey Ivanovskiy

12/23/2019

08:55 AM Support #2674: list of HTML5, CSS and Javascript features in use in the web client (both ChUI and...

Browser market share, as of November 2019:
* "StatCounter":https://gs.statcounter.com/browser-market-share
** All... Greg Shah

05:38 AM Feature #3856: implement a widget that replaces the Microsoft ImageList OCX (VB 6.0 common control)

Code review 3856b rev 11358, 11359 and 11360.
The changes are good. Please note that I removed added white spaces ... Hynek Cihlar