Bug #9629
Replace usage of deprecated mysql_* functions
| Status: | Verified | Start date: | 03/28/2016 | |
|---|---|---|---|---|
| Priority: | Medium | Due date: | ||
| Assignee: | Dan Gillean | % Done: | 0% | |
| Category: | Internals | |||
| Target version: | Release 2.3.0 | |||
| Google Code Legacy ID: | Tested version: | |||
| Sponsored: | No | Requires documentation: |
Description
PHP 5.5.0 (available in Ubuntu 14.04) deprecated the mysql extension. Instead, either the mysqli or PDO_MySQL extensions should be used.
PHP 7.0 removed the mysql extension.
We have a class that's still using mysql: lib/QubitCsvTransform.class.php.
Related issues
History
#1 Updated by Jesús García Crespo about 6 years ago
- Subject changed from Replace deprecated mysql_* functions to Replace usage of deprecated mysql_* functions
#2 Updated by Jesús García Crespo about 6 years ago
Btw docs won't need updates as the php5-mysql package in Ubuntu installs both mysql and mysqli extensions in 12.04/14.04.
#3 Updated by Jesús García Crespo about 6 years ago
- Status changed from In progress to Code Review
- Assignee changed from Jesús García Crespo to José Raddaoui Marín
#4 Updated by Jesús García Crespo about 6 years ago
- Related to Task #9616: Make AtoM PHP 5.6-compatible, PHP 7.0-compatible and docs for Ubuntu Xenial 16.04 added
#5 Updated by Jesús García Crespo about 6 years ago
- Assignee changed from José Raddaoui Marín to Mike Cantelon
#6 Updated by Mike Cantelon about 6 years ago
- Status changed from Code Review to Feedback
- Assignee changed from Mike Cantelon to Jesús García Crespo
Looks good to me!
#7 Updated by Jesús García Crespo about 6 years ago
- Status changed from Feedback to Verified
#8 Updated by Jesús García Crespo about 6 years ago
- Status changed from Verified to QA/Review
- Assignee changed from Jesús García Crespo to Dan Gillean
Dan, QubitCsvTransform seems to be used only from: lib/task/import/example/transformation/example.transform.php.
#9 Updated by Dan Gillean about 6 years ago
- Status changed from QA/Review to Verified
I'm not going to test this, since it is a courtesy example we leave for developers wishing to write transformation scripts, and I am not a developer. I'm not sure if any out there is using this, but we do have documentation that references it in our wiki here:
I suggest we leave it in place for now. And only pay it more attention if we get reports that it does not work.