Skip to content

Commit 4b080b5

Browse files
committed
Mark ob_*() functions has side-effects
1 parent 9cd58b5 commit 4b080b5

File tree

3 files changed

+25
-1
lines changed

3 files changed

+25
-1
lines changed

bin/functionMetadata_original.php

+12
Original file line numberDiff line numberDiff line change
@@ -98,6 +98,18 @@
9898
'mb_str_pad' => ['hasSideEffects' => false],
9999
'mkdir' => ['hasSideEffects' => true],
100100
'move_uploaded_file' => ['hasSideEffects' => true],
101+
'ob_clean' => ['hasSideEffects' => true],
102+
'ob_end_clean' => ['hasSideEffects' => true],
103+
'ob_end_flush' => ['hasSideEffects' => true],
104+
'ob_flush' => ['hasSideEffects' => true],
105+
'ob_get_clean' => ['hasSideEffects' => true],
106+
'ob_get_contents' => ['hasSideEffects' => true],
107+
'ob_get_length' => ['hasSideEffects' => true],
108+
'ob_get_level' => ['hasSideEffects' => true],
109+
'ob_get_status' => ['hasSideEffects' => true],
110+
'ob_list_handlers' => ['hasSideEffects' => true],
111+
'output_add_rewrite_var' => ['hasSideEffects' => true],
112+
'output_reset_rewrite_vars' => ['hasSideEffects' => true],
101113
'pclose' => ['hasSideEffects' => true],
102114
'popen' => ['hasSideEffects' => true],
103115
'readfile' => ['hasSideEffects' => true],

bin/generate-function-metadata.php

+1
Original file line numberDiff line numberDiff line change
@@ -86,6 +86,7 @@ public function enterNode(Node $node)
8686
'connection_aborted',
8787
'connection_status',
8888
'file_get_contents',
89+
'ob_get_contents',
8990
], true)) {
9091
continue;
9192
}

resources/functionMetadata.php

+12-1
Original file line numberDiff line numberDiff line change
@@ -1382,11 +1382,22 @@
13821382
'numfmt_get_symbol' => ['hasSideEffects' => false],
13831383
'numfmt_get_text_attribute' => ['hasSideEffects' => false],
13841384
'numfmt_parse' => ['hasSideEffects' => false],
1385+
'ob_clean' => ['hasSideEffects' => true],
1386+
'ob_end_clean' => ['hasSideEffects' => true],
1387+
'ob_end_flush' => ['hasSideEffects' => true],
13851388
'ob_etaghandler' => ['hasSideEffects' => false],
1386-
'ob_get_contents' => ['hasSideEffects' => false],
1389+
'ob_flush' => ['hasSideEffects' => true],
1390+
'ob_get_clean' => ['hasSideEffects' => true],
1391+
'ob_get_contents' => ['hasSideEffects' => true],
1392+
'ob_get_length' => ['hasSideEffects' => true],
1393+
'ob_get_level' => ['hasSideEffects' => true],
1394+
'ob_get_status' => ['hasSideEffects' => true],
13871395
'ob_iconv_handler' => ['hasSideEffects' => false],
1396+
'ob_list_handlers' => ['hasSideEffects' => true],
13881397
'octdec' => ['hasSideEffects' => false],
13891398
'ord' => ['hasSideEffects' => false],
1399+
'output_add_rewrite_var' => ['hasSideEffects' => true],
1400+
'output_reset_rewrite_vars' => ['hasSideEffects' => true],
13901401
'pack' => ['hasSideEffects' => false],
13911402
'pam_auth' => ['hasSideEffects' => false],
13921403
'pam_chpass' => ['hasSideEffects' => false],

0 commit comments

Comments
 (0)