cubrid_affected_rows() requires a different parameter than mysql_affected_rows()

Details

  • Type: Improvement Improvement
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: PHP-8.4.1
  • Fix Version/s: PHP-8.4.1, PHP-Apricot
  • Component/s: PHPDriver
  • Labels:
    None

Description

cubrid_affected_rows() has the following definition:

int cubrid_affected_rows ([ resource $result ] )

While mysql_affected_rows():

int mysql_affected_rows ([ resource $link_identifier ] )

Thus, they are incompatible as MySQL requires a connection resource, while CUBRID requires result resource. Is there any specific reason we require result resource instead of connection resource like in MySQL?

Issue Links

relates to

Improvement - An improvement or enhancement to an existing feature or task. APIS-114 Improve documentation for cubrid_affected_rows()

  • Minor - Minor loss of function, or other problem where easy workaround is present.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are closed can be reopened.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
张慧 added a comment -

Commit rev: 437

Show
张慧 added a comment - Commit rev: 437
Hide
Permalink
张慧 added a comment -

Commit rev: 452

Show
张慧 added a comment - Commit rev: 452
Hide
Permalink
张慧 added a comment -

We have resolved this issue on PHP-8.4.1 and PHP-Apricot. We also have update the document too, but there may be few days later that cubrid_affected_rows will display the new contents.

If you have any problem, please let me know. Thank you!

Show
张慧 added a comment - We have resolved this issue on PHP-8.4.1 and PHP-Apricot. We also have update the document too, but there may be few days later that cubrid_affected_rows will display the new contents. If you have any problem, please let me know. Thank you!
Hide
Permalink
Esen Sagynov added a comment -

Cool! Thank you for all improvements! When do you plan to release the new version of the driver?

Show
Esen Sagynov added a comment - Cool! Thank you for all improvements! When do you plan to release the new version of the driver?
Hide
Permalink
张慧 added a comment -

Once CUBRID releases the new version, we will release PHP-8.4.1.0002

Show
张慧 added a comment - Once CUBRID releases the new version, we will release PHP-8.4.1.0002
Hide
Permalink
Esen Sagynov added a comment -

I will release the new version 8.4.1 (patch 2) on Monday. Looking for your update after that, then.

Show
Esen Sagynov added a comment - I will release the new version 8.4.1 (patch 2) on Monday. Looking for your update after that, then.
Hide
Permalink
张慧 added a comment -

I have found the document of cubrid_affected_rows on php.net have been updated.

If you have any problem, please let me know.

Thank you!

Show
张慧 added a comment - I have found the document of cubrid_affected_rows on php.net have been updated. If you have any problem, please let me know. Thank you!

People

  • Assignee:
    张慧
    Reporter:
    Esen Sagynov
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: