extern/egroupware_official
1
0
Fork 0
mirror of https://github.com/EGroupware/egroupware.git synced 2024-10-06 18:12:22 +02:00
Code Issues Packages Projects Releases Wiki Activity

- re-write of the phpgwapi::accounts_sql::auto_add function

Browse Source 
- set a default account email address as well as a user preference email
address in the users' preferences
- actually check if the account creation succeeds before we commit the transaction to the database.
- set default name to New User if we have no other data
- remove some crap and make function readable
This commit is contained in:
mgalgoci 2004-08-30 15:15:59 +00:00
parent 7b6b6d5759
commit 99ffa6ebc1
1 changed files with 78 additions and 45 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

83
phpgwapi/inc/class.accounts_sql.inc.php
View File

@ -326,13 +326,9 @@
function auto_add($accountname, $passwd, $default_prefs = False, $default_acls = False, $expiredate = 0, $account_status = 'A') function auto_add($accountname, $passwd, $default_prefs = False, $default_acls = False, $expiredate = 0, $account_status = 'A')
{ {
if ($expiredate) if ($expiredate == 0)
{ {
$expires = mktime(2,0,0,date('n',$expiredate), (int)date('d',$expiredate), date('Y',$expiredate)); if(isset($GLOBALS['phpgw_info']['server']['auto_create_expire']) == True)
}
else
{
if($GLOBALS['phpgw_info']['server']['auto_create_expire'])
{ {
if($GLOBALS['phpgw_info']['server']['auto_create_expire'] == 'never') if($GLOBALS['phpgw_info']['server']['auto_create_expire'] == 'never')
{ {
@ -341,16 +337,17 @@
else else
{ {
$expiredate = time() + $GLOBALS['phpgw_info']['server']['auto_create_expire']; $expiredate = time() + $GLOBALS['phpgw_info']['server']['auto_create_expire'];
$expires = mktime(2,0,0,date('n',$expiredate), (int)date('d',$expiredate), date('Y',$expiredate)); }
} }
} }
else else
{ {
/* expire in 30 days by default */ /* expire in 30 days by default */
$expiredate = time() + ((60 * 60) * (30 * 24)); $expiredate = time() + ((60 * 60) * (30 * 24));
}
$expires = mktime(2,0,0,date('n',$expiredate), (int)date('d',$expiredate), date('Y',$expiredate)); $expires = mktime(2,0,0,date('n',$expiredate), (int)date('d',$expiredate), date('Y',$expiredate));
}
}
$default_group_id = $this->name2id($GLOBALS['phpgw_info']['server']['default_group_lid']); $default_group_id = $this->name2id($GLOBALS['phpgw_info']['server']['default_group_lid']);
if (!$default_group_id) if (!$default_group_id)
{ {
@ -365,49 +362,85 @@
'account_lid' => $accountname, 'account_lid' => $accountname,
'account_type' => 'u', 'account_type' => 'u',
'account_passwd' => $passwd, 'account_passwd' => $passwd,
'account_firstname' => $GLOBALS['auto_create_acct']['firstname'], 'account_firstname' => $GLOBALS['auto_create_acct']['firstname'] ? $GLOBALS['auto_create_acct']['firstname'] : 'New',
'account_lastname' => $GLOBALS['auto_create_acct']['lastname'], 'account_lastname' => $GLOBALS['auto_create_acct']['lastname'] ? $GLOBALS['auto_create_acct']['lastname'] : 'User',
'account_status' => $account_status, 'account_status' => $account_status,
'account_expires' => $expires, 'account_expires' => $expires,
'account_primary_group' => $primary_group, 'account_primary_group' => $primary_group,
'account_email' => $GLOBALS['auto_create_acct']['email'],
); );
/* attempt to set an email address */
if (isset($GLOBALS['auto_create_acct']['email']) == True && $GLOBALS['auto_create_acct']['email'] != '')
{
$acct_info['account_email'] = $GLOBALS['auto_create_acct']['email'];
}
elseif(isset($GLOBALS['phpgw_info']['server']['mail_suffix']) == True && $GLOBALS['phpgw_info']['server']['mail_suffix'] != '')
{
$acct_info['account_email'] = $accountname . '@' . $GLOBALS['phpgw_info']['server']['mail_suffix'];
}
$this->db->transaction_begin(); $this->db->transaction_begin();
$this->create($acct_info);
$accountid = $this->name2id($accountname); $this->create($acct_info); /* create the account */
// if we have a primary_group, add it as "regular" eGW group (via ACL) too
if ($accountid && $primary_group) $accountid = $this->name2id($accountname); /* grab the account id or an error code */
if ($accountid) /* begin account setup */
{
/* If we have a primary_group, add it as "regular" eGW group (via ACL) too. */
if ($primary_group)
{ {
$this->db->query("insert into phpgw_acl (acl_appname, acl_location, acl_account, acl_rights) values('phpgw_group', " $this->db->query("insert into phpgw_acl (acl_appname, acl_location, acl_account, acl_rights) values('phpgw_group', "
. $primary_group . ', ' . $accountid . ', 1)',__LINE__,__FILE__); . $primary_group . ', ' . $accountid . ', 1)',__LINE__,__FILE__);
} }
// if we have an mail address set it as email pref
if ($accountid && @$GLOBALS['auto_create_acct']['email']) /* if we have an mail address set it in the uesrs' email preference */
if (isset($GLOBALS['auto_create_acct']['email']) && $GLOBALS['auto_create_acct']['email'] != '')
{ {
$GLOBALS['phpgw']->acl->acl($accountid); // needed als preferences::save_repository calls acl $GLOBALS['phpgw']->acl->acl($accountid); /* needed als preferences::save_repository calls acl */
$GLOBALS['phpgw']->preferences->preferences($accountid); $GLOBALS['phpgw']->preferences->preferences($accountid);
$GLOBALS['phpgw']->preferences->read_repository(); $GLOBALS['phpgw']->preferences->read_repository();
$GLOBALS['phpgw']->preferences->add('email','address',$GLOBALS['auto_create_acct']['email']); $GLOBALS['phpgw']->preferences->add('email','address',$GLOBALS['auto_create_acct']['email']);
$GLOBALS['phpgw']->preferences->save_repository(); $GLOBALS['phpgw']->preferences->save_repository();
} }
// commit the new account transaction /* use the default mail domain to set the uesrs' email preference */
elseif(isset($GLOBALS['phpgw_info']['server']['mail_suffix']) && $GLOBALS['phpgw_info']['server']['mail_suffix'] != '')
{
$GLOBALS['phpgw']->acl->acl($accountid); /* needed als preferences::save_repository calls acl */
$GLOBALS['phpgw']->preferences->preferences($accountid);
$GLOBALS['phpgw']->preferences->read_repository();
$GLOBALS['phpgw']->preferences->add('email','address', $accountname . '@' . $GLOBALS['phpgw_info']['server']['mail_suffix']);
$GLOBALS['phpgw']->preferences->save_repository();
}
/* commit the new account transaction */
$this->db->transaction_commit(); $this->db->transaction_commit();
/* does anyone know what the heck this is required for? */
$GLOBALS['hook_values']['account_lid'] = $acct_info['account_lid']; $GLOBALS['hook_values']['account_lid'] = $acct_info['account_lid'];
$GLOBALS['hook_values']['account_id'] = $accountid; $GLOBALS['hook_values']['account_id'] = $accountid;
$GLOBALS['hook_values']['new_passwd'] = $acct_info['account_passwd']; $GLOBALS['hook_values']['new_passwd'] = $acct_info['account_passwd'];
$GLOBALS['hook_values']['account_status'] = $acct_info['account_status']; $GLOBALS['hook_values']['account_status'] = $acct_info['account_status'];
$GLOBALS['hook_values']['account_firstname'] = $acct_info['account_lid']; $GLOBALS['hook_values']['account_firstname'] = $acct_info['account_firstname'];
$GLOBALS['hook_values']['account_lastname'] = 'eGW Account'; $GLOBALS['hook_values']['account_lastname'] = $acct_info['account_lastname'];
$GLOBALS['hook_values']['account_email'] = $acct_info['account_passwd'];
$GLOBALS['phpgw']->hooks->process($GLOBALS['hook_values']+array( $GLOBALS['phpgw']->hooks->process($GLOBALS['hook_values']+array(
'location' => 'addaccount' 'location' => 'addaccount'
),False,True); // called for every app now, not only enabled ones ),False,True); /* called for every app now, not only enabled ones */
return $accountid; } /* end account setup */
else /* if no account id abort the account creation */
{
$this->db->transaction_abort();
} }
/*
* If we succeeded in creating the account (above), return the accountid, else,
* return the error value from $this->name2id($accountname)
*/
return $accountid;
} /* end auto_add() */
function get_account_name($accountid,&$lid,&$fname,&$lname) function get_account_name($accountid,&$lid,&$fname,&$lname)
{ {
$this->db->select($this->table,'account_lid,account_firstname,account_lastname',array('account_id'=>$accountid),__LINE__,__FILE__); $this->db->select($this->table,'account_lid,account_firstname,account_lastname',array('account_id'=>$accountid),__LINE__,__FILE__);