Simple Machines Community Forum

Archived Boards and Threads... => Archived Boards => Mambo Bridge Support => Topic started by: dcabbar on August 15, 2005, 12:36:28 AM

Title: com_smf_registration & aro+acl data
Post by: dcabbar on August 15, 2005, 12:36:28 AM
Hi,

I was looking at the com_smf_registration code, and saw couple of lines in saveRegistration method for creating ACL info in mos_aro* tables.

The thing I don't understand here is, mosUser->store method already takes care of this, why do you need to replicate the whole thing in a manual, hence less consistent way.

My dilemma here is, I want every new user registration to be defaulted to Author instead of Registered. Hence, when I change com_smf_registration line:

   $row->gid = $acl->get_group_id('Registered','ARO');

to

   $row->gid = $acl->get_group_id('Author','ARO');


the whole thing gets messed up because, mambo's mosUser->store creates a row in the acl tables as Author, and com_smf_registration creates yet another as Registered (because type is hardcoded to 18 there).

And the whole thing messes up.

Anyways, any idea/reason why this was replicated there?

Thanks...
Title: Re: com_smf_registration & aro+acl data
Post by: tentronik on August 15, 2005, 01:09:08 AM
Just a guess but cant you use the member registration settings and put a primary membergroup called Authors?
Title: Re: com_smf_registration & aro+acl data
Post by: Orstio on August 15, 2005, 07:02:01 AM
QuoteThe thing I don't understand here is, mosUser->store method already takes care of this, why do you need to replicate the whole thing in a manual, hence less consistent way.

I was finding that mosUser->store was not always working as it should.  It was sometimes missing entering new users into the ACL.  Then there would be multiple database errors when the user tried to login.  So, I put in the klunky code, just to make sure that it works every time.

Quotethe whole thing gets messed up because, mambo's mosUser->store creates a row in the acl tables as Author, and com_smf_registration creates yet another as Registered (because type is hardcoded to 18 there).

The value for author is 21, yes?  Just change it to that.