php-reports icon indicating copy to clipboard operation
php-reports copied to clipboard

Published 20 hours ago • verified publisher icon jdorn

Dynamic Header

Open davidmichaelakers opened this issue 8 years ago • 2 comments

Adding the ability to conditionally specify headers and macros using a new dynamic header. See provided examples for possible use case. This addresses ticket #218 and other similar requests.

davidmichaelakers avatar Jun 07 '16 03:06 davidmichaelakers

I tried this patch and it did allow me to add a dynamic header but not to include a "database_options" section in a PHP report. The dropdown for that was still empty.

The first issue I encountered was case sensitivity in the Header class names.

https://github.com/jdorn/php-reports/pull/230/commits/859a1b261186500e396e397d74fe373c10526279#diff-75bde2a484a48e29ed0ba2597e579ddbR36

needs to change to something like: $report->parseHeader(ucwords(strtolower($header['name'])), $header['params']);

lolaslade avatar Feb 06 '18 19:02 lolaslade

Hi,

@davidmichaelakers I'm not able to make it work on a php report, could you provide another example using database_options on a php report ?

apcloic avatar Feb 12 '19 08:02 apcloic