CMS MADE SIMPLE FORGE

CMS Users

 
Back Back to List

[#5522] Initial test

avatar
Created By: Deleted User (deleteduser_11200)
Date Submitted: Thu Oct 14 11:41:39 -0400 2010

Assigned To: Jean-Christophe Cuvelier (totophe)
Version: CMS Users 0.0.2
CMSMS Version: None
Severity: Trivial
Resolution: Accepted
State: Open
Summary:
Initial test
Detailed Description:
Dear Totophe,

I did a small test on {CMSUsers} your project looks promissing.  When there is a
new release I am eager to test again.

Some problems I spotted in /admin

- When you enter a group before there is any user available the /admin page does
not render ok. Only a white page is shown the module can not be used anymore.
During a second installation I first added a new user. After I added a new group
and everything went fine.

- Adding templates works fine although it's a slow process.Iif you need a lot of
'default' templates it takes a lot of clicks.

- Every template contains data although the default template for 'profile-form'
is empty

- When you do an uninstall the database is not removed 

On the frontend:

- Login / Logout works fine
- Lost password works but the confirmation mail does not contain a proper token
- Sign Up works but the confirm signup also does not have a confirmation token

I was not able to test profile-edit / password-change and many more functions.
When there is documentation it will probally be clear to me how to call specific
actions in the module tag. Like {CMSUsers action='....'}

Good luck with your further developments.

- Arnoud


History

Comments
avatar 
Date: 2010-10-15 03:14
Posted By: Jean-Christophe Cuvelier (totophe)

Dear Arnoud,

Thanks for your feedback ! Much appreciated ! :-)

So, most of your points has been fixed and will be included in the next release
(soon).

- Templates: I have to improve the default templates, but in theory, you do not
need to customize them a lot. So, it's true, it requires a lot of click when you
build one, but for the moment, I think it's ok :)

- Lost password/Sign up confirmation: On my side, I did have a proper token.
Strange...

Again, thanks for your feedback :)

- Totophe
avatar 
Date: 2010-10-15 05:04
Posted By: Deleted User (deleteduser_11200)

Totophe,

Looking forward to your next release >:)

* About the missing token:

In CMSUsers.module.php case 'password_reset' I printed some debugging info to
stdout.

On my side there seems to be a problem with the "CreatLink" method. Resulting in
a url without a token.

$url =
$this->CreateLink($id,'password_reset',$returnid,'',array(),'',true,false,'',false,'user/password_reset/'
. $user->getId() . '/' . $user->getToken() . '/' . $returnid);

===

print "<pre>\n";
print "id: $id \n";
print "returnid : $returnid" . "\n"; 
print "getId : ". $user->getId() . "\n";
print "getToken : ". $user->getToken(). "\n";
print "url: $url \n";
print "</pre>\n";

===

/* stdout */

id: cntnt01 
returnid : 15
getId : 4
getToken : 259ed15bbee3029f312de0ac37795843
url:
http://myhost.lan/index.php?mact=CMSUsers,cntnt01,password_reset,0&cntnt01returnid=15

===

- Arnoud
avatar 
Date: 2010-10-15 05:09
Posted By: Deleted User (deleteduser_11200)

By turning on mod_rewrite the url is generated ok:

url: http://myhost.lan/user/password_reset/4/259ed15bbee3029f312de0ac37795843/15
avatar 
Date: 2010-10-15 05:12
Posted By: Jean-Christophe Cuvelier (totophe)

Ok, great ! :-) It is fixed now :-) I forgot to make it work for non modrewrite
users :-)

Thanks !
Updates

Updated: 2010-10-15 03:14
resolution_id: => 6
severity_id: 12 => 4