A D6 version of this would be much appreciated!

CommentFileSizeAuthor
#1 1225822-browserid-backport-d6.patch9.6 KBpebosi

Comments

pebosi’s picture

pebosi commented
StatusFileSize
new 1225822-browserid-backport-d6.patch9.6 KB

Hi,

i created a patch for a backport to D6, please review.

Regards Peter

icecreamyou’s picture

icecreamyou commented
Status: Active » Needs work

Thanks for the patch, pebosi. Two corrections:

  • Function documentation should say Implementation of hook_HOOK(). instead of Implements hook_HOOK().
  • The #markup element doesn't exist in Drupal 6, it should be #value instead.

Additionally, for clarity, please don't mix in unrelated changes. The login form_alters don't need to be combined and browserid_admin() doesn't need to be moved to its own file in this issue.

Mad Maks’s picture

Mad Maks commented

Pebosi, did you work on this patch anymore? I really want to use browserid on my drupal 6 site.

icecreamyou’s picture

icecreamyou commented

FWIW, I have very little interest in creating a D6 backport myself, so any progress on this would need to be contributed by someone else. If no one steps up after awhile I will mark this won't fix.

icecreamyou’s picture

icecreamyou commented
Status: Needs work » Closed (won't fix)

This module is being deprecated in favor of Mozilla Persona.