Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Restore access to the mysqli object #5486

Merged
merged 1 commit into from Dec 16, 2022

Conversation

mc12345678
Copy link
Contributor

Allows access to the mysqli object of the database class. This permits the following mysqli_xxx functions to be usable:

mysqli_affected_rows
mysqli_autocommit
mysqli_begin_transaction
mysqli_change_user
mysqli_character_set_name
mysqli_close
mysqli_commit
mysqli_dump_debug_info
mysqli_errno
mysqli_error_list
mysqli_error
mysqli_execute_query
mysqli_field_count
mysqli_get_charset
mysqli_get_connection_stats
mysqli_get_host_info
mysqli_get_proto_info
mysqli_get_server_info
mysqli_get_server_version
mysqli_get_warnings
mysqli_info
mysqli_insert_id
mysqli_kill
mysqli_more_results
mysqli_multi_query
mysqli_next_result
mysqli_options
mysqli_ping
mysqli_prepare
mysqli_query
mysqli_real_connect
mysqli_real_escape_string
mysqli_real_query
mysqli_reap_async_query
mysqli_refresh
mysqli_release_savepoint
mysqli_rollback
mysqli_savepoint
mysqli_select_db
mysqli_set_charset
mysqli_sqlstate
mysqli_ssl_set
mysqli_stat
mysqli_stmt_init
mysqli_store_result
mysqli_thread_id
mysqli_use_result
mysqli_warning_count

An alternative to this would be to create an accessor function. Doing so would still break other code just like the member currently being private.

Fixes #5484

Allows access to the mysqli object of the database class. This permits
the following mysqli_xxx functions to be usable:

mysqli_affected_rows
mysqli_autocommit
mysqli_begin_transaction
mysqli_change_user
mysqli_character_set_name
mysqli_close
mysqli_commit
mysqli_dump_debug_info
mysqli_errno
mysqli_error_list
mysqli_error
mysqli_execute_query
mysqli_field_count
mysqli_get_charset
mysqli_get_connection_stats
mysqli_get_host_info
mysqli_get_proto_info
mysqli_get_server_info
mysqli_get_server_version
mysqli_get_warnings
mysqli_info
mysqli_insert_id
mysqli_kill
mysqli_more_results
mysqli_multi_query
mysqli_next_result
mysqli_options
mysqli_ping
mysqli_prepare
mysqli_query
mysqli_real_connect
mysqli_real_escape_string
mysqli_real_query
mysqli_reap_async_query
mysqli_refresh
mysqli_release_savepoint
mysqli_rollback
mysqli_savepoint
mysqli_select_db
mysqli_set_charset
mysqli_sqlstate
mysqli_ssl_set
mysqli_stat
mysqli_stmt_init
mysqli_store_result
mysqli_thread_id
mysqli_use_result
mysqli_warning_count

An alternative to this would be to create an accessor function. Doing so
would still break other code just like the member currently being private.

Fixes zencart#5484
@scottcwilson scottcwilson merged commit 0c7e217 into zencart:v158 Dec 16, 2022
@scottcwilson
Copy link
Sponsor Contributor

scottcwilson commented Jan 4, 2023

This change may be backed out in a future release after people have had time to update their software. The number of packages in the Plugin Library that use $db->link directly is not very large (and some of these may be old versions - my check was very superficial):

Block_Email_Manager
EasyPopulate-v4
advanced_cross_sell_151
ceon_back_in_stock_notifications
ebay
inventory
multi_cross_sell_v1
non_blocking_lock
paypay
simple-seo-url_1
snap_affiliates-4
store_credit

We'll discuss when a new full release is on the horizon.

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.

v158 use of (many) MySQLi_ functions disabled
2 participants