Project

General

Profile

Bug #7202

'accum' frame-level widget can have widget options

Added by Constantin Asofiei about 1 year ago. Updated about 1 year ago.

Status:
Closed
Priority:
Normal
Target version:
-
Start date:
Due date:
% Done:

100%

billable:
No
vendor_id:
GCD
case_num:
version:

History

#2 Updated by Constantin Asofiei about 1 year ago

This testcase shows that a 'accum' frame widget can have widget options:

def var i as int.

do i = 1 to 10 transaction:
   accum "" (count).
   form count no-label format "999" with frame f1 10 down.
   display count with frame f1.
   down with frame f1.
end.

This does not convert properly at the frame definitions. Also, FWD doesn't convert properly even without the widget-level options for 'count'.

#3 Updated by Constantin Asofiei about 1 year ago

  • Assignee set to Constantin Asofiei
  • Status changed from New to WIP

#4 Updated by Constantin Asofiei about 1 year ago

  • % Done changed from 0 to 100
  • Status changed from WIP to Review

This is fixed in 7199a rev 14512.

#5 Updated by Greg Shah about 1 year ago

Code Review Task Branch 7199a Revision 14512

Overall I don't have an objection.

In regard to the accumulate.rules change, what is the display count with frame f1. supposed to do if it doesn't display the result?

#6 Updated by Constantin Asofiei about 1 year ago

Greg Shah wrote:

In regard to the accumulate.rules change, what is the display count with frame f1. supposed to do if it doesn't display the result?

This is not related to displaying the result - FWD was emitting the accum.getResult() twice, like this:

            FrameElement[] elementList0 = new FrameElement[]
            {
               new Element(accumCount2.getResult(), f1Frame.widgetCount(accumCount2.getResult()))
            };

#7 Updated by Greg Shah about 1 year ago

  • Status changed from Review to Test

#8 Updated by Constantin Asofiei about 1 year ago

7199a was merged to trunk rev 14514 and archived.

#9 Updated by Greg Shah about 1 year ago

  • Status changed from Test to Closed

Also available in: Atom PDF