View Issue Details Jump to Notes ] Issue History ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0000089SoothERPSpécificationspublic2014-10-30 18:312015-09-21 23:37
ReporterYves 
Assigned ToYves 
PriorityhighSeveritymajorReproducibilityalways
StatusresolvedResolutionfixed 
PlatformOSOS Version
Product VersionRC1 
Target Version1.0Fixed in Version 
Summary0000089: Compatibilité Php 5.4 à valider
DescriptionLa version Php 5.3 étant arrivée en fin de vie (même si des distibutions du type LTS continueront à priori encore à backporter les correctifs de sécurité), il devient prioritaire de faire migrer le code pour compatibilité avec Php 5.4 (minimum)
TagsNo tags attached.
Attached Files

- Relationships

-  Notes
User avatar (0000160)
crepeausucre (developer)
2015-06-04 16:00

Yves,

Voici une erreur rencontrée sur la branche 5.4 compat avec une configuration Ubuntu Apache PHP classique:
(string) = ERREUR PHP: [2] require_once(/documents/_doc_fac.class.php): failed to open stream: No such file or directory /soothinstalls/sooth-php54compat/soothERP/src/2.071.0/core/libcore/_document.lib.php - ligne 23

En modifiant la ligne 23 ainsi c'est OK:
require_once ( dirname(__FILE__) . DIRECTORY_SEPARATOR . "/documents/_doc_".strtolower($_SESSION['types_docs'][$doc->id_type_doc]->code_doc).".class.php");

Avant de faire un pull request, est-ce que tu souhaites le tester sous windows ?
(normalement le correctif est compatible toute plateforme)
User avatar (0000161)
crepeausucre (developer)
2015-06-04 16:02

En dehors du bug remonté ci-dessus je n'ai rien rencontré de particulier, si ce n'est que la branche corrige en effet nombre de problèmes de compatibilité rencontrés en php 5.4 avec la branche 2.062.2/master !!
User avatar (0000162)
Yves (administrator)
2015-06-04 23:19

Erwan, oui testé sous Windows (Apache), ça marche tu peux soumettre un pull request je l’intégrerai.

Merci
User avatar (0000163)
Yves (administrator)
2015-06-04 23:39

Mais dans ce cas tu devrais avoir aussi le même souci ligne 41 du même fichier (création d'un document), non ?
User avatar (0000164)
crepeausucre (developer)
2015-09-21 10:02

Oui pour la ligne 41, mon pull request reprends les 2.
User avatar (0000165)
Yves (administrator)
2015-09-21 23:37

Erwan, merci, c'est intégré sur Github.

- Issue History
Date Modified Username Field Change
2014-10-30 18:31 Yves New Issue
2014-10-30 18:31 Yves Status new => assigned
2014-10-30 18:31 Yves Assigned To => Yves
2015-06-04 16:00 crepeausucre Note Added: 0000160
2015-06-04 16:02 crepeausucre Note Added: 0000161
2015-06-04 23:19 Yves Note Added: 0000162
2015-06-04 23:39 Yves Note Added: 0000163
2015-09-21 10:02 crepeausucre Note Added: 0000164
2015-09-21 23:37 Yves Note Added: 0000165
2015-09-21 23:37 Yves Status assigned => resolved
2015-09-21 23:37 Yves Resolution open => fixed


Copyright © 2000 - 2017 MantisBT Team
Powered by Mantis Bugtracker