phpgroupware-tracker
[Top][All Lists]
Advanced

[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/





reply via email to

[Prev in Thread] Current Thread [Next in Thread]