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

provide calc pdf export module constant #12

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

mjg
Copy link
Contributor

@mjg mjg commented Jun 7, 2017

pyoo provides FILTER_PDF_EXPORT which users may mistake as being the
(non-existing) general PDF export filter, whereas in fact it is the
writer export filter, which results in an "SfxBaseModel error" 0x81a
when it is used for anything else but writer documents.

Provide FILTER_WRITER_PDF_EXPORT and FILTER_CALC_PDF_EXPORT so that the
two most used export filters are there, and their sheer presence under
these names alerts the user to use the right one (or look at their
value, and infer what to use for "math", "impress" etc.).

FILTER_PDF_EXPORT could be deprecated but is left in for compatibility
reasons for now.

pyoo provides FILTER_PDF_EXPORT which users may mistake as being the
(non-existing) general PDF export filter, whereas in fact it is the
writer export filter, which results in an "SfxBaseModel error" 0x81a
when it is used for anything else but writer documents.

Provide FILTER_WRITER_PDF_EXPORT and FILTER_CALC_PDF_EXPORT so that the
two most used export filters are there, and their sheer presence under
these names alerts the user to use the right one (or look at their
value, and infer what to use for "math", "impress" etc.).

FILTER_PDF_EXPORT could be deprecated but is left in for compatibility
reasons for now.
@mila
Copy link
Contributor

mila commented Dec 5, 2017

Hi, unfortunately I can no longer merge this pull request as I no longer work in Seznam.

I merged this change to a fork in my personal profile and uploaded a new version to PyPI. Thank you and sorry for the long inactivity.

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.

2 participants