Project

General

Profile

Activity

From 01/10/2015 to 02/08/2015

02/07/2015

11:38 PM Database Bug #2488: replace runtime compilation of dynamic queries
Each OPEN QUERY in the following test case (@client-where-simple.p@) will force client-side where clause processing:
...
Eric Faulhaber
07:03 PM Database Bug #2488: replace runtime compilation of dynamic queries
The regression test was successful. Please commit and distribute the update. Eric Faulhaber

02/06/2015

04:15 PM Database Bug #2488: replace runtime compilation of dynamic queries
The conversion test was successful. There are no differences in the generated code with the attached update. Ovidiu Maxiniuc
02:07 PM Database Bug #2488: replace runtime compilation of dynamic queries
> The best alternative is to use your solution from note 43 and rename the new method to @putAnnotationObject()@ so t... Eric Faulhaber
02:00 PM Database Bug #2488: replace runtime compilation of dynamic queries
Eric,
The last updates fail to convert the majic sources. The reason is the @putAnnotation()@ method. Although I use...
Ovidiu Maxiniuc
12:38 PM Database Bug #2488: replace runtime compilation of dynamic queries
Thanks, Ovidiu. Please post your conversion results when that's done. I'll run my tests later today or overnight. Eric Faulhaber
07:31 AM Database Bug #2488: replace runtime compilation of dynamic queries
> * Why did you indent build.xml:265?
There were some TABs characters in that file that I replaced with plain spaces...
Ovidiu Maxiniuc
11:03 PM Database Bug #2488: replace runtime compilation of dynamic queries
Code review 0205a:
* Please put back the indent at @DynamicQueryHelper@:101.
* I think the logging levels used in...
Eric Faulhaber
03:48 PM User Interface Feature #2479: DEFINE IMAGE and image phrase support for both button and image
This update for review presents the approach for image-phrase implementation candidate. Needs to check and debug but ... Eugenie Lyzenko
02:20 PM User Interface Feature #1790: implement menu support
Vadim Gindin wrote:
> Will you also insist on using of @PaintEvent@? If yes could you help me to understand how it s...
Constantin Asofiei
12:28 PM User Interface Feature #1790: implement menu support
How to properly test static menus deletion? Vadim Gindin
02:13 PM User Interface Feature #2498: implement ADD-LAST() method
>
> > 6. The item ID generator should not be in the @ControlSetConfig@ class. We are trying to eliminate all logi...
Greg Shah
01:46 PM User Interface Feature #2498: implement ADD-LAST() method
Greg Shah wrote:
> 2. Is there any way to introduce problems in @ComboBoxModel.calcSelection()@ by relying upon the ...
Igor Skornyakov
01:14 PM User Interface Feature #2498: implement ADD-LAST() method
Code Review ias_upd20150205a.zip
I really like your nicely factored @ControlSetEntity@ @add*()@ implementation. I...
Greg Shah
12:27 PM User Interface Feature #2498: implement ADD-LAST() method
Got it. Thank you. Igor Skornyakov
12:24 PM User Interface Feature #2498: implement ADD-LAST() method
> I've also notices that all handle.unwrapXXX() methods work with interfaces while ControlEntity is a class. I don't ... Greg Shah
12:04 PM User Interface Feature #2498: implement ADD-LAST() method
Got it. Actually I've already stated to look at the conversion details.
I've added @unwrapConrolEntity()@ method to ...
Igor Skornyakov
11:52 AM User Interface Feature #2498: implement ADD-LAST() method
I will do the code review next. Sorry that I have been slow to respond. I've just been very busy.
In regard to t...
Greg Shah
10:26 AM User Interface Feature #2498: implement ADD-LAST() method
Regression test performed twice.
The following steps failed identically in both runs:
4 ctrlc_11_session1 CTRL...
Igor Skornyakov
08:47 PM Base Language Feature #1628: fix any incompatibilities or missing features of NO-ECHO support
>Even the character type seems incorrect as the default format for character is x(8). It is not based on the length o... Evgeny Kiselev

02/05/2015

06:02 PM Base Language Bug #2499: DST portion of the timezone offset is always included when it should be conditional
I agree. Well done. Greg Shah
06:00 PM Base Language Bug #2499: DST portion of the timezone offset is always included when it should be conditional
I believe I have good news.
The testing of 0204a finished with following result:
gso_ctrlc_tests: 2 fails: once th...
Ovidiu Maxiniuc
04:15 PM User Interface Feature #2478: button image support
The update @evl_upd20150204a.zip@ has been committed in bzr as 10744. Eugenie Lyzenko
04:08 PM User Interface Feature #2478: button image support
The testing completed. Results: @10742_5c96d7c_20150205_evl.zip@. No regressions confirmed.
I've stopped the serve...
Eugenie Lyzenko
10:53 AM User Interface Feature #2478: button image support
I appreciate that you are taking extra care. Thanks. Greg Shah
10:36 AM User Interface Feature #2478: button image support
The runtime testing almost completed. The only issue is the @GSO 269@ test. It failed with two main cycles but passed... Eugenie Lyzenko
03:06 PM User Interface Feature #2498: implement ADD-LAST() method
Just a notice. It seems that there are some problems with conversion.
1. Conversion of the attached @radio2.p@ progr...
Igor Skornyakov
01:58 PM User Interface Feature #2498: implement ADD-LAST() method
All done and is ready for the code review. I'm not sure however that frame overflow detection at @RadioSetWidget.addL... Igor Skornyakov
03:05 PM Database Bug #2488: replace runtime compilation of dynamic queries
Eric Faulhaber wrote:
> Before I write up the code review, I need a definitive answer on the client-side where claus...
Ovidiu Maxiniuc
01:01 AM Database Bug #2488: replace runtime compilation of dynamic queries
Before I write up the code review, I need a definitive answer on the client-side where clause processing question in ... Eric Faulhaber
11:16 AM Database Feature #2134: configurable denormalization of extent fields into individual fields in schema con...
Update that excludes temporary table fields from denormalization process. Committed to bzr revision 10743. Stanislav Lomany
08:19 PM User Interface Feature #2479: DEFINE IMAGE and image phrase support for both button and image
This update adds missing in previous update @ImageConfig@ class. Also it resolves the offset-size phrase issues found... Eugenie Lyzenko

02/04/2015

05:09 PM User Interface Feature #2478: button image support
The conversion testing completed. The generated code is identical. Continue with runtime testing. Eugenie Lyzenko
02:20 PM User Interface Feature #2478: button image support
> Please go ahead and do one more round of testing, using evl_upd20150204a.zip. If it passes, you can check it in.
...
Eugenie Lyzenko
01:46 PM User Interface Feature #2478: button image support
Please go ahead and do one more round of testing, using evl_upd20150204a.zip. If it passes, you can check it in. Greg Shah
11:16 AM User Interface Feature #2478: button image support
Merged with 10741 code base. Eugenie Lyzenko
02:45 PM Base Language Bug #2499: DST portion of the timezone offset is always included when it should be conditional
Yes, exactly. Greg Shah
02:44 PM Base Language Bug #2499: DST portion of the timezone offset is always included when it should be conditional
Greg Shah wrote:
> 2. Prepare a version of the fix that has the @date.java@ changes merged with the H063 version o...
Ovidiu Maxiniuc
02:31 PM Base Language Bug #2499: DST portion of the timezone offset is always included when it should be conditional
> 1. Check in and distribute om_upd20150203a.zip.
The update was committed to bzr renvo 10742 and distributed by m...
Ovidiu Maxiniuc
01:31 PM Base Language Bug #2499: DST portion of the timezone offset is always included when it should be conditional
Great!
Next steps:
1. Check in and distribute om_upd20150203a.zip.
2. Prepare a version of the fix that has ...
Greg Shah
01:13 PM Base Language Bug #2499: DST portion of the timezone offset is always included when it should be conditional
Greg Shah wrote:
> I'm fine with the changes. If it passes testing, you can check it in.
The @main@ passed the reg...
Ovidiu Maxiniuc
01:03 PM User Interface Feature #2498: implement ADD-LAST() method
Fixed @SelectionListBody@ to correctly restore selection after list modification Igor Skornyakov
10:54 AM User Interface Feature #2498: implement ADD-LAST() method
Just noticed that ListSelectionModel implementation should be revised - currently it behaves incorrectly after ADD-FI... Igor Skornyakov
10:43 AM User Interface Feature #2498: implement ADD-LAST() method
SELECTION-LIST behaviour fixed to be compatible with 4GL (both in SINGLE and MULTIPLE selection modes).
RADIO-SET ...
Igor Skornyakov
08:20 PM User Interface Feature #2479: DEFINE IMAGE and image phrase support for both button and image
This update for review includes suggest for runtime implementation for image phrase for image widget. In general it w... Eugenie Lyzenko

02/03/2015

03:22 PM User Interface Feature #2498: implement ADD-LAST() method
Implemented @ADD-LAST()@ and fixed @ADD-FIRST()@ for RADIO-SET. Igor Skornyakov
01:07 PM User Interface Feature #2498: implement ADD-LAST() method
Yes, please fix it. Greg Shah
12:40 PM User Interface Feature #2498: implement ADD-LAST() method
ADD-FIRST/ADD-LAST/DELETE functionality for SELECTION-LIST look OK.
However I've noticed that Java runtime doesn't a...
Igor Skornyakov
09:27 AM User Interface Feature #2498: implement ADD-LAST() method
An id field is added to ControlSetItem. Now the behaviour of COMBO-BOX widget after ADD-FIRST/ADD-LAST/DELETE operati... Igor Skornyakov
02:14 PM User Interface Feature #1790: implement menu support
Could you help me with initial focus setting. I'm trying to set focus on a first menu-item in a shown pop-up menu i.e... Vadim Gindin
12:45 PM Base Language Bug #2511: timestamp instance oddity when assigning to a datetime
Consider the following P4GL code fragment executed on windev01, configured with timezone UTC+02:00 (only need to be o... Ovidiu Maxiniuc
12:11 PM Base Language Bug #2511 (New): timestamp instance oddity when assigning to a datetime
Ovidiu Maxiniuc
11:21 AM Base Language Bug #2499: DST portion of the timezone offset is always included when it should be conditional
The om_upd20150203a.zip update has:
* 3 fails in CTRL+C
* 2 fails in gc_tests
* 6 fails in tc_tests
Except for ...
Ovidiu Maxiniuc
07:44 AM Base Language Bug #2499: DST portion of the timezone offset is always included when it should be conditional
Code Review om_upd20150203a.zip
I'm fine with the changes. If it passes testing, you can check it in.
Please a...
Greg Shah

02/02/2015

05:34 PM Base Language Bug #2499: DST portion of the timezone offset is always included when it should be conditional
I am putting this update to test.
It passed my tests regarding the DST. Yet, there is an issue regarding the assign...
Ovidiu Maxiniuc
03:55 PM User Interface Feature #2422: add features to BROWSE
Addition to note 63. Another 4GL bug which I'm not going to reproduce. Applies to ChUI and GUI. If the number of rows... Stanislav Lomany
02:50 PM User Interface Feature #2498: implement ADD-LAST() method
Most of the above is implemented (for COMBO-BOX). However the issue with the value shown in the box after @ADD-FIRST@... Igor Skornyakov
10:25 AM User Interface Feature #2478: button image support
The @evl_upd20150202a.zip@ also merged with 10739. Eugenie Lyzenko
09:15 AM User Interface Feature #2478: button image support
Update merged with 10738 code base. Eugenie Lyzenko

02/01/2015

03:52 PM User Interface Feature #2478: button image support
Update merged with the 10736 code base. Eugenie Lyzenko
11:02 PM User Interface Feature #2478: button image support
Sorry, there are 2 updates in front of you right now. Hynek's is first and then Constantin has one. Thanks for your... Greg Shah
11:50 AM User Interface Feature #2498: implement ADD-LAST() method
Good work in finding these issues.
> Should I start working on the issues mentioned above?
Yes, please do.
Greg Shah
11:15 AM User Interface Feature #2498: implement ADD-LAST() method
It is easy to implement section 2 (e.g by adding @realValue@ field to @ControlSetItem@ in addition to @value@ - for @... Igor Skornyakov
11:41 AM Base Language Bug #2510 (Closed): client-side context reset problem
Let's not worry about this now. Your update is a good, safe movement forward. Greg Shah
11:36 AM Base Language Bug #2510: client-side context reset problem
Greg Shah wrote:
> You can go ahead and check it in.
Released to rev 10734.
> My only question is whether the C...
Constantin Asofiei
11:30 AM Base Language Bug #2510: client-side context reset problem
You can go ahead and check it in. Greg Shah
11:30 AM Base Language Bug #2510: client-side context reset problem
Code Review ca_upd20150131c.zip
It looks good. My only question is whether the @ContextLocal@ should also have it...
Greg Shah
10:37 AM Base Language Bug #2510 (Review): client-side context reset problem
Attached update fixes the reset of client-side context data, when CTRL-C is pressed and ClientCore.start loop goes to... Constantin Asofiei
10:36 AM Base Language Bug #2510 (Closed): client-side context reset problem
Constantin Asofiei

01/31/2015

05:36 PM User Interface Feature #2478: button image support
Merged with recent code base. Is it time to start new round or there will be more updates? Eugenie Lyzenko
02:38 PM User Interface Feature #1790: implement menu support
I didn't find a way to get it work with posting @PaintEvent@. But I've fixed the bug adding @menu.requestSync()@ call... Vadim Gindin

01/30/2015

05:01 PM User Interface Feature #2478: button image support
> Normally, the answer would be yes. But right now we are in the middle of a number of conflicting updates that have... Eugenie Lyzenko
04:57 PM User Interface Feature #2478: button image support
Normally, the answer would be yes. But right now we are in the middle of a number of conflicting updates that have @... Greg Shah
11:43 AM User Interface Feature #2478: button image support
The @CTRL-C@ part completed without regression. 3-way tests was started in a separate session. The results file on sh... Eugenie Lyzenko
07:56 AM User Interface Feature #2478: button image support
Finally I have got the main part completed without regression. Starting the @CTRL-C@ tests. Eugenie Lyzenko
04:51 PM User Interface Feature #2479: DEFINE IMAGE and image phrase support for both button and image
This update for your review includes conversion support implementation candidate for @size-phrase@ and @image-phrase@... Eugenie Lyzenko
05:35 AM User Interface Feature #2479: DEFINE IMAGE and image phrase support for both button and image
Yes, please implement this. Greg Shah
05:25 AM User Interface Feature #2479: DEFINE IMAGE and image phrase support for both button and image
Question. Do we need th support for @FROM {X n Y n} | {ROW n COLUMN n}@ clause of the @image phrase@?
This sets th...
Eugenie Lyzenko
03:31 PM User Interface Feature #1790: implement menu support
I did as you advised. I used @menu.setVisible(true)@ that calls @repaint()@ method, that in turn call @EventManager.p... Vadim Gindin
11:22 AM User Interface Feature #1790: implement menu support
Greg Shah wrote:
> 1. I think your drawing issues are related to how you are directly calling @menu.draw()@ in @Thi...
Constantin Asofiei
08:44 AM User Interface Feature #1790: implement menu support
> but still got mbar.getSelf().readOnlyError("popup-only");.
You have to add the attribute into the writable attri...
Greg Shah
08:06 AM User Interface Feature #1790: implement menu support
# I've added POPUP-ONLY to @load_descriptiors@ function along with other menu related attributes, created @MenuInterf... Vadim Gindin
03:27 PM User Interface Feature #2498: implement ADD-LAST() method
Additional details about COMBO-BOX widget behaviour in presence of duplicated/"unknown" labels/values:
1. empty labe...
Igor Skornyakov
08:41 AM User Interface Feature #2498: implement ADD-LAST() method
> May be it makes sense to fix the implementation of ADD-FIRST as well (the current implementation also doesn't show ... Greg Shah
07:48 AM User Interface Feature #2498: implement ADD-LAST() method
If "unknown value is specified for @ADD-LAST(items)@ method it is accepted but on the screen a blank line is shown (i... Igor Skornyakov
12:40 PM Base Language Bug #2509 (Closed): problem converting INPUT x @ y constructs
Greg Shah
12:27 PM Base Language Bug #2509: problem converting INPUT x @ y constructs
P2J incorrectly resolves the frame of a @INPUT i@ expression, in a case like this:... Constantin Asofiei
12:23 PM Base Language Bug #2509 (Closed): problem converting INPUT x @ y constructs
Constantin Asofiei
08:16 AM User Interface Bug #2497: ANY-PRINTABLE only works for specific widget triggers, but not frame or global anywhere
LE: the trigger in note 22 was changed to be for the frame, not widget. Constantin Asofiei
08:12 AM User Interface Bug #2497: ANY-PRINTABLE only works for specific widget triggers, but not frame or global anywhere
I'm noting this here, but is a bug, not a regression; the following case doesn't work in P2J:... Constantin Asofiei
06:56 AM User Interface Bug #2497: ANY-PRINTABLE only works for specific widget triggers, but not frame or global anywhere
The changes are doing well in regression testing. I got timeout failures in tc_item_master_006, tc_item_master_104 a... Greg Shah
05:21 AM Base Language Bug #2507 (Closed): format validation with editing blocks
Greg Shah
01:11 AM Base Language Bug #2507: format validation with editing blocks
Greg Shah wrote:
> Code Review ca_upd20150129b.zip
>
> I'm fine with the changes.
Passed runtime testing, re...
Constantin Asofiei

01/29/2015

06:20 PM User Interface Feature #1790: implement menu support
Code Review vig_upd20150130a.zip
I think it is moving in the right direction.
1. I think your drawing issues ar...
Greg Shah
03:54 PM User Interface Feature #1790: implement menu support
The next update, containing the following changes.
# The first working (in testing mode) variant with popup-menu.
#...
Vadim Gindin
08:29 AM User Interface Feature #1790: implement menu support
OK. Keep this in mind when you are working on getting the event processing to work properly. Greg Shah
05:38 AM User Interface Feature #1790: implement menu support
Greg Shah wrote:

> Can you create a trigger on this event and use no-apply? Something like this:
>
> [...]
>...
Vadim Gindin
05:51 PM Base Language Bug #2499: DST portion of the timezone offset is always included when it should be conditional
Code Review om_upd20150129b.zip
There are no changes in the @datetime.java@ that was in the update zip. Otherwise...
Greg Shah
02:44 PM Base Language Bug #2499: DST portion of the timezone offset is always included when it should be conditional
This is most likely an RC.
I only done the 'theoretical' checking on @datetime@ and @datetimetz@. I am preparing no...
Ovidiu Maxiniuc
05:31 PM User Interface Feature #2252: implement GUI client support
Greg Shah wrote:
> Code Review hc_upd20150129b.zip
>
> My review of ZCL is not very very deep. But overall, I'm ...
Hynek Cihlar
05:07 PM User Interface Feature #2252: implement GUI client support
Code Review hc_upd20150129b.zip
My review of ZCL is not very very deep. But overall, I'm fine with the update. O...
Greg Shah
10:51 AM User Interface Feature #2252: implement GUI client support
Constantin Asofiei wrote:
>
> After re-enabling this code for the case oldLbl is null, the NPE is no longer there....
Hynek Cihlar
05:43 AM User Interface Feature #2252: implement GUI client support
Hynek Cihlar wrote:
> > > An update, the NPE is caused by my changes. I have changed the side-label management sligh...
Constantin Asofiei
05:06 AM User Interface Feature #2252: implement GUI client support
Constantin Asofiei wrote:
> Hynek Cihlar wrote:
> > Constantin Asofiei wrote:
> > > Hynek Cihlar wrote:
> > > > Const...
Hynek Cihlar
04:52 AM User Interface Feature #2252: implement GUI client support
Hynek Cihlar wrote:
> Constantin Asofiei wrote:
> > Hynek Cihlar wrote:
> > > Constantin, this also touches some o...
Constantin Asofiei
04:18 AM User Interface Feature #2252: implement GUI client support
Constantin Asofiei wrote:
> Hynek Cihlar wrote:
> > Constantin, this also touches some of your side-label changes, ...
Hynek Cihlar
03:24 AM User Interface Feature #2252: implement GUI client support
Hynek Cihlar wrote:
> Constantin, this also touches some of your side-label changes, please have a look too. Also no...
Constantin Asofiei
02:44 AM User Interface Feature #2252: implement GUI client support
Replacement for ca_upd20150123a.zip, side-labels in shared frames.
Passed runtime testing, released to rev 10725.
Constantin Asofiei
08:51 PM User Interface Feature #2252: implement GUI client support
The attached update adds support for moving frame widgets by setting col/row/x/y attributes from application code. Th... Hynek Cihlar
04:37 PM User Interface Feature #2508: test LIST-ITEMS support in P2J and confirm that it is complete, add or fix feature...
This attribute is available on selection list, combo-box and browse column. Please write use cases to test it, inclu... Greg Shah
04:33 PM User Interface Feature #2508 (Closed): test LIST-ITEMS support in P2J and confirm that it is complete, add or fi...
Greg Shah
04:28 PM Base Language Bug #2507: format validation with editing blocks
Code Review ca_upd20150129b.zip
I'm fine with the changes.
Greg Shah
03:57 PM Base Language Bug #2507: format validation with editing blocks
> This is going through runtime testing now.
CTRL-C passed, MAIN part was killed and restarted, as after ~5 hours ...
Constantin Asofiei
10:52 AM Base Language Bug #2507: format validation with editing blocks
This is replacement of 0128a.zip. Enhances the support for character formats and is a good step forward, but there ar... Constantin Asofiei
04:11 PM User Interface Bug #2497: ANY-PRINTABLE only works for specific widget triggers, but not frame or global anywhere
> there are the testcases/uast/*focus*.p tests.
Good idea. I'll see if they can be used to test this out.
Greg Shah
04:09 PM User Interface Bug #2497: ANY-PRINTABLE only works for specific widget triggers, but not frame or global anywhere
Greg Shah wrote:
> I wonder if it is worth testing my second approach with MAJIC.
Yes, I think this one is OK to te...
Constantin Asofiei
03:57 PM User Interface Bug #2497: ANY-PRINTABLE only works for specific widget triggers, but not frame or global anywhere
I've tried the following idea:... Greg Shah
03:54 PM User Interface Bug #2497: ANY-PRINTABLE only works for specific widget triggers, but not frame or global anywhere
Greg Shah wrote:
> That is a strange result. I wouldn't have expected that without NO-APPLY. The entry/leave proce...
Constantin Asofiei
03:37 PM User Interface Bug #2497: ANY-PRINTABLE only works for specific widget triggers, but not frame or global anywhere
That is a strange result. I wouldn't have expected that without NO-APPLY. The entry/leave processing must be respon... Greg Shah
03:26 PM User Interface Bug #2497: ANY-PRINTABLE only works for specific widget triggers, but not frame or global anywhere
@refocusTarget@ might have been used to solve a case like this:... Constantin Asofiei
02:43 PM User Interface Bug #2497: ANY-PRINTABLE only works for specific widget triggers, but not frame or global anywhere
The change to add @refocusTarget()@ came in rev 9907 (H649 on 20101015 by SIY). There is no indication of why this w... Greg Shah
02:26 PM User Interface Bug #2497: ANY-PRINTABLE only works for specific widget triggers, but not frame or global anywhere
I stepped over that @refocusTarget()@ call because I knew the target widget was already in focus.
I don't fully un...
Greg Shah
01:25 PM User Interface Bug #2497: ANY-PRINTABLE only works for specific widget triggers, but not frame or global anywhere
Greg Shah wrote:
> Part of the problem is that the screen refreshes are deferred so I can't see when the cursor move...
Constantin Asofiei
01:02 PM User Interface Bug #2497: ANY-PRINTABLE only works for specific widget triggers, but not frame or global anywhere
This update fixes the trigger recursion but the cursor is still always forced back to the first char position. I hav... Greg Shah
07:11 PM User Interface Bug #2497: ANY-PRINTABLE only works for specific widget triggers, but not frame or global anywhere
Actually, we do have generic protection logic for trigger recursion. It is located in @LogicalTerminal.trigger()@:
...
Greg Shah
02:46 PM Database Bug #2488: replace runtime compilation of dynamic queries
This update fixes the copy/paste bug revealed by issues #2500 and #2501. Ovidiu Maxiniuc
02:36 PM User Interface Feature #2494 (Closed): DELETE() method runtime implementation
Greg Shah
02:34 PM User Interface Feature #2494: DELETE() method runtime implementation
Passed runtime regression testing and checked in as bzr rev 10727. Igor Skornyakov
02:05 PM User Interface Feature #2494: DELETE() method runtime implementation
You can check in the update and send the associated notification email.
Please make sure you have no other changes...
Greg Shah
01:48 PM User Interface Feature #2494: DELETE() method runtime implementation
Igor Skornyakov wrote:
> Actually the is one test which failed in exactly the same way in both runs:
>
> failure ...
Constantin Asofiei
01:44 PM User Interface Feature #2494: DELETE() method runtime implementation
Actually the is one test which failed in exactly the same way in both runs:
failure in step 40: 'Unexpected EOF in...
Igor Skornyakov
01:05 PM User Interface Feature #2494: DELETE() method runtime implementation
Are there any specific tests that failed in both runs (even though the failure was at a different point)? The only w... Greg Shah
12:41 PM User Interface Feature #2494: DELETE() method runtime implementation
Second run of the regression test failed:
2 gso_tests Specific GSO testcases to regression test. FAILED NONE gs...
Igor Skornyakov
08:34 AM User Interface Feature #2494: DELETE() method runtime implementation
Merged with recent changes Igor Skornyakov
01:05 PM User Interface Feature #2498: implement ADD-LAST() method
Yes and yes. Greg Shah
12:28 PM User Interface Feature #2498: implement ADD-LAST() method
I have a question.
As far as I understand there is no "null" value in ABL - it has "unknown" (literal ?) instead. If...
Igor Skornyakov
07:02 PM User Interface Feature #2478: button image support
Conversion testing completed. The codes are identical. Starting the runtime testing. Eugenie Lyzenko

01/28/2015

05:21 PM User Interface Feature #2494: DELETE() method runtime implementation
1 tests failed:
2 gso_tests Specific GSO testcases to regression test. FAILED NONE gso_tests 2314.617 ...
Igor Skornyakov
11:16 AM User Interface Feature #2494: DELETE() method runtime implementation
Greg Shah wrote:
> Code Review ias_upd20150128a.zip (which is really ias_upd20150128b.zip)
>
> I'm OK with the ch...
Igor Skornyakov
11:09 AM User Interface Feature #2494: DELETE() method runtime implementation
Code Review ias_upd20150128a.zip (which is really ias_upd20150128b.zip)
I'm OK with the changes. Please get them ...
Greg Shah
11:08 AM User Interface Feature #2494: DELETE() method runtime implementation
Sorry, I wasn't looking carefully enough. I see that you defined your own interface to match the concept of the @Fun... Greg Shah
11:07 AM User Interface Feature #2494: DELETE() method runtime implementation
Also, please don't upload modified updates as the same update zip file name. The update in note 80 is your second up... Greg Shah
11:05 AM User Interface Feature #2494: DELETE() method runtime implementation
I don't understand. Isn't @Function@ a Java 8 thing or a Guava thing? Neither should be used in our code. Where in... Greg Shah
10:24 AM User Interface Feature #2494: DELETE() method runtime implementation
Greg Shah wrote:
> 1. We do not yet support Java 8. We want to do so, but it takes some real planning because it re...
Igor Skornyakov
09:55 AM User Interface Feature #2494: DELETE() method runtime implementation
Code Review ias_upd20150128a.zip
1. We do not yet support Java 8. We want to do so, but it takes some real planni...
Greg Shah
06:27 AM User Interface Feature #2494: DELETE() method runtime implementation
Java code updated: use value instead of label in the @DELETE(list-items)@ for COMBO-BOX and SELECTION-LIST widgets.
Igor Skornyakov
05:11 PM User Interface Feature #2498: implement ADD-LAST() method
Do you have testcases for the errors and edge cases?
Please collect all of your testcases and check them into the ...
Greg Shah
04:57 PM User Interface Feature #2498: implement ADD-LAST() method
Greg Shah wrote:
> Please check how this works with parameters that are unknown value and empty string.
>
> Pleas...
Igor Skornyakov
04:48 PM User Interface Feature #2498: implement ADD-LAST() method
Please check how this works with parameters that are unknown value and empty string.
Please also check how it work...
Greg Shah
03:36 PM User Interface Feature #2498: implement ADD-LAST() method
Greg,
1. Can I start Java runtime implementation?
2. Should I try to implement the described "strange" behaviour if...
Igor Skornyakov
03:16 PM User Interface Feature #2498: implement ADD-LAST() method
Sample 4GL programs Igor Skornyakov
01:55 PM User Interface Feature #2498: implement ADD-LAST() method
The RADIO-SET widget has only @ADD-LAST(label, value)@ method.
It just adds the new label/value to the widget.
If AUT...
Igor Skornyakov
10:42 AM User Interface Feature #2498: implement ADD-LAST() method
For COMBO-BOX and SELECTION-LIST:
1. If LIST-ITEM-PAIRS is set then @ADD-LAST(items)@ is rejected with two warnings (...
Igor Skornyakov
05:06 PM User Interface Feature #1790: implement menu support
> Assume we have a button with popup-menu assigned. We must press ESC-U for CHUI or F4 for WINDOWS to call this menu ... Greg Shah
04:37 PM User Interface Feature #1790: implement menu support
I also want to add, that I faced with conversion bug for frames during conversion of @chui_menu_2_purpose.p@. Extra f... Vadim Gindin
04:31 PM User Interface Feature #1790: implement menu support
I need an advice. Assume we have a button with popup-menu assigned. We must press ESC-U for CHUI or F4 for WINDOWS to... Vadim Gindin
03:02 PM User Interface Feature #1790: implement menu support
I'm working on showing POPUP-MENU for widgets. I'd like to add this too.
About @MenuElementConfig@. There is some ...
Vadim Gindin
12:56 PM User Interface Feature #1790: implement menu support
I'd like to see this next set of functionality stabilized and put into testing. What is left to do? Greg Shah
12:55 PM User Interface Feature #1790: implement menu support
Code Review vig_upd20150128a.zip
Overall, this is MUCH better than the previous approach. Well done.
1. Menus n...
Greg Shah
04:52 PM Base Language Bug #2499: DST portion of the timezone offset is always included when it should be conditional
I'm good with this approach if you find the the @datetime@ and @datetimetz@ are OK too. Greg Shah
04:37 PM Base Language Bug #2499 (WIP): DST portion of the timezone offset is always included when it should be conditional
Most likely, the code inside @getDefaultOffset@ should be fixed:... Ovidiu Maxiniuc
04:21 PM Database Bug #2488 (WIP): replace runtime compilation of dynamic queries
Added 'dumb' cache implementation and some additional fixes. Ovidiu Maxiniuc
04:03 PM User Interface Bug #2497: ANY-PRINTABLE only works for specific widget triggers, but not frame or global anywhere
This update (which does make ANY-PRINTABLE more widely functional), can also cause problems. This code that applies ... Greg Shah
03:01 PM User Interface Feature #2478: button image support
>It looks good. Please do get it tested.
OK. Starting the conversion testing stage(we have the rule changed).
Eugenie Lyzenko
02:56 PM User Interface Feature #2478: button image support
Code Review evl_upd20150128a.zip
It looks good. Please do get it tested.
Greg Shah
02:50 PM User Interface Feature #2478: button image support
Reworked image loading support.
- New interfaces added, reworked conversion rule for methods.
- bit change for imag...
Eugenie Lyzenko
12:56 PM User Interface Feature #2478: button image support
> So we can run testing now or postpone to the time the interface rework is ready. What do you think?
Do the rewor...
Greg Shah
12:25 PM User Interface Feature #2478: button image support
>This update seems pretty close to something that can go into regression testing. Is there anything else needed excep... Eugenie Lyzenko
12:13 PM User Interface Feature #2478 (WIP): button image support
> Is it answering to your question?
Yes.
This update seems pretty close to something that can go into regressio...
Greg Shah
11:24 AM User Interface Feature #2478: button image support
>1. There are no real changes in ButtonWidget, just a history entry. I think that can be removed safely. :)
OK. Done...
Eugenie Lyzenko
11:01 AM User Interface Feature #2478: button image support
Code Review evl_upd20150127a.zip
I like it.
1. There are no real changes in @ButtonWidget@, just a history entr...
Greg Shah
08:27 PM User Interface Feature #2478: button image support
Small update to add method interfaces to support dynamic button/image widgets. Eugenie Lyzenko
11:55 AM Base Language Bug #2507: format validation with editing blocks
Greg Shah wrote:
> Code Review ca_upd20150128a.zip
>
> My only concern is with the use of @checkFormat()@ instead...
Constantin Asofiei
08:31 AM Base Language Bug #2507: format validation with editing blocks
Code Review ca_upd20150128a.zip
My only concern is with the use of @checkFormat()@ instead of @validateFormat()@ o...
Greg Shah
04:19 AM Base Language Bug #2507: format validation with editing blocks
From what I see, the format validation is done when LEAVE is sent for the widget. More, when EDITING mode is enabled... Constantin Asofiei
04:05 AM Base Language Bug #2507: format validation with editing blocks
Constantin,
Consider this:...
Constantin Asofiei
04:04 AM Base Language Bug #2507 (Closed): format validation with editing blocks
Constantin Asofiei

01/27/2015

03:46 PM User Interface Feature #2494: DELETE() method runtime implementation
It appears that my understanding of the DELETE() method semantics was incorrect -the @DELETE(list-items)@ for COMBO-B... Igor Skornyakov
08:47 AM User Interface Feature #2494: DELETE() method runtime implementation
Code Review ias_upd20150127a.zip
The changes look good.
I agree that the testing failures are most likely not d...
Greg Shah
06:00 AM User Interface Feature #2494: DELETE() method runtime implementation
Some regression tests failed:
1. 1. failure in step 13: 'timeout before the specific screen buffer became available ...
Igor Skornyakov
02:31 PM User Interface Feature #1790: implement menu support
1) I've prepared the next update with separate menu description class. Does it preferable than previous updates with ... Vadim Gindin
04:26 AM User Interface Bug #2495: APPLY event TO handle does not work
ca_upd20150127a.zip passed conversion testing and was released to bzr rev 10723 Constantin Asofiei
04:26 AM User Interface Bug #2495: APPLY event TO handle does not work
Fixed some conversion issues related to APPLY TO handle.
Anything which has a frame-id needs to convert as a Gener...
Constantin Asofiei
08:29 PM User Interface Feature #2478: button image support
Had to modify @CommonWidget/GenericWidget@ classes to add @loadImage*()@ methods support to handle dynamically alloca... Eugenie Lyzenko

01/26/2015

02:44 PM User Interface Feature #2494: DELETE() method runtime implementation
Thank you. Igor Skornyakov
02:24 PM User Interface Feature #2494: DELETE() method runtime implementation
Igor Skornyakov wrote:
> I tried to run regression test, but it failed (runtime-regression).
The problem is with y...
Constantin Asofiei
02:22 PM User Interface Feature #2494: DELETE() method runtime implementation
Got it. Thank you for your patience. I've configured Eclipse to avoid specific explicit imports. BTW: what about stat... Igor Skornyakov
01:37 PM User Interface Feature #2494: DELETE() method runtime implementation
Code Review ias_upd20150124d.zip
Almost there. 5 small problems are remaining. Sorry that we are so specific in o...
Greg Shah
06:51 AM User Interface Feature #2494: DELETE() method runtime implementation
I'm very sorry, but I've dropped the entire content of the repo by a stupid mistake. How this can be repaired?
Igor Skornyakov
06:08 AM User Interface Feature #2494: DELETE() method runtime implementation
Something is wrong with your repo, see this like:... Constantin Asofiei
06:03 AM User Interface Feature #2494: DELETE() method runtime implementation
I've tried to run regression testing but the compilation failed. Please find the build logs attached. As I can see th... Igor Skornyakov
02:42 PM Base Language Bug #2499: DST portion of the timezone offset is always included when it should be conditional
Some ideas about automating the DST testing (in junits or date.main):
> Ovidiu Maxiniuc wrote:
Java 8 has some new A...
Constantin Asofiei
12:35 PM Base Language Feature #1628: fix any incompatibilities or missing features of NO-ECHO support
Evgeny Kiselev wrote:
> My changes in @Window@ class is issue for gso_276 test. Looking for approach.
Did you int...
Greg Shah
12:34 PM Base Language Feature #1628: fix any incompatibilities or missing features of NO-ECHO support
Code Review evk_upd20150126a.zip
1. Please remove @import org.apache.commons.logging.*;@ from @Stream.java@.
2....
Greg Shah
07:46 AM Base Language Feature #1628: fix any incompatibilities or missing features of NO-ECHO support
My changes in @Window@ class is issue for gso_276 test. Looking for approach. Evgeny Kiselev
09:01 PM Base Language Feature #1628: fix any incompatibilities or missing features of NO-ECHO support
Tests gso_17 and gso_488 has been fixed in @Stream@ class. Even if output is unformatted we still need to use formatt... Evgeny Kiselev
12:01 PM Runtime Infrastructure Feature #2506: provide support for substitution variables which can be inserted in PROPATH entrie...
In the 4GL, the PROPATH is often specified in scripts which modify the environment before launching the Progress clie... Greg Shah
11:53 AM Runtime Infrastructure Feature #2506 (New): provide support for substitution variables which can be inserted in PROPATH ...
Greg Shah
11:51 AM Base Language Feature #2505: SEARCH and loading of static resources from the server-side application jar
Today our SEARCH and any file reading occurs exclusively on the client side. As part of #2476 we are implementing su... Greg Shah
11:47 AM Base Language Feature #2505 (New): SEARCH and loading of static resources from the server-side application jar
Greg Shah
11:46 AM Base Language Feature #2504: real environment access for OS-GETENV
Today we map OS-GETENV to the JVM @System.getProperty()@ which does not real from real environment variables, but ins... Greg Shah
11:40 AM Base Language Feature #2504 (Closed): real environment access for OS-GETENV
Greg Shah
11:38 AM Conversion Tools Feature #2503: improve ConversionDriver.main() to provide back a return code and/or better handle...
From a customer:
> Conversion failed last night (because we checked something in that we shouldn't have done - a c...
Greg Shah
11:37 AM Conversion Tools Feature #2503 (WIP): improve ConversionDriver.main() to provide back a return code and/or better ...
Greg Shah
11:35 AM Conversion Tools Feature #2502: using a file list as input for the ConversionDriver fails if there is trailing space
Even a single trailing space will cause the file to be silently not found, leading to unexpected results. I haven't ... Greg Shah
11:33 AM Conversion Tools Feature #2502 (New): using a file list as input for the ConversionDriver fails if there is traili...
Greg Shah
03:14 AM Database Bug #2488: replace runtime compilation of dynamic queries
Eric,
If you did not started the review, please find attached an update that adds the missing javadocs to @RuntimeJa...
Ovidiu Maxiniuc

01/24/2015

02:53 PM User Interface Bug #2497 (Closed): ANY-PRINTABLE only works for specific widget triggers, but not frame or globa...
Passed regression testing and checked in as bzr rev 10720. Greg Shah
07:56 PM User Interface Feature #2478: button image support
This updates has code merged with recent code base.
One interesting thing I've found. If the method @LOAD-IMAGE()@...
Eugenie Lyzenko

01/23/2015

06:00 PM User Interface Feature #1790: implement menu support
I've fixed this bug. It was some sort of serialization bug. I also fixed a drawing of menu-item labels for menubar. H... Vadim Gindin
12:06 PM User Interface Feature #1790: implement menu support
Hopefully you are continuing to debug. I have no idea what is causing the problem. Greg Shah
10:02 AM User Interface Feature #1790: implement menu support
I did it and nevertheless the constructor of @SilentUnwindException@ is called from client-side with the error @com.g... Vadim Gindin
09:31 AM User Interface Feature #1790: implement menu support
I can't tell anything useful from the logs. You'll have to set breakpoints in the @SilentUnwindException@ constructo... Greg Shah
09:27 AM User Interface Feature #1790: implement menu support
In the future, when posting something that isn't large, please post it in-line here. Don't attach a file for such th... Greg Shah
09:23 AM User Interface Feature #1790: implement menu support
I need some advice. I got an error, that happen right in the moment of client call. I've filled @ScreenDefinition@ wi... Vadim Gindin
04:24 PM User Interface Feature #2494: DELETE() method runtime implementation
Done. I did it with meld - it appears to be almost as good as Araxis Merge I used before. Igor Skornyakov
04:14 PM User Interface Feature #2494: DELETE() method runtime implementation
The class definition itself has been changed:... Greg Shah
03:56 PM User Interface Feature #2494: DELETE() method runtime implementation
I see tow issues.
1. Invalid braces in constructor (fixed)
2. Reformatted comments/Javadocs.
I can revert my chang...
Igor Skornyakov
03:39 PM User Interface Feature #2494: DELETE() method runtime implementation
Code Review ias_upd20150123c.zip
> I'm sorry.
No worries. Learning the coding standards is part of the ramp-up...
Greg Shah
03:07 PM User Interface Feature #2494: DELETE() method runtime implementation
Oh, I'm sorry. It appears that one have to set this in two places in Eclipse now. Igor Skornyakov
02:44 PM User Interface Feature #2494: DELETE() method runtime implementation
Code Review ias_upd20150123b.zip
Actually, this update is very hard to compare with the first for @ComboBoxWidget@...
Greg Shah
02:27 PM User Interface Feature #2494: DELETE() method runtime implementation
Greg Shah wrote:
> FYI, we prefer to minimize the linkages because there can be unintended consequences. For exampl...
Igor Skornyakov
02:06 PM User Interface Feature #2494: DELETE() method runtime implementation
FYI, we prefer to minimize the linkages because there can be unintended consequences. For example, some of our code ... Greg Shah
02:05 PM User Interface Feature #2494: DELETE() method runtime implementation
Sadly, I see that it has also sneaked its way into @LogicalTerminal@ and @ChuiWebPageHandler@. We'll need to clean t... Greg Shah
02:00 PM User Interface Feature #2494: DELETE() method runtime implementation
Igor Skornyakov wrote:
> Got it.
> Regarding 3rd party dependencies. I used Google Guava classes only because the c...
Constantin Asofiei
01:53 PM User Interface Feature #2494: DELETE() method runtime implementation
Got it.
Regarding 3rd party dependencies. I used Google Guava classes only because the corresponding jar is already ...
Igor Skornyakov
01:14 PM User Interface Feature #2494: DELETE() method runtime implementation
Code Review ias_upd20150123.zip
Functionally, it is quite good. Most of the feedback relates to coding standards....
Greg Shah
12:54 PM User Interface Feature #2494: DELETE() method runtime implementation
Please find updated classes attached. Igor Skornyakov
12:27 PM User Interface Feature #2494: DELETE() method runtime implementation
That should not be needed.
OK, so you should do the following:
1. Upload the update zip file here so that we ca...
Greg Shah
12:19 PM User Interface Feature #2494: DELETE() method runtime implementation
Thank you - it works. I've finished testing in ChUI mode. Shall I test on Windows as well? Igor Skornyakov
11:59 AM User Interface Feature #2494: DELETE() method runtime implementation
It's not the best way, but for now you can use this: @frame.getName(this)@ from within a widget. This corresponds to... Greg Shah
10:55 AM User Interface Feature #2494: DELETE() method runtime implementation
I've started testing in ChUI mode. Basically all works, but there is one minor(?) issues:
The type() return not th...
Igor Skornyakov
09:44 AM User Interface Feature #2494: DELETE() method runtime implementation
It's OK - I understand. Thank you, Igor Skornyakov
09:43 AM User Interface Feature #2494: DELETE() method runtime implementation
Good. Sorry for the old docs. We are busy and haven't gotten them updated. And those docs reference an internal bz... Greg Shah
09:35 AM User Interface Feature #2494: DELETE() method runtime implementation
Yes, it works, thank you. Igor Skornyakov
09:23 AM User Interface Feature #2494: DELETE() method runtime implementation
The documentation is out of date. The @ServerDriver@ itself has no understanding of a @-z@ parameter. The @server.s... Greg Shah
09:12 AM User Interface Feature #2494: DELETE() method runtime implementation
I got the command from /books/conversion_handbook/p2j_conversion_handbook.pdf page 221 Igor Skornyakov
09:03 AM User Interface Feature #2494: DELETE() method runtime implementation
I don't fully understand what you are saying you did or what the problem is.
When I run a server for my converted ...
Greg Shah
06:43 AM User Interface Feature #2494: DELETE() method runtime implementation
I've tried to launch the P2J Server as described in the "P2J Conversion Handbook", but got an error @"Too many config... Igor Skornyakov
03:46 PM User Interface Bug #2497: ANY-PRINTABLE only works for specific widget triggers, but not frame or global anywhere
No, here it is. Greg Shah
03:45 PM User Interface Bug #2497: ANY-PRINTABLE only works for specific widget triggers, but not frame or global anywhere
Good catch. Here is the version with javadoc. Greg Shah
03:03 PM User Interface Bug #2497: ANY-PRINTABLE only works for specific widget triggers, but not frame or global anywhere
Greg Shah wrote:
> Merged update to bzr 10719 which includes the fix from #2495. I'm going to put this into testing...
Constantin Asofiei
01:45 PM User Interface Bug #2497: ANY-PRINTABLE only works for specific widget triggers, but not frame or global anywhere
Merged update to bzr 10719 which includes the fix from #2495. I'm going to put this into testing. Greg Shah
01:57 PM Base Language Bug #2499: DST portion of the timezone offset is always included when it should be conditional
I took a quick look at this. Outside the util package, @date.getDefaultOffset()@ is used from @persist/HQLPreprocess... Greg Shah
01:51 PM Base Language Bug #2499: DST portion of the timezone offset is always included when it should be conditional
TIMCO reported a problem with transactions starting at midnight. Constantin tracked the problem back to the TODAY fu... Greg Shah
01:46 PM Base Language Bug #2499 (WIP): DST portion of the timezone offset is always included when it should be conditional
Greg Shah
01:34 PM User Interface Bug #2495 (Closed): APPLY event TO handle does not work
Greg Shah
01:29 PM User Interface Bug #2495 (Review): APPLY event TO handle does not work
Greg Shah wrote:
> Please check it in.
Released to rev 10719.
Constantin Asofiei
08:28 AM User Interface Bug #2495: APPLY event TO handle does not work
Please check it in. Greg Shah
07:13 AM User Interface Bug #2495: APPLY event TO handle does not work
Greg Shah wrote:
> Code Review ca_upd20150122c.zip
>
> It looks good.
0122c.zip passed runtime/conversion testing.
Constantin Asofiei
12:31 PM User Interface Feature #2498: implement ADD-LAST() method
This method is available on selection list, combo-box, radio-set and browse column. Follow the same approach as for ... Greg Shah
12:30 PM User Interface Feature #2498 (Closed): implement ADD-LAST() method
Greg Shah
10:05 AM User Interface Feature #2478: button image support
This update for review adds @DEFINE IMAGE@ handling for deferred data loading for image widget.
Continue working.
Eugenie Lyzenko
09:29 AM Database Bug #2488: replace runtime compilation of dynamic queries
Greg, you are right.
We do parse the initial query using the @progress.g@ (@ProgressParser@ from the code in note ...
Ovidiu Maxiniuc
08:58 AM Database Bug #2488: replace runtime compilation of dynamic queries
When I refer to the Progress AST, I mean the original AST that is first created for the provided expression in a @QUE... Greg Shah
08:55 AM Database Bug #2488: replace runtime compilation of dynamic queries
A few thoughts on the caching:
I agree with Greg that the Progress AST will be much more reliable. However, it is ...
Eric Faulhaber
08:53 AM Database Bug #2488: replace runtime compilation of dynamic queries
Not sure I understand. Of course, the trees will be the except for the string value at some point. But later, during ... Ovidiu Maxiniuc
08:26 AM Database Bug #2488: replace runtime compilation of dynamic queries
> and realize that they are the same ?
Don't forget that in the Progress AST the tree structure is in fact:
* ...
Greg Shah
05:01 AM Database Bug #2488: replace runtime compilation of dynamic queries
I analyzed the logs with the queries from #1868. Here are some results/thoughts:
* there are ~5500 @FIND FIRST@ of w...
Ovidiu Maxiniuc
09:29 AM Base Language Feature #1628: fix any incompatibilities or missing features of NO-ECHO support
>Eugenie Lyzenko: is there anything about GSO 488 which was not captured in the 4GL?
I think yes. The 4GL system @...
Eugenie Lyzenko
08:39 AM Base Language Feature #1628: fix any incompatibilities or missing features of NO-ECHO support
Code Review evk_upd20150121a.zip
The changes are fine. In the future, please post the update without the @01_@ pr...
Greg Shah
07:46 AM Base Language Feature #1628: fix any incompatibilities or missing features of NO-ECHO support
Greg Shah wrote:
> Are the ChUI-specific changes to @Window.message()@ still needed (to set the cursor position to c...
Evgeny Kiselev
09:18 AM User Interface Feature #2252: implement GUI client support
Code Review ca_upd20150123a.zip
The changes look good. The result is cleaner too from a design perspective, so it...
Greg Shah
07:48 AM User Interface Feature #2252: implement GUI client support
Fix for side-label in shared frames. Built on top of #2495 ca_upd20150122c.zip Constantin Asofiei

01/22/2015

06:15 PM User Interface Feature #2478: button image support
This update for review contains prototype for deferred image loading in button widget. Some notes:
1. There is only ...
Eugenie Lyzenko
05:55 PM User Interface Bug #2497: ANY-PRINTABLE only works for specific widget triggers, but not frame or global anywhere
The testcase is checked in as @testcases/uast/any_printable_event.p@. Greg Shah
05:53 PM User Interface Bug #2497: ANY-PRINTABLE only works for specific widget triggers, but not frame or global anywhere
This is the fix. It ensures that ANY-PRINTABLE and ANY-KEY are searched on each pass through (specific-widget, frame... Greg Shah
05:49 PM User Interface Bug #2497: ANY-PRINTABLE only works for specific widget triggers, but not frame or global anywhere
The following testcase illustrates this problem:... Greg Shah
05:48 PM User Interface Bug #2497 (Closed): ANY-PRINTABLE only works for specific widget triggers, but not frame or globa...
Greg Shah
04:56 PM Database Bug #2488: replace runtime compilation of dynamic queries
Uploaded update for review, together a file with my important testcases.
* I tried to detect as much as possible i...
Ovidiu Maxiniuc
06:02 AM Database Bug #2488: replace runtime compilation of dynamic queries
After I got satisfactory results with simpler queries, I started to increase the complexity of the dynamic queries in... Ovidiu Maxiniuc
03:38 PM Database Bug #2496: Static conversion of legacy field names does not preserve original case
Legacy field names annotations for static conversion are now in low case. TableMapper seems works properly, also ther... Vadim Nebogatov
03:07 PM Database Bug #2496 (New): Static conversion of legacy field names does not preserve original case
Vadim Nebogatov
02:20 PM User Interface Feature #2494: DELETE() method runtime implementation
Greg Shah wrote:
> For this task I think you can ignore the browse column aspects.
OK. Thank you.
Igor Skornyakov
02:17 PM User Interface Feature #2494: DELETE() method runtime implementation
> If I understand correctly this means that in the scope of this task there is nothing to do specifically for the BRO... Greg Shah
02:11 PM User Interface Feature #2494: DELETE() method runtime implementation
Here it is Igor Skornyakov
02:09 PM User Interface Feature #2494: DELETE() method runtime implementation
> If I understand correctly this means that in the scope of this task there is nothing to do specifically for the BRO... Greg Shah
01:43 PM User Interface Feature #2494: DELETE() method runtime implementation
Please note the generated code for the BROWSE widget contains nothing related to a combo-box column except @addWidget... Igor Skornyakov
01:22 PM User Interface Feature #2494: DELETE() method runtime implementation
Indeed - after commenting out p2j_test namespace the browse2.p converted OK (even with @f0+cb@)
Thanks a lot!
Igor Skornyakov
01:20 PM User Interface Feature #2494: DELETE() method runtime implementation
Greg Shah wrote:
> The "b" reference in the define frame is being matched with a field in the book table of p2j_test...
Eric Faulhaber
01:15 PM User Interface Feature #2494: DELETE() method runtime implementation
I guess that P2J is having a problem with @DEFINE FRAME f b.@ from line 10 of @r-browse2.p@, if that is the same prog... Greg Shah
12:55 PM User Interface Feature #2494: DELETE() method runtime implementation
Well, the conversion passed for combo, radio and select. I had to use @f2+m0+cb@ as @f0+cb@ disn't work (at some stag... Igor Skornyakov
12:39 PM User Interface Feature #2494: DELETE() method runtime implementation
Thank you very much.
Now the conversion started. However there are other problems at the later steps. I'll try to un...
Igor Skornyakov
11:37 AM User Interface Feature #2494: DELETE() method runtime implementation
I believe that the current @testcases/uast/@ environment was broken at some point by some kind of metadata -nonsense-... Greg Shah
11:28 AM User Interface Feature #2494: DELETE() method runtime implementation
Greg,
I cannot perform the conversion:
Exception in thread "main" java.lang.ExceptionInInitializerError
Caused by:...
Igor Skornyakov
09:57 AM User Interface Feature #2494: DELETE() method runtime implementation
> I understand that now I have to convert my 4GL sample programs and test. Is that correct?
Exactly right.
Greg Shah
09:25 AM User Interface Feature #2494: DELETE() method runtime implementation
I see. Thank you.
I've implemented @DELETE()@ for RADIO-SET, SELECTION-LIST and COMBO-BOX. I understand that now I...
Igor Skornyakov
08:44 AM User Interface Feature #2494: DELETE() method runtime implementation
Each resource type has a TYPE attribute. It comes from the @CommonHandle@ interface and is implemented for widgets a... Greg Shah
07:30 AM User Interface Feature #2494: DELETE() method runtime implementation
I have a question.
In 4GL warning contains a name of the widget - e.g. "The DELETE attribute on the SELECTION-LIST ...
Igor Skornyakov
01:37 PM User Interface Bug #2495: APPLY event TO handle does not work
Code Review ca_upd20150122c.zip
It looks good.
Greg Shah
01:19 PM User Interface Bug #2495: APPLY event TO handle does not work
Attached fixes the issues in notes 3 and 5. Testing is under way. Constantin Asofiei
12:52 PM User Interface Bug #2495: APPLY event TO handle does not work
> Should I continue fixing this?
Yes, if it can be done in a reasonable amount of time. I think it is time to get...
Greg Shah
12:37 PM User Interface Bug #2495: APPLY event TO handle does not work
There is this case also:... Constantin Asofiei
10:24 AM User Interface Bug #2495: APPLY event TO handle does not work
Greg Shah wrote:
> 3. In @LogicalTerminal.apply(WrappedResource res, double eventCode)@, could a non-widget/non-fram...
Constantin Asofiei
09:56 AM User Interface Bug #2495: APPLY event TO handle does not work
Code Review ca_upd20150122a.zip
This is really good. I like how you solved the ID lookup problem by using a diffe...
Greg Shah
09:00 AM User Interface Bug #2495: APPLY event TO handle does not work
-------- Forwarded Message --------
Subject: Re: common idiom which we don't handle properly
Date: Thu, 22 Jan 2015...
Greg Shah
08:59 AM User Interface Bug #2495: APPLY event TO handle does not work
I have found a common idiom which we don't handle properly. Here is an example:... Greg Shah
08:56 AM User Interface Bug #2495 (Closed): APPLY event TO handle does not work
Greg Shah

01/21/2015

06:17 PM User Interface Feature #2476: window icon support improvements
Committed in bzr as 10716. Eugenie Lyzenko
06:03 PM User Interface Feature #2476: window icon support improvements
Yes, go ahead. Greg Shah
05:47 PM User Interface Feature #2476: window icon support improvements
The @CTRL-C@ part completed without regressions. Had to start 3-way tests separately. The results file is on the shar... Eugenie Lyzenko
09:31 AM User Interface Feature #2476: window icon support improvements
The main part of the regression testing has been completed without regressions, starting the @CTRL-C@ tests. Eugenie Lyzenko
06:06 PM User Interface Feature #2478: button image support
OK, please take this into account in your design.
1. I think that the loadImage() methods should still be used by ...
Greg Shah
05:32 PM User Interface Feature #2478: button image support
>Eugenie: please determine the answer to this question.
The answer is the moment when button is realized is the po...
Eugenie Lyzenko
01:57 PM User Interface Feature #2478: button image support
> But if the image is loaded only when the frame (or button?) is realised, unless I've missed some discussion on the... Greg Shah
01:26 PM User Interface Feature #2478: button image support
>> Postpone the image loading to the moment when the button widget is created on the client side. This will requir... Eugenie Lyzenko
01:25 PM User Interface Feature #2478: button image support
Greg Shah wrote:
> > Postpone the image loading to the moment when the button widget is created on the client side. ...
Constantin Asofiei
01:15 PM User Interface Feature #2478: button image support
> Postpone the image loading to the moment when the button widget is created on the client side. This will require ad... Greg Shah
12:56 PM User Interface Feature #2478: button image support
The button widget image processing note. There are two places when we need to set/load image integration into button ... Eugenie Lyzenko
01:29 PM User Interface Feature #2494: DELETE() method runtime implementation
Greg Shah wrote:
> I have not yet gotten the time to finish the code review on those changes (and on the other fix...
Igor Skornyakov
01:06 PM User Interface Feature #2494: DELETE() method runtime implementation
> So, it seems that I can start coding now. Is that correct?\
Yes.
> Albeit these changes do not affect the cla...
Greg Shah
01:01 PM User Interface Feature #2494: DELETE() method runtime implementation
Greg Shah wrote:
> Just change the @delete(int)@ to be @delete(long)@
Got it. So, it seems that I can start codin...
Igor Skornyakov
12:51 PM User Interface Feature #2494: DELETE() method runtime implementation
> I see. However at this moment we have @delete(int)@ method in the @ControlSetEntity@ (and even in @GenericWidget@).... Greg Shah
12:22 PM User Interface Feature #2494: DELETE() method runtime implementation
Greg Shah wrote:
> > But I see entry(int) and entry(int64) in ControlSetEntity, but there is no entry(long)
>
> I...
Igor Skornyakov
12:08 PM User Interface Feature #2494: DELETE() method runtime implementation
Constantin Asofiei wrote:
> To determine what is the message shown by the DELETE method (in this case), usually I ch...
Igor Skornyakov
11:08 AM User Interface Feature #2494: DELETE() method runtime implementation
Constantin Asofiei wrote:
> On a side note, how do you use the @r-radio.p@ test? I've tried switching focus to the f...
Igor Skornyakov
10:41 AM User Interface Feature #2494: DELETE() method runtime implementation
Greg Shah wrote:
> Constantin/Stanislav: can you think of other ways we code these warning cases in the P2J runtime?...
Constantin Asofiei
10:21 AM User Interface Feature #2494: DELETE() method runtime implementation
> Does it just mean that there is no need to add delete(decimal idx) method?
Yes. Although how we would handle a ...
Greg Shah
09:30 AM User Interface Feature #2494: DELETE() method runtime implementation

> This means we need some extra investigation. Some warnings are just formatted messages displayed in the message ...
Igor Skornyakov
08:52 AM User Interface Feature #2494: DELETE() method runtime implementation
> method generates warning if
This means we need some extra investigation. Some warnings are just formatted mess...
Greg Shah
07:29 AM User Interface Feature #2494: DELETE() method runtime implementation
Questions.
1. I understand that we should not care about thread safety of widgets' methods as they are supposed to b...
Igor Skornyakov
07:09 AM User Interface Feature #2494: DELETE() method runtime implementation
Runtime support.
1. Server side.
a. Implement @delete(int idx)@, @delete(int64 idx)@, @delete(String item)@ and @de...
Igor Skornyakov
06:04 AM User Interface Feature #2494: DELETE() method runtime implementation
1. The statement about raising error is not correct. Actually the @DELETE(list-index)@ method generates warning if ar... Igor Skornyakov
10:28 AM Database Bug #2488: replace runtime compilation of dynamic queries
Eric Faulhaber wrote:
> Don't we convert these kinds of expressions into HQL today, using PL/Java UDFs to represent ...
Ovidiu Maxiniuc
10:27 AM Database Bug #2488: replace runtime compilation of dynamic queries
Ovidiu Maxiniuc wrote:
> On the other hand, if we do to many semantic annotations, the tree will grow - and the mana...
Eric Faulhaber
10:19 AM Database Bug #2488: replace runtime compilation of dynamic queries
Ovidiu Maxiniuc wrote:
> The static call is emitted, relying on java compiler to resolve the actual method to be inv...
Eric Faulhaber
02:58 AM Database Bug #2488: replace runtime compilation of dynamic queries
This is true, we don't do the constant expression evaluation at this moment neither in static conversion but it would... Ovidiu Maxiniuc
08:57 AM User Interface Feature #2422: add features to BROWSE
About this particular feature:
>If the previous page is fully deleted, you have to press Page Up twice in order to m...
Stanislav Lomany
08:14 AM User Interface Feature #2422: add features to BROWSE
You'll be working on GUI soon, so do make sure that your approach can be easily adapted for those cases. Greg Shah

01/20/2015

05:59 PM User Interface Feature #2422: add features to BROWSE
Some differences in handling of Page Up/Down in ChUI and UI versions.
# About Page Down key (in ChUI it's not actu...
Stanislav Lomany
04:36 PM Database Bug #2488: replace runtime compilation of dynamic queries
Certainly static expressions will be more efficient to convert, execute, and store at conversion. I'm pretty sure we ... Eric Faulhaber
03:55 PM Database Bug #2488: replace runtime compilation of dynamic queries
> Don't abandon it just yet...it will not need to do as much as it does now, but this is a good candidate to morph in... Ovidiu Maxiniuc
08:08 PM Database Bug #2488: replace runtime compilation of dynamic queries
Ovidiu Maxiniuc wrote:
> Attached you can find a first version of 'runtime/interpret_find_query.xml'. It only handle...
Eric Faulhaber
04:31 PM User Interface Feature #2494: DELETE() method runtime implementation
Good results.
Please make sure you have tested the following:
* For @DELETE(list-index)@: @INTEGER@ inputs of 0...
Greg Shah
02:59 PM User Interface Feature #2494: DELETE() method runtime implementation
1. The @DELETE()@ method exists for COMBO-BOX, RADIO-SET, SELECTION-LIST and BROWSE column widgets. Its semantics and... Igor Skornyakov
11:16 AM User Interface Feature #2494: DELETE() method runtime implementation
You can edit it manually like a normal cell. I'm not sure if drop-down list can actually appear in browse in ChUI. Po... Stanislav Lomany
07:45 AM User Interface Feature #2494: DELETE() method runtime implementation
Stanislav: in the 4GL, how does one get the ChUI browse cell for a combo-box into editing mode? Greg Shah
07:42 AM User Interface Feature #2494: DELETE() method runtime implementation
I failed to activate editing in Linux character mode. In Windows GUI mode the corresponding cell had standard clickab... Igor Skornyakov
07:38 AM User Interface Feature #2494: DELETE() method runtime implementation
FYI, LIST-ITEMS support in P2J may be partial (I think it is missing at least for browse columns).
When running th...
Greg Shah
03:40 PM User Interface Feature #2476: window icon support improvements
The conversion testing completed. Generated codes are identical. Starting the runtime testing.
And I have committe...
Eugenie Lyzenko
10:59 AM User Interface Feature #2476: window icon support improvements
> I suggest to run regression and commit icon related changes to have some intermediate work stage to shift the next... Greg Shah
10:59 AM User Interface Feature #2476: window icon support improvements
It is not a bad idea to load resources from a specific part of the jar, but I am OK with the current implementation a... Greg Shah
10:56 AM User Interface Feature #2476: window icon support improvements
Eugenie Lyzenko wrote:
> Regarding icon loading. Everything looks OK except one point I need to clarify:
> 1. Assum...
Constantin Asofiei
10:42 AM User Interface Feature #2476: window icon support improvements
Regarding icon loading. Everything looks OK except one point I need to clarify:
1. Assume we start the server from @...
Eugenie Lyzenko
08:02 PM User Interface Feature #2476: window icon support improvements
> I'm good with this.
>
> Does this pass all of your 4GL testcases?
>
> If so, please put it into conversion an...
Eugenie Lyzenko
07:11 PM User Interface Feature #2476: window icon support improvements
Code Review evl_upd20150119c.zip
I'm good with this.
Does this pass all of your 4GL testcases?
If so, please...
Greg Shah
07:26 AM Base Language Feature #1628: fix any incompatibilities or missing features of NO-ECHO support
Code Review evk_upd20150119a.zip
It looks good.
Are the ChUI-specific changes to @Window.message()@ still neede...
Greg Shah
09:18 PM Base Language Feature #1628: fix any incompatibilities or missing features of NO-ECHO support
Explicit flush stream is helped, thanks.
That's why different test in regression testing were failed from time to t...
Evgeny Kiselev

01/19/2015

06:53 PM User Interface Feature #2476: window icon support improvements
Greg Shah wrote:
>
> The only thing I see is that @FileSystemOps.searchPath()@ should not be called from the clien...
Eugenie Lyzenko
06:15 PM User Interface Feature #2476: window icon support improvements
Code Review evl_upd20150119b.zip
This is really good!
The only thing I see is that @FileSystemOps.searchPath(...
Greg Shah
05:43 PM User Interface Feature #2476: window icon support improvements
Small change to eliminate point #4 from the notes. Eugenie Lyzenko
02:28 PM User Interface Feature #2476: window icon support improvements
The new update for review. The fixes for notes to the previous one. Also the @gclogo.ico@ default icon has been moved... Eugenie Lyzenko
11:44 AM User Interface Feature #2476: window icon support improvements
> Does it mean the @FileSystemOps.searchPath()@ must be called on client side?
Not exactly. Instead, the client ...
Greg Shah
11:35 AM User Interface Feature #2476: window icon support improvements
One question.
>...If the app jar load fails, then it calls the client with a filename. It is the ThinClient.set*(S...
Eugenie Lyzenko
03:33 PM User Interface Feature #2494: DELETE() method runtime implementation
> How the COMBO-BOX widget for the BROWSE column can be activated in the Linux character mode?
This is close, but...
Greg Shah
08:07 PM User Interface Feature #2494: DELETE() method runtime implementation
Methods and attributes are functions and properties that can be invoked/read/written using a @HANDLE@ type. Each wid... Greg Shah
07:54 PM User Interface Feature #2494 (Closed): DELETE() method runtime implementation
Greg Shah
02:36 PM Database Bug #2488: replace runtime compilation of dynamic queries
Attached you can find a first version of 'runtime/interpret_find_query.xml'. It only handles the simple @find-query@ ... Ovidiu Maxiniuc
08:07 PM Database Bug #2488: replace runtime compilation of dynamic queries
Ovidiu Maxiniuc wrote:
> If you have the chance to re-run it, please add the string @predicate@ before and, to simpl...
Eric Faulhaber
01:27 PM Base Language Feature #1628: fix any incompatibilities or missing features of NO-ECHO support
> So we need always NL before message if message is not on the first line(or we need to calculate NL character as not... Greg Shah
09:06 PM Base Language Feature #1628: fix any incompatibilities or missing features of NO-ECHO support
Greg Shah wrote:
> Please post the corresponding 4GL output so that I can better understand what this code is suppose...
Evgeny Kiselev

01/18/2015

09:11 AM Base Language Feature #1628: fix any incompatibilities or missing features of NO-ECHO support
Please post the corresponding 4GL output so that I can better understand what this code is supposed to do. Greg Shah

01/17/2015

05:57 PM Base Language Feature #1628: fix any incompatibilities or missing features of NO-ECHO support
Constantin Asofiei wrote:
> Evgeny Kiselev wrote:
> > - Message should be always on New Line if current position is n...
Evgeny Kiselev
05:27 PM Database Bug #2488: replace runtime compilation of dynamic queries
Eric Faulhaber wrote (from Constantin's email):
> The problem here is that at conversion-time the SchemaDictionary (...
Eric Faulhaber
10:35 AM User Interface Feature #2476: window icon support improvements
Code Review evl_upd20150116b.zip
1. You can't load @LogicalTerminal@ on the client, but you are trying to do so. ...
Greg Shah
07:50 PM User Interface Feature #2476: window icon support improvements
This update has sources merged with the recent code base. Also cleanups, some common code rework(@Image@ class remove... Eugenie Lyzenko
08:15 AM User Interface Bug #2491 (Closed): SCREEN-VALUE returns a formatted value based on the widget's format string
CTRL-C tests passed (without any failures, including the 3-way ones). The attached version is checked in as bzr rev ... Greg Shah
06:40 AM User Interface Bug #2491: SCREEN-VALUE returns a formatted value based on the widget's format string
In regard to the use of screen-value in validation expressions, at least the basic functionality works as expected wi... Greg Shah
06:18 AM User Interface Bug #2491 (WIP): SCREEN-VALUE returns a formatted value based on the widget's format string
The @DateFormat@ class needed more changes to properly match the original behavior. With the attached changes, this ... Greg Shah
08:06 AM User Interface Feature #1790: implement menu support
Code Review vig_upd20150117a.zip
As a general approach, I'm not entirely convinced it is the best idea to add menu...
Greg Shah

01/16/2015

05:46 PM Database Bug #2488: replace runtime compilation of dynamic queries
BTW, we have implemented a lot of constructor and method matching support already for TRPL itself. Perhaps we can lev... Eric Faulhaber
05:37 PM Database Bug #2488: replace runtime compilation of dynamic queries
I've taken a look at the code so far and it is a great start! Yes, I agree that the final tree is simpler to deal wit... Eric Faulhaber
03:55 PM Database Bug #2488: replace runtime compilation of dynamic queries
Eric Faulhaber wrote:
> Sure, I can do that, but are you certain you want to use the current, final version of the J...
Ovidiu Maxiniuc
03:41 PM Database Bug #2488: replace runtime compilation of dynamic queries
Ovidiu Maxiniuc wrote:
> I did some researches on windev01. Progress is not very permissive. The static statements c...
Eric Faulhaber
03:30 PM Database Bug #2488: replace runtime compilation of dynamic queries
Related to the @createAlias@ topic, here's an email exchange I had with Constantin recently:
> Eric,
>
> The prob...
Eric Faulhaber
03:28 PM Database Bug #2488: replace runtime compilation of dynamic queries
> The text of the error message suggests you are passing the same parameter (@dictdb@) for both the @alias@ and @ldbN... Ovidiu Maxiniuc
02:38 PM Database Bug #2488: replace runtime compilation of dynamic queries
While I am excited that you've begun implementing a solution, I'm curious whether you've found the answers to the que... Eric Faulhaber
02:31 PM Database Bug #2488: replace runtime compilation of dynamic queries
Ovidiu Maxiniuc wrote:
> I started implementing a small 'interpreting' engine, targeting only the required node/item...
Eric Faulhaber
02:25 PM Database Bug #2488: replace runtime compilation of dynamic queries
I started implementing a small 'interpreting' engine, targeting only the required node/items. In fact at this time, t... Ovidiu Maxiniuc
04:54 PM User Interface Feature #2476: window icon support improvements
I'm more worried about the client falling back to the default icon when the @setWindowDefaultIcon()@ fails to load a ... Greg Shah
04:13 PM User Interface Feature #2476: window icon support improvements
>I think there is still a problem when the client side is told to do something (like load a filename) and that fails.... Eugenie Lyzenko
03:52 PM User Interface Feature #2476: window icon support improvements
Eugenie Lyzenko wrote:
> >3. Fallback to the default icon when other specified options fail.
>
> This is handled ...
Greg Shah
03:50 PM User Interface Feature #2476: window icon support improvements
>3. Fallback to the default icon when other specified options fail.
This is handled in LT.setWindowDefaultIcon():
...
Eugenie Lyzenko
02:27 PM User Interface Feature #2476: window icon support improvements
Code Review evl_upd20150116a.zip
This is really good. Keep going as per your plan.
I realize this is unfinishe...
Greg Shah
02:05 PM User Interface Feature #2476: window icon support improvements
This update for review is not the final version to test. Just for you to see the default icon implementation and new ... Eugenie Lyzenko
03:58 PM User Interface Feature #1790: implement menu support
Here is the next update. I used @ScreenDefinition@ (not @ScreenBuffer@). Please confirm, that I'm on right way. Vadim Gindin
02:12 AM User Interface Feature #1790: implement menu support
> Constantin: thoughts?
# I suspect that showing the sub-menu (when a menu is activated in the menubar) will need to...
Constantin Asofiei
02:11 PM Base Language Feature #1628: fix any incompatibilities or missing features of NO-ECHO support
Evgeny Kiselev wrote:
> - Message should be always on New Line if current position is not in the start of line. I th...
Constantin Asofiei
08:04 AM User Interface Feature #2252: implement GUI client support
Constantin Asofiei wrote:
> Greg Shah wrote:
> > Code Review ca_upd20150115b.zip
> >
> > I'm fine with the chang...
Constantin Asofiei
05:42 AM User Interface Feature #2252: implement GUI client support
Greg Shah wrote:
> Code Review ca_upd20150115b.zip
>
> I'm fine with the changes.
0115b.zip passed runtime testing.
Constantin Asofiei
06:40 AM Base Language Bug #2485: Invalid THIS-PROCEDURE in AdminClient
Committed to bzr rev 10711. Stanislav Lomany
05:36 AM Base Language Bug #2485: Invalid THIS-PROCEDURE in AdminClient
Stanislav Lomany wrote:
> MAJIC converted OK. Please review the update so I can distribute it.
I'm OK with the chan...
Constantin Asofiei
05:28 AM Base Language Bug #2485: Invalid THIS-PROCEDURE in AdminClient
MAJIC converted OK. Please review the update so I can distribute it. Stanislav Lomany
02:18 AM Base Language Bug #2485: Invalid THIS-PROCEDURE in AdminClient
Fixed conversion. MAJIC conversion is running. Stanislav Lomany
01:58 AM User Interface Bug #2491: SCREEN-VALUE returns a formatted value based on the widget's format string
Greg, I recall that SCREEN-VALUE was problematic in the widget validation expressions, too. It can be postponed if th... Constantin Asofiei
09:20 PM User Interface Bug #2491: SCREEN-VALUE returns a formatted value based on the widget's format string
It turns out that our SCREEN-VALUE implementation was nearly completely broken. MAJIC only uses a small number of ca... Greg Shah
09:15 PM User Interface Bug #2491 (Closed): SCREEN-VALUE returns a formatted value based on the widget's format string
Greg Shah

01/15/2015

06:35 PM User Interface Feature #2479: DEFINE IMAGE and image phrase support for both button and image
>Add @type == prog.define_image@ to that and I think the problem will be resolved.
Thank, this works.
Eugenie Lyzenko
06:08 PM User Interface Feature #2479: DEFINE IMAGE and image phrase support for both button and image
> I've found I've missed one rule file update for image widget conversion support: convert/frame_generator.xml. Witho... Greg Shah
06:04 PM User Interface Feature #2479: DEFINE IMAGE and image phrase support for both button and image
Greg,
I've found I've missed one rule file update for image widget conversion support: convert/frame_generator.xml...
Eugenie Lyzenko
06:08 PM Base Language Bug #2485 (Closed): Invalid THIS-PROCEDURE in AdminClient
Greg Shah
05:42 PM Base Language Bug #2485 (Review): Invalid THIS-PROCEDURE in AdminClient
Stanislav Lomany
05:40 PM Base Language Bug #2485: Invalid THIS-PROCEDURE in AdminClient
svl_upd20150115a.zip was committed into bzr rev 10710.
svl_upd20150114b.zip was committed to staging on devsrv01.
Stanislav Lomany
10:45 AM Base Language Bug #2485: Invalid THIS-PROCEDURE in AdminClient
Stanislav Lomany wrote:
> So, should I distribute the P2J update without regression testing
Run only the CTRL-C pa...
Constantin Asofiei
10:43 AM Base Language Bug #2485: Invalid THIS-PROCEDURE in AdminClient
So, should I distribute the P2J update without regression testing and check in MAJIC part into staging branch on devs... Stanislav Lomany
10:07 AM Base Language Bug #2485: Invalid THIS-PROCEDURE in AdminClient
Stanislav Lomany wrote:
> Fixed getting "headless" attribute.
If this solves the admin client problem without regre...
Constantin Asofiei
09:34 AM Base Language Bug #2485: Invalid THIS-PROCEDURE in AdminClient
Fixed getting "headless" attribute. Stanislav Lomany
03:53 AM Base Language Bug #2485: Invalid THIS-PROCEDURE in AdminClient
Stanislav Lomany wrote:
> Looks like with this call is disturbing natural order of loading @TransactionManager@ and ...
Constantin Asofiei
05:04 PM User Interface Feature #1790: implement menu support
> ScreenBuffer is not really useful because it linked to a frame and we don't know what frame is the main (common). S... Greg Shah
03:17 PM User Interface Feature #1790: implement menu support
I trying to decide how to transfer information about main menu (menubar=true) to the client. @ScreenBuffer@ is not re... Vadim Gindin
11:35 AM User Interface Feature #1790: implement menu support
No, that is not something that is specific to the p2j project. It is a tool that we would add to the standard workst... Greg Shah
11:09 AM User Interface Feature #1790: implement menu support
Should I commit (without @work@ folder as you wrote) the script from the note #152? If yes - can I commit it right to... Vadim Gindin
11:03 AM User Interface Feature #1790: implement menu support
@vig_upd20150114d.zip@ is committed to bzr with rev №10709 Vadim Gindin
09:42 AM User Interface Feature #1790: implement menu support
Yes, please do check in vig_upd20150114d.zip and distribute it. Greg Shah
09:37 AM User Interface Feature #1790: implement menu support
The current update has passed conversion and regression testing. Can I commit it? Vadim Gindin
03:41 PM User Interface Feature #2252: implement GUI client support
Constantin Asofiei wrote:
> Hynek, please check your tests, too.
Yes, works ok.
Hynek Cihlar
01:58 PM User Interface Feature #2252: implement GUI client support
Code Review ca_upd20150115b.zip
I'm fine with the changes.
Greg Shah
01:51 PM User Interface Feature #2252: implement GUI client support
Greg Shah wrote:
> > If down frame widgets are not moved by changing their ROW/COLUMN attribute, then I think we sh...
Constantin Asofiei
01:23 PM User Interface Feature #2252: implement GUI client support
> If down frame widgets are not moved by changing their ROW/COLUMN attribute, then I think we should leave this as i... Greg Shah
01:14 PM User Interface Feature #2252: implement GUI client support
Greg,
> * moving widgets in a down frame; not sure yet, might be related to the layout issue.
This is related to th...
Constantin Asofiei
10:12 AM User Interface Feature #2252: implement GUI client support
Greg Shah wrote:
> Code Review ca_upd20150115a.zip
>
> It's good. Please check it in.
Released to rev 10708.
Constantin Asofiei
09:44 AM User Interface Feature #2252: implement GUI client support
Code Review ca_upd20150115a.zip
It's good. Please check it in.
Greg Shah
04:46 AM User Interface Feature #2252: implement GUI client support
This is replacement for ca_upd20150114b.zip and passed runtime testing. The @Assigning row/col attributes of a widg... Constantin Asofiei
03:12 PM Database Bug #2488: replace runtime compilation of dynamic queries
Ovidiu Maxiniuc wrote:
> Yes, @query-prepare()@ is the big issue. I believe there are no constrains here and any num...
Eric Faulhaber
02:58 PM Database Bug #2488: replace runtime compilation of dynamic queries
Yes, @query-prepare()@ is the big issue. I believe there are no constrains here and any number of buffers can be adde... Ovidiu Maxiniuc
02:53 PM Database Bug #2488: replace runtime compilation of dynamic queries
OK, so no calls to UDFs, either? That's another way to trigger client-side where clause processing.
If not, it wou...
Eric Faulhaber
02:42 PM Database Bug #2488: replace runtime compilation of dynamic queries
Looking at @find-first()@ method in manual, the predicate expression is very simple, just ... Ovidiu Maxiniuc
02:34 PM Database Bug #2488: replace runtime compilation of dynamic queries
I think the following would give us a reasonably complex @WhereExpression@:... Eric Faulhaber
01:05 PM Database Bug #2488: replace runtime compilation of dynamic queries
> I'm not sure how well an interpreted WhereExpression would work. We may still have to fall back to a compiled appro... Ovidiu Maxiniuc
12:47 PM Database Bug #2488: replace runtime compilation of dynamic queries
Ovidiu Maxiniuc wrote:
> The java/TRLP debate. It's clear that TRPL has the some advantages, but I think that we sho...
Eric Faulhaber
12:41 PM Database Bug #2488: replace runtime compilation of dynamic queries
Ovidiu Maxiniuc wrote:
> The difficulty appears when there are another buffers (and aliases)...
Please explain your...
Eric Faulhaber
06:41 AM Database Bug #2488: replace runtime compilation of dynamic queries
I understand the principle of this approach.
At first sight, it looks rather easy, the where clause and sorting ar...
Ovidiu Maxiniuc
02:04 AM Database Bug #2488: replace runtime compilation of dynamic queries
The basic idea behind this improvement is to continue to use runtime TRPL to convert a query to an in-memory, Java AS... Eric Faulhaber
01:40 PM User Interface Feature #2490: fix movement of widgets part of a down frame's body
The widgets part of the down body for a down frame can't be safely moved (by changing their ROW/COLUMN attributes) be... Constantin Asofiei
01:36 PM User Interface Feature #2490 (New): fix movement of widgets part of a down frame's body
Constantin Asofiei
12:53 PM User Interface Feature #2476: window icon support improvements
Actually, let's name @setDefaultWindowIcon()@ as @setWindowDefaultIcon()@ instead. I think it is more descriptive.
...
Greg Shah
12:31 PM User Interface Feature #2476: window icon support improvements
>1. We want to minimize unnecessary calls from the client to the server. Generally, we want the server to dictate the... Eugenie Lyzenko
11:33 AM User Interface Feature #2476: window icon support improvements
Code Review evl_upd20150114a.zip
This is moving in the right direction, but I do have some questions/comments:
...
Greg Shah
07:43 PM User Interface Feature #2476: window icon support improvements
The update includes icon related part of the icon/image implementation for review. Not completed testing but approach... Eugenie Lyzenko
09:38 AM Base Language Feature #1628: fix any incompatibilities or missing features of NO-ECHO support
I think your idea of changing the approach seems to be on the right track.
> Message should be always on New Line ...
Greg Shah
05:21 AM Base Language Feature #1628: fix any incompatibilities or missing features of NO-ECHO support
At the moment regression tests finished with failed status.
5 tests are failed because of different numbers of rows ...
Evgeny Kiselev

01/14/2015

03:14 PM Base Language Bug #2485: Invalid THIS-PROCEDURE in AdminClient
Heh, there is a problem with @TransactionManager.isHeadless@: normal clients do not start.... Stanislav Lomany
02:20 PM Base Language Bug #2485: Invalid THIS-PROCEDURE in AdminClient
The query changes are the same that were sent to Scott. Stanislav Lomany
12:48 PM Base Language Bug #2485: Invalid THIS-PROCEDURE in AdminClient
Code Review svl_upd20150114b.zip
I'm fine with the external procedure wrapping parts of this.
Eric: please revi...
Greg Shah
12:48 PM Base Language Bug #2485: Invalid THIS-PROCEDURE in AdminClient
Code Review svl_upd20150114a.zip
I'm fine with this.
Greg Shah
11:40 AM Base Language Bug #2485: Invalid THIS-PROCEDURE in AdminClient
Updates for review. Stanislav Lomany
08:22 AM Base Language Bug #2485: Invalid THIS-PROCEDURE in AdminClient
Yes, I think that is fine. We just need to put some clear comments into there to explain why this is being done. Greg Shah
01:59 AM Base Language Bug #2485: Invalid THIS-PROCEDURE in AdminClient
Greg, can we assume that if we are in headless mode (@TransactionManager.isHeadless()@ returns true), we are under li... Constantin Asofiei
12:06 PM User Interface Feature #2252: implement GUI client support
Constantin Asofiei wrote:
> Hynek, I'm planning on looking into solving these problems (based on my previous update)...
Hynek Cihlar
11:58 AM User Interface Feature #2252: implement GUI client support
Hynek, I'm planning on looking into solving these problems (based on my previous update):
* Assigning row/col attrib...
Constantin Asofiei
11:11 AM User Interface Feature #2252: implement GUI client support
Constantin Asofiei wrote:
> Hynek Cihlar wrote:
> > Constantin Asofiei wrote:
> > > Second part of the SIDE-LABEL-...
Hynek Cihlar
11:08 AM User Interface Feature #2252: implement GUI client support
Hynek Cihlar wrote:
> Constantin Asofiei wrote:
> > Second part of the SIDE-LABEL-HANDLE support, to allow changing...
Constantin Asofiei
09:09 AM User Interface Feature #2252: implement GUI client support
Greg Shah wrote:
> Code Review ca_upd20150114b.zip
>
> It looks like there were some tricky latent bugs that you ...
Hynek Cihlar
09:08 AM User Interface Feature #2252: implement GUI client support
Constantin Asofiei wrote:
> Second part of the SIDE-LABEL-HANDLE support, to allow changing the ROW/COLUMN attribute...
Hynek Cihlar
09:06 AM User Interface Feature #2252: implement GUI client support
> The remaining issue is related to moving widgets in a down frame; root cause: server-side has only 1 widget which i... Greg Shah
09:05 AM User Interface Feature #2252: implement GUI client support
Code Review ca_upd20150114b.zip
It looks like there were some tricky latent bugs that you had to solve here. I se...
Greg Shah
08:58 AM User Interface Feature #2252: implement GUI client support
The label moves ok and the position attributes are properly reported back to the server.
Btw. there are some (chui...
Hynek Cihlar
06:51 AM User Interface Feature #2252: implement GUI client support
Second part of the SIDE-LABEL-HANDLE support, to allow changing the ROW/COLUMN attributes from the server-side. Runt... Constantin Asofiei
10:09 AM User Interface Feature #1790: implement menu support
> Can It be useful for somebody?
Very cool! I like it.
The only thing we need to do here is to eliminate the d...
Greg Shah
10:03 AM User Interface Feature #1790: implement menu support
I was using an eclipse plugin to merge files last time and didn't accustom to this plugin probably. The reason of the... Vadim Gindin
09:54 AM User Interface Feature #1790: implement menu support
Thank you. These folders are different. diff*_ddl.txt is empty, but diff_*_src.txt is not empty. I'm trying to interp... Vadim Gindin
09:35 AM User Interface Feature #1790: implement menu support
As noted in the @timco.html@:
*Build and Save a Comparison Version*
Before running any conversion regression te...
Greg Shah
09:30 AM User Interface Feature #1790: implement menu support
I've ran conv-regression task. It executed, but failed at the end. Here is the end of the log:... Vadim Gindin
08:35 AM User Interface Feature #1790: implement menu support
Code Review vig_upd20150114c.zip
This is really good. Please put it into both conversion and runtime regression t...
Greg Shah
05:31 AM User Interface Feature #1790: implement menu support
I missed several files. Vadim Gindin
04:48 AM User Interface Feature #1790: implement menu support
Thank you. Here is the current update with these changes. Vadim Gindin
04:18 AM User Interface Feature #1790: implement menu support
Vadim Gindin wrote:
> Please, recall me, should I change a year of a Copyright block in each modified file to 2015?
...
Constantin Asofiei
03:16 AM User Interface Feature #1790: implement menu support
I've merged @method_attributes.rules@ and changed @text@ with @names.convert(text, names.variable)@ corresponding to ... Vadim Gindin
10:07 AM Base Language Bug #2489: FileSystemOps.searchPath(String) should return the 4GL procedure name for external pro...
It is common for 4GL code to search for 4GL external procedure names before executing those programs. Such searches ... Greg Shah
09:54 AM Base Language Bug #2489 (Closed): FileSystemOps.searchPath(String) should return the 4GL procedure name for ext...
Greg Shah
01:26 AM Database Bug #2488: replace runtime compilation of dynamic queries
Dynamically defined queries are parsed and converted at runtime by P2J, then compiled into discrete Java classes usin... Eric Faulhaber
12:48 AM Database Bug #2488 (Closed): replace runtime compilation of dynamic queries
Eric Faulhaber
09:06 PM User Interface Feature #2487: implement READ-FILE() and SAVE-FILE() methods in the editor widget
The @READ-FILE()@ method is already supported in @methods_attributes.rules@ (and the runtime stubs are in the @Editor... Greg Shah
08:32 PM User Interface Feature #2487 (Closed): implement READ-FILE() and SAVE-FILE() methods in the editor widget
Greg Shah
08:29 PM User Interface Feature #2486: enhance editor functionality
This task is just here to aggregate the related sub-tasks. Greg Shah
08:28 PM User Interface Feature #2486 (Closed): enhance editor functionality
Greg Shah

01/13/2015

04:06 PM Base Language Bug #2485 (WIP): Invalid THIS-PROCEDURE in AdminClient
Constantin, thanks, that helped.
Guys, there is one more issue: if we connect to a fresh server - no standards cli...
Stanislav Lomany
09:42 AM Base Language Bug #2485: Invalid THIS-PROCEDURE in AdminClient
Stanislav,
Your stacktrace is hiding another NPE, when the scope is initially pushed:...
Constantin Asofiei
08:54 AM Base Language Bug #2485: Invalid THIS-PROCEDURE in AdminClient
I think all of the issues have the same cause. Reproduction:
1. Log into admin client.
2. MAJIC -> Printer Maintena...
Stanislav Lomany
07:18 AM Base Language Bug #2485 (Closed): Invalid THIS-PROCEDURE in AdminClient
Stanislav Lomany
01:19 PM User Interface Feature #1790: implement menu support
Code Review vig_upd20150113a.zip
The only problem I see is that you still have a duplicate section for @kw_full_h_...
Greg Shah
01:14 PM User Interface Feature #1790: implement menu support
I think the triggers code looks fine. Is there anything else to do for the conversion? Greg Shah
12:48 PM User Interface Feature #1790: implement menu support
Here are the source and generated files for menu references in triggers as you asked earlier. Vadim Gindin
12:44 PM User Interface Feature #1790: implement menu support
The next update.
# Fixed remarks.
# Fixed @lvalue@ rule in progress.g (added KW_SUB_MENU as possible widget qualifi...
Vadim Gindin
09:39 AM User Interface Feature #1790: implement menu support
It is a good question. We already have quite a bit of logic that handles these cases differently. Right now they ca... Greg Shah
09:38 AM User Interface Feature #1790: implement menu support
I suppose I should make the common AST subtree for both cases. How can I make it and where? May be it is also a parse... Vadim Gindin
09:33 AM User Interface Feature #1790: implement menu support
After I fixed some parsing problem I faced with the following difficulty during conversion of the following trigger:
...
Vadim Gindin
07:18 AM User Interface Feature #2252: implement GUI client support
Committed to bzr revision 10704.
Greg Shah wrote:
> Code Review hc_upd20150112a.zip
>
> It looks good. Pleas...
Hynek Cihlar
08:37 PM User Interface Feature #2252: implement GUI client support
Code Review hc_upd20150112a.zip
It looks good. Please check it in.
Greg Shah
08:41 PM User Interface Feature #2476: window icon support improvements
OK, do whatever matches the behavior of the 4GL. Greg Shah
08:30 PM User Interface Feature #2476: window icon support improvements
The important finding. Progress does not load the icon from current directory if the current directory is not in @PRO... Eugenie Lyzenko

01/12/2015

06:30 PM User Interface Feature #2252: implement GUI client support
In ref to #2424.
The attached change set resolves widget positioning in a frame and passed regression testing. Ple...
Hynek Cihlar
12:29 PM User Interface Feature #2422: add features to BROWSE
Here is the list of 4GL bugs which I'm not going to reproduce. It's obviously not full and may expand it as I'll find... Stanislav Lomany
12:13 PM User Interface Feature #1790: implement menu support
I'm OK with what I saw in the code.
Please post some examples here (both the 4GL and then the converted Java for t...
Greg Shah
11:54 AM User Interface Feature #1790: implement menu support
The only triggers references are left. What do you think about approach I used for that? Vadim Gindin
11:10 AM User Interface Feature #1790: implement menu support
Code Review vig_upd20150112a.zip
Generally, the changes are quite good. I see the following cleanup items. It se...
Greg Shah
09:39 AM User Interface Feature #1790: implement menu support
Here is the current update with the following changes.
# Fixed extra ';' for EXPRESSION/NUM_LITERAL. Check please is...
Vadim Gindin
10:45 AM User Interface Feature #2479: DEFINE IMAGE and image phrase support for both button and image
Please add @CREATE IMAGE@ support by adding the widget to the list @DynamicWidgetFactory.java@. Greg Shah

01/10/2015

05:21 PM User Interface Feature #2476: window icon support improvements
>1. In ImageGuiImpl, should line 258 be if (readers.hasNext()) or should it be while (readers.hasNext())?
I use fi...
Eugenie Lyzenko
01:07 PM User Interface Feature #2476: window icon support improvements
Code Review evl_upd20150109b.zip
1. In @ImageGuiImpl@, should line 258 be @if (readers.hasNext())@ or should it be...
Greg Shah
 

Also available in: Atom