abies           Fri May 21 09:46:04 2004 EDT

  Modified files:              
    /php-src/ext/interbase      ibase_query.c 
  Log:
  Added test for duplicate field names when returning rows as hashes or objects
  # This change should be in 5.0.0, as it might break BC
  
  
http://cvs.php.net/diff.php/php-src/ext/interbase/ibase_query.c?r1=1.6&r2=1.7&ty=u
Index: php-src/ext/interbase/ibase_query.c
diff -u php-src/ext/interbase/ibase_query.c:1.6 php-src/ext/interbase/ibase_query.c:1.7
--- php-src/ext/interbase/ibase_query.c:1.6     Wed May 12 11:16:51 2004
+++ php-src/ext/interbase/ibase_query.c Fri May 21 09:46:04 2004
@@ -16,7 +16,7 @@
    +----------------------------------------------------------------------+
  */
 
-/* $Id: ibase_query.c,v 1.6 2004/05/12 15:16:51 abies Exp $ */
+/* $Id: ibase_query.c,v 1.7 2004/05/21 13:46:04 abies Exp $ */
 
 #ifdef HAVE_CONFIG_H
 #include "config.h"
@@ -1504,7 +1504,31 @@
        array_init(return_value);
 
        for (i = 0; i < ib_result->out_sqlda->sqld; ++i) {
+               char alias[METADATALENGTH+4];
                XSQLVAR *var = &ib_result->out_sqlda->sqlvar[i];
+               
+               if (! (fetch_type & FETCH_ROW)) {
+                       int i = 0;
+                       char const *base = "FIELD"; /* use 'FIELD' if name is empty */
+                       
+                       /**
+                       * Ensure no two columns have identical names: 
+                       * keep generating new names until we find one that is unique.
+                       */
+                       switch (*var->aliasname) {
+                               void *p;
+                               
+                               default:
+                                       i = 1;
+                                       strcpy(alias, base = var->aliasname);
+                                       
+                                       while (SUCCESS == 
zend_symtable_find(Z_ARRVAL_P(return_value),alias,strlen(alias)+1,&p)) {
+                               
+                               case '\0':
+                                               sprintf(alias, "%s_%02d", base, i++);
+                                       }
+                       }
+               }
 
                if (((var->sqltype & 1) == 0) || *var->sqlind != -1) {
                        zval *result;
@@ -1614,13 +1638,13 @@
                        if (fetch_type & FETCH_ROW) {
                                add_index_zval(return_value, i, result);
                        } else {
-                               add_assoc_zval(return_value, var->aliasname, result);
+                               add_assoc_zval(return_value, alias, result);
                        }
                } else {
                        if (fetch_type & FETCH_ROW) {
                                add_index_null(return_value, i);
                        } else {
-                               add_assoc_null(return_value, var->aliasname);
+                               add_assoc_null(return_value, alias);
                        }
                }
        } /* for field */

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

Reply via email to