Beefy Boxes and Bandwidth Generously Provided by pair Networks
good chemistry is complicated,
and a little bit messy -LW
 
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
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?
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 romping around the Monastery: (6)
As of 2015-07-30 04:12 GMT
Sections?
Information?
Find Nodes?
Leftovers?
    Voting Booth?

    The top three priorities of my open tasks are (in descending order of likelihood to be worked on) ...









    Results (269 votes), past polls