Re: [Phpsqliteadmin-devel] edit row patch

[Yves G. <yg...@mi...>]

On Wednesday 10 November 2004 16:05, Steve Dibb wrote:
> > * what is "import_request_variables("pg");" good for? At first
> > sight I could not see any reason :-(
>
> import_request_variables dumps all $_GET and $_POST variables to
> their own variables.  So if you have db.php?database=psa, a variable
> is automatically set for querystring $database to value $psa.  It's
> the same as saying $_GET['database'], but without $_GET['...'] or
> $_POST['...'].

OK I suspected that... ;-)

I fact I always liked to user the $_GET['...'] and $_POST['...'] way of 
writing because you can immediataly see where the variable comes from, 
cookie, session, get or post, somehow they stay in their "namespace".

> Also a cool thing you can do with it is prepend variable names to the
> variables you're importing.
>
> For example:
>
> page: view.php?table=users&limit=50
> code: import_request_variables("g", "psa_"); ("g" for get, "p" for
> post") result: $psa_table = "users", $psa_limit = 50;

This is also a possibility to have some kind of "namespace" in the 
variable name... In that case we should probably have get_, post_, 
session_ and cookie_ as "prefix".

If you could use that in your code it would be really appreciated :-)
Maybe we can agree on either using the $_GET or $_POST notation, or in 
case you work with miport_request_variables, use the get_ and post_ 
notation. What do you think?

(I know dbaction.php is bad in terms of code in that way, my bad...)



> > * I like the "require_once "inc.header.php";", it was longely
> > overdue :-)
> >
> > * I like the class, it's overdue as well
>
> Well, like I said, there's still a lot I'm hacking on.  Dont think of
> my change as final at all.  Transitional to say the least. :)
>
> Steve
>
>
> -------------------------------------------------------
> This SF.Net email is sponsored by:
> Sybase ASE Linux Express Edition - download now for FREE
> LinuxWorld Reader's Choice Award Winner for best database on Linux.
> http://ads.osdn.com/?ad_id=5588&alloc_id=12065&op=click
> _______________________________________________
> Phpsqliteadmin-devel mailing list
> Php...@li...
> https://lists.sourceforge.net/lists/listinfo/phpsqliteadmin-devel

-- 
Linux 2.6.9 #1 Mon Nov 1 12:01:23 CET 2004 i686
 10:07:47 up 12:16,  1 user,  load average: 1.04, 0.71, 0.62