chrismessina / diso

Automatically exported from code.google.com/p/diso
0 stars 0 forks source link

[OpenID-Wordpress] Patch to add admin option "Users can only login using OpenIDs." #100

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago

Maybe not so useful to most people, but I thought I'd submit it: 
Implemented this for a client who will be using OpenID exclusively for
wordpress logins.  Patch adds a new option, openid_required_for_login,
which functions similarly to openid_required_for_registration

Attached is the output of "svn diff".  It should patch the trunk version as
of 7-November.

My apologies if there is an alternate way of submitting patches that I'm
not aware of.

Original issue reported on code.google.com by ryan.dra...@gmail.com on 7 Nov 2008 at 7:36

Attachments:

GoogleCodeExporter commented 9 years ago
actually this is the best way to submit a patch, thanks.   I'll take a look at 
it and see about applying.

Original comment by wnorris on 7 Nov 2008 at 9:24

GoogleCodeExporter commented 9 years ago
+1 for this - we really need this functionality. Any chance of getting this 
patch
applied soon? Thanks.

Original comment by scot.hac...@gmail.com on 10 Jul 2009 at 1:06

GoogleCodeExporter commented 9 years ago
The posted patch will not apply cleanly to the current version of the plugin:

shackmac-2:openid shacker$ patch -p0 < require-openid.patch 
patching file admin_panels.php
Hunk #1 FAILED at 93.
Hunk #2 FAILED at 166.
2 out of 2 hunks FAILED -- saving rejects to file admin_panels.php.rej
patching file login.php
patch unexpectedly ends in middle of line
Hunk #1 FAILED at 68.
1 out of 1 hunk FAILED -- saving rejects to file login.php.rej

Original comment by scot.hac...@gmail.com on 10 Jul 2009 at 1:10

GoogleCodeExporter commented 9 years ago
Looks like the patch got quite out of date.  I'll have a look at updating it to 
patch
the latest version cleanly, when I have time, but it's really a pretty simple 
change.

Original comment by ryan.dra...@gmail.com on 4 Aug 2009 at 3:03