[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Phpgroupware-tracker] [bug #3723] deleting user w/ all records fails b
From: |
nobody |
Subject: |
[Phpgroupware-tracker] [bug #3723] deleting user w/ all records fails because of error in calendar's hook |
Date: |
Thu, 03 Jul 2003 13:54:18 -0400 |
User-agent: |
Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.3) Gecko/20030313 |
=================== BUG #3723: LATEST MODIFICATIONS ==================
http://savannah.gnu.org/bugs/?func=detailbug&bug_id=3723&group_id=509
Changes by: Ralf Becker <address@hidden>
Date: Thu 07/03/2003 at 19:54 (Europe/Berlin)
What | Removed | Added
---------------------------------------------------------------------------
Resolution | None | Invalid
Status | Open | Closed
------------------ Additional Follow-up Comments ----------------------------
Why do you think the param has to be an array. If you look at the code, you see
it gets directly passed as parameter to the function.
function ExecMethod($method, $functionparams = '_UNDEF_', $loglevel = 3,
$classparams = '_UNDEF_')
{
//....//
if ((is_array($functionparams) || $functionparams != '_UNDEF_') &&
($functionparams || $functionparams != 'True'))
{
return $GLOBALS[$classname]->$functionname($functionparams);
}
=================== BUG #3723: FULL BUG SNAPSHOT ===================
Submitted by: ccwf Project: phpGroupWare
Submitted on: Wed 05/21/2003 at 23:48
Category: calendar Bug Group: devel cvs
Severity: 5 - Major Priority: High
Resolution: Invalid Assigned to: ralfbecker
Status: Closed Component Version: DEB
Platform Version: Linux - Debian Reproducibility: Every Time
Summary: deleting user w/ all records fails because of error in calendar's hook
Original Submission: Line 17 of hook_deleteaccount.inc.php calls ExecMethod
with an integer second argument instead of an array. Putting array(...) around
the argument appears to fix the problem.
Follow-up Comments
*******************
-------------------------------------------------------
Date: Thu 07/03/2003 at 19:54 By: ralfbecker
Why do you think the param has to be an array. If you look at the code, you see
it gets directly passed as parameter to the function.
function ExecMethod($method, $functionparams = '_UNDEF_', $loglevel = 3,
$classparams = '_UNDEF_')
{
//....//
if ((is_array($functionparams) || $functionparams != '_UNDEF_') &&
($functionparams || $functionparams != 'True'))
{
return $GLOBALS[$classname]->$functionname($functionparams);
}
-------------------------------------------------------
Date: Sat 05/31/2003 at 23:44 By: ccwf
No, the argument to bocalendar::delete_calendar() indeed should not be an
array. However, delete_calendar() is being invoked via ExecMethod, and it is
ExecMethod() which is being called with an integer second argument when the
second argument to ExecMethod() must be an *array* consisting of the arguments
to bocalendar::delete_calendar(). Since bocalendar::delete_calendar takes a
single integer argument, ExecMethod must be passed an array containing a single
integer.
-------------------------------------------------------
Date: Sat 05/31/2003 at 10:45 By: ralfbecker
I dont understand why you mean the param to bocalendar::delete_calendar()
should be an array, it calls socalendar_sql::delete_calendar:
function delete_calendar($calendar='')
{
$this->stream->query('SELECT cal_id FROM phpgw_cal WHERE
owner='.intval($calendar),__LINE__,__FILE__);
As you can see in the code-piece it is an integer. If you make it an array it
will be casted back to int and end up as 0, which means nothing gets delete.
Your problem must be somewhere else ...
-------------------------------------------------------
Date: Thu 05/29/2003 at 03:25 By: ccwf
Further investigation revealed that attempting to delete LDAP users with all
records not only fails because of the calendar hook error, but leaves accounts
partially deleted and in a wedged state with inconsistentency between LDAP and
(in our case) PostgreSQL. After manually deleting remnants of the wedged,
partially deleted accounts and making the array(...) change above, phpGW
account deletion through the UI appears to be working well.
-------------------------------------------------------
Date: Wed 05/21/2003 at 23:55 By: ccwf
*sigh*
The change above successfully gets rid of my error messages when performing
account deletion; but deletion of LDAP user accounts still isn't working,
probably for reasons unrelated to the calendar.
CC list is empty
No files currently attached
For detailed info, follow this link:
http://savannah.gnu.org/bugs/?func=detailbug&bug_id=3723&group_id=509
_______________________________________________
Message sent via/by Savannah
http://savannah.gnu.org/
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- [Phpgroupware-tracker] [bug #3723] deleting user w/ all records fails because of error in calendar's hook,
nobody <=