Add 'none' border for JasperStudio XLSX Exports

Issue #573 wontfix
flo west created an issue

Hi,

i got a commit request attached referring to the borders.py. To work with the XLSX files from JasperStudio it is needed to include also the string! value border style 'none'.

Find attached my updated version.

Would be cool to have this in future releases, so that i don't have to patch this manually for each new update.

Bye,

Florian

Comments (5)

  1. CharlieC

    I don't see what the issue is Do you want support for <border style="none" />? We won't be supporting this because it means that the value "none" has to equal None in Python. This would add unnecessary complexity. Jasper produces some pretty weird XML.

    We also don't accept patches like this, especially without tests.

  2. flo west reporter

    Yeah, apparently that is what I need. The border 'none' - and i am aware that this is weird.

    Thanks anyway! You do a great job!

  3. CharlieC

    Please provide a test file. But it would also make sense to file an issue with Jasper so they don't create files like this in the first place.

  4. Log in to comment