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
Welcome back! If you're the original bug submitter, here's where you can edit the bug or add additional notes.
If you forgot your password, you can retrieve your password here.
Password:
Status:
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: Sun Dec 22 02:01:28 2024 UTC