Skip to content

Conversation

@JayPHP
Copy link
Contributor

@JayPHP JayPHP commented Apr 25, 2017

@morrisonlevi
Copy link
Contributor

morrisonlevi commented Apr 25, 2017

Will you please add a comment in the code that links to the corresponding bug report in case the contains was intentional? That way when we re-discover why it uses contains we can properly fix both.

@JayPHP
Copy link
Contributor Author

JayPHP commented Apr 26, 2017

@morrisonlevi sure, all done.

@JayPHP
Copy link
Contributor Author

JayPHP commented Jul 20, 2017

Any chance of getting this merged?

@php-pulls php-pulls merged commit d897477 into php:master Jul 20, 2017
@peterdd
Copy link
Contributor

peterdd commented Sep 3, 2021

On some pages this does not work: https://www.php.net/manual/en/mysqli.affected-rows.php

Maybe because 'mysql' === '\nmysql' is false..?

Related: https://bugs.php.net/bug.php?id=77036

parameterscroll

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants