Drupal 7 release for OpenID Selector [#1512328]

Comment	File	Size	Author
#3	openid_selector-1512328-d7_menu.patch	2.19 KB	xurizaemon
#2	openid_selector-1092262-librarians_get_path.patch	1.7 KB	xurizaemon

Comment #1

BarisW CreditAttribution: BarisW commented 3 April 2012 at 08:36

No commits in almost a year. I doubt if this every will happen.

Log in or register to post comments

Comment #2

xurizaemon

he/him

Ōtepoti, Aotearoa 🏝

CreditAttribution: xurizaemon commented 22 June 2012 at 21:24

Title:

Finish D7 version

» Drupal 7 release for OpenID Selector

File	Size
openid_selector-1092262-librarians_get_path.patch	1.7 KB

~~Updates menu for D7.~~

EDT: Oops uploaded wrong patch, it belongs on #1092262: Fatal error: Call to undefined function libraries_get_path() in openid_selector/openid_selector.install.

Log in or register to post comments

Comment #3

xurizaemon

he/him

Ōtepoti, Aotearoa 🏝

CreditAttribution: xurizaemon commented 21 July 2013 at 21:16

Status:

Reviewed & tested by the community

» Needs review

File	Size
openid_selector-1512328-d7_menu.patch	2.19 KB

Updates menu for D7.

(EDIT: This code was committed in e0bbc8f.)

Log in or register to post comments

Comment #4

ram4nd CreditAttribution: ram4nd commented 19 July 2013 at 17:14

Status:

Active

» Needs review

Definitely not under active development as the project page states. Maybe seek for an active co-maintainer?

Log in or register to post comments

Comment #5

ram4nd CreditAttribution: ram4nd commented 19 July 2013 at 17:20

Worked for me, but made a lot of errors exactly like the one I pasted on installation. At least it installed the module.

Warning: preg_match() [function.preg-match]: Delimiter must not be alphanumeric or backslash in file_scan_directory() (line 2042 of /home/ram4nd/public_html/includes/file.inc).

Log in or register to post comments

Comment #6

ram4nd CreditAttribution: ram4nd commented 19 July 2013 at 17:23

Status:

Needs review

» Reviewed & tested by the community

As the error seems to come from some other page. And the menu patch seems fine as well.

Log in or register to post comments

Comment #7

xurizaemon

he/him

Ōtepoti, Aotearoa 🏝

CreditAttribution: xurizaemon commented 21 July 2013 at 21:19

Status:

Needs review

» Reviewed & tested by the community

Thanks for RTBC ram4and.

Note that patch from #3 was committed in e0bbc8f already, and patch from #2 belongs to #1092262: Fatal error: Call to undefined function libraries_get_path() in openid_selector/openid_selector.install. There is no remaining RTBC code in this issue, so does your RTBC say 'this branch is ready for official release'?

If you think this module is worth maintaining, put your name on #1673150: Is OpenID Selector still being maintained? and move it to the project "Webmasters" while doing so. (I'll happily co-maintain but can't commit enough time to adopt on my own.)