Implement AD_FieldStyle for Info Windows

Description

Color lines in info window as discussed here:
https://groups.google.com/forum/#!topic/idempiere/7G5b3_xNfFk

And also implement about Special Form: we fixed this to workd with info windows related process - IDEMPIERE-3953

Environment

None

Activity

Show:
Carlos Ruiz
December 15, 2018, 2:19 PM

Hi ,

Attached two files:

- your patch converted to version 6.1z

- a small peer review from my side:

  • fixing the name of migration script (yyyymmdd)

  • fixing the oracle migration script (it was different than postgres version)

  • creating the sysconfig key and changing it to client scope

I'm not sure if is good to have the field style for the whole line instead of individual cells. Maybe field style would go better to AD_InfoColumn?

Also, on the summary, what I expected is to show the summary of a column, but I noticed is showing the summary just for selected records. That sounds useful for some cases, but I think what would be expected is the total of a column, maybe as an additional line showing totals (in case more than one column is totalized).

For the piece implementing opening forms from info window process - maybe we can integrate that to 6.1 in a different ticket - sounds like a bug fix.

Regards,

Carlos Ruiz

Murilo Habermann Torquato
May 28, 2019, 12:32 AM

Hi ,

Attached a new patch for default branch with original code + review + changes suggested:

Summary and style on column:

On CSS Style we can use info window columns as variables:

Final result with sumaary on header for selection and on last line for total amt/qty for defined columns:

On future i will extend the summary function to work in a similar way of "PrintDataGroup" letting users select more operations than only sum, but for now i think that this is enought and usefull.

Carlos Ruiz
September 17, 2019, 5:13 PM

Hi

I was reviewing this.

My old patches are obsolete, so I uploaded fresher patches:

I could not review your new patch

When downloaded it's downloading a patch from ticket - could be problem with JIRA or maybe you uploaded wrong patch?

Regards,

Carlos Ruiz

Murilo Habermann Torquato
September 17, 2019, 6:08 PM

Hi ,

sorry, it was my mistake… i'll check it later and attach an updated patch here

Murilo Habermann Torquato
September 18, 2019, 3:43 AM

Hi ,

Here you can find a newer patch with a different implementation for info column style.

An usage example on some BPartner Info columns:

This is the dictionary definition:

CSS Style uses Info Columns “ColumnName” field as context variables for conditional:

NOTE: I'm working o a better implementation for summary fields (and other operations) for info windows. I'll keep this out of this patch for now, and when it's ready i'll open another ticket (as you suggested some time ago).

Fixed

Assignee

Murilo Habermann Torquato

Reporter

Murilo Habermann Torquato

Labels

Tested By

None

Priority

Major