php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login
Bug #79540 Tests shouldn't be using functions removed in PHP 7
Submitted: 2020-04-29 17:08 UTC Modified: 2020-06-17 18:39 UTC
From: calvin at cmpct dot info Assigned:
Status: Open Package: ibm_db2 (PECL)
PHP Version: 7.3.17 OS: IBM i 7.2
Private report: No CVE-ID: None
View Add Comment Developer Edit
Welcome! If you don't have a Git account, you can't do anything here.
You can add a comment by following this link or if you reported this bug, you can edit this bug over here.
Block user comment
Status: Assign to:
Package:
Bug Type:
Summary:
From: calvin at cmpct dot info
New email:
PHP Version: OS:

 

 [2020-04-29 17:08 UTC] calvin at cmpct dot info
Description:
------------
eregi was removed. It should be using preg_match with an i regex instead.

Affected:

* tests/test_V6_060_Tables_01.phpt
* tests/test_V6_061_Tables_02.phpt
* tests/test_V6_062_Tables_03.phpt
* tests/test_V6_064_Tables_05.phpt
* tests/test_V6_066_TableObjects.phpt


Patches

tests-shouldnt-use-a-function-removed-in-php-7.patch (last revision 2020-04-29 17:08 UTC by calvin at cmpct dot info)

Add a Patch

Pull Requests

Add a Pull Request

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2020-06-17 18:35 UTC] alexdowad@php.net
This patch looks good to me. Could you submit it as a pull request to php-src in GitHub?
 [2020-06-17 18:39 UTC] calvin at cmpct dot info
The extension is in PECL instead of php-src/ext; is it OK to submit to the PECL repo on GH?
 [2020-06-18 17:52 UTC] alexinbeijing at gmail dot com
Calvin: Thanks very much for pointing that out! I'm sorry I didn't note that the patch is not for a built-in extension.

It looks like the repo for this extension is here:

https://github.com/php/pecl-database-ibm_db2

Could you try submitting the patch as a PR there?
 
PHP Copyright © 2001-2020 The PHP Group
All rights reserved.
Last updated: Wed Jul 08 01:01:25 2020 UTC