Correct drupal_system_listing() $mask param for D7 preg_match() [#1673186]

Status page in D7 shows a bunch of errors due to ereg() syntax.

Comment	File	Size	Author
#1	openid_selector-1673186-drupal_system_listing.patch	837 bytes	xurizaemon

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

xurizaemon

he/him

Ōtepoti, Aotearoa 🏝

CreditAttribution: xurizaemon commented 6 July 2012 at 00:36

Title:	Correct file_scan_directory() $mask param for D7 preg_match()	» Correct drupal_system_listing() $mask param for D7 preg_match()
Status:	Active	» Needs review

File	Size
openid_selector-1673186-drupal_system_listing.patch	837 bytes

patch

Comment #2

xurizaemon

he/him

Ōtepoti, Aotearoa 🏝

CreditAttribution: xurizaemon commented 16 August 2012 at 09:59

Error message (so I can find this) -

Warning: preg_match(): Delimiter must not be alphanumeric or backslash in file_scan_directory() (line 2044 of /includes/file.inc).

Merged this fix into patch on #1092262: Fatal error: Call to undefined function libraries_get_path() in openid_selector/openid_selector.install.

Comment #3

andriy_gerasika CreditAttribution: andriy_gerasika commented 22 February 2013 at 01:03

Status:

Needs review

» Fixed

fixed in http://drupalcode.org/project/openid_selector.git/commit/e341c65
thanks for patch

Comment #4

xurizaemon

he/him

Ōtepoti, Aotearoa 🏝

CreditAttribution: xurizaemon commented 22 February 2013 at 04:39

No thanks necessary, but I'd love it if assigned credit for the backlog of patches you're committing at the moment.

See Maintaining a Drupal.org project with Git and Adding a commit author for instructions. Ex -

git commit -m "Issue #1673186 by grobot: Correct drupal_system_listing() $mask param for D7 preg_match()." --author="grobot <grobot@76026.no-reply.drupal.org>"

You can click through from name of the patch submitter to get the attribution address.

No stress - this is a recent addition to Drupal.org and I understand you've been away :)