FWD » Core Development » Base Language

12/19/2024

01:29 PM Feature #9488: implement all built-in OO classes using the open source ADE code from 12.8.4

Please checkout again the project fully, there was some references from the source project I used to build it, which ... Constantin Asofiei

12:40 PM Feature #9488: implement all built-in OO classes using the open source ADE code from 12.8.4

OK, is there. I've included the @abl/ADE@ checkout, but couldn't include the .git folder, too. The commit message is... Constantin Asofiei

12:25 PM Feature #9488: implement all built-in OO classes using the open source ADE code from 12.8.4

> Please let me know if you want to upload the ADE project I used.
Yes, we should have it located in @~/secure/cod... Greg Shah

11:47 AM Feature #9488: implement all built-in OO classes using the open source ADE code from 12.8.4

I've ran the reports on the current parsed .cls files; these are some notes:
* unsupported handle attributes: ROLES
... Constantin Asofiei

10:23 AM Feature #9488: implement all built-in OO classes using the open source ADE code from 12.8.4

I've started parsing the ADE .cls files - there are usages like this:
* generic usage... Constantin Asofiei

09:14 AM Feature #9488: implement all built-in OO classes using the open source ADE code from 12.8.4

> To make this work, we will have to implement anything missing in @progress.lang.*@ since those are not written in ... Greg Shah

07:20 AM Feature #9488: implement all built-in OO classes using the open source ADE code from 12.8.4

Greg Shah wrote:
> There are less than 450 classes in @src/corelib/@ and @src/netlib/@. Isn't that all we're worrie... Constantin Asofiei

07:14 AM Feature #9488: implement all built-in OO classes using the open source ADE code from 12.8.4

There are less than 450 classes in @src/corelib/@ and @src/netlib/@. Isn't that all we're worried about? I don't th... Greg Shah

12/18/2024

05:31 PM Feature #9488: implement all built-in OO classes using the open source ADE code from 12.8.4

Greg Shah wrote:
> Constantin: Please evaluate the gaps that need to be resolved to convert all the OO built-in clas... Constantin Asofiei

04:42 PM Feature #9488: implement all built-in OO classes using the open source ADE code from 12.8.4

As of 12.7.0, PSC has licensed all of its ADE code under the Apache 2.0 license (see https://github.com/progress/ADE/... Greg Shah

04:34 PM Feature #9488 (WIP): implement all built-in OO classes using the open source ADE code from 12.8.4

Greg Shah

09:46 AM Bug #9411: Implement SESSION:STREAM

Please update the gap marking for this attribute (mark it full/partial the same way that cpstream is marked).
Then... Greg Shah

03:13 AM Bug #9411 (Internal Test): Implement SESSION:STREAM

Alexandru Lungu

08:42 AM Bug #8944: FINALLY block transaction processing

There is another potential issue that maybe should be discussed.
In @notifyMasterFinish@ and @processFinalizables@, ... Artur Școlnic

07:43 AM Bug #8944 (Review): FINALLY block transaction processing

I tried to solve the issue without changing the existing 'infrastructure' and overall logic.
As I stated in note 29,... Artur Școlnic

12/17/2024

08:19 AM Bug #9482: Reduce logging for warnings coming outside FWD.

CentralLogger logs warning with throwables and their respective stack trace, if the throwable was passed as a paramet... Artur Școlnic

08:15 AM Bug #9482 (New): Reduce logging for warnings coming outside FWD.

Artur Școlnic

04:40 AM Bug #9037: Two timers problem: only one fires

Review:
Not sure if locking in @ContextLocal@ would introduce even more problems, but I've found that out while wor... Tomasz Domin

12/16/2024

09:37 AM Bug #9354: Invalid handle (3135) error is thrown insted of the 10068 error.

There are some prints for debugging reasons.... Eduard Marcu

12/13/2024

02:13 PM Bug #9354: Invalid handle (3135) error is thrown insted of the 10068 error.

> I think what is needed is to put an annotation when @chainedRef@ should be true and after a @REFERENCE@ gets added ... Greg Shah

09:48 AM Bug #9354: Invalid handle (3135) error is thrown insted of the 10068 error.

For the progress code given by Alex: @obj:gen():gen():test():test().@... Eduard Marcu

12/12/2024

10:49 AM Bug #9037: Two timers problem: only one fires

> @SecurityContext@ instance (Greg, IIRC this is a singleton for each FWD context, right?)
In most cases, yes it i... Greg Shah

08:13 AM Bug #9037: Two timers problem: only one fires

Vladimir Tsichevski wrote:
> Constantin Asofiei wrote:
>
> > Regarding the changes in 9037a...
>
> Branch 9037... Constantin Asofiei

08:13 AM Bug #9037: Two timers problem: only one fires

Constantin Asofiei wrote:
> Regarding the changes in 9037a...
Branch 9037a never existed :-(. The change is in ... Vladimir Tsichevski

07:54 AM Bug #9037: Two timers problem: only one fires

Greg/Vladimir: I agree that the synchronization in ContextLocal is faulty. But I propose to improve and fix the sync... Constantin Asofiei

10:01 AM Bug #9354: Invalid handle (3135) error is thrown insted of the 10068 error.

> I like the second version more. Maybe use @ref()@, @chainedRef()@ (which will call @chainedRef(false)@ underneath) ... Greg Shah

09:36 AM Bug #9354: Invalid handle (3135) error is thrown insted of the 10068 error.

I think your approach is right. Please consider:... Alexandru Lungu

08:54 AM Bug #9354: Invalid handle (3135) error is thrown insted of the 10068 error.

... Eduard Marcu

04:29 AM Bug #9396: Improve Incompatible data types in expression or assignment. (223) error handling

I thought about this and found that we have a map storing function definitions in @SignatureHelper@ (@map_func@), thi... Dănuț Filimon

12/11/2024

03:57 AM Bug #9411: Implement SESSION:STREAM

I am OK with the changes in 9411a. Alexandru Lungu

09:01 PM Feature #9448: compatibility test security features

Check the tests in Testcases to ensure they are complete and comprehensive. Add tests if needed. Convert and run the ... Greg Shah

08:58 PM Feature #9448 (Closed): compatibility test security features

Greg Shah

08:53 PM Feature #9447: XML support feature improvements and testing

* @SAX-READER@ (also should be added to @X-DOCUMENT@ if easily done, but only the @SAX-READER@ version is used in the... Greg Shah

08:52 PM Feature #9447 (Closed): XML support feature improvements and testing

Greg Shah

08:20 PM Support #9446: compatibility test PUT-DOUBLE

Check the @PUT-DOUBLE@ tests in [[p2j:Testcases]] to ensure they are complete and comprehensive. Add tests if needed... Greg Shah

08:20 PM Support #9446 (Closed): compatibility test PUT-DOUBLE

Greg Shah

08:18 PM Support #9445: compatibility test FIX-CODEPAGE

Check the @FIX-CODEPAGE@ tests in [[p2j:Testcases]] to ensure they are complete and comprehensive. Add tests if need... Greg Shah

08:13 PM Support #9445 (Closed): compatibility test FIX-CODEPAGE

Greg Shah

12/10/2024

12:35 PM Feature #9442: implement limited support for RCODE-INFORMATION

Greg Shah wrote:
> Both start with an assignment of @RCODE-INFORMATION:FILENAME@ to some program name or class name.... Constantin Asofiei

12:32 PM Feature #9442: implement limited support for RCODE-INFORMATION

There are 2 patterns used that should be supported.
Both start with an assignment of @RCODE-INFORMATION:FILENAME@ ... Greg Shah

12:29 PM Feature #9442 (Closed): implement limited support for RCODE-INFORMATION

Greg Shah

09:59 AM Bug #9411: Implement SESSION:STREAM

Greg Shah wrote:
> Eugenie: Please review and comment on #9411-14.
>
> > is not needed then we can either change ... Eugenie Lyzenko

09:35 AM Bug #9411: Implement SESSION:STREAM

It is already committed! Eduard Marcu

09:28 AM Bug #9411: Implement SESSION:STREAM

Please apply the changes to the branch. Greg Shah

09:26 AM Bug #9411: Implement SESSION:STREAM

Committed 9411a revision 15609. ... Eduard Marcu

09:04 AM Bug #9411: Implement SESSION:STREAM

Perfect. OE treats these as synonyms too. Go ahead with that idea. Greg Shah

08:52 AM Bug #9411: Implement SESSION:STREAM

!6.png!
I think it's worth mentioning that even if I set @-stream "someEncoder"@ the error message contains @-cps... Eduard Marcu

08:27 AM Bug #9411: Implement SESSION:STREAM

Alexandru Lungu wrote:
> Running a 4GL application with @-cpstream@ option at CLI is also changing @STREAM@?
Yes.
... Eduard Marcu

08:12 AM Bug #9411: Implement SESSION:STREAM

> As such I think we can go ahead and make SESSION:STREAM be SESSION:CPSTREAM in conversion.
Running a 4GL applica... Alexandru Lungu

08:10 AM Bug #9411: Implement SESSION:STREAM

I tested with every encoder and I found the following:
* the encoder is the key of @convmap2JavaDefault@
* @SESSION... Eduard Marcu

06:47 AM Bug #9411: Implement SESSION:STREAM

Eugenie: Please review and comment on #9411-14.
> is not needed then we can either change the rules to call @getCP... Greg Shah

06:12 AM Bug #9411: Implement SESSION:STREAM

Alexandru Lungu wrote:
> Functional: I might not properly understand the use of @convmap2JavaDefault@. What is the g... Eduard Marcu

05:03 AM Bug #9411: Implement SESSION:STREAM

Review of 9411a:
* please keep the naming consistent with the rest of CommonSession API (aka use @get@ prefix)
* ... Alexandru Lungu

04:26 AM Bug #9411: Implement SESSION:STREAM

Since we moved the @stream()@ method we should also do the following changes:... Eduard Marcu

04:06 AM Bug #9411: Implement SESSION:STREAM

Committed 9411a revision 15610. Eduard Marcu

04:05 AM Bug #9411 (Review): Implement SESSION:STREAM

Greg Shah wrote:
> Code Review Task Branch 9411a Revision 15609
>
> 1. Instead of @SessionUtils@, the getter shou... Eduard Marcu

09:57 AM Bug #9350 (Internal Test): gap marking for LOG-MANAGER attributes and methods is probably wrong

Code Review Task Branch 9198a Revision 15486
The changes are good.
I've checked in rev 15487 with minor comment... Greg Shah

09:27 AM Bug #9350 (Review): gap marking for LOG-MANAGER attributes and methods is probably wrong

Galya B

09:27 AM Bug #9350: gap marking for LOG-MANAGER attributes and methods is probably wrong

Up in 9198a r15486.
Diff:... Galya B

08:52 AM Bug #9350 (WIP): gap marking for LOG-MANAGER attributes and methods is probably wrong

The current state should be:
1. LOG-MANAGER attributes:
* LOGFILE-NAME (KW_LOGF_NAM) - conversion full/runtime fu... Galya B

07:44 AM Bug #9354: Invalid handle (3135) error is thrown insted of the 10068 error.

I can reproduce the issue. The problem is in @object.ref@:... Eduard Marcu

04:28 AM Bug #9354 (WIP): Invalid handle (3135) error is thrown insted of the 10068 error.

Eduard Marcu

07:38 AM Feature #4392 (Closed): add -pf support for appserver CONNECT() method

Greg Shah

08:34 PM Feature #9437: enhancements for Progress.Lang.Class

... Greg Shah

08:33 PM Feature #9437 (Closed): enhancements for Progress.Lang.Class

Greg Shah

08:26 PM Feature #9436: add built-in OO classes

... Greg Shah

08:24 PM Feature #9436 (WIP): add built-in OO classes

Greg Shah

12/09/2024

06:50 PM Feature #9435: class-level virtual procedure/function definitions

Some procedures and functions can be defined in classes.
* @PROCEDURE EXTERNAL@
* @PROCEDURE/FUNCTION ... IN hand... Greg Shah

06:49 PM Feature #9435 (Closed): class-level virtual procedure/function definitions

Greg Shah

06:44 PM Bug #4602: fixes for OO 4GL and structured error handling

Some things to ensure are fully working:
* @CATCH@ with inner @UNDO, THROW@
* @FINALLY@ with @DELETE OBJECT@ (wit... Greg Shah

04:56 PM Bug #9411: Implement SESSION:STREAM

Code Review Task Branch 9411a Revision 15609
1. Instead of @SessionUtils@, the getter should be in @I18nOps@.
2... Greg Shah

10:05 AM Bug #9411: Implement SESSION:STREAM

Committed 9411a revision 15609. Eduard Marcu

09:39 AM Bug #9411: Implement SESSION:STREAM

That seems reasonable. Please put the changes in a branch. Greg Shah

09:24 AM Bug #9411: Implement SESSION:STREAM

... Eduard Marcu

09:20 AM Bug #9411: Implement SESSION:STREAM

I tested the OE application and introduced other encoding formats in startup. @SESSION:STREAM@ can also output other ... Eduard Marcu

09:07 AM Bug #9411: Implement SESSION:STREAM

In @method_attributes.rules@:... Eduard Marcu

03:03 AM Bug #9411 (WIP): Implement SESSION:STREAM

Eduard Marcu

11:55 AM Bug #6640: parsing bug for initial ",0000" for decimal variables or fields

What happens in OE when you set the decimal separator or group separator to different values?
You can set this on ... Greg Shah

10:44 AM Bug #6640: parsing bug for initial ",0000" for decimal variables or fields

Seems, the decimal separators are ignored completely when parsing a string to a number.
In OE this works as if the i... Vladimir Tsichevski

12/05/2024

09:47 AM Bug #8309: Do the error handling for r-index and extend implementation to INDEX and LONGCHAR

Greg Shah wrote:
> Yuck. I'm not OK with error messages being dependent upon files independently installed or edite... Marian Edu

08:27 AM Bug #8309: Do the error handling for r-index and extend implementation to INDEX and LONGCHAR

Marian Edu wrote:
> Greg Shah wrote:
> > Yes, exactly this. Luckily, this is a pretty rare case because they try t... Greg Shah

09:02 AM Bug #9417: Errors might not be logged by the LegacyLogManager.

Apart from the example in note 1, I have not been able to find cases where errors are not logged, but there might be.... Artur Școlnic

08:56 AM Bug #9417: Errors might not be logged by the LegacyLogManager.

How might you test this bug? Galya B

08:38 AM Bug #9417: Errors might not be logged by the LegacyLogManager.

+Galya, +Constantin Greg Shah

08:36 AM Bug #9417: Errors might not be logged by the LegacyLogManager.

... Artur Școlnic

08:33 AM Bug #9417 (New): Errors might not be logged by the LegacyLogManager.

Artur Școlnic

05:34 AM Bug #9396: Improve Incompatible data types in expression or assignment. (223) error handling

Alexandru Lungu wrote:
> Danut, I am trying to drag the discussion to the point where proving that the error occurs ... Dănuț Filimon

05:26 AM Bug #9396: Improve Incompatible data types in expression or assignment. (223) error handling

Danut, I am trying to drag the discussion to the point where proving that the error occurs only when @var4glType@ and... Alexandru Lungu

03:55 AM Bug #9396: Improve Incompatible data types in expression or assignment. (223) error handling

Alexandru Lungu wrote:
> Please investigate why @tt.f1 = LENGTH('abc')@ is hitting the switch-case, making @var4glTy... Dănuț Filimon

03:20 AM Bug #9396 (WIP): Improve Incompatible data types in expression or assignment. (223) error handling

Resuming work for #9396-25. Dănuț Filimon

12/04/2024

08:01 AM Bug #9396: Improve Incompatible data types in expression or assignment. (223) error handling

> My conclusion is that we need to add the full type checking during standard conversion with error reporting if we w... Greg Shah

06:43 AM Bug #9396: Improve Incompatible data types in expression or assignment. (223) error handling

Alexandru Lungu wrote:
> Ovidiu, the work-flow of static and dynamic queries in FWD differs a bit, especially throug... Ovidiu Maxiniuc

04:55 AM Bug #9396: Improve Incompatible data types in expression or assignment. (223) error handling

*Review of 8396a*
* While the approach should fix the evident problem, I think it is too relaxed. Mind that @1 + 1... Alexandru Lungu

04:17 AM Bug #9396: Improve Incompatible data types in expression or assignment. (223) error handling

Ovidiu, the work-flow of static and dynamic queries in FWD differs a bit, especially through @DynamicQueryHelper@. @f... Alexandru Lungu

04:34 AM Bug #8309: Do the error handling for r-index and extend implementation to INDEX and LONGCHAR

Greg Shah wrote:
> Yes, exactly this. Luckily, this is a pretty rare case because they try to keep OE backwards com... Marian Edu

12/03/2024

05:41 PM Bug #9396: Improve Incompatible data types in expression or assignment. (223) error handling

Alexandru Lungu wrote:
> *Ovidiu*, can you provide an insight for this matter?
First, a bit of wisdom :) from my ... Ovidiu Maxiniuc

08:49 AM Bug #9396 (Review): Improve Incompatible data types in expression or assignment. (223) error hand...

Committed the changes to *9396a/15588*. Partially reverted changes from 15583 and 15584 revisions.
Please review.
... Dănuț Filimon

07:58 AM Bug #9396: Improve Incompatible data types in expression or assignment. (223) error handling

Alexandru Lungu wrote:
> > 528 - 4359 times (ProgressParserTokenTypes.KW_AND)
>
> I wonder if this is something l... Dănuț Filimon

07:03 AM Bug #9396: Improve Incompatible data types in expression or assignment. (223) error handling

> 528 - 4359 times (ProgressParserTokenTypes.KW_AND)
I wonder if this is something line @tt.f1 = 1 and true@ kind ... Alexandru Lungu

06:59 AM Bug #9396: Improve Incompatible data types in expression or assignment. (223) error handling

Alexandru Lungu wrote:
> Can you check what the construct is in that case? I expect to see UPPER(RTRIM()) kind of co... Dănuț Filimon

06:55 AM Bug #9396: Improve Incompatible data types in expression or assignment. (223) error handling

Can you check what the construct is in that case? I expect to see UPPER(RTRIM()) kind of combination in 99% of that c... Alexandru Lungu

06:12 AM Bug #9396: Improve Incompatible data types in expression or assignment. (223) error handling

I've run the analysis on how often we get @FUNCTION(literal)@ constructs (the scenario involves the extended performa... Dănuț Filimon

04:47 AM Bug #9396: Improve Incompatible data types in expression or assignment. (223) error handling

I also committed the logical(int) constructor change to *9396a/15586*.
Adjusted Done%. Dănuț Filimon

04:45 AM Bug #9396: Improve Incompatible data types in expression or assignment. (223) error handling

Alexandru Lungu wrote:
> Danut, lets go for to the more evident fix of not allowing the literals to be extracted in ... Dănuț Filimon

04:42 AM Bug #9396 (WIP): Improve Incompatible data types in expression or assignment. (223) error handling

Danut, lets go for to the more evident fix of not allowing the literals to be extracted in such complex scenarios. Th... Alexandru Lungu

04:33 AM Bug #9396: Improve Incompatible data types in expression or assignment. (223) error handling

Alexandru, you are right. The problem also happens with other functions, the example:... Dănuț Filimon

04:15 AM Bug #9396: Improve Incompatible data types in expression or assignment. (223) error handling

> Is the literal extraction happening only in f1 = 1 kind is situations? Or f1 + 1 = f2 is also a valid case in which... Alexandru Lungu

03:51 AM Bug #9396: Improve Incompatible data types in expression or assignment. (223) error handling

Somehow I modified this, sorry. Dănuț Filimon

03:47 AM Bug #9396: Improve Incompatible data types in expression or assignment. (223) error handling

> where the injected variable is actually a CHARACTER but the conversion should fail because of the datetime function... Dănuț Filimon

05:39 AM Bug #9411: Implement SESSION:STREAM

This task is created to cover the implementation details of SESSION:STREAM. Alexandru Lungu

05:39 AM Bug #9411 (Closed): Implement SESSION:STREAM

Alexandru Lungu

12/02/2024

10:29 AM Bug #9396: Improve Incompatible data types in expression or assignment. (223) error handling

*Review of 9396a*
* The logical constructor can be condensed: @assign(value != 0)@. I agree that the constructor s... Alexandru Lungu

08:19 AM Support #9398 (Review): Incomplete SESSION:DATE-FORMAT implementation in FWD and setting session ...

Vladimir Tsichevski

08:19 AM Support #9398 (WIP): Incomplete SESSION:DATE-FORMAT implementation in FWD and setting session dat...

The issue fixed in 9398a rev. 15587.
Now the @date-format@ client option can be used to set the @SESSION:DATA-FORM... Vladimir Tsichevski

11/29/2024

10:20 AM Support #9398: Incomplete SESSION:DATE-FORMAT implementation in FWD and setting session date form...

I've created the 9398a branch for fixing this. Vladimir Tsichevski

09:35 AM Bug #9396 (Review): Improve Incompatible data types in expression or assignment. (223) error hand...

Committed *9396a/15585*. Added logical(int) constructor to fix #9396-7.
All written tests passed (tests cover dyna... Dănuț Filimon

09:13 AM Bug #9396: Improve Incompatible data types in expression or assignment. (223) error handling

Found the following test:... Dănuț Filimon

07:01 AM Bug #9396: Improve Incompatible data types in expression or assignment. (223) error handling

Dănuț Filimon wrote:
> This turns out to be more complicated for certain cast functions, for example you can have @D... Dănuț Filimon

03:43 AM Bug #9396: Improve Incompatible data types in expression or assignment. (223) error handling

This turns out to be more complicated for certain cast functions, for example you can have @DATETIME(DATE(3,3,2001), ... Dănuț Filimon

06:08 AM Bug #8309 (Internal Test): Do the error handling for r-index and extend implementation to INDEX a...

I am OK with the 8309a changes. Alexandru Lungu

11/28/2024

08:39 AM Support #9398: Incomplete SESSION:DATE-FORMAT implementation in FWD and setting session date form...

In FWD, the command line options that can be specified in OE are handled using our @com.goldencode.p2j.main.StartupPa... Greg Shah

08:12 AM Support #9398: Incomplete SESSION:DATE-FORMAT implementation in FWD and setting session date form...

Progress allows to set the @SESSION:DATE-FORMAT@ from command-line using the @Date Format@ (@-d@) option.
Does FWD... Vladimir Tsichevski

08:09 AM Support #9398 (Test): Incomplete SESSION:DATE-FORMAT implementation in FWD and setting session da...

Vladimir Tsichevski

07:08 AM Bug #9396: Improve Incompatible data types in expression or assignment. (223) error handling

*Committed 9396a/15583*. This commit fixes all of my testcases related to #9396-2 table (except the 3rd bullet point)... Dănuț Filimon

06:42 AM Bug #9396: Improve Incompatible data types in expression or assignment. (223) error handling

The third bullet point is a regression added in trunk/15456, @recordOrShowError@ throws a @DeferredLegacyErrorExcepti... Dănuț Filimon

06:10 AM Bug #9396 (WIP): Improve Incompatible data types in expression or assignment. (223) error handling

From #9377-8:
I've created unit tests for testing the behavior of *temp-tables* fields and literals:
|_. Field ty... Dănuț Filimon

06:04 AM Bug #9396 (WIP): Improve Incompatible data types in expression or assignment. (223) error handling

Dănuț Filimon

04:38 AM Bug #8643 (Test): ErrorManager not handling caught errors and error-status:error attribute properly

Eduard Soltan

04:38 AM Bug #8643: ErrorManager not handling caught errors and error-status:error attribute properly

Branch 8643a was merged into trunk 15582 and archived. Eduard Soltan

11/27/2024

01:18 PM Bug #8309: Do the error handling for r-index and extend implementation to INDEX and LONGCHAR

Greg Shah wrote:
> > Does this mean we shall make FWD runtime honor the version returned by @EnvironmentOps@?
>
>... Vladimir Tsichevski

12:37 PM Bug #8309: Do the error handling for r-index and extend implementation to INDEX and LONGCHAR

> Does this mean we shall make FWD runtime honor the version returned by @EnvironmentOps@?
Yes, exactly this. Luc... Greg Shah

12:35 PM Bug #8309: Do the error handling for r-index and extend implementation to INDEX and LONGCHAR

Greg Shah wrote:
> I know it isn't nice to write, but in the 4GL code we can use @PROVERSION@ and have conditional m... Vladimir Tsichevski

12:32 PM Bug #8309: Do the error handling for r-index and extend implementation to INDEX and LONGCHAR

Greg Shah wrote:
> I know it isn't nice to write, but in the 4GL code we can use @PROVERSION@ and have conditional m... Vladimir Tsichevski

10:59 AM Bug #8309: Do the error handling for r-index and extend implementation to INDEX and LONGCHAR

I know it isn't nice to write, but in the 4GL code we can use @PROVERSION@ and have conditional matching code. On th... Greg Shah

08:55 AM Bug #8309: Do the error handling for r-index and extend implementation to INDEX and LONGCHAR

Greg Shah wrote:
> Marian's team currently writes everything for 12.2 since that is what they have. That doesn't me... Marian Edu

08:46 AM Bug #8309: Do the error handling for r-index and extend implementation to INDEX and LONGCHAR

Marian's team currently writes everything for 12.2 since that is what they have. That doesn't meant that that is the... Greg Shah

08:29 AM Bug #8309: Do the error handling for r-index and extend implementation to INDEX and LONGCHAR

Greg Shah wrote:
> We MUST support both. Please add runtime version checking using the @EnvironmentOps@ "Progress" ... Vladimir Tsichevski

08:22 AM Bug #8309: Do the error handling for r-index and extend implementation to INDEX and LONGCHAR

We MUST support both. Please add runtime version checking using the @EnvironmentOps@ "Progress" version helpers. Greg Shah

07:42 AM Bug #8309: Do the error handling for r-index and extend implementation to INDEX and LONGCHAR

Some implication found:
The @INDEX/R-INDEX@ tests now existing in the testcases assume the error message is:
<p... Vladimir Tsichevski

05:53 AM Bug #8309: Do the error handling for r-index and extend implementation to INDEX and LONGCHAR

Alexandru Lungu wrote:
> When updating I get:
> [...]
>
> The revisions you are mentioning look newer; maybe you... Vladimir Tsichevski

02:58 AM Bug #8309: Do the error handling for r-index and extend implementation to INDEX and LONGCHAR

When updating I get:... Alexandru Lungu

11:31 AM Bug #8643 (Merge Pending): ErrorManager not handling caught errors and error-status:error attribu...

You can merge after 9342a. Greg Shah

09:46 AM Bug #8643: ErrorManager not handling caught errors and error-status:error attribute properly

Greg, can I merge 8643a? Eduard Soltan

07:50 AM Bug #8643: ErrorManager not handling caught errors and error-status:error attribute properly

Run CHUI regression tests, and they are passing. Eduard Soltan

11/26/2024

02:28 PM Bug #8309: Do the error handling for r-index and extend implementation to INDEX and LONGCHAR

Greg Shah wrote:
> > > * there are many places where @final@ was added to the method parameters. This remembered me ... Vladimir Tsichevski

02:23 PM Bug #8309: Do the error handling for r-index and extend implementation to INDEX and LONGCHAR

> > * there are many places where @final@ was added to the method parameters. This remembered me of the "Overuse-of-f... Greg Shah

11:23 AM Bug #8309 (Review): Do the error handling for r-index and extend implementation to INDEX and LONG...

Alexandru Lungu wrote:
> Review of 8309a/rev. 15573:
>
> * I am OK with the functionality of the changes and the ... Vladimir Tsichevski

08:53 AM Bug #8309 (WIP): Do the error handling for r-index and extend implementation to INDEX and LONGCHAR

Alexandru Lungu

08:00 AM Bug #8309: Do the error handling for r-index and extend implementation to INDEX and LONGCHAR

Review of 8309a/rev. 15573:
* I am OK with the functionality of the changes and the overall refactoring!
* minor ... Alexandru Lungu

09:15 AM Bug #8643: ErrorManager not handling caught errors and error-status:error attribute properly

I have run again *one by one* the fwd tests that failed only with @8643a@ and they passed. Andrei Plugaru

07:51 AM Bug #8643: ErrorManager not handling caught errors and error-status:error attribute properly

Andrei Plugaru wrote:
> *The testing I have done yesterday was probably flawed.*
>
> The problem was that I didn'... Andrei Plugaru

06:23 AM Bug #8643: ErrorManager not handling caught errors and error-status:error attribute properly

Constatin wrote:
> Did you restore the database before running the tests?
yes, a couple of times.
Andrei Plu... Eduard Soltan

06:09 AM Bug #8643: ErrorManager not handling caught errors and error-status:error attribute properly

*The testing I have done yesterday was probably flawed.*
The problem was that I didn't reimport the DB before runn... Andrei Plugaru

05:56 AM Bug #8643: ErrorManager not handling caught errors and error-status:error attribute properly

ETF testing passed Constantin Asofiei

04:11 AM Bug #8643: ErrorManager not handling caught errors and error-status:error attribute properly

Eduard Soltan wrote:
> But the baseline result for fwd tests that I obtained it is a bit worse that the result Andre... Constantin Asofiei

11/25/2024

02:13 PM Bug #8643: ErrorManager not handling caught errors and error-status:error attribute properly

Andrei Plugaru wrote:
> I have run the fwdtests and, unfortunately, *it seems there are 5 regressions*. I would want... Eduard Soltan

01:53 PM Bug #8643: ErrorManager not handling caught errors and error-status:error attribute properly

Andrei Plugaru wrote:
> I have run the fwdtests and, unfortunately, *it seems there are 5 regressions*. I would want... Eric Faulhaber

10:28 AM Bug #8643: ErrorManager not handling caught errors and error-status:error attribute properly

I have run the fwdtests and, unfortunately, *it seems there are 5 regressions*. I would want, however, to make anothe... Andrei Plugaru

08:48 AM Bug #8643: ErrorManager not handling caught errors and error-status:error attribute properly

Andrei Plugaru wrote:
> Constantin Asofiei wrote:
> > Andrei: did you run both fwdtests and unittests?
>
> No, I... Constantin Asofiei

08:30 AM Bug #8643: ErrorManager not handling caught errors and error-status:error attribute properly

Constantin Asofiei wrote:
> Andrei: did you run both fwdtests and unittests?
No, I have only run the exact test t... Andrei Plugaru

08:29 AM Bug #8643: ErrorManager not handling caught errors and error-status:error attribute properly

I run unitest for a large application.
Also run unittest of a large GUI app, and harness of a big rest app. Eduard Soltan

08:28 AM Bug #8643 (Internal Test): ErrorManager not handling caught errors and error-status:error attribu...

Review is OK. I can run ETF - what other testing is needed?
Andrei: did you run both fwdtests and unittests? Constantin Asofiei

03:27 AM Bug #8643: ErrorManager not handling caught errors and error-status:error attribute properly

*Eduard*, I retested #9353 with @8643a rev. 15530@ and the exception is still handled right. ✅ Andrei Plugaru

03:24 AM Bug #8643: ErrorManager not handling caught errors and error-status:error attribute properly

Constantin Asofiei wrote:
> Eduard, so there are no changes required for the branch?
No, there should be a change... Eduard Soltan

03:21 AM Bug #8643: ErrorManager not handling caught errors and error-status:error attribute properly

Eduard, so there are no changes required for the branch? Constantin Asofiei

02:59 AM Bug #8643: ErrorManager not handling caught errors and error-status:error attribute properly

I run the example from #9298-38 using command line, and got the same result as if running from OE.
Got the excepti... Eduard Soltan

10:04 AM Bug #6389: LENGTH function with COLUMN option

My understanding from that time is the 'column' length is to align the text for i.e. printing. So this can not depen... Constantin Asofiei

10:02 AM Bug #6389: LENGTH function with COLUMN option

Maybe there is one more thing we can try, I was thinking that the @column@ length is determined by the width of the c... Artur Școlnic

11/22/2024

10:22 AM Bug #8309: Do the error handling for r-index and extend implementation to INDEX and LONGCHAR

Please coordinate with Marian to check these into the [[p2j:Testcases]] project. Greg Shah

06:17 AM Bug #8309: Do the error handling for r-index and extend implementation to INDEX and LONGCHAR

Attached are unit tests for testing case-sensitivity. Vladimir Tsichevski

04:10 AM Bug #8643: ErrorManager not handling caught errors and error-status:error attribute properly

Actully I think that the problematic line in this code from #8643-38.... Eduard Soltan

11/21/2024

03:10 PM Bug #8309 (Review): Do the error handling for r-index and extend implementation to INDEX and LONG...

Issue fixed in rev. 15573. Review, please. Vladimir Tsichevski

01:31 PM Bug #8309: Do the error handling for r-index and extend implementation to INDEX and LONGCHAR

Vladimir Tsichevski wrote:
> Another issue discovered: the @R-INDEX@ implementation does not support @CHARACTER@ cas... Vladimir Tsichevski

07:28 AM Bug #8309: Do the error handling for r-index and extend implementation to INDEX and LONGCHAR

Another issue discovered: the @R-INDEX@ implementation does not support @CHARACTER@ case sensitivity, the comparison ... Vladimir Tsichevski

10:13 AM Bug #8643: ErrorManager not handling caught errors and error-status:error attribute properly

Constantin Asofiei wrote:
> But a question: does this test still work in FWD with 8643a?
Well, not really. I cha... Eduard Soltan

03:41 AM Bug #8643: ErrorManager not handling caught errors and error-status:error attribute properly

The changes make sense, we've seen weird behavior for db-related errors in other places.
But a question: does this... Constantin Asofiei

11/20/2024

05:37 PM Bug #8309: Do the error handling for r-index and extend implementation to INDEX and LONGCHAR

Branch 8309a rev. 15571 now handles invalid index argument for @R-INDEX@ correctly.
ABLUnit tests attached for the @... Vladimir Tsichevski

12:14 PM Bug #8309 (WIP): Do the error handling for r-index and extend implementation to INDEX and LONGCHAR

I've created the 8309a branch for this. Vladimir Tsichevski

09:55 AM Bug #6389: LENGTH function with COLUMN option

I am sorry, I forgot to add the 'destination' parameter for @chr@ function. I updated the procedures above. It turn o... Ovidiu Maxiniuc

09:19 AM Bug #6389: LENGTH function with COLUMN option

Constantin Asofiei wrote:
> But: are we sure characters in a certain string are not 'merged' into i.e. 3 columns eve... Artur Școlnic

09:10 AM Bug #6389: LENGTH function with COLUMN option

I extended the extractor to:... Ovidiu Maxiniuc

09:07 AM Bug #6389: LENGTH function with COLUMN option

We can keep this data in a file (or an array) directly, no need to complicate things with intervals.
But: are we s... Constantin Asofiei

09:02 AM Bug #6389: LENGTH function with COLUMN option

Correct me if I am wrong, but the main problem with an approach like this is the fact that we can't rely on intervals... Artur Școlnic

08:49 AM Bug #6389: LENGTH function with COLUMN option

To extract the column lengths for all 64Ki Unicode characters we can use the following procedure (do not forget to st... Ovidiu Maxiniuc

07:59 AM Bug #6389: LENGTH function with COLUMN option

Artur Școlnic wrote:
> Ovidiu, won't manually scanning the Unicode planes for any type of characters limit the funct... Ovidiu Maxiniuc

07:08 AM Bug #6389: LENGTH function with COLUMN option

Yes, the results are from OE. In FWD the @length@ function does not work with any of the parameters if multi byte cha... Artur Școlnic

07:03 AM Bug #6389: LENGTH function with COLUMN option

Are these results from OE? Greg Shah

06:12 AM Bug #6389: LENGTH function with COLUMN option

I was operating under the assumption that @chr(, "utf-8")@ encodes a character using utf-8 and 4gl will calculate the... Artur Școlnic