Uploaded image for project: 'CUBRID APIs'
  1. CUBRID APIs
  2. APIS-106

cubrid_affected_rows() requires a different parameter than mysql_affected_rows()

    Details

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

      Description

      cubrid_affected_rows() has the following definition:

      {code}
      int cubrid_affected_rows ([ resource $result ] ){code}

      While mysql_affected_rows():

      {code}
      int mysql_affected_rows ([ resource $link_identifier ] ){code}

      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

          Activity

          Hide
          hellohuizhang 张慧 added a comment -

          Commit rev: 437

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

          Commit rev: 452

          Show
          hellohuizhang 张慧 added a comment - Commit rev: 452
          Hide
          hellohuizhang 张慧 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
          hellohuizhang 张慧 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
          kadishmal Esen Sagynov added a comment -

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

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

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

          Show
          hellohuizhang 张慧 added a comment - Once CUBRID releases the new version, we will release PHP-8.4.1.0002
          Hide
          kadishmal 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
          kadishmal 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
          hellohuizhang 张慧 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
          hellohuizhang 张慧 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:
              hellohuizhang 张慧
              Reporter:
              kadishmal Esen Sagynov
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: