ab Sun Mar 18 23:52:04 2001 EDT Modified files: (Branch: PHP_4_0_5) /php4/ext/midgard .cvsignore article.c attachment.c config.m4 config.m4.session midgard.c php_midgard.h snippet.c Log: Bringing fixed code to stable branch Index: php4/ext/midgard/.cvsignore diff -u php4/ext/midgard/.cvsignore:1.5 php4/ext/midgard/.cvsignore:1.5.2.1 --- php4/ext/midgard/.cvsignore:1.5 Tue Feb 20 03:00:23 2001 +++ php4/ext/midgard/.cvsignore Sun Mar 18 23:52:03 2001 @@ -3,7 +3,6 @@ *.lo *.slo *.la -CVS Makefile acinclude.m4 aclocal.m4 Index: php4/ext/midgard/article.c diff -u php4/ext/midgard/article.c:1.15 php4/ext/midgard/article.c:1.15.2.1 --- php4/ext/midgard/article.c:1.15 Mon Mar 12 02:19:37 2001 +++ php4/ext/midgard/article.c Sun Mar 18 23:52:03 2001 @@ -1,4 +1,4 @@ -/* $Id: article.c,v 1.15 2001/03/12 10:19:37 davidg Exp $ +/* $Id: article.c,v 1.15.2.1 2001/03/19 07:52:03 ab Exp $ Copyright (C) 1999 Jukka Zitting <[EMAIL PROTECTED]> Copyright (C) 2000 The Midgard Project ry Copyright (C) 2000 Emile Heyns, Aurora SA <[EMAIL PROTECTED]> @@ -780,6 +780,7 @@ zval **url, **calstart, **caldays, **icon, **view, **print; zval **extra1, **extra2, **extra3; zval **type, **score, **up, *self; + long upval; RETVAL_FALSE; CHECK_MGD; @@ -852,8 +853,11 @@ if (!mgd_exists_id(mgd_handle(), "topic", "id=$d", (*topic)->value.lval)) RETURN_FALSE_BECAUSE(MGD_ERR_NOT_EXISTS); - if ((*up)->value.lval != 0 && !mgd_exists_id(mgd_handle(), "article", "id=$d", (*up)->value.lval)) - RETURN_FALSE_BECAUSE(MGD_ERR_NOT_EXISTS); + if (up) { + upval = atol((*up)->value.str.val); + if (upval != 0 && !mgd_exists_id(mgd_handle(), "article", "id=$d", upval)) + RETURN_FALSE_BECAUSE(MGD_ERR_NOT_EXISTS); + } if (!istopicowner((*topic)->value.lval)) { RETURN_FALSE_BECAUSE(MGD_ERR_ACCESS_DENIED); Index: php4/ext/midgard/attachment.c diff -u php4/ext/midgard/attachment.c:1.10 php4/ext/midgard/attachment.c:1.10.2.1 --- php4/ext/midgard/attachment.c:1.10 Sun Mar 11 15:30:36 2001 +++ php4/ext/midgard/attachment.c Sun Mar 18 23:52:03 2001 @@ -1,4 +1,4 @@ -/* $Id: attachment.c,v 1.10 2001/03/11 23:30:36 davidg Exp $ +/* $Id: attachment.c,v 1.10.2.1 2001/03/19 07:52:03 ab Exp $ Copyright (C) 1999 Jukka Zitting <[EMAIL PROTECTED]> Copyright (C) 2000 The Midgard Project ry Copyright (C) 2000 Emile Heyns, Aurora SA <[EMAIL PROTECTED]> @@ -434,6 +434,7 @@ if (sapi_send_headers() != SUCCESS) { mgd_free_pool(pool); mgd_release(res); + fclose(fp); RETURN_FALSE_BECAUSE(MGD_ERR_INTERNAL); } @@ -442,6 +443,7 @@ PHPWRITE(buf, b); } + fclose(fp); mgd_free_pool(pool); mgd_release(res); RETVAL_TRUE; Index: php4/ext/midgard/config.m4 diff -u php4/ext/midgard/config.m4:1.13 php4/ext/midgard/config.m4:1.13.2.1 --- php4/ext/midgard/config.m4:1.13 Sun Mar 11 15:45:57 2001 +++ php4/ext/midgard/config.m4 Sun Mar 18 23:52:03 2001 @@ -1,4 +1,4 @@ -dnl $Id: config.m4,v 1.13 2001/03/11 23:45:57 davidg Exp $ +dnl $Id: config.m4,v 1.13.2.1 2001/03/19 07:52:03 ab Exp $ dnl Copyright (C) 1999 Jukka Zitting <[EMAIL PROTECTED]> dnl Copyright (C) 2000 The Midgard Project ry @@ -66,4 +66,12 @@ if test "$PHP_MIDGARD_EXPERIMENTAL" != "no"; then AC_DEFINE(YOU_WANT_TO_TEST, 1, [ ]) fi + + if test "$php_always_shared" = "yes"; then + MIDCONFIG_FILE="config.h" + else + MIDCONFIG_FILE="php_config.h" + fi + PHP_SUBST_OLD(MIDCONFIG_FILE) + AC_OUTPUT(php_midgard.h, [], []) fi Index: php4/ext/midgard/config.m4.session diff -u php4/ext/midgard/config.m4.session:1.3 php4/ext/midgard/config.m4.session:1.3.2.1 --- php4/ext/midgard/config.m4.session:1.3 Sun Mar 11 15:45:57 2001 +++ php4/ext/midgard/config.m4.session Sun Mar 18 23:52:03 2001 @@ -1,4 +1,4 @@ -dnl $Id: config.m4.session,v 1.3 2001/03/11 23:45:57 davidg Exp $ +dnl $Id: config.m4.session,v 1.3.2.1 2001/03/19 07:52:03 ab Exp $ dnl Copyright (C) 1999 Jukka Zitting <[EMAIL PROTECTED]> dnl Copyright (C) 2000 The Midgard Project ry @@ -74,4 +74,11 @@ if test "$PHP_MIDGARD_SESSION" != "no"; then AC_DEFINE(HAVE_MIDGARD_SESSION, 1, [ ]) fi + if test "$php_always_shared" = "yes"; then + MIDCONFIG_FILE="config.h" + else + MIDCONFIG_FILE="php_config.h" + fi + PHP_SUBST_OLD(MIDCONFIG_FILE) + AC_OUTPUT(php_midgard.h, [], []) fi Index: php4/ext/midgard/midgard.c diff -u php4/ext/midgard/midgard.c:1.22 php4/ext/midgard/midgard.c:1.22.2.1 --- php4/ext/midgard/midgard.c:1.22 Sat Mar 10 14:43:10 2001 +++ php4/ext/midgard/midgard.c Sun Mar 18 23:52:03 2001 @@ -1,4 +1,4 @@ -/* $Id: midgard.c,v 1.22 2001/03/10 22:43:10 emile Exp $ +/* $Id: midgard.c,v 1.22.2.1 2001/03/19 07:52:03 ab Exp $ Copyright (C) 1999 Jukka Zitting <[EMAIL PROTECTED]> Copyright (C) 2000 The Midgard Project ry Copyright (C) 2000 Emile Heyns, Aurora SA <[EMAIL PROTECTED]> @@ -390,9 +390,10 @@ MGDG(rcfg) = (midgard_request_config *) ap_get_module_config(r->request_config, midgard_module); if(MGDG(rcfg) == NULL) { + /* Midgard is probably not enabled for this host, only log debug info */ MGD_LOG_START("Cannot get midgard module config") MGD_LOG_END() - php_error(E_NOTICE, "Cannot get midgard module config"); + MGDG(rcfg) = NULL; MGDG(dcfg) = NULL; return SUCCESS; @@ -403,6 +404,10 @@ if(MGDG(dcfg) == NULL) { MGD_LOG_START("Cannot get midgard module directory config") MGD_LOG_END() + + /* this is an error, since if we can get the module config we should + also be able to get this + */ php_error(E_NOTICE, "Cannot get midgard module directory config"); MGDG(rcfg) = NULL; MGDG(dcfg) = NULL; Index: php4/ext/midgard/php_midgard.h diff -u php4/ext/midgard/php_midgard.h:1.8 php4/ext/midgard/php_midgard.h:1.8.2.1 --- php4/ext/midgard/php_midgard.h:1.8 Sat Mar 10 14:43:10 2001 +++ php4/ext/midgard/php_midgard.h Sun Mar 18 23:52:03 2001 @@ -1,4 +1,4 @@ -/* $Id: php_midgard.h,v 1.8 2001/03/10 22:43:10 emile Exp $ +/* $Id: php_midgard.h,v 1.8.2.1 2001/03/19 07:52:03 ab Exp $ Copyright (C) 1999 Jukka Zitting <[EMAIL PROTECTED]> Copyright (C) 2000 The Midgard Project ry Copyright (C) 2000 Emile Heyns, Aurora SA <[EMAIL PROTECTED]> @@ -24,11 +24,11 @@ /* You should tweak config.m4 so this symbol (or some else suitable) gets defined. */ -#include <php_config.h> +#include "php.h" +#include "config.h" #if HAVE_MIDGARD -#include <php.h> #include <midgard/midgard.h> #include <midgard/apache.h> /* #include <Zend/zend_modules.h> */ Index: php4/ext/midgard/snippet.c diff -u php4/ext/midgard/snippet.c:1.7 php4/ext/midgard/snippet.c:1.7.2.1 --- php4/ext/midgard/snippet.c:1.7 Sat Mar 10 14:43:10 2001 +++ php4/ext/midgard/snippet.c Sun Mar 18 23:52:03 2001 @@ -1,4 +1,4 @@ -/* $Id: snippet.c,v 1.7 2001/03/10 22:43:10 emile Exp $ +/* $Id: snippet.c,v 1.7.2.1 2001/03/19 07:52:03 ab Exp $ Copyright (C) 1999 Jukka Zitting <[EMAIL PROTECTED]> Copyright (C) 2000 The Midgard Project ry Copyright (C) 2000 Emile Heyns, Aurora SA <[EMAIL PROTECTED]> @@ -124,9 +124,6 @@ convert_to_string_ex(code); convert_to_string_ex(doc); convert_to_string_ex(author); - - if (!mgd_exists_id(mgd_handle(), "person", "id=$d", (*author)->value.lval)) - RETURN_FALSE_BECAUSE(MGD_ERR_NOT_EXISTS); if (!mgd_exists_id(mgd_handle(), "snippetdir", "id=$d", (*snippetdir)->value.lval)) RETURN_FALSE_BECAUSE(MGD_ERR_NOT_EXISTS); -- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] To contact the list administrators, e-mail: [EMAIL PROTECTED]