Beefy Boxes and Bandwidth Generously Provided by pair Networks
"be consistent"
 
PerlMonks  

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

Ron,

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:

Before:

gen_files => { 'my_thumb' => gen_thumb({ w => 100, h => 100 }), }
After:
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
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?
Username:
Password:

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

How do I use this? | Other CB clients
Other Users?
Others taking refuge in the Monastery: (10)
As of 2014-12-17 21:43 GMT
Sections?
Information?
Find Nodes?
Leftovers?
    Voting Booth?

    Is guessing a good strategy for surviving in the IT business?





    Results (34 votes), past polls