[Phpgroupware-tracker] [bugs #9786] (need confirmation) reas_repository/save_repository seems not working in LDAP accounts

[Philipp Kamps]

This mail is an automated notification from the bugs tracker
 of the project: phpGroupWare.

/**************************************************************************/
[bugs #9786] Latest Modifications:

Changes by: 
                Philipp Kamps <address@hidden>
'Date: 
                Thu 07/29/04 at 07:38 (GMT)

            What     | Removed                   | Added
---------------------------------------------------------------------------
              Status | Open                      | Closed


------------------ Additional Follow-up Comments ----------------------------
i just hate these additional workarounds - but we don't have another chance. 
Your code is in CVS, thx






/**************************************************************************/
[bugs #9786] Full Item Snapshot:

URL: <http://savannah.gnu.org/bugs/?func=detailitem&item_id=9786>
Project: phpGroupWare
Submitted by: Caeies
On: Tue 07/27/04 at 14:58

Category:  API - Admin
Item Group:  0.9.16.000 release
Severity:  3 - Ordinary
Priority:  7 - High
Resolution:  Fixed
Assigned to:  fipsfuchs
Status:  Closed
Component Version:  CVS
Platform Version:  GNU/Linux - Debian
Reproducibility:  Every Time
Planned Release:  0.9.16.001+
Fixed Release:  


Summary:  (need confirmation) reas_repository/save_repository seems not working 
in LDAP accounts

Original Submission:  Hi,

I'm trying to do the following :

$account = CreateObject('phpgwapi.accounts',$account_id);
$account->read_repository();
$account->save_repository();

and I get the following error:
ldap_modified FAILED: [21] Invalid syntax

Modifies: uid=caeiesATcaeies.org,ou=users,dc=proget,dc=test,dc=fr

Array
(
    [objectclass] => Array
        (
            [0] => phpgwAccount
            [1] => phpgwContact
            [2] => posixAccount
        )

    [phpgwaccountid] => 21304
    [phpgwaccountexpires] => 
    [phpgwaccountstatus] => I
    [phpgwquota] => -1
    [phpgwcontactid] => 160
    [cn] => Caeies
    [uidnumber] => 21304
    [uid] => caeiesATcaeies.org
    [description] => compte créé dans phpgw
    [givenname] => Caeies
    [sn] => Caeies
    [homedirectory] => /dev/null
    [gidnumber] => 21304
)

1) Did you have the same problem,
2) I will try to understand what is the problem.

Thanks in avance,

Caeies

Follow-up Comments
------------------


-------------------------------------------------------
Date: Thu 07/29/04 at 07:38         By: fipsfuchs
i just hate these additional workarounds - but we don't have another chance. 
Your code is in CVS, thx

-------------------------------------------------------
Date: Wed 07/28/04 at 12:30         By: Caeies
Hi fips,

I reopen it since I found another problem with this read/save repository 
function,

that the correction :

account_expires is one problem and you solved it nicely :)

but there's another one : account_status.

So I modify the code to obtain :

$entry['phpgwaccountexpires'] = isset($account_info['account_expires'])  ? 
$account_info['account_expires'] : $account_info['expires'] ;
if ($account_info['account_status'] || $account_info['status'])
            {
                $entry['phpgwaccountstatus'] = 
isset($account_info['account_status']) ? $account_info['account_status'] : 
$account_info['status'];
            }

Near line 749 in account_ldap.

What do you think about that ?

(If the code is not set for the status, when read/save repository you will have 
an inactive account).

Cheers 
Caeies





-------------------------------------------------------
Date: Wed 07/28/04 at 08:25         By: fipsfuchs
it was a problem with the damn object var names in the account backend. 
$this->data['expire'] and $this->data['account_expire'] is used....

I changed this in head - so we won't get such problems.

fixed in CVS - thx Caeies












For detailed info, follow this link:
<http://savannah.gnu.org/bugs/?func=detailitem&item_id=9786>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.gnu.org/