On 11/15/2012 04:18 PM, Sherif Ramadan wrote: > On Thu, Nov 15, 2012 at 6:59 PM, Anthony Ferrara <ircmax...@gmail.com> wrote: > >> >> >> If it wasn't that active open source projects still have ext/mysql as their >> primary connection today, I would agree with you. But we still have open >> source projects, major ones, that still rely on it even in their dev >> versions. Sure Wordpress may have a patch available, but will it work with >> extensions? Likely not. >> >> Right now, it's just "discouraged for new development". Not "you really need >> to get off it now". By adding E_DEPRECATED, you're basically saying "see, we >> told you to get off it", when we never did. I think controlled communication >> and collaboration with the involved projects would go a long way to >> maintaining (and restoring to some extent) the health of the community. >> >> And it's one thing to have the warning that says "It's discouraged", and >> having a warning that says "It's officially deprecated. In the next release, >> we're going to start throwing deprecated notices if you use it. And the >> release after that, it's going to disappear. So get migrating off it now". >> They convey completely different things. The first sounds like "best >> practices", where the second shows the gravity of the situation... >> >> > > I'm just trying to understand your reasoning behind this view. > > How is "telling people it's deprecated, but only in the manual" going > to be any different than "putting warnings to discourage future use, > but only in the manual"? > > 1) People who use mysql_* will rarely ever look at the manual to see them. > 2) Even if they do they do what makes you think this is more effective > than seeing the warning in their logs? > 3) Even if it was more effective, what makes you think it will result > in any kind of action beyond that of which the warnings would result? > > Sorry, I don't see any kind of indication that putting things in the > manual is going to show "the gravity of the situation" any more than > the deprecated errors will.
The main argument is that we haven't actually made it clear in the manual that the extension is going to be deprecated. There is a user note on http://www.php.net/manual/en/book.mysql.php and a "This extension is not recommended for writing new code" on http://www.php.net/manual/en/intro.mysql.php As Anthony mentioned, this is a very weak best-practice type of message that we are only barely whispering here. To go from that to start throwing deprecation warnings on what is perhaps the most heavily used PHP extension ever is a rather drastic step. Your argument seems to be, "Well, there is no point documenting it because people won't read it anyway", which may be true, but we haven't actually tried that yet and for something like this I think it should be the first step. -Rasmus -- PHP Internals - PHP Runtime Development Mailing List To unsubscribe, visit: http://www.php.net/unsub.php
