Feature request

  • Resolved Jory Hogeveen

    (@keraweb)


    8 years, 9 months ago

    I hope you guys are willing to come up with this:

    Enable non-destructive watermarking of the original images as an option.

    Problem:
    When watermarking images exept the original, these images can still be found and indexed.

    Sollution:
    Create an option to store all original images in a backup folder within the uploads folder that the image is located.
    The user can then use .htaccess to fully protect their original content.

    Todo: (when the option is enabled)

    • Create backupfolder “watermark-backup” within the images uploads folder (example: **/2016/02/watermark-backup/)
    • When saving the original image with a watermark, create a backup in the backup folder
    • When saving images with a new watermark, allways use the backup image if one is available. (htaccess wont be a problem since you’ll be working server-side).

    I can provide help on this if that is ok!
    Also, I you won’t implement this. Can you provide hooks in the plugin that I can use you implement this myself?

    https://www.ads-software.com/plugins/image-watermark/

Viewing 12 replies - 16 through 27 (of 27 total)
  • Plugin Author dFactory

    (@dfactory)

    Hello Jory,

    Yea, we’ve done some test and except from UI improvements which we will handle (don’t worry about that) there are some functional suggestions (if you’d like to do them)

    1. make the iw-backup folder reside inside Uploads dir (as defined in WP) instead of wp-content.
    2. privide a check if the created backup folder is writtable, as it turns out in some scenarios the files are not being backed up (stored in the iw-folder) even though the functionality is enabled

    But generally, it’s a very nice step forward for IW.

    Thanks,
    Bartosz / dfactory team

    Thread Starter Jory Hogeveen

    (@keraweb)

    Hi Bartosz,

    Great!

    Actually I was thinking about changing the backup folder already. It’s most probably also better for the folder permissions (your issue 2).

    Also, I will create a check if the folder is writeable and create a notice if it isn’t.

    Will let know when I’ve created a new commit!

    Regards, Jory

    Thread Starter Jory Hogeveen

    (@keraweb)

    Hi Bartosz,

    Updated the Github repository with the new features.

    – Reloate the backup folder under the wp_uploads_dir() location
    – Create a notice and disable the backup function when the filder isn’t writeable

    Let me know!

    Plugin Author dFactory

    (@dfactory)

    Great Jory,

    Will check and get back to you.

    @jory

    Getting 500 error in ajax request after adding then removing and then adding watermark again

    Thread Starter Jory Hogeveen

    (@keraweb)

    Hi Karanits,

    Do you also get some debug info from PHP?

    @jory

    Sorry for the late reply. There is no debug info which comes in cPanel error log for this. FYI, I am using Multisite.

    Thread Starter Jory Hogeveen

    (@keraweb)

    Hmm, that is strange, a 500 error should be logged in the error log of the server.
    I’m using a network aswell for development so that shouldn’t be a problem.
    I can’t reproduce your error so it’s impossible for me to debug. If you have any info please let me know.

    Plugin Author dFactory

    (@dfactory)

    Hey Jory,

    I think we can move on with this. The functionality seems to be stable, we would just make some UI improvements. How can we do that? Are you goin to make a pull request in Github?

    Thread Starter Jory Hogeveen

    (@keraweb)

    Great to hear that! And no problem, I’ll make a pull request when I get home again!

    Thread Starter Jory Hogeveen

    (@keraweb)

    Here is the pull request!
    https://github.com/dfactoryplugins/image-watermark/pull/1

    Thread Starter Jory Hogeveen

    (@keraweb)

    Any news on this? ??

Viewing 12 replies - 16 through 27 (of 27 total)
  • The topic ‘Feature request’ is closed to new replies.