The long awaited SMF Module!

I think they both should be case sensitive.

Hi,

Remon,

I think I’ve solved the problem. It was caused from a bug in the smf api. The api is still in beta but I’ve managed to fix a few things here and there which would make it a little stable for the purpose of integrating SMF with MODx.

Please see the attached

Cool Raymond, we can share the API fixes back with them, and generate some buzz over there while we’re at it!

Just so you know...the files aren’t very compatible (I presume they are created on a Windows machine), and I have to convert them all to use Unix line endings or they either come out double-spaced, which really doesn’t hurt anything, or with no line endings at all, which breaks the scripts horribly when there are single-line comments!

Other than that, everything seems to be working beautifully now.

Oh, one thing, I noticed that the newest download archive doesn’t have that patched version of weblogin.processor.inc.php. Does it not need that any more?

Quote from: xwisdom at Jan 31, 2006, 08:28 PM

On the issue of user names... Should user names be case-sensitive? I think only passwords should be case-sensitive. This is so because a user might signup as "Mary" but later enters "mary" whe logging in. Or another might sign up as "ADmin"

What do you think?

Well, as far as I know, user names are always treaded case sensitive on all OS’es. So you can’t make a user with (login) name Remon and login with remon....
So, to avoid confusion, I would suggest to use case sensitive comparisons when syncing users between smf and modx.

After I uploaded your fix, login/out works like a charm now, very impressive!

Remon

Thanks Remon. I’ll consider the case-sensitive user name issue.

Anyone else feels the same as Ryan and Remon?

PS. I’ll give this module another day then flag it as stable.

I say make it case sensitive and it’s pretty close to stable (after a brief beating upon with that change).

definitely should be case sensitive.

Currently MODx is case-insensitive while SMF is case-sensitive. I’ll change the Sync to support case-sensitive lookups