Moved responsive images rewrite to sl_images

[lowki]

Je ne suis pas certain que ce soit une bonne idée de le mettre là, mais je n'ai pas trouvé mieux. Au passage la méthode prend un paramètre de plus pour spécifier les tailles et points de bascule que l'on souhaite avoir et sert également des images adaptées aux écrans à haute densité de pixels (Retina). A terme, il faudrait tout intégrer dans ckeditor, ou configurer les points de rupture et tailles d'images dans le template. PS: Je ne comprends pas pourquoi, mes anciens commit apparaissent toujours dans le pulll request.

[wakdev]

Oula ! Effectivement. Tu as mis à jour ton dépôt avant ?

[wakdev]

Possibilité de m'arranger ça ? car là j'ai un peu de mal à y voir clair ><

[lowki]

Oui j'ai fais un merge juste avant. Je ne sais pas si on peut effacer des commit. Je vais me renseigner :)

[wakdev]

Sinon au pire je fais un diff quand j'ai un peu de temps, puis tu vois après le merge pour rétablir ton fork à jour.

[wakdev]

J'ai un problème avec la règle htaccess qui me pète les miniatures lors de l'upload coté back-office. Aussi, je me demande si cela ne serai pas mieux d'intégrer le paramètre "adapter les images à l'écran" au niveau global (table sl_config).

[wakdev]

Ah et il manque aussi dans le dump le champ responsive_images ;) Que pense tu de l'intégration global de ce paramètre ?

[lowki]

Ha, pas moi, tu utilises IE?

[wakdev]

Le troll quoi ^^ Non je parle de la ligne permettant de créer le champ responsive_images dans le dump SQL lors de l'installation ;)

[wakdev]

Ah bin si il est là : https://github.com/lowki/slash-cms/commit/834d032c9d88ccceb1047e61df0896cc5d1aa8e2#L8L78

[wakdev]

Pour la miniature dans l'ajax_upload, j'ai bien un problème avec le htaccess ;)

[lowki]

J'étais en train de chercher la ligne :) Je te trollais sur les miniatures qui ne fonctionnent pas dans le back. Sinon, je pense qu'il faut que ça reste paramétrable pour chaque page/article. Ha ok +1 pour l'ajaxupload.

[wakdev]

Coté front j'ai aussi des problèmes d'affichages >< Et un truc bizarre...

[wakdev]

Rah, je n'ai plus d'affichage sur les images responsive ><

[lowki]

Bizarre tout ça, je n'ai pas de bug sur le front.

[wakdev]

J'ai un problème avec la règle : RewriteRule (medias/(?:images|attachments)/..(?:jpe?g|gif|png|JPE?G|GIF|PNG))(?:/(\d\w))?(?:/(\d\w*))? core/common/class/functions/includes/show_image.php?url=$1&width=$2&height=$3

Ne fonctionne pas si le site n'est pas à la racine.

[lowki]

Ca ne match pas ou ça donne une mauvaise url?

[wakdev]

Mauvaise url, du coup bin... pas d'image ^^

[lowki]

Marche impec chez moi. Quelle url il te donne? T'as quoi comme serveur?

[wakdev]

Je vais re-vérifier quand j'aurai un peu de temps, si j'ai pas une merde sur mon serveur. Ou alors j'ai pas vidé mon cache mdr ^^

[wakdev]

C'est la même :(

[wakdev]

Obligé de rajouter /slash-cms/ devant la règle. Du coup : /slash-cms/core/common/class/functions/includes/show_image.php?url=$1&width=$2&height=$3

[wakdev]

De plus forcement, cela pose aussi problème sur les lignes ErrorDocument

[wakdev]

Putain, je suis une buse en htaccess >< Je trouve pas la solution lol

[wakdev]

Bon sinon il y a la solution du RewriteBase /slash-cms/ Mais cela oblige l'utilisateur à modifier le htaccess, ce qui est pas terrible.

[lowki]

Colle ça die($url); à la ligne 249 de show_image.php et dis moi ce que ça te sort s'il te plait.

[wakdev]

http://localhost/slash-cms/medias/images/wakdev.jpg/2000 => ../../../../../medias/images/wakdev.jpg

Avec le RewriteBase /slash-cms/ Sinon c'est 404

[wakdev]

Nan, c'est vraiment la règle de ré-écriture qui merde. (voir image sans RewriteBase /slash-cms/)

[lowki]

C'est quoi ce chemin? /sources/master/

[wakdev]

C'est mon chemin local vers mes sources : E:\slash-cms\sources\master\slash-cms J'ai crée un alias Apache :

Alias /slash-cms/ "e:/slash-cms/sources/master/slash-cms/" <Directory "e:/slash-cms/sources/master/slash-cms/"> Options Indexes FollowSymLinks MultiViews AllowOverride all Order allow,deny Allow from all

[lowki]

Ok je vais essayer avec ça

[wakdev]

Sinon tant pis, on utilise le RewriteBase.

[wakdev]

En même temps, c'est un peu son rôle...

[lpotherat]

Je m'incruste, le RewriteBase peut être généré à l'installation ? ;) .htaccess n'est qu'un fichier après tout.

[wakdev]

C'est jouable je pense. ^^

[wakdev]

Après si il y a moyen de l'éviter, c'est mieux

[lowki]

Oui bien sur c'est possible, mais comme le dit wak, je préfère éviter. Pour une fois qu'on a le temps, faisons les choses bien :)

[lowki]

Bon finalement j'ai cédé au RewriteBase, je l'ai inclus au script d'installation. Le pull request arrive.

[wakdev]

Oui, enfin bon c'est un cas bien particulier, ce n'est pas bien grave. Merci pour le pull request, j'ai test et merge :)