Jump to content
horst

Croppable Image 3

Recommended Posts

I don't understand. Please can you clarify, maybe with a short explanation, what you want to do?

Share this post


Link to post
Share on other sites

Yes of course.

When the module is cropping an image, a variation image is created in /assets/files/id by CroppableImage3Helpers::renderImage method. I want to hook to the method like this to have treatments on the newly created image. Right now it is not hookable. My use case is to upload the image to AWS S3 if I want to make a stateless webserver. I have made the same request to processwire-request for your information.

I think there can be more use cases. One of them always on my mind is generating a small placeholder image for lazy loading.

Share this post


Link to post
Share on other sites

I think you don't need to hook into this module. You need to hook into Pageimage, doing the following:

  • check if the requested variation is available on your AWS S3 remote storage,
  • and if yes, just link to it,
  • and if no, create a new one, upload it and finally link to it.

This is independent from the different ImageRenderingEngines and extra modules.

EDIT:

Have you seen this:

 

Share this post


Link to post
Share on other sites

Thanks for your suggestion.

I have tried what you have suggested before. The problem of this approach is that checking remote asset availability needs an extra round trip and that increases the page loading time a lot. If there are many images in a page, every PageImage call would have to check if the remote image is available or not. This is not feasible. Please correct me if I am wrong.

I think of a workaround for handling this module. As page editor will save the page, most likely, after adding new crop, I can hook to page save to sync with AWS S3, although the module does not require hitting save to make changes.

Share this post


Link to post
Share on other sites

What is the primary benefit for hosting images in the S3 cloud?

Or the other way round: do you need to save diskspace on your local server?

Share this post


Link to post
Share on other sites

For webapp and e-commerce shops, high availability is critical. To achieve this, one of the approaches is using multiple stateless web servers running in parallel, by keeping all assets and database away from the web servers. In this way, we can add or remove as many web server as we need. Assets can be kept safe and synchronized across all web servers.

I use AWS's Elastic Beanstalk to achieve this infrastructure. The ec2 servers controlled by Elastic Beanstalk will be terminated and rebuilt from time to time, so stateless servers are needed to use this kind of service or infrastructure.

  • Like 2

Share this post


Link to post
Share on other sites
48 minutes ago, horst said:

Have you seen this:

Yes. I have tried this module. It is working great with normal file, but using this would lose the image inputfiled editor, and the feature of Croppable Image 3, which I am using in my every project.😭

Share this post


Link to post
Share on other sites

I've never dealt with AWS, and although I'm sure you've searched around, I found some forum threads dealing with similar issues.

https://processwire.com/blog/posts/amazon-aws-now-powering-processwire.com-sites/

^ nothing specific about image-cropping (or images in general), but maybe @ryan has built something which would help you find a solution?

(don't want to pollute this thread, just thinking maybe you'll find some hints or code-snippets...)

Share this post


Link to post
Share on other sites

Thanks @dragan. I don't want to off topic much too. I am actually forking the module you mentioned to make it a more robust one. I would share the module once it is completed. It will be working silently and seamlessly with other modules. As Croppable Image 3 is my must use module, I just seek any chance that I can make the integration better.😅(trying to relate Croppable Image 3 more)

Share this post


Link to post
Share on other sites

Hhm, so you will have one or multiple webservers with different states of variations, but only the AWS S3 cloud is always containing everything.

What's about your own idea, hooking into $page->save():

  • check if imagefield(s) exist in that edited page
  • if yes, compare all image and variation timestamps and upload those from within the last x minutes.

This way, you are completly free to use what ever image tools you like.

  • Like 1

Share this post


Link to post
Share on other sites

Hi Horst

Possible bug...

When I'm in the popup window which displays the crop preview, there's a sharpening dropdown. If I select None as the sharpening value, the next screen (the Save screen) lists the sharpening as Soft.

It also does this when I set the sharpening to Strong. I haven't checked the others (Medium, Hard etc)

ProcessWire: 3.0.119
CroppableImage3: Latest
 

Share this post


Link to post
Share on other sites

@horst How can I add a current timestamp at the end of the suffix everytime a crop is saved? With @adrian's module ProcessCustomUploadNames it renames all image variations so I'm wondering if it's possible to only rename the variations with the suffix we're having in Crop Settings:

So instead of:

my_image.jpg
my_image.-suffix.jpg

We should have something like this:

my_image.jpg
my_image.-suffix-1543078585.jpg

Share this post


Link to post
Share on other sites

Replacing the 221 line of ProcessCroppableImage3.module with this:

$basename = basename($img->basename, '.' . $img->ext) . '.-' . strtolower($suffix) . '-' . $img->mtime . '.' . $img->ext;

and the 224 line of FieldtypeCroppableImage3.module with this:

$basename = basename($img->basename, '.' . $img->ext) . ".-$suffix" . '-' . $img->mtime . "." . $img->ext;

it adds the current timestamp everytime the cropped has changed but the problem is that variations with previous timestamps are not deleted. @horst can you please help?

Share this post


Link to post
Share on other sites
On 11/22/2018 at 6:08 PM, Peter Knight said:

Hi Horst

Possible bug...

When I'm in the popup window which displays the crop preview, there's a sharpening dropdown. If I select None as the sharpening value, the next screen (the Save screen) lists the sharpening as Soft.

It also does this when I set the sharpening to Strong. I haven't checked the others (Medium, Hard etc)

ProcessWire: 3.0.119
CroppableImage3: Latest
 

@Peter Knight, please can you try the following fix manually on your side?

In the file site/modules/CroppableImage3/ProcessCroppableImage3/ProcessCroppableImage3.module on the lines 234 & 235 you must remove the wrapping isset(...) on each first checked param per line!

This

233            // get quality & sharpening from post, sanitize and validate it
234            $quality = isset($this->input->post->quality) && is_numeric($this->input->post->quality) && 0<intval($this->input->post->quality) && 101>intval($this->input->post->quality) ? intval($this->input->post->quality) : $options['quality'];
235            $sharpening = isset($this->input->post->sharpening) && ImageSizer::sharpeningValueStr($this->input->post->sharpening)==$this->input->post->sharpening ? $this->input->post->sharpening : $options['sharpening'];

should become this:

233            // get quality & sharpening from post, sanitize and validate it
234            $quality = $this->input->post->quality && is_numeric($this->input->post->quality) && 0 < intval($this->input->post->quality) && 101 > intval($this->input->post->quality) ? intval($this->input->post->quality) : $options['quality'];
235            $sharpening = $this->input->post->sharpening && ImageSizer::sharpeningValueStr($this->input->post->sharpening)==$this->input->post->sharpening ? $this->input->post->sharpening : $options['sharpening'];

 

If it fixes it on your side too, I will release a new version of the module.

  • Like 1

Share this post


Link to post
Share on other sites
5 hours ago, PWaddict said:

t adds the current timestamp everytime the cropped has changed but the problem is that variations with previous timestamps are not deleted. @horst can you please help?

Sorry @PWaddict, I only can support standard-conform variation names. But I'm wondering why you need to hack the code.

I don't know what the custom upload names module does, but if it does something like striping complete variation names including suffixes, I have no option to help.

Share this post


Link to post
Share on other sites
1 hour ago, horst said:

Sorry @PWaddict, I only can support standard-conform variation names. But I'm wondering why you need to hack the code.

I don't know what the custom upload names module does, but if it does something like striping complete variation names including suffixes, I have no option to help.

The reason I want to add timestamps in the cropped variations is because I'm also using them as og:image and Facebook will not refresh the image if I changed the crop unless the filename or page url are renamed. So, adding timestamps inside filename is the perfect solution for this.

@adrian's module renames all image variations (normal & cropped ones) and It doesn't affect CroppableImage3's suffixes.

Share this post


Link to post
Share on other sites
1 hour ago, PWaddict said:

@adrian's module renames all image variations (normal & cropped ones) and It doesn't affect CroppableImage3's suffixes.

Couldn't CustumUploadNames be modified to handle this need?

Share this post


Link to post
Share on other sites
11 minutes ago, adrian said:

Couldn't CustumUploadNames be modified to handle this need?

I don't know. I tried to modified CroppableImage3 because it seems easier since it only renames the cropped variations.

Share this post


Link to post
Share on other sites

Croppableimage does not use own suffixes. It uses pw cores default image variation suffixes. 

Can you not simply add the timestamp via the options array?

Or what is the part that I do not understand?

Maybe you can give me a variation name as example, show what custom upload names do with it, and what cai3 does. 

Also please tell me when the modules affect the variation names, in which order. 

Share this post


Link to post
Share on other sites
8 hours ago, horst said:

@Peter Knight, please can you try the following fix manually on your side?

In the file site/modules/CroppableImage3/ProcessCroppableImage3/ProcessCroppableImage3.module on the lines 234 & 235 you must remove the wrapping isset(...) on each first checked param per line!

 

Thanks @horst
that worked. 

When I set sharpening to none, it was reflected on the final save window.

I'm still getting a slight pixelation on the source jpg but it's probably non Module related. It's hardly noticeable but would be keen to hear if you think it could be.

 

  • Like 1

Share this post


Link to post
Share on other sites

@Peter Knight, thanks for testing. 

With php 5.x versions it works using isset(). With php 7.x versions it doesn't work anymore. 

About the pixelation, it also may have to do with the scaling factor. Also, if possible, you may compare gd and imagick engines. 

Share this post


Link to post
Share on other sites
18 hours ago, horst said:

Can you not simply add the timestamp via the options array?

Can you show me an example how can I do that? On Crop Settings I have "thumbnail,1200,600" so the main variation image is like myimage.-thumbnail.jpg. How can I rename it to myimage.-thumbnail-1543078585.jpg with the options array?

Share this post


Link to post
Share on other sites

Using the options array it creates another variation file with the manual suffix I mention below. It doesn't rename the main variation image.

$og_image->getCrop('thumbnail', "width=1200, suffix=test")->httpUrl;

 

Share this post


Link to post
Share on other sites

@PWaddict

I may seem a little impatient now, but I still don't understand what you want after all your posts. Please just write down 2 or three variation names where I can see what is and what should be. Otherwise I cannot deal with it any further. (I have three deadlines this week!)

Share this post


Link to post
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.


  • Recently Browsing   0 members

    No registered users viewing this page.

  • Similar Content

    • By Mike Rockett
      Jumplinks for ProcessWire
      Release: 1.5.56
      Composer: rockett/jumplinks
      Jumplinks is an enhanced version of the original ProcessRedirects by Antti Peisa.
      The Process module manages your permanent and temporary redirects (we'll call these "jumplinks" from now on, unless in reference to redirects from another module), useful for when you're migrating over to ProcessWire from another system/platform. Each jumplink supports wildcards, shortening the time needed to create them.
      Unlike similar modules for other platforms, wildcards in Jumplinks are much easier to work with, as Regular Expressions are not fully exposed. Instead, parameters wrapped in curly braces are used - these are described in the documentation.
      Under Development: 2.0, to be powered by FastRoute
      As of version 1.5.0, Jumplinks requires at least ProcessWire 2.6.1 to run.
      View on GitLab
      Download via the Modules Directory
      Read the docs
      Features
      The most prominent features include:
      Basic jumplinks (from one fixed route to another) Parameter-based wildcards with "Smart" equivalents Mapping Collections (for converting ID-based routes to their named-equivalents without the need to create multiple jumplinks) Destination Selectors (for finding and redirecting to pages containing legacy location information) Timed Activation (activate and/or deactivate jumplinks at specific times) 404-Monitor (for creating jumplinks based on 404 hits) Additionally, the following features may come in handy:
      Stale jumplink management Legacy domain support for slow migrations An importer (from CSV or ProcessRedirects) Feedback & Feature Requests
      I’d love to know what you think of this module. Please provide some feedback on the module as a whole, or even regarding smaller things that make it whole. Also, please feel free to submit feature requests and their use-cases.
      Note: Features requested so far have been added to the to-do list, and will be added to 2.0, and not the current dev/master branches.
      Open Source

      Jumplinks is an open-source project, and is free to use. In fact, Jumplinks will always be open-source, and will always remain free to use. Forever. If you would like to support the development of Jumplinks, please consider making a small donation via PayPal.
      Enjoy! :)
    • By BitPoet
      As threatened in Ryan's announcement for 3.0.139, I built a little module for sliding toggles as a replacement for checkboxes. Styling of the input is CSS3 only (with all the usual caveats about older browsers), no JS necessary, and may still be a bit "rough around the edges", so to speak, since I didn't have much time for testing on different devices or brushing things up enough so I'd feel comfortable pushing it to the module directory. But here's the link to the GitHub repo for now:
      InputfieldSlideToggle
      Fieldtype and Inputfield that implements smartphone-style toggles as replacement for checkbox inputs. The visualization is CSS-only, no additional JS necessary.
      Status
      Still very alpha, use with caution!
      Features / Field Settings
      Size
      You can render the toggles in four different sizes: small, medium, large and extra large.
      Off Color
      Currently, "unchecked" toggles can be displayed either in grey (default) or red.
      On Color
      "Checked" toggles can be rendered in one of these colors: blue (default), black, green, grey, orange or red.
      Screenshots

      Some examples with checkbox label


      View all Size and Color Combinations
      Small toggles Medium toggles Big toggles Extra big toggles  









    • By Orkun
      Hi Guys
      I needed to add extended functionalities for the InputfieldDatetime Module (module is from processwire version 2.7.3) because of a Request of Customer.
      So I duplicated the module and placed it under /site/modules/.
      I have added 3 new Settings to the InputfieldDatetime Module.
      1. Day Restriction - Restrict different days based on weekdays selection (e.g. saturday, sunday) - WORKING

       
      2. Time Slots - Define Time slots based on custom Integer Value (max is 60 for 1 hour) - WORKING

       
      3. Time Range Rules per Weekday - Define a minTime and MaxTime per Weekday (e.g. Opening Hours of a Restaurant) - NOT WORKING PROPERLY

       
      The Problem
      Time Slots and Day Restriction working fine so far. But the Time Range Rules per Weekday doesn't work right.
      What should happen is, that when you click on a date, it should update the minTime and maxTime of the Time Select.
      But the change on the select only happens if you select a date 2 times or when you select a date 1 time and then close the datepicker and reopen it again.
      The time select doesn't get change when you select a date 1 time and don't close the picker.
      Here is the whole extended InputfieldDatetime Module.
      The Files that I have changed:
      InputfieldDatetime.module InputfieldDatetime.js jquery-ui-timepicker-addon.js (https://trentrichardson.com/examples/timepicker/) - updated it to the newest version, because minTime and maxTime Option was only available in the new version  
      Thats the Part of the JS that is not working correctly:
      if(datetimerules && datetimerules.length){ options.onSelect = function(date, inst) { var day = $(this).datetimepicker("getDate").getDay(); day = day.toString(); var mintime = $(this).attr('data-weekday'+day+'-mintime'); var maxtime = $(this).attr('data-weekday'+day+'-maxtime'); console.log("weekday: "+day); console.log("minTime: "+mintime); console.log("maxTime: "+maxtime); var optionsAll = $(this).datetimepicker( "option", "all" ); optionsAll.minTime = mintime; optionsAll.maxTime = maxtime; $(this).datetimepicker('destroy'); $(this).datetimepicker(optionsAll); $(this).datetimepicker('refresh'); //$.datepicker._selectDate($(this).attr("id"),date); //$.datepicker._base_getDateDatepicker(); // var inst = $.datepicker._getInst($(this)); // $.datepicker._updateDatepicker(inst); /*$(this).datetimepicker('destroy'); InputfieldDatetimeDatepicker($(this), mintime, maxtime); $(this).datetimepicker('refresh'); */ // $(this).datetimepicker('option', {minTime: mintime, maxTime: maxtime}); } } Can you have a look and find out what the Problem is?
      InputfieldDatetime.zip
       
      Kind Regards
      Orkun
    • By teppo
      This module tracks changes, additions, removals etc. of public (as in "not under admin") pages of your site. Like it's name says, it doesn't attempt to be a version control system or anything like that - just a log of what's happened.
      At the moment it's still a work in progress and will most likely be a victim of many ruthless this-won't-work-let's-try-that-instead cycles, but I believe I've nailed basic functionality well enough to post it here.. so, once again, I'll be happy to hear any comments you folks can provide
      https://modules.processwire.com/modules/process-changelog/
      https://github.com/teppokoivula/ProcessChangelog
      How does it work?
      Exactly like it's (sort of) predecessor, Process Changelog actually consists of two modules: Process Changelog and Process Changelog Hooks. Hooks module exists only to serve main module by hooking into various functions within Pages class, collecting data of performed operations, refining it and keeping up a log of events in it's own custom database table (process_changelog.) Visible part is managed by Process Changelog, which provides users a (relatively) pretty view of the contents of said log table.
      How do you use it?
      When installed this module adds new page called Changelog under Admin > Setup which provides you with a table view of collected data and basic filtering tools See attached screenshots to get a general idea about what that page should look like after a while.
      For detailed installation instructions etc. see README.md.
       


    • By Gadgetto
      Status update links (inside this thread) for SnipWire development will be always posted here:
      2019-08-08
      2019-06-15
      2019-06-02
      2019-05-25
      If you are interested, you can test the current state of development:
      https://github.com/gadgetto/SnipWire
      Please note that the software is not yet intended for use in a production system (alpha version).
      If you like, you can also submit feature requests and suggestions for improvement. I also accept pull requests.
      ---- INITIAL POST FROM 2019-05-25 ----
      I wanted to let you know that I am currently working on a new ProcessWire module that fully integrates the Snipcart Shopping Cart System into ProcessWire. (this is a customer project, so I had to postpone the development of my other module GroupMailer).
      The new module SnipWire offers full integration of the Snipcart Shopping Cart System into ProcessWire.
      Here are some highlights:
      simple setup with (optional) pre-installed templates, product fields, sample products (quasi a complete shop system to get started immediately) store dashboard with all data from the snipcart system (no change to the snipcart dashboard itself required) Integrated REST API for controlling and querying snipcart data webhooks to trigger events from Snipcart (new order, new customer, etc.) multi currency support self-defined/configurable tax rates etc. Development is already well advanced and I plan to release the module in the next 2-3 months.
      I'm not sure yet if this will be a "Pro" module or if it will be made available for free.
      I would be grateful for suggestions and hints!
      (please have a look at the screenshots to get an idea what I'm talking about)
       




×
×
  • Create New...