Beefy Boxes and Bandwidth Generously Provided by pair Networks
Pathologically Eclectic Rubbish Lister
 
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.

Replies are listed 'Best First'.
Re^2: RFC: CGI::Uploader V 2.90_01
by ron.savage (Acolyte) 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
Domain Nodelet?
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 meditating upon the Monastery: (2)
As of 2023-03-30 05:26 GMT
Sections?
Information?
Find Nodes?
Leftovers?
    Voting Booth?
    Which type of climate do you prefer to live in?






    Results (73 votes). Check out past polls.

    Notices?