Skip to content

Watermarks in extension - #12

Open
bchoquet-heliopsis wants to merge 4 commits into
ezsystems:masterfrom
heliopsis:watermark_repos
Open

Watermarks in extension#12
bchoquet-heliopsis wants to merge 4 commits into
ezsystems:masterfrom
heliopsis:watermark_repos

Conversation

@bchoquet-heliopsis

Copy link
Copy Markdown
Contributor

Hi,
watermarks were to be put only in ezie extension, this patch allows to define other extensions as watermark repositories.

Also, watermark filter do not work properly with eZComponents (see https://issues.apache.org/jira/browse/ZETACOMP-55) so I took the liberty to override watermarkAbsolute() in the dedicated ezie image magick handler

Comment thread classes/image_tool_watermark.php Outdated

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wouldn't it be better to use eZURLOperator::eZImage() to get the path of the file, thus allowing overriding images in custom designs? There would be no need for WatermarkExtensions setting at all, just put the images in watermarks directory in any of designs in any of extensions?

@bchoquet-heliopsis

Copy link
Copy Markdown
Contributor Author

Hi,
it's simpler indeed, thanks for your comment. The template parameter is a bit crappy though ;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants