Beefy Boxes and Bandwidth Generously Provided by pair Networks
go ahead... be a heretic

Re: RFC: CGI::Uploader V 2.90_01

by markjugg (Curate)
on Dec 02, 2008 at 02:32 UTC ( #727279=note: print w/ replies, xml ) Need Help??

in reply to RFC: CGI::Uploader V 2.90_01


Thanks your interest in this. I'm curious about the decision to move the imaging code into the core. Uploading could be about a lot of other transformations besides image resizing, and the old design of handing image transformations through a helper module seemed like a reasonable interface:


gen_files => { 'my_thumb' => gen_thumb({ w => 100, h => 100 }), }
transform => { imager => Imager -> new(), options => {xpixels => 400, ypixels => 500}, }
That looks worse to me, and sets a precedent for supporting imagers internally, while before there was example of adding new transformers via plugins.

Comment on Re: RFC: CGI::Uploader V 2.90_01
Select or Download Code
Replies are listed 'Best First'.
Re^2: RFC: CGI::Uploader V 2.90_01
by ron.savage (Novice) on Dec 04, 2008 at 02:46 UTC
    Hi Mark. Actually, I'm very uneasy about embedding such restricted transformation logic in the module. The feedback has confirmed my suspicions that that part needs to be re-implemented. I was a good idea to call this one 2.90_01 and not 3.00 :-). Cheers

Log In?

What's my password?
Create A New User
Node Status?
node history
Node Type: note [id://727279]
and the web crawler heard nothing...

How do I use this? | Other CB clients
Other Users?
Others chanting in the Monastery: (5)
As of 2015-12-02 02:09 GMT
Find Nodes?
    Voting Booth?

    My keyboard shows this many letters:

    Results (31 votes), past polls