php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login
Bug #53630 optional prompt argument is treated as non-optional
Submitted: 2010-12-30 00:38 UTC Modified: 2011-01-10 19:19 UTC
From: jo at feuersee dot de Assigned: iliaa (profile)
Status: Closed Package: Readline related
PHP Version: 5.3.4 OS: Linux
Private report: No CVE-ID: None
Welcome back! If you're the original bug submitter, here's where you can edit the bug or add additional notes.
If you forgot your password, you can retrieve your password here.
Password:
Status:
Package:
Bug Type:
Summary:
From: jo at feuersee dot de
New email:
PHP Version: OS:

 

 [2010-12-30 00:38 UTC] jo at feuersee dot de
Description:
------------
The readline functions prompt argument is supposed to be optional. But omitting it raises a PHP warning.

Test script:
---------------
php -r 'printf("%s\n", readline());'

Replace single and double quotes vice versa for Win platform

Actual result:
--------------
PHP Warning:  readline() expects exactly 1 parameter, 0 given in Command line code on line 1


Patches

53630.diff (last revision 2010-12-30 05:53 UTC by dave dot tajchreber at gmail dot com)

Pull Requests

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2010-12-30 06:52 UTC] dtajchreber@php.net
-Status: Open +Status: Verified
 [2011-01-10 19:19 UTC] iliaa@php.net
Automatic comment from SVN on behalf of iliaa
Revision: http://svn.php.net/viewvc/?view=revision&revision=307343
Log: Fixed bug #53630 (Fixed parameter handling inside readline() function).
 [2011-01-10 19:19 UTC] iliaa@php.net
-Status: Verified +Status: Closed -Assigned To: +Assigned To: iliaa
 [2011-01-10 19:19 UTC] iliaa@php.net
This bug has been fixed in SVN.

Snapshots of the sources are packaged every three hours; this change
will be in the next snapshot. You can grab the snapshot at
http://snaps.php.net/.
 
Thank you for the report, and for helping us make PHP better.


 
PHP Copyright © 2001-2024 The PHP Group
All rights reserved.
Last updated: Tue Dec 03 16:01:33 2024 UTC