Newsgroups: php.internals Path: news.php.net Xref: news.php.net php.internals:54541 Return-Path: Mailing-List: contact internals-help@lists.php.net; run by ezmlm Delivered-To: mailing list internals@lists.php.net Received: (qmail 46313 invoked from network); 12 Aug 2011 14:13:03 -0000 Received: from unknown (HELO lists.php.net) (127.0.0.1) by localhost with SMTP; 12 Aug 2011 14:13:03 -0000 Authentication-Results: pb1.pair.com smtp.mail=laruence@gmail.com; spf=pass; sender-id=pass Authentication-Results: pb1.pair.com header.from=laruence@gmail.com; sender-id=pass Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.212.42 as permitted sender) X-PHP-List-Original-Sender: laruence@gmail.com X-Host-Fingerprint: 209.85.212.42 mail-vw0-f42.google.com Received: from [209.85.212.42] ([209.85.212.42:62610] helo=mail-vw0-f42.google.com) by pb1.pair.com (ecelerity 2.1.1.9-wez r(12769M)) with ESMTP id 4B/F1-40016-CE4354E4 for ; Fri, 12 Aug 2011 10:13:02 -0400 Received: by vwl1 with SMTP id 1so2862831vwl.29 for ; Fri, 12 Aug 2011 07:12:57 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=mime-version:sender:in-reply-to:references:date :x-google-sender-auth:message-id:subject:from:to:cc:content-type :content-transfer-encoding; bh=N7iePyl9reUyFYHDWD0Khc/KTmsjBij+kee20kp09uo=; b=PTM1xZ35ovA2inEK7xieyMbPCIr3mKos9P/2XHIIp7vznvddvZp9rlhldKW0jlkRUD OpNyMgNYMOk5PBdE73cjVVFqnDc/GGQ3u2S+Mk5wtu+2D3vkEjS0oHQwtx5gKROGYgUW um3A95qyK8LMWTYpT+4yE2fPD/8lGdJvX/67A= MIME-Version: 1.0 Received: by 10.220.152.147 with SMTP id g19mr226920vcw.260.1313158377290; Fri, 12 Aug 2011 07:12:57 -0700 (PDT) Sender: laruence@gmail.com Received: by 10.220.184.76 with HTTP; Fri, 12 Aug 2011 07:12:57 -0700 (PDT) In-Reply-To: References: Date: Fri, 12 Aug 2011 22:12:57 +0800 X-Google-Sender-Auth: nNHRNPKHKj6oZApdNa9RFsbmhdM Message-ID: To: Ferenc Kovacs Cc: Derick Rethans , PHP Internals Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Subject: Re: [PHP-DEV] behavior of translating dot in the query variable name into underscore character From: laruence@php.net (Laruence) s/goot at/not good at/ sorry 2011/8/12 Laruence : > Hi: > =C2=A0I am goot at debate as my poor english, > =C2=A0I try to make my point clearly, > > =C2=A0that is , I think there is one reason to do this change, that is > "what you saw should be what you get", > > =C2=A0this behavior confused any pepole who meet this issue first, and > need explained by document, > > =C2=A0I agree with Tyral:"it would be really awkward keeping this > workaround much longer for =C2=A0a design problem which isn't there > anymore." > > thanks > > 2011/8/12 Ferenc Kovacs : >> On Fri, Aug 12, 2011 at 3:20 PM, Derick Rethans wrote: >>> On Fri, 12 Aug 2011, Laruence wrote: >>> >>>> =C2=A0 =C2=A0 =C2=A0a request uri : http://***/a.b=3D2 =C2=A0will resu= lt $_GET["a_b"] =3D 2 ( >>>> the dot was translated into underscore char); >>>> >>>> =C2=A0 =C2=A0 =C2=A0my point is , since register_globals has be remove= d already( in >>>> my opinion this behavior only make sence when register_globals on), >>>> should we remove this translating behavior too? >>> >>> No. No reason to change anything that isn't broken. Chanhing this will >>> mean a BC break, even for people that don't use register_globals. >>> >> >> it is broken, we just sucked it up. >> if we would both provide the foo_bar and foo.bar: >> >> - if your code expects foo.bar to be foo_bar, it's there. >> - if you expect foo.bar, it's also fine. >> - if you did a workaround in the past (parse the query string >> manually), that would also work, in the worst case, you would >> overwrite the foo.bar with the same value. >> >> so it's not much userland BC (not until we remove the foo_bar syntax), >> except maybe if you do some really strict input validation (abort if >> there is an request variable not specified in the whitelist). >> I don't mind if this only ships with the next minor or major version, >> but it would be really awkward keeping this workaround much longer for >> a design problem which isn't there anymore. >> >> -- >> Ferenc Kov=C3=A1cs >> @Tyr43l - http://tyrael.hu >> > > > > -- > Laruence =C2=A0Xinchen Hui > http://www.laruence.com/ > --=20 Laruence =C2=A0Xinchen Hui http://www.laruence.com/