mboeren         Thu May  6 08:22:14 2004 EDT

  Modified files:              
    /php-src/ext/dbx    dbx_oci8.c dbx_pgsql.c 
  Log:
  Call functions by their new, proper names instead of what are now aliases.
  Change OCIFetchInto (deprecated) to oci_fetch_array.
  
  
http://cvs.php.net/diff.php/php-src/ext/dbx/dbx_oci8.c?r1=1.17&r2=1.18&ty=u
Index: php-src/ext/dbx/dbx_oci8.c
diff -u php-src/ext/dbx/dbx_oci8.c:1.17 php-src/ext/dbx/dbx_oci8.c:1.18
--- php-src/ext/dbx/dbx_oci8.c:1.17     Thu Jan  8 03:15:09 2004
+++ php-src/ext/dbx/dbx_oci8.c  Thu May  6 08:22:14 2004
@@ -20,7 +20,7 @@
    +----------------------------------------------------------------------+
 */
 
-/* $Id: dbx_oci8.c,v 1.17 2004/01/08 08:15:09 andi Exp $ */
+/* $Id: dbx_oci8.c,v 1.18 2004/05/06 12:22:14 mboeren Exp $ */
 
 #include "dbx.h"
 #include "dbx_oci8.h"
@@ -40,7 +40,7 @@
        arguments[0]=username;
        arguments[1]=password;
        arguments[2]=db;
-       dbx_call_any_function(INTERNAL_FUNCTION_PARAM_PASSTHRU, "OCILogon", 
&returned_zval, number_of_arguments, arguments);
+       dbx_call_any_function(INTERNAL_FUNCTION_PARAM_PASSTHRU, "oci_connect", 
&returned_zval, number_of_arguments, arguments);
        if (!returned_zval || Z_TYPE_P(returned_zval)!=IS_RESOURCE) {
                if (returned_zval) zval_ptr_dtor(&returned_zval);
                return 0;
@@ -59,7 +59,7 @@
        arguments[0]=username;
        arguments[1]=password;
        arguments[2]=db;
-       dbx_call_any_function(INTERNAL_FUNCTION_PARAM_PASSTHRU, "OCIPLogon", 
&returned_zval, number_of_arguments, arguments);
+       dbx_call_any_function(INTERNAL_FUNCTION_PARAM_PASSTHRU, "oci_pconnect", 
&returned_zval, number_of_arguments, arguments);
        if (!returned_zval || Z_TYPE_P(returned_zval)!=IS_RESOURCE) {
                if (returned_zval) zval_ptr_dtor(&returned_zval);
                return 0;
@@ -71,14 +71,14 @@
 int dbx_oci8_close(zval **rv, zval **dbx_handle, INTERNAL_FUNCTION_PARAMETERS)
 {
        /* returns 1 as long on success or 0 as long on failure */
-       /* actually, ocilogoff officially does nothing, so what should I return? */
+       /* actually, oci_close officially does nothing, so what should I return? */
        /* I will just return NULL right now and change the test accordingly */
        int number_of_arguments=1;
        zval **arguments[1];
        zval *returned_zval=NULL;
 
        arguments[0]=dbx_handle;
-       dbx_call_any_function(INTERNAL_FUNCTION_PARAM_PASSTHRU, "OCILogOff", 
&returned_zval, number_of_arguments, arguments);
+       dbx_call_any_function(INTERNAL_FUNCTION_PARAM_PASSTHRU, "oci_close", 
&returned_zval, number_of_arguments, arguments);
        if (!returned_zval || Z_TYPE_P(returned_zval)!=IS_BOOL) {
                if (returned_zval) zval_ptr_dtor(&returned_zval);
                return 0;
@@ -98,16 +98,16 @@
 
        arguments[0]=dbx_handle;
        arguments[1]=sql_statement;
-       dbx_call_any_function(INTERNAL_FUNCTION_PARAM_PASSTHRU, "OCIParse", 
&returned_zval, number_of_arguments, arguments);
-       /* OCIParse returns a bool for failure, or a statement_identifier for valid 
sql_statements */
+       dbx_call_any_function(INTERNAL_FUNCTION_PARAM_PASSTHRU, "oci_parse", 
&returned_zval, number_of_arguments, arguments);
+       /* oci_parse returns a bool for failure, or a statement_identifier for valid 
sql_statements */
        if (!returned_zval || (Z_TYPE_P(returned_zval)!=IS_BOOL && 
Z_TYPE_P(returned_zval)!=IS_RESOURCE)) {
                if (returned_zval) zval_ptr_dtor(&returned_zval);
                return 0;
        }
        number_of_arguments=1;
        arguments[0]=&returned_zval;
-       dbx_call_any_function(INTERNAL_FUNCTION_PARAM_PASSTHRU, "OCIExecute", 
&execute_zval, number_of_arguments, arguments);
-       /* OCIExecute returns a bool for success or failure */
+       dbx_call_any_function(INTERNAL_FUNCTION_PARAM_PASSTHRU, "oci_execute", 
&execute_zval, number_of_arguments, arguments);
+       /* oci_execute returns a bool for success or failure */
        if (!execute_zval || Z_TYPE_P(execute_zval)!=IS_BOOL || 
Z_BVAL_P(execute_zval)==0) {
                if (execute_zval) zval_ptr_dtor(&execute_zval);
                zval_ptr_dtor(&returned_zval);
@@ -115,8 +115,8 @@
        }
        number_of_arguments=1;
        arguments[0]=&returned_zval;
-       dbx_call_any_function(INTERNAL_FUNCTION_PARAM_PASSTHRU, "OCIStatementType", 
&statementtype_zval, number_of_arguments, arguments);
-       /* OCIStatementType returns a string. 'SELECT' means there are results */
+       dbx_call_any_function(INTERNAL_FUNCTION_PARAM_PASSTHRU, "oci_statement_type", 
&statementtype_zval, number_of_arguments, arguments);
+       /* oci_statement_type returns a string. 'SELECT' means there are results */
        if (!statementtype_zval || Z_TYPE_P(statementtype_zval)!=IS_STRING) {
                if (statementtype_zval) zval_ptr_dtor(&statementtype_zval);
                if (execute_zval) zval_ptr_dtor(&execute_zval);
@@ -148,7 +148,7 @@
        zval *returned_zval=NULL;
 
        arguments[0]=result_handle;
-       dbx_call_any_function(INTERNAL_FUNCTION_PARAM_PASSTHRU, "OCINumCols", 
&returned_zval, number_of_arguments, arguments);
+       dbx_call_any_function(INTERNAL_FUNCTION_PARAM_PASSTHRU, "oci_num_fields", 
&returned_zval, number_of_arguments, arguments);
        if (!returned_zval || Z_TYPE_P(returned_zval)!=IS_LONG) {
                if (returned_zval) zval_ptr_dtor(&returned_zval);
                return 0;
@@ -170,8 +170,8 @@
        ZVAL_LONG(zval_column_index, column_index+1);
        arguments[0]=result_handle;
        arguments[1]=&zval_column_index;
-       dbx_call_any_function(INTERNAL_FUNCTION_PARAM_PASSTHRU, "OCIColumnName", 
&returned_zval, number_of_arguments, arguments);
-       /* OCIColumnName returns a string */
+       dbx_call_any_function(INTERNAL_FUNCTION_PARAM_PASSTHRU, "oci_field_name", 
&returned_zval, number_of_arguments, arguments);
+       /* oci_field_name returns a string */
        if (!returned_zval || Z_TYPE_P(returned_zval)!=IS_STRING) {
                if (returned_zval) zval_ptr_dtor(&returned_zval);
                FREE_ZVAL(zval_column_index);
@@ -195,8 +195,8 @@
        ZVAL_LONG(zval_column_index, column_index+1);
        arguments[0]=result_handle;
        arguments[1]=&zval_column_index;
-       dbx_call_any_function(INTERNAL_FUNCTION_PARAM_PASSTHRU, "OCIColumnType", 
&returned_zval, number_of_arguments, arguments);
-       /* OCIColumnType returns a string??? */
+       dbx_call_any_function(INTERNAL_FUNCTION_PARAM_PASSTHRU, "oci_field_type", 
&returned_zval, number_of_arguments, arguments);
+       /* oci_field_type returns a string??? */
        if (!returned_zval || Z_TYPE_P(returned_zval)!=IS_STRING) {
                if (returned_zval) zval_ptr_dtor(&returned_zval);
                FREE_ZVAL(zval_column_index);
@@ -211,31 +211,23 @@
 int dbx_oci8_getrow(zval **rv, zval **result_handle, long row_number, 
INTERNAL_FUNCTION_PARAMETERS)
 {
        /* returns array[0..columncount-1] as strings on success or 0 as long on 
failure */
-       int number_of_arguments=3;
-       zval **arguments[3];
+       int number_of_arguments=2;
+       zval **arguments[2];
        zval *zval_resulttype=NULL;
-       zval *zval_returned_array=NULL;
        zval *returned_zval=NULL;
 
-       MAKE_STD_ZVAL(zval_returned_array); /* no value needed, it will be overwritten 
anyway */
-       ZVAL_EMPTY_STRING(zval_returned_array); /* there seems to be some weird 
mem-bug, so assigning a value anyway */
        MAKE_STD_ZVAL(zval_resulttype);
        ZVAL_LONG(zval_resulttype, OCI_NUM | OCI_RETURN_NULLS | OCI_RETURN_LOBS); /* 
no ASSOC, dbx handles that part */
        arguments[0]=result_handle;
-       arguments[1]=&zval_returned_array;
-       arguments[2]=&zval_resulttype;
-       dbx_call_any_function(INTERNAL_FUNCTION_PARAM_PASSTHRU, "OCIFetchInto", 
&returned_zval, number_of_arguments, arguments);
-       /* OCIFetchInto returns the number of columns as an integer on success and 
FALSE */
-       /* on failure. The actual array is passed back in arg[1] */
-       if (!returned_zval || Z_TYPE_P(returned_zval)!=IS_LONG || 
Z_LVAL_P(returned_zval)==0) {
+       arguments[1]=&zval_resulttype;
+       dbx_call_any_function(INTERNAL_FUNCTION_PARAM_PASSTHRU, "oci_fetch_array", 
&returned_zval, number_of_arguments, arguments);
+       if (!returned_zval || Z_TYPE_P(returned_zval)!=IS_ARRAY) {
                if (returned_zval) zval_ptr_dtor(&returned_zval);
                FREE_ZVAL(zval_resulttype);
-               FREE_ZVAL(zval_returned_array);
                return 0;
        }
        FREE_ZVAL(zval_resulttype);
-       zval_ptr_dtor(&returned_zval);
-       MOVE_RETURNED_TO_RV(rv, zval_returned_array);
+       MOVE_RETURNED_TO_RV(rv, returned_zval);
        return 1;
 }
 
@@ -251,8 +243,8 @@
        zval *returned_message_zval=NULL;
        arguments[0]=dbx_handle;
        if (!dbx_handle) number_of_arguments=0;
-       dbx_call_any_function(INTERNAL_FUNCTION_PARAM_PASSTHRU, "OCIError", 
&returned_zval, number_of_arguments, arguments);
-       /* OCIError should returns an assoc array containing code & message, dbx needs 
the message */
+       dbx_call_any_function(INTERNAL_FUNCTION_PARAM_PASSTHRU, "oci_error", 
&returned_zval, number_of_arguments, arguments);
+       /* oci_error should returns an assoc array containing code & message, dbx 
needs the message */
        if (!returned_zval || Z_TYPE_P(returned_zval)!=IS_ARRAY) {
                if (returned_zval) zval_ptr_dtor(&returned_zval);
                return 0;
http://cvs.php.net/diff.php/php-src/ext/dbx/dbx_pgsql.c?r1=1.24&r2=1.25&ty=u
Index: php-src/ext/dbx/dbx_pgsql.c
diff -u php-src/ext/dbx/dbx_pgsql.c:1.24 php-src/ext/dbx/dbx_pgsql.c:1.25
--- php-src/ext/dbx/dbx_pgsql.c:1.24    Thu Jan  8 03:15:10 2004
+++ php-src/ext/dbx/dbx_pgsql.c Thu May  6 08:22:14 2004
@@ -16,7 +16,7 @@
    +----------------------------------------------------------------------+
  */
 
-/* $Id: dbx_pgsql.c,v 1.24 2004/01/08 08:15:10 andi Exp $ */
+/* $Id: dbx_pgsql.c,v 1.25 2004/05/06 12:22:14 mboeren Exp $ */
 
 #include "dbx.h"
 #include "php_dbx.h"
@@ -148,7 +148,7 @@
        /* db_name is not used in this function */
        args[0]=dbx_handle;
        args[1]=sql_statement;
-       dbx_call_any_function(INTERNAL_FUNCTION_PARAM_PASSTHRU, "pg_exec", 
&returned_zval, nargs, args);
+       dbx_call_any_function(INTERNAL_FUNCTION_PARAM_PASSTHRU, "pg_query", 
&returned_zval, nargs, args);
        /* pg_query returns a bool for success or failure, 
           or a result_identifier for select statements */
        if (!returned_zval || (Z_TYPE_P(returned_zval)!=IS_BOOL && 
Z_TYPE_P(returned_zval)!=IS_RESOURCE)) {
@@ -167,7 +167,7 @@
        zval *returned_zval=NULL;
 
        arguments[0]=result_handle;
-       dbx_call_any_function(INTERNAL_FUNCTION_PARAM_PASSTHRU, "pg_numfields", 
&returned_zval, number_of_arguments, arguments);
+       dbx_call_any_function(INTERNAL_FUNCTION_PARAM_PASSTHRU, "pg_num_fields", 
&returned_zval, number_of_arguments, arguments);
        if (!returned_zval || Z_TYPE_P(returned_zval)!=IS_LONG) {
                if (returned_zval) zval_ptr_dtor(&returned_zval);
                return 0;
@@ -188,7 +188,7 @@
        ZVAL_LONG(zval_column_index, column_index);
        arguments[0]=result_handle;
        arguments[1]=&zval_column_index;
-       dbx_call_any_function(INTERNAL_FUNCTION_PARAM_PASSTHRU, "pg_fieldname", 
&returned_zval, number_of_arguments, arguments);
+       dbx_call_any_function(INTERNAL_FUNCTION_PARAM_PASSTHRU, "pg_field_name", 
&returned_zval, number_of_arguments, arguments);
        /* pg_fieldname returns a string */
        if (!returned_zval || Z_TYPE_P(returned_zval)!=IS_STRING) {
                if (returned_zval) zval_ptr_dtor(&returned_zval);
@@ -212,7 +212,7 @@
        ZVAL_LONG(zval_column_index, column_index);
        arguments[0]=result_handle;
        arguments[1]=&zval_column_index;
-       dbx_call_any_function(INTERNAL_FUNCTION_PARAM_PASSTHRU, "pg_fieldtype", 
&returned_zval, number_of_arguments, arguments);
+       dbx_call_any_function(INTERNAL_FUNCTION_PARAM_PASSTHRU, "pg_field_type", 
&returned_zval, number_of_arguments, arguments);
        /* pg_fieldtype returns a string */
        if (!returned_zval || Z_TYPE_P(returned_zval)!=IS_STRING) {
                if (returned_zval) zval_ptr_dtor(&returned_zval);
@@ -252,7 +252,7 @@
 
        arguments[0]=dbx_handle;
        if (!dbx_handle) number_of_arguments=0;
-       dbx_call_any_function(INTERNAL_FUNCTION_PARAM_PASSTHRU, "pg_errormessage", 
&returned_zval, number_of_arguments, arguments);
+       dbx_call_any_function(INTERNAL_FUNCTION_PARAM_PASSTHRU, "pg_last_error", 
&returned_zval, number_of_arguments, arguments);
        if (!returned_zval || Z_TYPE_P(returned_zval)!=IS_STRING) {
                if (returned_zval) zval_ptr_dtor(&returned_zval);
                return 0;

-- 
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php

Reply via email to