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-12-23 17:58 UTC
From: calvin at cmpct dot info Assigned: calvinb (profile)
Status: Closed Package: ibm_db2 (PECL)
PHP Version: 7.3.17 OS: IBM i 7.2
Private report: No CVE-ID: None
View Developer Edit
Welcome! If you don't have a Git account, you can't do anything here.
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)

Pull Requests

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?
 [2020-12-23 17:58 UTC] calvinb@php.net
-Status: Open +Status: Closed -Assigned To: +Assigned To: calvinb
 
PHP Copyright © 2001-2024 The PHP Group
All rights reserved.
Last updated: Sat Dec 21 15:01:29 2024 UTC