Removed ibase_timefmt() - use ini_set() instead

Improved phpinfo() appearance

diff --git a/ext/interbase/interbase.c b/ext/interbase/interbase.c
index 150b62058bc2e6069611d61201cc63d208c3d43d..e26eaf53836e050fad0f81dcd737948ebd4cae44 100644 (file)
--- a/ext/interbase/interbase.c
+++ b/ext/interbase/interbase.c
@@ -59,7 +59,6 @@ function_entry ibase_functions[] = {
        PHP_FE(ibase_prepare, NULL)
        PHP_FE(ibase_execute, NULL)
        PHP_FE(ibase_free_query, NULL)
-       PHP_FE(ibase_timefmt, NULL)
        PHP_FE(ibase_gen_id, NULL)
        PHP_FE(ibase_num_fields, NULL)
        PHP_FE(ibase_num_params, NULL)
@@ -124,7 +123,6 @@ function_entry ibase_functions[] = {
        PHP_FALIAS(fbird_prepare,ibase_prepare, NULL)
        PHP_FALIAS(fbird_execute,ibase_execute, NULL)
        PHP_FALIAS(fbird_free_query,ibase_free_query, NULL)
-       PHP_FALIAS(fbird_timefmt,ibase_timefmt, NULL)
        PHP_FALIAS(fbird_gen_id,ibase_gen_id, NULL)
        PHP_FALIAS(fbird_num_fields,ibase_num_fields, NULL)
        PHP_FALIAS(fbird_num_params,ibase_num_params, NULL)
@@ -186,13 +184,6 @@ zend_module_entry ibase_module_entry = {
 
 #ifdef COMPILE_DL_INTERBASE
 ZEND_GET_MODULE(ibase)
-#define DL_MALLOC(size) malloc(size)
-#define DL_STRDUP(str) strdup(str)
-#define DL_FREE(ptr) free(ptr)
-#else
-#define DL_MALLOC(size) emalloc(size)
-#define DL_STRDUP(str) estrdup(str)
-#define DL_FREE(ptr) efree(ptr)
 #endif
 
 /* True globals, no need for thread safety */
@@ -429,6 +420,16 @@ static void _php_ibase_free_trans(zend_rsrc_list_entry *rsrc TSRMLS_DC) /* {{{ *
 }
 /* }}} */
 
+static ZEND_INI_DISP(php_ibase_password_displayer_cb)
+{
+       if ((type == ZEND_INI_DISPLAY_ORIG && ini_entry->orig_value) 
+                       || (type != ZEND_INI_DISPLAY_ORIG && ini_entry->value)) {
+               ZEND_PUTS("********");
+       } else {
+               ZEND_PUTS("no value");
+       }
+}
+
 /* {{{ startup, shutdown and info functions */
 PHP_INI_BEGIN()
        STD_PHP_INI_BOOLEAN("ibase.allow_persistent", "1", PHP_INI_SYSTEM, OnUpdateBool, allow_persistent, zend_ibase_globals, ibase_globals)
@@ -436,18 +437,15 @@ PHP_INI_BEGIN()
        STD_PHP_INI_ENTRY_EX("ibase.max_links", "-1", PHP_INI_SYSTEM, OnUpdateLong, max_links, zend_ibase_globals, ibase_globals, display_link_numbers)
        STD_PHP_INI_ENTRY("ibase.default_db", NULL, PHP_INI_SYSTEM, OnUpdateString, default_db, zend_ibase_globals, ibase_globals)
        STD_PHP_INI_ENTRY("ibase.default_user", NULL, PHP_INI_ALL, OnUpdateString, default_user, zend_ibase_globals, ibase_globals)
-       STD_PHP_INI_ENTRY("ibase.default_password", NULL, PHP_INI_ALL, OnUpdateString, default_password, zend_ibase_globals, ibase_globals)
+       STD_PHP_INI_ENTRY_EX("ibase.default_password", NULL, PHP_INI_ALL, OnUpdateString, default_password, zend_ibase_globals, ibase_globals,php_ibase_password_displayer_cb)
        STD_PHP_INI_ENTRY("ibase.default_charset", NULL, PHP_INI_ALL, OnUpdateString, default_charset, zend_ibase_globals, ibase_globals)
-       STD_PHP_INI_ENTRY("ibase.timestampformat", IB_DEF_DATE_FMT " " IB_DEF_TIME_FMT, PHP_INI_ALL, OnUpdateString, cfg_timestampformat, zend_ibase_globals, ibase_globals)
-       STD_PHP_INI_ENTRY("ibase.dateformat", IB_DEF_DATE_FMT, PHP_INI_ALL, OnUpdateString, cfg_dateformat, zend_ibase_globals, ibase_globals)
-       STD_PHP_INI_ENTRY("ibase.timeformat", IB_DEF_TIME_FMT, PHP_INI_ALL, OnUpdateString, cfg_timeformat, zend_ibase_globals, ibase_globals)
+       STD_PHP_INI_ENTRY("ibase.timestampformat", IB_DEF_DATE_FMT " " IB_DEF_TIME_FMT, PHP_INI_ALL, OnUpdateString, timestampformat, zend_ibase_globals, ibase_globals)
+       STD_PHP_INI_ENTRY("ibase.dateformat", IB_DEF_DATE_FMT, PHP_INI_ALL, OnUpdateString, dateformat, zend_ibase_globals, ibase_globals)
+       STD_PHP_INI_ENTRY("ibase.timeformat", IB_DEF_TIME_FMT, PHP_INI_ALL, OnUpdateString, timeformat, zend_ibase_globals, ibase_globals)
 PHP_INI_END()
 
 static void php_ibase_init_globals(zend_ibase_globals *ibase_globals)
 {
-       ibase_globals->timestampformat = NULL;
-       ibase_globals->dateformat = NULL;
-       ibase_globals->timeformat = NULL;
        ibase_globals->num_persistent = 0;
        ibase_globals->sql_code = 0;
 }
@@ -467,9 +465,6 @@ PHP_MINIT_FUNCTION(ibase)
        REGISTER_LONG_CONSTANT("IBASE_FETCH_BLOBS", PHP_IBASE_FETCH_BLOBS, CONST_PERSISTENT);
        REGISTER_LONG_CONSTANT("IBASE_FETCH_ARRAYS", PHP_IBASE_FETCH_ARRAYS, CONST_PERSISTENT);
        REGISTER_LONG_CONSTANT("IBASE_UNIXTIME", PHP_IBASE_UNIXTIME, CONST_PERSISTENT);
-       REGISTER_LONG_CONSTANT("IBASE_TIMESTAMP", PHP_IBASE_TIMESTAMP, CONST_PERSISTENT);
-       REGISTER_LONG_CONSTANT("IBASE_DATE", PHP_IBASE_DATE, CONST_PERSISTENT);
-       REGISTER_LONG_CONSTANT("IBASE_TIME", PHP_IBASE_TIME, CONST_PERSISTENT);
 
        /* transactions */
        REGISTER_LONG_CONSTANT("IBASE_WRITE", PHP_IBASE_WRITE, CONST_PERSISTENT);
@@ -495,21 +490,6 @@ PHP_RINIT_FUNCTION(ibase)
        IBG(default_link)= -1;
        IBG(num_links) = IBG(num_persistent);
 
-       if (IBG(timestampformat)) {
-               DL_FREE(IBG(timestampformat));
-       }
-       IBG(timestampformat) = DL_STRDUP(IBG(cfg_timestampformat));
-
-       if (IBG(dateformat)) {
-               DL_FREE(IBG(dateformat));
-       }
-       IBG(dateformat) = DL_STRDUP(IBG(cfg_dateformat));
-
-       if (IBG(timeformat)) {
-               DL_FREE(IBG(timeformat));
-       }
-       IBG(timeformat) = DL_STRDUP(IBG(cfg_timeformat));
-
        RESET_ERRMSG;
 
        return SUCCESS;
@@ -542,21 +522,6 @@ PHP_MSHUTDOWN_FUNCTION(ibase)
 
 PHP_RSHUTDOWN_FUNCTION(ibase)
 {
-       if (IBG(timestampformat)) {
-               DL_FREE(IBG(timestampformat));
-       }
-       IBG(timestampformat) = NULL;
-
-       if (IBG(dateformat)) {
-               DL_FREE(IBG(dateformat));
-       }
-       IBG(dateformat) = NULL;
-
-       if (IBG(timeformat)) {
-               DL_FREE(IBG(timeformat));
-       }
-       IBG(timeformat) = NULL;
-
        return SUCCESS;
 } 
  
@@ -565,7 +530,12 @@ PHP_MINFO_FUNCTION(ibase)
        char tmp[64], *s;
 
        php_info_print_table_start();
-       php_info_print_table_row(2, "Interbase Support", "enabled");
+       php_info_print_table_row(2, "Firebird/InterBase Support", 
+#ifdef COMPILE_DL_INTERBASE
+               "dynamic");
+#else
+               "static");
+#endif
 
 #ifdef FB_API_VER
        sprintf( (s = tmp), "Firebird API version %d", FB_API_VER);
@@ -600,29 +570,7 @@ PHP_MINFO_FUNCTION(ibase)
        } while (0);
 #endif                 
 
-       php_info_print_table_row(2, "Revision", FILE_REVISION);
-#ifdef COMPILE_DL_INTERBASE
-       php_info_print_table_row(2, "Dynamic Module", "Yes");
-#endif
-       php_info_print_table_row(2, "Allow Persistent Links", (IBG(allow_persistent) ? "Yes" : "No"));
-
-       if (IBG(max_persistent) == -1) {
-               sprintf(tmp, "%ld/unlimited", IBG(num_persistent));
-       } else {
-               sprintf(tmp, "%ld/%ld", IBG(num_persistent), IBG(max_persistent));
-       }
-       php_info_print_table_row(2, "Persistent Links", tmp);
-
-       if (IBG(max_links) == -1) {
-               sprintf(tmp, "%ld/unlimited", IBG(num_links));
-       } else {
-               sprintf(tmp, "%ld/%ld", IBG(num_links), IBG(max_links));
-       }
-       php_info_print_table_row(2, "Total Links", tmp);
-
-       php_info_print_table_row(2, "Timestamp Format", IBG(timestampformat));
-       php_info_print_table_row(2, "Date Format", IBG(dateformat));
-       php_info_print_table_row(2, "Time Format", IBG(timeformat));
+       DISPLAY_INI_ENTRIES();
 
        php_info_print_table_end();
 }
@@ -1191,45 +1139,6 @@ PHP_FUNCTION(ibase_rollback_ret)
 }
 /* }}} */
 
-/* {{{ proto bool ibase_timefmt(string format [, int type ])
-   Sets the format of timestamp, date and time columns returned from queries */
-PHP_FUNCTION(ibase_timefmt)
-{
-       char *fmt;
-       int fmt_len;
-       long type = PHP_IBASE_TIMESTAMP;
-       
-       if (FAILURE == zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "s|l", &fmt, &fmt_len, &type)) {
-               RETURN_FALSE;
-       }
-
-       switch (type) {
-               case PHP_IBASE_TIMESTAMP:
-                       if (IBG(timestampformat)) {
-                               DL_FREE(IBG(timestampformat));
-                       }
-                       IBG(timestampformat) = DL_STRDUP(fmt);
-                       break;
-               case PHP_IBASE_DATE:
-                       if (IBG(dateformat)) {
-                               DL_FREE(IBG(dateformat));
-                       }
-                       IBG(dateformat) = DL_STRDUP(fmt);
-                       break;
-               case PHP_IBASE_TIME:
-                       if (IBG(timeformat)) {
-                               DL_FREE(IBG(timeformat));
-                       }
-                       IBG(timeformat) = DL_STRDUP(fmt);
-                       break;
-               default:
-                       RETURN_FALSE;
-       }
-
-       RETURN_TRUE;
-}
-/* }}} */
-
 /* {{{ proto int ibase_gen_id(string generator [, int increment [, resource link_identifier ]])
    Increments the named generator and returns its new value */
 PHP_FUNCTION(ibase_gen_id)

diff --git a/ext/interbase/php_ibase_includes.h b/ext/interbase/php_ibase_includes.h
index fb488b6741b2283bab05c2baf542f1108d13a4a5..f92a4f7ab971a83d3e3be3f5d9e5bcf2feb488dd 100755 (executable)
--- a/ext/interbase/php_ibase_includes.h
+++ b/ext/interbase/php_ibase_includes.h
@@ -64,7 +64,6 @@ ZEND_BEGIN_MODULE_GLOBALS(ibase)
        long allow_persistent;
        char *default_db, *default_user, *default_password, *default_charset;
        char *timestampformat, *dateformat, *timeformat;
-       char *cfg_timestampformat, *cfg_dateformat, *cfg_timeformat;
        char errmsg[MAX_ERRMSG];
        long sql_code;
 ZEND_END_MODULE_GLOBALS(ibase)
@@ -115,10 +114,6 @@ enum php_interbase_option {
        PHP_IBASE_FETCH_BLOBS           = 1,
        PHP_IBASE_FETCH_ARRAYS      = 2,
        PHP_IBASE_UNIXTIME                      = 4,
-       /* timefmt flags */
-       PHP_IBASE_TIMESTAMP             = 1,
-       PHP_IBASE_DATE                          = 2,
-       PHP_IBASE_TIME                          = 4,
        /* transaction access mode */
        PHP_IBASE_WRITE                         = 1,
        PHP_IBASE_READ                          = 2,

diff --git a/ext/interbase/tests/003.phpt b/ext/interbase/tests/003.phpt
index fe0074e518396fe84ee2e5f8153466530b9980ef..0263010b0236fe24135f414d496d46aa5a4bbeb2 100644 (file)
--- a/ext/interbase/tests/003.phpt
+++ b/ext/interbase/tests/003.phpt
@@ -31,7 +31,7 @@ InterBase: misc sql types (may take a while)
        @ibase_query("insert into test3 (iter) values (?)", null);
 
     /* if timefmt is not supported, suppress error here */
-    @ibase_timefmt("%m/%d/%Y %H:%M:%S");
+    ini_set('ibase.timestampformat',"%m/%d/%Y %H:%M:%S");
 
     for($iter = 0; $iter < 10; $iter++){
        /* prepare data  */

diff --git a/ext/interbase/tests/006.phpt b/ext/interbase/tests/006.phpt
index 82be48084a596a52b50158dc44145a2b1c9b8406..8155c9f41e853ea1a67d83b7067d0341e5370f95 100644 (file)
--- a/ext/interbase/tests/006.phpt
+++ b/ext/interbase/tests/006.phpt
@@ -32,7 +32,7 @@ InterBase: binding (may take a while)
        ibase_commit();
 
        /* if timefmt not supported, hide error */
-       @ibase_timefmt("%m/%d/%Y %H:%M:%S");
+       ini_set('ibase.timestampformat',"%m/%d/%Y %H:%M:%S");
 
        echo "insert\n";
 
@@ -54,7 +54,7 @@ InterBase: binding (may take a while)
                        values (?,?,?,?,?,?,?,?,?,?)",
                        $iter, $v_char, $v_date, $v_decimal, $v_double, $v_float,
                        $v_integer, $v_numeric, $v_smallint, $v_varchar);
-               $sel = ibase_query("select * from test6 where iter = $iter");
+               $sel = ibase_query("select * from test6 where iter = ?", $iter);
 
                $row = ibase_fetch_object($sel);
                if(substr($row->V_CHAR,0,strlen($v_char)) != $v_char) {

diff --git a/ext/interbase/tests/007.phpt b/ext/interbase/tests/007.phpt
index bc654f14627d13bcd09c6db1065988dd08b096c0..069b7eda26cec1944524509eee0c11973a3dde4f 100644 (file)
--- a/ext/interbase/tests/007.phpt
+++ b/ext/interbase/tests/007.phpt
@@ -26,7 +26,7 @@ InterBase: array handling
        ibase_commit();
 
        /* if timefmt not supported, hide error */
-       @ibase_timefmt("%m/%d/%Y %H:%M:%S");
+       ini_set('ibase.timestampformat',"%m/%d/%Y %H:%M:%S");
 
        echo "insert\n";