I also added myself to relicensecheck with 3f8ef39767c7d05672032cd2b56a0d9286a39a36

I'm all for it!

More details:

Thanks! I had no idea what was causing the error.

There is a side effect here that AF_SimpleCalculate now returns a localized number which will have group Seperators according to the system locale. See my inline comment about this.

Force consistent locale in calculate text test to avoid
different results.

Update the calculatetexttest and simpleCalculate PDF with
formatting removed from the fields in simple calculate.

Just noticed that this breaks the calculatetexttest as I've accidentally also added Formatting (which Acrobat did automatically) on the calculated fields in that test document.

The test is D13588

Apologies for the delay. Some travel, some sick days and some other work and the weeks fly by ;-)
I'm getting back to this now.

Yes, I plan to add a test for this.

• Simplyfy page lookup in processFormatAction
• Clarify comment about the need to refresh calculated

fields in processFormatAction.

Remove spurious comment

Removed to currency and extended numberToString to handle this, too.

In D13170#270082, @aheinecke wrote:

In D13170#269657, @aacid wrote:

QTest::qWait( 100 ); is bad as you probably already guessed since it's "machine dependant".

Yeah, looked fishy ;-) Although stuff like that is used in part test on other places so i've copied it.

Use QTRY instead of QTest::wait. This is acutally much faster.

In D13170#269657, @aacid wrote:

QTest::qWait( 100 ); is bad as you probably already guessed since it's "machine dependant".

In D13171#269646, @aacid wrote:

I'm not very happy of the internalText method/storage in form, ideally, the text in core/form should always be the internal text, and if there's a formatted text that needs to be different that should only be as part of the widget itself. i.e. the FormLineEdit would do something like

setText( formatText( form->text() ) );

Do you think it could be implemented this way?