Page 2 sur 3

Re: Nouveau bouton à disposition de tous : Cloud

Publié : 16 aou 2015, 00:19
par papajoke
@Loubrix
justement dans la première soluce, c'est juste un filtre a la réception du post. tu n'as que str_replace(...) a ajouter au code existant.

Re: Nouveau bouton à disposition de tous : Cloud

Publié : 16 aou 2015, 10:23
par Loubrix
il se trouve que le bug a déjà été soumis chez PhpBB mais qu'hélas la correction n'a été faite que dans la version 3.1 (pas dans la 3.0 que nous utilisons). (bug dupliqué).
c'est quoi le code html pour le â ?

edit: je me demande si à titre de patch, ce serait pas une idée plus simple de créer une nouvelle balise BBcode (comme celle pour Youtube).

Re: Nouveau bouton à disposition de tous : Cloud

Publié : 16 aou 2015, 10:34
par cellix
C’est cela que tu cherches ?
http://www.commentcamarche.net/contents ... ciaux-html

â → â (ou en code ISO â )

Re: Nouveau bouton à disposition de tous : Cloud

Publié : 16 aou 2015, 11:31
par Loubrix
oui, mais ça marche pas...

mais je vais attendre qu'on aie fait la mise à jour avant d'aller faire une modif dans le code; donc pour le moment, utilisez le lien alternatif.

Re: Nouveau bouton à disposition de tous : Cloud

Publié : 16 aou 2015, 11:52
par cellix
Ce code n’est valable pour les pages html. Pour les url, c’est un autre encodage qui est utilisé : http://coin.des.experts.pagesperso-oran ... q9_29.html

Et encore, cela ne concerne que la partie après le nom de domaine (ie., après le premier / de l’url).

Pour ce qui est du nom de domaine proprement dit, c’est un truc aléatoire (mais reproductible) de la forme :

xn--<le.DN.avec.que.ses.caracteres.asccii>-<unesequencealeatoire>.fr


Pour le cas que âpp.com, le nom de domaine normalisé est "pp" et la séquence aléatoire est "oia".

Pour plus d’explications : https://fr.wikipedia.org/wiki/Nom_de_domaine_internationalisé

Re: Nouveau bouton à disposition de tous : Cloud

Publié : 16 aou 2015, 12:27
par Loubrix
oui, j'ai fait plein de tests et je me rends compte que quand on arrive à faire accepter l'URL par PhpBB, ça coince quand même avec la localisation de l'image; par exemple, ceci:

[minia=500]http://%E2pp.com/dl/Hgj8m[/minia]

est bien compris par PhpBB comme une image liée (contrairement à l'URL normale où la balise n'est même pas interprétée), mais le cadre image apparait vide, car l'image n'est pas trouvée.

le problème, c'est que quand on uploade une image sur ce site, il donne les liens à utiliser en âpp.com; ça semble difficile d'expliquer à des utilisateurs qui ont du mal à comprendre les balises du forum que copier-coller le lien ne suffit pas, et qu'il faut remplacer âpp.com dans l'URL par xn--pp-oia.com

Re: Nouveau bouton à disposition de tous : Cloud

Publié : 16 aou 2015, 14:08
par papajoke
je viens de tester mon patch donné + haut chez moi (phpbb 3.0.12) il fonctionne très bien :
fichier includes/message_parser.php
la class parse_message->parse() # ligne 1113 chez moi et cette version github ...
ajouter dans cette méthode la ligne :

$this->message = str_replace('://âpp.com/', '://xn--pp-oia.com/', $this->message);

Re: Nouveau bouton à disposition de tous : Cloud

Publié : 16 aou 2015, 14:16
par cellix
Ne vous embêtez pas, Trent va rajouter un bouton pour passer du mode âpp.com à xn--pp-oia.com

Plus de précisions à venir…

Si vous voulez discuter avec lui, n’hésitez pas, il est présent sur freenode (canal #vivaldi-fr).

Re: Nouveau bouton à disposition de tous : Cloud

Publié : 16 aou 2015, 16:56
par bonyred
essai

http://xn--pp-oia.com/Hgkd

Pas évident évident.

A plus.

Re: Nouveau bouton à disposition de tous : Cloud

Publié : 16 aou 2015, 17:10
par lemust83
Ça ne serait pas plutôt une lionne....
En tout cas ,j'aime beaucoup :bien

Re: Nouveau bouton à disposition de tous : Cloud

Publié : 16 aou 2015, 17:40
par bonyred
Modifier

http://xn--pp-oia.com/Hgkk

Avec tag.

Le xfc

http://xn--pp-oia.com/Hgkl

Un peu batailler mais c'est bon.

A plus.

Re: Nouveau bouton à disposition de tous : Cloud

Publié : 16 aou 2015, 17:49
par papajoke
@bonyred
oops au jeu des 7 erreurs j'ai rien vu :pleure:

ps: ajoute un tag "manjaro" histoire d'avoir un bon moyen de retrouver les images de la communauté.

Re: Nouveau bouton à disposition de tous : Cloud

Publié : 16 aou 2015, 19:02
par Loubrix
papajoke a écrit :je viens de tester mon patch donné + haut chez moi (phpbb 3.0.12) il fonctionne très bien :
fichier includes/message_parser.php
la class parse_message->parse() # ligne 1113 chez moi et cette version github ...
ajouter dans cette méthode la ligne :

$this->message = str_replace('://âpp.com/', '://xn--pp-oia.com/', $this->message);


justement, ton idée m'en donne d'autres; déjà, tu mets ça dans la class parse_message, mais ne serait il pas plus approprié de mettre ça dans function validate_url($var1, $var2) (ligne 936), vu qu'on y trouve déjà une règle qui me semble résoudre le problème des espaces dans les URLs:

$url = str_replace(' ', '%20', $url);

et alors ça, ça fait quoi ?

if (!preg_match('#^[a-z][a-z\d+\-.]*:/{2}#i', $url))


bref, je me demandais s'il était possible d'écrire une règle générale qui permet à PhpBB de traduire automatiquement les URLs comportant des accents ou autres caractères spéciaux en URL IDN, d'autant que les serveurs DNS connaissent l'adresse IDN (un whois sur âpp.com ne renvoie d'ailleurs que l'adresse xn--pp-oia.com).
c'est d'ailleurs ce que faisait Firefox quand il ne supportait pas encore les URLs accentuées: il remplaçait par l'adresse IDN automatiquement (il devait bien la trouver quelque part).

Cellix a écrit :Ne vous embêtez pas, Trent va rajouter un bouton pour passer du mode âpp.com à xn--pp-oia.com

si tu me dis que c'est une question de jours, on se prendra pas la tête, mais un jour ou l'autre, on tombera sur un autre site avec le même problème, surtout qu'on est un site francophone, ce qui augmente nos chances de tomber sur des URLs avec des accents.

Re: Nouveau bouton à disposition de tous : Cloud

Publié : 16 aou 2015, 19:58
par papajoke
pour validate_url() il valide une url passée en paramètre cela implique qu'il a déjà "parser"(retrouver) toutes les url dans le message. Mais avec des regex du type [a-z0-1] il ne risque pas de la trouver comme une url et donc même pas de validation !
moi, j'applique la modif avant qu'il ne parse le message (c'est ca le truc (de facilité :clindoeil: ))
De plus, ici, c'est la méthode : bbcode_firstpass->validate_url() donc ceci ne s'applique que a un bbcode et pas a une url écrite dans le texte

Pour convertir automatiquement les nom de domaine, il existe la fonction idn_to_ascii

on tombera sur un autre site avec le même problème

Ce n'est pas uniquement avec le nom de domaine, si l'url suivant le nom de domaine à un accent ça marche pas non plus ;rale:
http://www.larousse.fr/dictionnaires/francais/été/31337
https://fr.wikipedia.org/wiki/%C3%8Atre
https://fr.wikipedia.org/wiki/Être

Re: Nouveau bouton à disposition de tous : Cloud

Publié : 16 aou 2015, 20:27
par Loubrix
oui, mais les URLs avec des accents dans le chemin, ça on sait faire:
http://www.larousse.fr/dictionnaires/francais/%C3%A9t%C3%A9/31337

et sinon, idn_to_ascii, ça semble faire ce que je veux, mais comment je l'intègre dans PhpBB ?

Re: Nouveau bouton à disposition de tous : Cloud

Publié : 16 aou 2015, 20:33
par cellix
Ça y est, c’est intégré sur âpp.com :
Image

Re: Nouveau bouton à disposition de tous : Cloud

Publié : 16 aou 2015, 20:41
par papajoke
Merci à Trent :maître:
note: [url] ca passe pas :clindoeil: il faut impérativement des minuscules [URL=http://xn--pp-oia.com/Hgkn]âpp.com/...[/url]
note 2 : il manque un contenu entre [url]et[/url] et première balise non fermée

Loubrix a écrit :et sinon, idn_to_ascii, ça semble faire ce que je veux, mais comment je l'intègre dans PhpBB ?

:saispas: il faudrait faire un regex dans me même fonction parse() qui recherche des http(s)://caractere_different_[a-z0-1]/ puis une boucle sur les résultats (extraite le sous-domaine ?) et leurs appliquer la fonction et changer le message

Re: Nouveau bouton à disposition de tous : Cloud

Publié : 16 aou 2015, 20:48
par lemust83
Image

Yes , mais pas en copiant le lien fournit "http://âpp.com/Hgkn" J'ai du bidouiller...
Edit: L'image intégrée n'est plus un lien comme pour les autres hébergeurs. Si on clic dessus ,rien....

Re: Nouveau bouton à disposition de tous : Cloud

Publié : 16 aou 2015, 20:54
par papajoke
lemust83 a écrit :Edit: L'image intégrée n'est plus un lien comme pour les autres hébergeurs. Si on clic dessus ,rien....

colonne "lien miniature"

et pour la première colonne nous devrions plutot avoir : [ url=http://xn--pp-oia.com/Hgkn]âpp.com/Hgkn[ /url], ce qui donne: âpp.com/Hgkn un nini bug :)

-------------------------------------
un test : http://%C3%A2ap.com/Hgkn qui fonctionne donc il est possible sur le serveur de remplacer aussi "â" par "%C3%A2" mais que c'est beau comme url :rigole:

Re: Nouveau bouton à disposition de tous : Cloud

Publié : 16 aou 2015, 21:12
par lemust83
Merci , mais ça reste un peu compliqué: il ne faut pas copier le lien proposé initialement ,mais cliquer sur go pour avoir les liens dispos. Ça reste en miniature et s'intègre pas (du moins pas encore trouvé comment) dans les balises [minia]

Image