Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

phpthumb - alternative solutions/libraries #285

Closed
stefanorosanelli opened this issue Sep 5, 2012 · 7 comments
Closed

phpthumb - alternative solutions/libraries #285

stefanorosanelli opened this issue Sep 5, 2012 · 7 comments

Comments

@stefanorosanelli
Copy link
Member

alternatives to http://phpthumb.sourceforge.net

@batopa
Copy link
Member

batopa commented Sep 5, 2012

Cose da verificare:

  1. compatibilit con i vari metodi usati al momento da beThumb (crop, croponly, fill, ...)
  2. compatibilit con Imagemagick (per thumbnail di formati non supportati dalla GD library tipo TIFF, EPS, etc...)
  3. possibilit di generare immagini con watermark

Al momento in http://phpthumb.gxdlabs.com/ mancano il punto 2. (non c la classe per interfacciarsi ad ImageMagick) e il 3. (ma c' una issue con un metodo che potrebbe essere usato)

Se si vuole usare questa libreria quindi andrebbe implementata la classe per imagemagick magari utilizzando la classe php http://www.php.net/manual/en/class.imagick.php (da installare il modulo php). Bisogna valutare se ne vale la pena.

Ho anche visto che hanno aggiornato http://phpthumb.sourceforge.net/ ma mi sembra faccia schifo come prima. Magari darci un occhio.

@batopa
Copy link
Member

batopa commented Sep 5, 2012

Per interfacciarsi ad imagemagick volendo c' anche questa libreria http://www.francodacosta.com/phmagick che sembra fatta bene. Ha gi metodi pronti per fare crop/resize, watermark etc...

@stefanorosanelli
Copy link
Member Author

un errore visibile su:
http://frontends.test.bedita.com/mimbo/sketch-comedies

Fatal error: Call to protected method GdThumb::setDestination() from context 'BeThumbHelper' in /var/www/test.bedita.com/bedita-app/views/helpers/be_thumb.php on line 282

@qwerg
Copy link
Member

qwerg commented Sep 5, 2012

Mi sa che questa la feature pi urgente.
Al momento il dettaglio di un media immagine non viene visualizzato.
A meno che non venga esplicitata nei params anche l'altezza, il che non va bene.

@qwerg
Copy link
Member

qwerg commented Sep 5, 2012

Replying to [comment:4 andrea]:

Mi sa che questa la feature pi urgente.
Al momento il dettaglio di un media immagine non viene visualizzato.
A meno che non venga esplicitata nei params anche l'altezza, il che non va bene.

esempio
http://test.bedita.com/multimedia/view/30

@nikazzio
Copy link
Member

nikazzio commented Sep 5, 2012

Replying to [comment:5 andrea]:

Replying to [comment:4 andrea]:

Mi sa che questa la feature pi urgente.
Al momento il dettaglio di un media immagine non viene visualizzato.
A meno che non venga esplicitata nei params anche l'altezza, il che non va bene.

esempio
http://test.bedita.com/multimedia/view/30

Ho messo a posto la visualizzazione del dettaglio dell'immagine. Era un problema di compatibilit
con il nuovo be_thumb.
In pratica ora i parametri sono:

width, height, longside: se non ne viene specificato nessunodi questi usa le dimensioni di default su bedita.ini.php altrimenti:

  • longside'' fa l'override degli altri e forza il mode in automatico a ''resize
  • se viene specificato solo uno tra w/h'' viene settato l'altro a 0 e forzato il mode a ''resize

il parametro mode'' invece funziona con: ''crop, croponly, resize
e nel caso di resize'' viene usato il ''modeparam'' per specificare se ''fill'' o ''stretch (altrimenti se assente viene fatto il resize semplice).
fill'' e ''stretch'' nel ''mode sono stati mantenuti solo per retrocompatibilit.

crop'' e ''croponly funzionano come prima.

in form_file.tpl ora c':
{assign_associative var="params" width=500 longside=false mode="fill" modeparam="000000" type=null upscale=false}

che teoricamente incongruente perch il mode fill prevedere sia altezza che larghezza. Quindi attualmente essendo specificata soltanto una coordinata funziona come se fosse un resize semplice ignorando gli altri parametri.

altre info qui: http://docs.bedita.com/frontends/embedding-images-with-beembedmedia

@batopa
Copy link
Member

batopa commented Sep 5, 2012

Andrebbe fatto un po' di refactoring su BeThumbHelper che un bel casino... ma... per il momento lascerei cos.
Chiudo ticket e apriamo eventuali ticket specifici per bug, refactoring e altro.

nikazzio pushed a commit that referenced this issue Sep 5, 2012
… - switch definitivo ed eliminazione delle vecchie librerie
nikazzio pushed a commit that referenced this issue Sep 5, 2012
… - introduzione handler per imageMagick - funzioni complementari completate.
nikazzio pushed a commit that referenced this issue Sep 5, 2012
nikazzio pushed a commit that referenced this issue Sep 5, 2012
nikazzio pushed a commit that referenced this issue Sep 5, 2012
nikazzio pushed a commit that referenced this issue Sep 5, 2012
nikazzio pushed a commit that referenced this issue Sep 5, 2012
nikazzio pushed a commit that referenced this issue Sep 5, 2012
nikazzio pushed a commit that referenced this issue Sep 5, 2012
nikazzio pushed a commit that referenced this issue Sep 5, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants