- Issue created by @DrDam
The current state of the solution is based on some hacks on Core and CROP, with some evolution on theses elements, the whole integration will be "cleaner".
- Find a way to "split" some control elements out of the "deliver" method into protected methods in order to more easily extends the method.
- exemple : L114-125 => check for scheme throwing NotFoundHttpException, can be a protected method out side of the deliver method
On ImageStyleInterface and ImageStyle class
- Add an optional string parmeter "context" on the "createDerivative" method, this parameter will be passed to effects.
On ImageEffectInterface and subclasses
- Add an optional string parmeter "context" on the "applyEffect" method (get from ImageStyle::createDerivative method)
On Crop API & plugins
- Add an optional string parmeter "context" on the Crop::findCrop method, in order to load contextual CROP (currently : context = crop_id)
- In applyEffect method on Effect Plugin Classes, send the "context" to the Crop::findCrop method
On CropEntity
- Find a better way to "encode" the context (not using the crop_id anymore) that will use as less resources as possible on crop search/load, and not a long hash (to have the "cleanest" filename).
Active
2.1
Code