Req #31424 [Opn]: Make mode optional on mkdir

Edit report at https://bugs.php.net/bug.php?id=31424&edit=1 ID: 31424 Updated by: cmb@php.net Reported by: aidan@php.net Summary: Make mode optional on mkdir Status: Open Type: Feature/Change Request -Package: Feature/Change Request +Package: Directory function related PHP Version: 5CVS-2005-01-06 (dev) Block user comment: N Private report: N New Comment: I fail to see why NULL would be preferable to the more explicit 0777 here. IMHO: won't fix. Previous Comments: ------------------------------------------------------------------------ [2005-01-06 03:07:53] aidan@php.net Description: ------------ As all but the first parameter of mkdir are optional and non sequential, it makes sense to be able to pass NULL to each if a later param is required. E.g. mkdir('some/path/foobar', null, true); Reproduce code: --------------- This patch adds this functionality: http://rafb.net/paste/results/0QWc5F15.html Included in the report: --- \repository\php-src\ext\standard\file.1.395.c Mon Dec 13 10:21:53 2004 +++ \repository\php-src\ext\standard\file.c Thu Jan 06 12:29:41 2005 @@ -1372,7 +1372,7 @@ char *dir; php_stream_context *context; - if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "s|lbr", &dir, &dir_len, &mode, &recursive, &zcontext) == FAILURE) { + if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "s|!lbr", &dir, &dir_len, &mode, &recursive, &zcontext) == FAILURE) { RETURN_FALSE; } ------------------------------------------------------------------------ -- Edit this bug report at https://bugs.php.net/bug.php?id=31424&edit=1