Ticket #1309 (closed defect: fixed)
Avertissement argument passé par référence
Reported by: | nikrou | Owned by: | xave |
---|---|---|---|
Priority: | normal | Milestone: | 2.5 |
Component: | module:core | Version: | 2.3 |
Severity: | normal | Keywords: | |
Cc: |
Description
Certains appels de fonctions passent les arguments par référence. Cette "facilité" du langage ne sera plus possible avec PHP 5.4.
Un bug a été soumis dans le bugtracker debian montrant deux appels de ce type : http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=658868
Un appel ligne 38 de inc/dbschema/upgrade-cli.php :
$changes = dotclearUpgrade(&$core);
Je pense qu'on peut simplement enlever le &. Les autres appels à cette fonction se faisant normalement. On peut au besoin forcer le passage par adresse dans la déclaration de la fonction dans le fichier inc/dbschema/upgrade.php
Un appel ligne 327 de plugins/widgets/index.php :
'<div class="widgetSettings">'.$w->formSettings($iname,&$j).'</div>'.
La correction et la même. La déclaration de la méthode attend bien une adresse et pas une valeur.
Je n'ai pas mis le patch.
(In [0e7dec7b7202]) Removed by-reference call in upgrade-cli.php, fixes #1309