intive-FDV / DynamicJasper

Dynamic Reports using Jasper Reports
http://intive-fdv.github.io/DynamicJasper/
GNU Lesser General Public License v3.0
242 stars 127 forks source link

Bug fix on the calculus of the group footer band height. #1

Closed mdaloia closed 12 years ago

mdaloia commented 12 years ago

Bug fix on the calculus of the group footer band height (was using isFitHeaderHeightToContent instead of isFitFooterHeightToContent).

juanalvarezg commented 12 years ago

thano you very much!!!!

will merge raight away!

Juan Manuel Alvarez Gimenez tel: (54-11) 5239-9899 | cel: (11) 15-4400-0852 Av. Belgrano 1580 2do piso (C1093AAQ) Buenos Aires, Argentina | www.fdvsolutions.com

On Thu, Aug 2, 2012 at 10:47 AM, Martin D'Aloia < reply@reply.github.com

wrote:

Bug fix on the calculus of the group footer band height (was using isFitHeaderHeightToContent instead of isFitFooterHeightToContent).

You can merge this Pull Request by running:

git pull https://github.com/mdaloia/DynamicJasper jasper4_1

Or you can view, comment on it, or merge it online at:

https://github.com/FDVSolutions/DynamicJasper/pull/1

-- Commit Summary --

  • * Bug fix on the calculus of the group footer band height (was using isFitHeaderHeightToContent instead of isFitFooterHeightToContent).

-- File Changes --

M src/main/java/ar/com/fdvs/dj/core/layout/AbstractLayoutManager.java (2)

-- Patch Links --

https://github.com/FDVSolutions/DynamicJasper/pull/1.patch https://github.com/FDVSolutions/DynamicJasper/pull/1.diff


Reply to this email directly or view it on GitHub: https://github.com/FDVSolutions/DynamicJasper/pull/1

juanalvarezg commented 12 years ago

I have to check if this is also happening in version 3.x

mdaloia commented 12 years ago

Yes, it is also happening in version 3.x. I will create another pull request to change it on java5 (DJ 3.x) branch.