[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
Patch nksessions.php

J'ai installé ce patch : http://code.google.com/p/nuked-klan/source/detail?r=20dd09ac12548587d7bd22c12e0ec44dcf5c76c1

Et ça me donne ce bug à la connexion au site :

N'étant qu'une quiche en codage, je ne sais pas ce qui cloche à la ligne 151 (un espace ajouté dans le patch)



Posté le 07/04/2013 - 21:04:36   Revenir en haut | Permalien
France.gif Zdav
Responsable Qualité

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

Salut keneda,

 

Effectivement la correction n'a jamais été commité

 

Voici la correction a c e souci :

 

http://www.nuked-klan.org/upload/Forum/a4bb91653f.zip

 

Cdt



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



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

Merci bien, bug plus là, nickel

Sinon mis à part ce patch, j'ai installé manuellement tous les patchs "post 1.7.9 release finale" présents sur le google_code, tous ce qui n'était pas déjà dedans la finale, depuis le 03/12/2012 inclus si je me souviens bien, y'en a-t-il d'autres dans ce lot qui ne sont pas "commités" (késako?) et qui risquent de poser problème?

 



Posté le 08/04/2013 - 01:25:29   Revenir en haut | Permalien
France.gif Zdav
Responsable Qualité

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

Yop,

 

Oui il y aurait bien un autre patch, un qui devrait te plaire d'ailleurs : compatibilité NK avec les hébergeurs en php 5.4.

Pour l'instant pas de nouvelles de ce dernier, dès que j'en saurai un peu plus il est fort probable que je reprenne contact avec toi pour les tests vu que tu es l'un des premiers à avoir remonté ce bug.

 

Bonne journée

 

Cordialement



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



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

Oui j'avais cru comprendre ça sur le topic en post-it.

ZE patch

 



Posté le 08/04/2013 - 11:40:39   Revenir en haut | Permalien
France.gif Sam
Développeur

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

Salut Keneda,

 

le dépôt de la 1.7.9  a été déplacé sur GitHub : https://github.com/Nuked-Klan/nk1.7.9

 

j'ai ajouté le patch pour PHP 5.4, si tu peux tester et me faire un feedback ça serait cool



Posté le 09/04/2013 - 04:58:10   Revenir en haut | Permalien
Belgium.gif Sekuline
Ancien

Messages : 3349
Inscrit(e) le: 04/09/2008
RE : Patch nksessions.php

Il est pas HOT ce se SAMHOT?



développeur web auto-didacte (xHTML, HTML5, CSS3, PHP, MySQL, JS : jQuery)

Espace de stockage portable ! Sekuline

Posté le 09/04/2013 - 13:10:22   Revenir en haut | Permalien
Belgium.gif PePeRePeRveRs
Vice-Président Association

Messages : 3672
Inscrit(e) le: 13/06/2004
RE : Patch nksessions.php

Je dirais même que SAMHOT une épine du pied



Avant d'ouvrir un sujet, n'oubliez pas de lire le règlement et faire une recherche sur le forum !

Posté le 09/04/2013 - 14:20:42   Revenir en haut | Permalien
France.gif Zdav
Responsable Qualité

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

Perso je suis Fan

 

 



Edité par Zdav le 09/04/2013 - 15:57:50

Posté le 09/04/2013 - 15:57:24   Revenir en haut | Permalien
Belgium.gif PePeRePeRveRs
Vice-Président Association

Messages : 3672
Inscrit(e) le: 13/06/2004
RE : Patch nksessions.php

Mdrrr, Zdav ! J'ai mis quelques secondes à comprendre ton jeu de mots



Avant d'ouvrir un sujet, n'oubliez pas de lire le règlement et faire une recherche sur le forum !

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



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

Ok je test et je vous tiens au courant, merci (et maintenant je comprends mieux le "commité")

 

Edit : oui mais alors là je me retrouve un peu con, vu la taille des modifs à faire (fallait s'en douter c'est sûr), les faire à la main serait le meilleur moyen de sécher devant mon PC avant même l'arrivée de l'été.

Je fais comment pour récup tous les fichiers modifiés?

 

Edit2 : bon j'ai trouvé pour récupérer tous les fichiers modifiés ou non (cliquer sur ZIP), par contre vais ne pouvoir tester que sur une installation fraiche et pas sur notre site en "production" (vous me direz c'est moins risqué ).

 

Edit3 : sinon pourquoi êtes-vous passé de googlecode à github, mis à part que c'est plus pratique pour les noobs comme moi de récupérer les fichiers vu que pas besoin de tortoise?

 

Edit 4 : sur site de test http://forum.stars-team.com/

1er test, php5.4 actif, sur bdd effacée sauf les forum_threads / cat et messages, tout s'est bien passé à l'install jusqu'à la page récapitulative où j'avais un message d'erreur (qui date d'avant il me semble) : "Une erreur est survenue lors de la récupération de la liste des partenaires..."

Mais quand j'ai voulu finaliser, ce qui renvoie sur le site fraichement créé, là, c'est le drame :

 

Fatal error: Allowed memory size of 134217728 bytes exhausted (tried to allocate 65484 bytes) in /homez.152/starstea/www/forum/nuked.php on line 34

 

Et la ligne concernée dans le fichiers :

 

return nkHtmlEntities($var,ENT_XHTML,'ISO-8859-1'

 

J'ai essayé en réactivant php5.3 histoire de voir si ça changeait quelque-chose (erreur différente ou quoi) mais non, pareil.

Je vais réessayer en supprimant toute la BDD mais je doute un peu que ça vienne des 3 catégories que j'avais laissé de mon ancien site de test.

 

Edit 5 :

Bon ben comme je m'y attendais, c'était pas dû aux trois bouts de bdd laissées.

Bon courage à toi Samoth et t'hésites pas si t'as besoin d'aide pour des tests, ou de l'accès à mon site de test (bdd / ftp).

En attendant je repasse le serveur en 5.3 histoire qu'on puisse utiliser normalement le site principal.



Edité par Keneda le 10/04/2013 - 11:25:38
Posté le 10/04/2013 - 09:35:36   Revenir en haut | Permalien
France.gif Zdav
Responsable Qualité

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

Yop Keneda,

 

J'avais effectivement les mêmes erreurs que toi.

Samoth les as corrigé.

Tu peux donc re télécharger le zip si tu le souhaites

 

Merci!



Posté le 10/04/2013 - 17:52:50   Revenir en haut | Permalien
France.gif Keneda
Member



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

Et bien il semble que ça fonctionne

J'ai testé avec les données "news" et les catégories forum du site en "production" et je n'ai plus les erreurs de titre ne s'affichant pas quand ils ont des caractères spéciaux.

J'ai aussi essayé l'ajout d'une news ainsi que d'un topic "test" et cela fonctionne parfaitement.

 

Bien sûr faudra voir dans la durée et avec une utilisation "normale".

Puis vous êtes pas à l'abris que je trouve un ou deux bug dâtant d'avant ou non (j'en ai sous le coude de toute façon )

 

En tous cas GG Samoth (et la team NK) pour ce suivi hors pair !

Sacré boulot que cette modif!

En attendant un patch NK PHP5.4, je repasse en 5.3 le serveur.

 

PS : par contre bien sûr (fallait s'y attendre) la version PHP5.4 de NK1.7.9 est totalement incompatible avec PHP5.3 !!

 



Posté le 10/04/2013 - 19:54:33   Revenir en haut | Permalien
France.gif Zdav
Responsable Qualité

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

Merci du retour Keneda

 

Citation par Keneda :

PS : par contre bien sûr (fallait s'y attendre) la version PHP5.4 de NK1.7.9 est totalement incompatible avec PHP5.3 !!

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



Posté le 10/04/2013 - 20:56:07   Revenir en haut | Permalien
France.gif Sam
Développeur

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

Salut Keneda,

 

j'ai rajouté la retro-compatibilité du Patch PHP5.4, tu peux d'ores et déjà écraser ta 1.7.9 avec celle de Git à condition bien sur de ne pas avoir de fichier modifié.

 

Pour info, nous sommes passé sur GitHub, car ce dernier est plus "User Friendly" que google code comme tu as pu t'en rendre compte

 

Cdt,

Sam

 



Posté le 11/04/2013 - 00:27:24   Revenir en haut | Permalien
Page : [1] 2