Bug #17428 Updated: Fix the tutorial about global variables

ID: 17428 Updated by: philip@php.net Reported By: jim.min@sivacorp.com -Status: Open +Status: Closed Bug Type: Website problem -Operating System: Windows XP +Operating System: all PHP Version: 4.2.1 New Comment: This bug has been fixed in CVS. You can grab a snapshot of the CVS version at http://snaps.php.net/. In case this was a documentation problem, the fix will show up soon at http://www.php.net/manual/. In case this was a PHP.net website problem, the change will show up on the PHP.net site and on the mirror sites. Thank you for the report, and for helping us make PHP better. On a related note, today tut.php was moved to phpdoc (chapters/tutorial.xml). It has been reworded to not rely on register_globals. Previous Comments: ------------------------------------------------------------------------ [2002-07-08 01:42:59] philip@php.net It's not a matter of being bashed, it's a matter of tut.php working with register_globals on or off. The example code should work with the default settings of php.ini-dist, and it will (but does not currently as of PHP 4.2.0). Users can read about register_globals if they want to enable, and code accordingly. ------------------------------------------------------------------------ [2002-07-07 22:06:27] imajes@php.net can we please stop with the Register Globals bashing? there is nothing wrong with it. We simply switch it of by default as there __could__ be security risks if users are not aware of it's awesome power. Please also do not 'fix' phpweb to not use register globals. if you must, add a comment on the show source page. ------------------------------------------------------------------------ [2002-07-07 22:01:45] jacques@php.net Okay I've just done a cvs commit on tut.php which has links to the strstr manual page and the control structure page, as well as changed the variables listed to way they would be set if register_globals=off in the php.ini file. I've implemented some of philip@php.net's suggestions so far. I'll get around to going thru the entire phpweb and changing all the code to use the register_globals=off since newbies also like to view the source code to see how the webteam do various things on the site. ------------------------------------------------------------------------ [2002-07-07 21:04:04] jacques@php.net I'm currently going to fix this page now. I'm going to ensure that all the examples on the page work. Expect a commit a bit later this morning. ------------------------------------------------------------------------ [2002-07-04 21:54:27] philip@php.net This page needs a complete rewrite, this is still open. Many paragraphs are outdated and written during a time when the register_globals directive didn't even exist (always on). A simple note on the bottom that essentially says: "the above method is not preferred." isn't enough. Complete newbies read tut.php Also, superglobals becaming available in 4.1.0 should be mentioned and linked. And, all examples should work with register_globals = off. Also, for example, $HTTP_USER_AGENT is mentioned in a few places yet this isn't available with register_globals off. And lastly, tut.php needs more links to related man pages. Like the used functions and predefined variables, if statement, forms, etc. ------------------------------------------------------------------------ The remainder of the comments for this report are too long. To view the rest of the comments, please view the bug report online at http://bugs.php.net/17428 -- Edit this bug report at http://bugs.php.net/?id=17428&edit=1