Re: Evaluation

On Sat, 2009-05-16 at 12:12 -0400, Adam Tauno Williams wrote:
> On Sat, 2009-05-16 at 10:46 -0400, Dirk H Bartley wrote:
> > > > Yes, as I said, the ZideStore is too old. It had quite a set of 
> > > I believe his version has the GroupDAVv2 fixes from r2207.
> > > The contents of the public/Contacts folder is initially derived from the
> > > query -
> > > SELECT c1.company_id AS pkey, c1.name AS sn, c1.firstname AS givenname,
> > > c1.object_version AS version
> > > FROM company c1
> > > WHERE c1.is_person=1
> > > AND (c1.is_account=0 OR c1.is_account IS NULL)
> > > AND (c1.is_private=0 OR c1.is_private IS NULL);
> > > [pg. 62 of the current WMOGAG, the "Folder Hierarchy" section covers how
> > > each folder in ZideStore is derived from the database.]
> > > I assume that 10121 is presented by this query?
> > OGo=# SELECT c1.company_id AS pkey, c1.name AS sn, c1.firstname AS
> > givenname,
> > OGo-# c1.object_version AS version
> > OGo-# FROM company c1
> > OGo-# WHERE c1.is_person=1
> > OGo-# AND (c1.is_account=0 OR c1.is_account IS NULL)
> > OGo-# AND (c1.is_private=0 OR c1.is_private IS NULL);
> >  pkey  |      sn      | givenname | version 
> > -------+--------------+-----------+---------
> >  10121 | Dirk Bartley | Dirk      |       2
> >  36560 | Berner       | Sean      |       1
> >  36573 | Bosnjak      | Jill      |       2
> >  36586 | Pennington   | Joe       |       2
> >  36643 | Carlson      | John      |       1
> > (5 rows)
> > The contact in question is me.
> 
> That is weird.  Do you have ACLs defined on your own object?  (I assume
> 10,121 is also is_account=1 [ your account object ]).  

If I do, I have not added any.  My account was added from active
directory.

> Check the
> object_acl table for object_id=10,121;

My understanding is that that would be a negative.

OGo=# SELECT c1.company_id AS pkey, c1.name AS sn, c1.firstname AS
givenname,
OGo-# c1.object_version AS version, is_account, account
OGo-# FROM company c1
OGo-# WHERE c1.is_person=1
OGo-# AND (c1.is_account=0 OR c1.is_account IS NULL)
OGo-# AND (c1.is_private=0 OR c1.is_private IS NULL);
 pkey  |      sn      | givenname | version | is_account | account 
-------+--------------+-----------+---------+------------+---------
 10121 | Dirk Bartley | Dirk      |       2 |          0 | 
 36560 | Berner       | Sean      |       1 |          0 | 
 36573 | Bosnjak      | Jill      |       2 |          0 | 
 36586 | Pennington   | Joe       |       2 |          0 | 
 36643 | Carlson      | John      |       1 |          0 | 
(5 rows)

I would have guessed that is_account would be true.

OGo=# select * from object_acl;
 object_acl_id | sort_key | action | object_id | auth_id | permissions 
---------------+----------+--------+-----------+---------+-------------
(0 rows)


>   there the auth_id is the
> objectId (pkey) of either an account (person where is_account = 1) or a
> team (table: team).
> 
> But I can duplicate your issue in both /Accounts and /public/Enterprises
> where the PROPFIND identifies resources that are hidden.  In /Accounts
> it is showing accounts that archived, and in /public/Enterprises it is
> showing accounts that are hidden by ACL.  The first (/Accounts) should
> be a trivial fix,  and I'll have to work on the second one.  I assume
> the same issue applies to both Enterprises/Contacts as internally they
> are both "Company" objects and handled by much the same code.
> 

-- 
OpenGroupware.org Users
[email protected]
http://mail.opengroupware.org/mailman/listinfo/users