Drupal 7.x module now available

[Steve Moitozo II]

After a bit of housekeeping I have committed the Drupal 7 compatible
code and created a release for download. I have also updated the Drupal
6.x download and branched the SVN tree to prepare for maintenance of the
D6 module.

Please test and send feedback.

-S2
--
Organizational structure, by itself, is no solution to anything.
--Eugene Wiemers

Steve Moitozo II
Engineer
SIL Global Technology Services
office: +1 (704) 256-0407
skype: smoitozo

[Halvor Utby]

Hi,

I have now testet the module with d7 and it works great. Some of the hacks I put in the d6 version has been solved. Thanks.

However, there is a few things I miss or that isn't working properly:
- An easy way of changing the link "Federated Log In" to something else. In production we will only allow Simplesaml-login, hence the link-name can for us be "Login".
- The "Allow authentication with local Drupal accounts"-option does not work for our drupal-root-user. Disabling simplesamlphp_auth through drush is the only way I can get in as the "drupal-root-user" after the module has been enabled.
- "Error message Notice: Undefined index: mail in user_save() (line 567 of /path/to/drupal-7.0/modules/user/user.module)." first time a user logs in (this might be a d7 problem and not simplesamlphp).
- I.e. the smartname-fullname option we had in d6 is no longer mentioned. (Yes, I know it still works, but I think it is beneficial to inform other users about this option, I may be wrong)
- The option to send users that log out to a spesific page is missing. Sending users to "/" is not always optimal, in our multi-site environment. This might also be a d7 problem.
- "Instalation directory" has a typo. ;-)

That said, I have to add that you have done a great job on this. I am especially happy with the added block feature. Thank you.

Some of the problems we experience might be to the fact that we have upgraded for d6 to d7 for this test-blog.

--
You received this message because you are subscribed to the Google Groups "Drupal SimpleSAMLphp integration" group.
To post to this group, send email to drupalsi...@googlegroups.com.
To unsubscribe from this group, send email to drupalsimplesa...@googlegroups.com.
For more options, visit this group at http://groups.google.com/group/drupalsimplesaml?hl=en.

--
Halvor Utby

[Steve Moitozo II]

On Feb 24, 2011, at 3:35 AM, Halvor Utby wrote:

> Hi,
>
> I have now testet the module with d7 and it works great. Some of the
> hacks I put in the d6 version has been solved. Thanks.
>
> However, there is a few things I miss or that isn't working properly:
> - An easy way of changing the link "Federated Log In" to something
> else. In production we will only allow Simplesaml-login, hence the
> link-name can for us be "Login".

I understand exactly what you mean. On one of my sites I put together
a simple custom block that links people to saml_login. That way I
could control all the verbiage.

> - The "Allow authentication with local Drupal accounts"-option does
> not work for our drupal-root-user. Disabling simplesamlphp_auth
> through drush is the only way I can get in as the "drupal-root-user"
> after the module has been enabled.

What version of Drupal are you running (6.x or 7.x)? Hmm. It works for
me. Did you put something in the "Which users should be allowed to
login with local accounts?" If you specified any IDs but left out the
drupal-root-user then that'll keep it from logging in. Otherwise it
should work.

> - "Error message Notice: Undefined index: mail in user_save() (line
> 567 of /path/to/drupal-7.0/modules/user/user.module)." first time a
> user logs in (this might be a d7 problem and not simplesamlphp).

This is a bug in Drupal. Changing your drupal error reporting setting
to "Errors and warnings" will suppress this.

> - I.e. the smartname-fullname option we had in d6 is no longer
> mentioned. (Yes, I know it still works, but I think it is beneficial
> to inform other users about this option, I may be wrong)

Good point. We need to expand the context help on this module. This
should be included.

> - The option to send users that log out to a spesific page is
> missing. Sending users to "/" is not always optimal, in our multi-
> site environment. This might also be a d7 problem.

This was removed at some point but I can't remember why. Send me a
patch if you're so inclined.

> - "Instalation directory" has a typo. ;-)

Thanks for this. I'll get this fixed.

>
> That said, I have to add that you have done a great job on this. I
> am especially happy with the added block feature. Thank you.
>
> Some of the problems we experience might be to the fact that we have
> upgraded for d6 to d7 for this test-blog.
>

Thank you for your feedback. It's going to make it better!

[Halvor Utby]

On 25 February 2011 14:55, Steve Moitozo II <steve_...@sil.org> wrote:

On Feb 24, 2011, at 3:35 AM, Halvor Utby wrote:

Hi,

I have now testet the module with d7 and it works great. Some of the hacks I put in the d6 version has been solved. Thanks.

However, there is a few things I miss or that isn't working properly:
- An easy way of changing the link "Federated Log In" to something else. In production we will only allow Simplesaml-login, hence the link-name can for us be "Login".

I understand exactly what you mean. On one of my sites I put together a simple custom block that links people to saml_login. That way I could control all the verbiage.

- The "Allow authentication with local Drupal accounts"-option does not work for our drupal-root-user. Disabling simplesamlphp_auth through drush is the only way I can get in as the "drupal-root-user" after the module has been enabled.

What version of Drupal are you running (6.x or 7.x)? Hmm. It works for me. Did you put something in the "Which users should be allowed to login with local accounts?" If you specified any IDs but left out the drupal-root-user then that'll keep it from logging in. Otherwise it should work.

With d7 I ment Drupal-7.0

- "Error message Notice: Undefined index: mail in user_save() (line 567 of /path/to/drupal-7.0/modules/user/user.module)." first time a user logs in (this might be a d7 problem and not simplesamlphp).

This is a bug in Drupal. Changing your drupal error reporting setting to "Errors and warnings" will suppress this.

Yes, I suspected such. Thank you.

 

- I.e. the smartname-fullname option we had in d6 is no longer mentioned. (Yes, I know it still works, but I think it is beneficial to inform other users about this option, I may be wrong)

Good point. We need to expand the context help on this module. This should be included.

- The option to send users that log out to a spesific page is missing. Sending users to "/" is not always optimal, in our multi-site environment. This might also be a d7 problem.

This was removed at some point but I can't remember why. Send me a patch if you're so inclined.

- "Instalation directory" has a typo. ;-)

Thanks for this. I'll get this fixed.

That said, I have to add that you have done a great job on this. I am especially happy with the added block feature. Thank you.

Some of the problems we experience might be to the fact that we have upgraded for d6 to d7 for this test-blog.

Thank you for your feedback. It's going to make it better!

 

My pleasure. If I get the time to produce patches I'll let you know.

 

--
You received this message because you are subscribed to the Google Groups "Drupal SimpleSAMLphp integration" group.
To post to this group, send email to drupalsi...@googlegroups.com.
To unsubscribe from this group, send email to drupalsimplesa...@googlegroups.com.
For more options, visit this group at http://groups.google.com/group/drupalsimplesaml?hl=en.

--
Halvor Utby