unnötige variablen definition in config.php

Beitrag von **emergence** » Mi 8. Dez 2004, 12:39

und zwar

/* Set global vars which are missing on some server configurations */

$HTTP_HOST    = $_SERVER['HTTP_HOST'];
$PHP_SELF     = $_SERVER['PHP_SELF'];
$QUERY_STRING = $_SERVER["QUERY_STRING"];

der grund warum mich das stört ist folgender

in der globals_off.inc.php werden sowieso alle variablen via EXTR_OVERWRITE zur verfügung gestellt...

sollte nun $HTTP_SERVER_VARS["HTTP_HOST"] nicht gesetzt sein ist es $_SERVER['HTTP_HOST'] auch nicht...

somit bringts die zweite definition nicht...
sie hat sogar einen nachteil zb bei php 4.0.6 steht $_SERVER nicht zur verfügung... (ja leider ich muss mich zeitweise noch damit herumquälen)
die session verwaltung geht sonst flöten...

Beitrag von **emergence** » Do 9. Dez 2004, 10:51

ad. globals_off.inc.php

man sollte die unterscheidung einbauen von welchen variablen typ die extraktion der werte erfolgen soll...
ich würde empfehlen entweder die unterscheidung ab php 4.1.0 $_SERVER etc.. zu verwenden

oder falls $_SERVER nicht zur verfügung steht die variablen $HTTP_*_VARS auf die andere schreibweise umzumünzen...

ab php 5.0.1 gibts ja noch die möglichkeit das die lange schreibweise deaktiviert werden kann...
da sollte man sich vielleicht überlegen alle langen schreibweisen
muster $HTTP_*_VARS in contenido auf die neue zu ändern...

Beitrag von **emergence** » Do 30. Dez 2004, 12:35

in globals_off.inc.php

Code: Alles auswählen

/* Register post and get variables into $FORM */
$FORM = array_merge($HTTP_POST_VARS, $HTTP_GET_VARS);

gefällt mir überhaupt nicht...

da sollte man sich eher der variable $_REQUEST bedienen...

falls man sicher gehen möchte das diese gesetzt ist könnte man folgendes einbauen....

Code: Alles auswählen

	if (!isset($_REQUEST)) {
		/* Register post,get and cookie variables into $_REQUEST */
		$_REQUEST = array_merge($HTTP_GET_VARS, $HTTP_POST_VARS, $HTTP_COOKIE_VARS);
	}

ansonsten kann es zu wunderbaren überschneidungen kommen...

geändert werden müssten folgende dateien die sich im CVS $FORM bedienen....

include.frontend.user_menu.php
include.html_tpl_edit_form.php
include.html_tpl_files_overview.php
include.js_edit_form.php
include.js_files_overview.php
include.style_edit_form.php
include.style_files_overview.php

die variable $form wird ansonsten noch in ca. 51 dateien für andere zwecke verwendet.... zum debuggen wird das sonst ne katastrophe...

timo · Beitrag von **timo** » Do 20. Jan 2005, 11:34

hmm ich hab jetzt ne weile gesucht...

das Problem ist, daß es ein Riesenaufwand wäre alles durchzugehen und danach noch (was quasi ein Ding der unmöglichkeit ist) alles zu testen...d.h. ich würde es lieber so lassen...

z.b. wird bei der conlib Session-Verwaltung auf $HTTP_HOST zurückgegriffen...und ich weiß nicht wo das noch überall verwendet wird (also die anderen Variablen)...

Beitrag von **emergence** » Do 20. Jan 2005, 11:56

der massivste punkt ist leider der mit $FORM
das sollte umbedingt auf $_REQUEST geändert werden...

timo · Beitrag von **timo** » Do 20. Jan 2005, 11:59

ich habe es jetzt mal geändert...sollte es aber damit Probleme geben, werde ich es wieder zurückändern...

Beitrag von **emergence** » Do 20. Jan 2005, 12:03

glaub mir timo da kommt es zu keinen problemen..