I think a lot of site will need a version of this module for Drupal 6

Do you plan to create such version?

If not - i can help you to do this, as i need it

CommentFileSizeAuthor
#13 loginza.tar_.gz6.82 KBIRuslan

Comments

vgarvardt’s picture

vgarvardt commented

I'm not planning to use Loginza on my current projects, only for further projects I'm working on, so I'm not planning to backport to D6.

Current D6 version that can be found on drupal.ru works in the different way - it does not create real Drupal users, and mine D7 version does. Actually I copied auth code from core OpenID module with minor modifications, so it works almost in the same way.

You can backport my D7 to D6 and I'll add you as a co-maintainer so you could handle D6 version support.

Eugene Fidelin’s picture

Eugene Fidelin commented

Ok, Vladimir, please add me as co-maintainer

mattcasey’s picture

mattcasey commented

This module looks really great, subscribing

BenK’s picture

BenK commented

Subscribing

alexbk66-’s picture

alexbk66- commented

Subscribing

ZPRaven’s picture

ZPRaven commented

Subscribing

sashken2’s picture

sashken2 commented

Subscribing

RedRat’s picture

RedRat commented

Subscribing

Siegfrid’s picture

Siegfrid commented

2Eugef: Можно у вас уточнить, когда м.б. ожидать первой версии под 6ку? Я ведь правильно понял, что вы хотите сделать backporting с 7ки на 6ку?

Eugene Fidelin’s picture

Eugene Fidelin commented

Siegfrid, сейчас проект, которому был нужен этот модуль, временно отложил внедрение этой функции.
Поэтому в ближайшее время, к сожалению, портирование на 6-ку я делать не буду

IRuslan’s picture

IRuslan commented

Сделайте ко-мейнтейнером, я выложу порт под D6.

vgarvardt’s picture

vgarvardt commented
Assigned: Eugene Fidelin » Unassigned

Прикрепи модуль к комментарию. Я посмотрю, если все ок - добавлю.

IRuslan’s picture

IRuslan commented
StatusFileSize
new loginza.tar_.gz6.82 KB
vgarvardt’s picture

vgarvardt commented
Status: Active » Needs work

Добавил ко-мантейнером.

Несколько небольших замечаний, которые увидел сходу:
- не хватает drupal_uninstall_schema в hook_uninstall
- настройки для Д6 должны быть admin/user/loginza вместо admin/config/people/loginza

Поправь и заливай.

mattcasey’s picture

mattcasey commented
Status: Needs work » Active

I Just tried this on a test site. On the user account page, Loginza says I'm logged in with a Gmail account but it doesn't refresh the page, and it appears that I'm not actually logged in to Drupal because I can still see the login form, etc.

IRuslan’s picture

IRuslan commented

@vgarvardt Fixed and pushed to new branch.

IRuslan’s picture

IRuslan commented
Status: Active » Closed (fixed)

@mattwad
I think this issue should be closed.

If you have some problems with D6 version, create new issues for 6.x-2.x branch.