Inline cropping, filter and validate file types

[fuenfzehndreissig]

The following features has been added:

• Inline cropping and scaling with canvas (in addition to inline processing)
• Filter and validate file types to prevent upload of non image files

[sconsult]

Hi @fuenfzehndreissig

Very good additions, thank you for your effort. Filter and Refactoring are good to merge, i would like to ask one modification for the inlineCrop. Please add a dedicated option to enable inlineCrop.

Quality can in most of the cases only ensured if your using a high level image library such as imagemagick, thats why we have not added inlinecrop in the past, only the inline upload.

However i would merge the changes into the master if this option is added.

Cheers

Mat

[fuenfzehndreissig]

Hi Mat,

sorry for my late response. I spended the last weeks on holiday.

Glad to here that my additions can be useful. I will add the dedicated
option to enable inlineCrop in the next days.

Best regards
Christian

Am 06.03.2015 um 21:02 schrieb Mat Steinlin:

Hi @fuenfzehndreissig https://github.com/fuenfzehndreissig

Very good additions, thank you for your effort. Filter and Refactoring
are good to merge, i would like to ask one modification for the
inlineCrop. Please add a dedicated option to enable inlineCrop.

Quality can in most of the cases only ensured if your using a high
level image library such as imagemagick, thats why we have not added
inlinecrop in the past, only the inline upload.

However i would merge the changes into the master if this option is added.

Cheers

Mat

—
Reply to this email directly or view it on GitHub
#78 (comment).

[felipeduardo]

@fuenfzehndreissig @sconsult this PR is ok?

[bindesh]

Hi @fuenfzehndreissig

when i use inlineprocess for croppic it gives me an error "Image type is not supported" for only large images i have made changes into php.ini file but still getting this error
Croppic is working nice in it's normal process but i need to use the process inline functionality in this ..
also this image is supported in normal process but in process inline it is creating a big problem.
can you help me out of this problem as soon as possible?

here i attach my error screen shot

please reply me it's very urgent