ID:               41375
 Updated by:       [EMAIL PROTECTED]
 Reported By:      bugs dot php dot net at kreidenweis dot com
 Status:           Open
 Bug Type:         Documentation problem
 Operating System: Any
 PHP Version:      5.2.2
 New Comment:

Yes, of course it should have been caught in the RC stage, but it
wasn't - in fact it took me more than one final release to track it
down.

Why is what's done, done, when the behavior that was changed existed
for years and years, and the new behavior has been around for 2 point
releases?


Previous Comments:
------------------------------------------------------------------------

[2007-09-07 14:59:08] [EMAIL PROTECTED]

Don't you think such questions should have been asked during RC stage?
It's a bit too late to change it back, what's done is done.

------------------------------------------------------------------------

[2007-09-07 14:46:42] [EMAIL PROTECTED]

Okay, documenting it at least makes code and documentation consistent,
but why the change in the first place? Why not avoid the break and leave
it consistent?

------------------------------------------------------------------------

[2007-09-07 09:40:38] [EMAIL PROTECTED]

Reclassified.


------------------------------------------------------------------------

[2007-09-07 06:48:16] [EMAIL PROTECTED]

"If start is negative, the returned string will start at the start'th
character from the end of string."

The function still accepts a negative start. What changed is if the
start is longer than the string length, the test (user friendly
version):

 if (from > strlen(str) || (from < 0 && -from > strlen(str)))

As I agree it may break some scripts, the new behavior should be
documented, as a bug fix.

Can we move this bug to the doc category?

------------------------------------------------------------------------

[2007-09-07 02:22:57] [EMAIL PROTECTED]

Seems to me as if this was introduced with 5.2.3, not 5.2.2 per the
original submitter. Also see
http://us.php.net/manual/en/function.substr.php#75999 - this isn't
something that only affects me.

------------------------------------------------------------------------

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/41375

-- 
Edit this bug report at http://bugs.php.net/?id=41375&edit=1

Reply via email to