[Résolu] Patch nksessions.php< Sujet précédent  Sujet suivant >
Index du Forum -> NUKED-KLAN VERSION > 1.7 -> Généralités

Page : 1 [2]
AuteurMessage
France.gif Keneda
Member



Messages : 197
Inscrit(e) le: 09/05/2008
RE : Patch nksessions.php
Citation par Zdav :

L'oncle Sam va remédier a ce petit souci

Je ne considérais pas vraiment ça comme un soucis.

 

@Samoth, malheureusement j'ai pas mal de modifs faites manuellement sur le site (et comme un couillon j'ai pas tout noté ) donc je vais faire ça au calme, tranquilou, ou alors j'attendrai un patch/zip avec juste les fichiers modifiés pour la compatibilité 5.4.

En attendant je vais tester là où ça va bien avec la dernière modif.

Merci pour ton travail acharné

 

Prochaine étape, après ça, test de la 1.8 (compatible "nativement" php5.4 je présume?)

 

EDIT :

Encore une fois j'apporte une mauvaise nouvelle...

 

Le site de test fonctionne en php 5.3 et 5.4, par contre si les titres / corps des news, ainsi que les titres des catégories / sous-catégories / topics du forum s'affichent correctement, avec caractères spéciaux ou non, je n'ai en revanche aucun contenu enregistré pour les messages forum (pourtant tout se passe comme si), que ce soit en 5.3 ou en 5.4, et la BDD ne contient rien en colonne "txt" des messages.

Et ce, que le texte comporte ou non des caractères spéciaux.

 

Et je suis confus car en fait je ne suis même plus sûr que le corps des nouveaux messages s'affichait avant la rétrocompatibilité php (les yeux dans le fion ou le cerveau en bouillie, au choix)...

Par contre si je met du contenu dans la colonne "txt" directement dans la BDD le site affiche bien son contenu dans le corps des messages, caractères spéciaux compris (comme quand j'ai importé d'anciens messages du forum en prod).

 

EDIT2 : j'ai testé en modifiant le fichier nuked.php pour revenir à sa version non rétrocompatible et j'ai toujours le soucis de non affichage des messages, car non enregistrement du texte dans la bdd.



Edité par Keneda le 11/04/2013 - 07:10:07
Posté le 11/04/2013 - 05:28:35   Revenir en haut | Permalien
France.gif Sam
Développeur

Messages : 1310
Inscrit(e) le: 19/11/2007
RE : Patch nksessions.php

Salut Keneda,

 

le problème des messages du forum vient d'une erreur de syntaxe de ma part en effet il y a un "s" en trop à la variable "$var" de la fonction nkHtmlSpecialChars()

 

Je commit la correction et je t'invite encore une fois à nous remonter tout autre problème que tu trouveras.

 

Merci encore pour ta contribution



Posté le 11/04/2013 - 11:04:09   Revenir en haut | Permalien
France.gif Keneda
Member



Messages : 197
Inscrit(e) le: 09/05/2008
RE : Patch nksessions.php

Et bien ma foie tu as trouvé le serpent dans le fichier

Place maintenant à la mise à jour du site de prod

Et si tu souhaites un ou deux ptits bugs pour te passer le temps, hop, tu sais à qui demander (sisi j'en ai) !



Posté le 11/04/2013 - 13:44:10   Revenir en haut | Permalien
France.gif Keneda
Member



Messages : 197
Inscrit(e) le: 09/05/2008
RE : Patch nksessions.php

DSL pour le double post mais je voulais mettre un fichier joint donc pas le choix on ne peux l'ajouter en mode edit.

Donc j'ai fait la mise à jour du site de prod, plus qu'à tester tout ça en réel, pour l'instant tout à l'air de marcher comme avant j'ai pu poster une news et un topic pour les retours.

 

Nickel Samoth !

 

Je met en fichier joint, pour qui serait intéressé, une archive contenant les fichiers à uplaoder sur son FTP pour faire la mise à jour simplement, attention, ce n'est QUE pour le patch PHP5.4, et il faut extraire simplement l'archive, et uploader son contenu à la racine du site.



Posté le 11/04/2013 - 15:15:25  1801d2eda8.7z (250 Ko) Revenir en haut | Permalien
France.gif Sam
Développeur

Messages : 1310
Inscrit(e) le: 19/11/2007
RE : Patch nksessions.php

Merci Keneda, je pense que d'autres membres apprécieront.

 

Par contre si tu as d'autres bugs tu peux nous les remonter à cette adresse, ce sera plus facile pour nous d'y répondre rapidement

https://github.com/Nuked-Klan/nk1.7.9/issues?page=1&state=open

ATTENTION : ce lien ne concerne que la 1.7.9 pour la 1.8 il te suffit de changer de dépôt et d'aller dans l'onglet "issues".

 

Merci encore de ta participation



Posté le 11/04/2013 - 15:23:03   Revenir en haut | Permalien
France.gif Keneda
Member



Messages : 197
Inscrit(e) le: 09/05/2008
RE : Patch nksessions.php

Merci à toi pour tes talents de codage

Je vais te laisser ainsi que Seku et les autres, un peu tranquil pour le moment, je garde sous le coude et reviens à la charge plus tard

 

Me suis enregistré quand même sur GH en attendant.



Posté le 11/04/2013 - 15:34:47   Revenir en haut | Permalien
France.gif Keneda
Member



Messages : 197
Inscrit(e) le: 09/05/2008
RE : Patch nksessions.php

Petit bug en rapport avec cette mise à jour je pense.

Le text-align d'un message ne fonctionne plus.

Quand on choisis le type, par exemple center, le texte est entourré par :

Une fois le message envoyé le texte n'est pas centré (ou aligné à droite, etc...) et quand on retourne éditer le message, la feuille source ne montre plus que :

Enfin, si on met autour du texte :

ou

Là ça marche, il semble que ce soit l'espace entre les":" et le "center" qui plante la fonction.

Je met [point virgule] comme ça à la place du signe car il ne passe pas à cet endroit sur votre forum malgré la fonction code.



Posté le 11/04/2013 - 18:59:40   Revenir en haut | Permalien
France.gif Keneda
Member



Messages : 197
Inscrit(e) le: 09/05/2008
RE : Patch nksessions.php

DSL du méchant triple post mais je n'édite pas mon message au dessus sinon tout ce que j'ai mis en "code" se barre et faut tout refaire.

Pour préciser que ce problème ne survient que sur le forum, le text align fonctionne bien sur les news.

 

Et aussi sur les anciens messages, le text align est toujours pris en compte mais quand j'édite je constate que l'espace entre le":" et le "center" ainsi que le "point virgule" derrière ne sont pas [plus] là.

 

 

 



Posté le 11/04/2013 - 19:18:24   Revenir en haut | Permalien
France.gif Zdav
Responsable Qualité

Messages : 2751
Inscrit(e) le: 06/08/2010
RE : Patch nksessions.php

Il semblerait que ce souci soit lié a php5.4, j'arrive bien a centrer en php5.3.1 par contre pas en php5.4



Posté le 11/04/2013 - 21:25:05   Revenir en haut | Permalien
France.gif Sam
Développeur

Messages : 1310
Inscrit(e) le: 19/11/2007
RE : Patch nksessions.php

Encore un petit commit qui corrige le dernier problème



Posté le 11/04/2013 - 21:45:12   Revenir en haut | Permalien
France.gif Keneda
Member



Messages : 197
Inscrit(e) le: 09/05/2008
RE : Patch nksessions.php

Alors après essai cela marche bien à première vue.

Par contre, et ça je pense que ça date d'avant PHP5_4 mais sans en être sûr, le [point virgule] se trouvant après l'attribut du text-align (ex "center" ) disparaît systématiquement de la feuille source sur le forum à l'édition du message, sans toutefois entraver le bon alignement du texte, alors que sur les news, il est bien conservé.

Un merci de plus à ton compteur en tous cas.

 

PS : tests effectués en PHP5_3 et 5_4 ;

PS2 :

test alignement

Et bien ça fait pareil ici

 

PS3 : je mettrai à jour l'archive du patch manuel que j'avais donné au-dessus en temps voulu, pour les acharnés, ils sauront comme moi comment faire sans

 



Posté le 12/04/2013 - 06:16:54   Revenir en haut | Permalien
France.gif Keneda
Member



Messages : 197
Inscrit(e) le: 09/05/2008
RE : Patch nksessions.php

Version "2" du patch PHP5_4.

Décompresser et coller les fichiers à la racine du site sur le FTP.



Posté le 13/04/2013 - 17:39:04  361632bb97.7z (250 Ko) Revenir en haut | Permalien
Page : 1 [2]