put this in a branch
authorhttps://id.koumbit.net/anarcat <https://id.koumbit.net/anarcat@web>
Sat, 7 Sep 2013 18:17:02 +0000 (14:17 -0400)
committeradmin <admin@branchable.com>
Sat, 7 Sep 2013 18:17:02 +0000 (14:17 -0400)
doc/bugs/notifyemail_fails_with_some_openid_providers.mdwn

index 5c5b01431f0af81db8d7c635095beb9f1f1ccde1..a135b061fba7f9848a2ba59a498f60a460d7d235 100644 (file)
@@ -56,22 +56,9 @@ sub formbuilder_setup (@) {
 
 I believe this could be worked around simply by re-enabling that field and allowing the user to specify an email there by hand, making a note that the OpenID provider's email is used by default.
 
-Here's the dumbest [[!taglink patch]] that actually fixes the problem for me:
+[[!template  id=gitbranch branch=anarcat/openid_email author="[[anarcat]]"]]
 
-[[!format diff """
-diff --git a/IkiWiki/Plugin/openid.pm b/IkiWiki/Plugin/openid.pm
-index e3b9982..df87ba3 100644
---- a/IkiWiki/Plugin/openid.pm
-+++ b/IkiWiki/Plugin/openid.pm
-@@ -104,7 +104,6 @@ sub formbuilder_setup (@) {
-                       size => 1, force => 1,
-                       fieldset => "login",
-                       comment => $session->param("name"));
--              $form->field(name => "email", type => "hidden");
-       }
- }
-"""]]
+The dumbest [[!taglink patch]] that actually fixes the problem for me is in the branch mentionned here.
 
 It would probably be better to add a comment on the field as indicated above, but it's a good proof of concept.