Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Defective PPTX format in BIRT #813 #818

Merged

Conversation

claesrosell
Copy link
Contributor

Null guard for when DimensionType.parseUnit() fails in
PPTXUtil.convertCssToEnum(). This happens typically when the
cssDimension is "auto" for margins, which really should have a special
case treatment in the TableWriter.endCell(). However, from the look of
it, the "margin" code in endCell needs a lot of refactoring and should
perhaps be done in other places in PPTXCanvas instead ( .drawText() etc
)

Null guard for when DimensionType.parseUnit() fails in
PPTXUtil.convertCssToEnum(). This happens typically when the
cssDimension is "auto" for margins, which really should have a special
case treatment in the TableWriter.endCell(). However, from the look of
it, the "margin" code in endCell needs a lot of refactoring and should
perhaps be done in other places in PPTXCanvas instead ( .drawText() etc
)
@wimjongman
Copy link
Contributor

Claes, there is something wrong with your account: https://accounts.eclipse.org/legal/eca/validation/92768

@wimjongman
Copy link
Contributor

Claes, there is something wrong with your account: https://accounts.eclipse.org/legal/eca/validation/92768

Never mind, I revalidated and now it is fine. There was a strange drop in the internet. I saw several independent servers failing at the same time. Maybe a glitch in the matrix...

@claesrosell
Copy link
Contributor Author

I needed to re-validate for some reason.

@joachim-wesenberg
Copy link

The change in 4.9.0-20220208-win32 has solved the problem (in the pptx format for Powerpoint as well as LibreOffice). Could you please close this issue.
Thank you very much to all of you (especially Claes)!

@wimjongman
Copy link
Contributor

@claesrosell can you squash and merge?

@claesrosell
Copy link
Contributor Author

As soon as I get my account properly configured. Hopefully I will have time to fix that tonight. 😎

@wimjongman
Copy link
Contributor

As soon as I get my account properly configured. Hopefully I will have time to fix that tonight. 😎

Cool. Make sure that you always commit with your committer email address and not accidentally with your gmail.

@claesrosell claesrosell merged commit 178ca77 into eclipse-birt:master Feb 9, 2022
@claesrosell claesrosell deleted the Defective_PPTX_format_#813 branch February 9, 2022 21:02
@wimjongman
Copy link
Contributor

Woot!!!! 🎉🎉🎉

@wimjongman wimjongman added this to the 4.9 milestone Feb 9, 2022
@wimjongman wimjongman linked an issue Feb 9, 2022 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Defective PPTX format in BIRT 4.9.0-20220204-win32
3 participants