A developmental branch is available as of today. If you find any Drupal 7 compatibility issues, post them here.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

sun’s picture

Priority: Normal » Critical
Status: Active » Needs review
FileSize
5.25 KB
sun’s picture

FileSize
5.13 KB

piratized.png

sun’s picture

FileSize
16.81 KB

piratized.modules.png

sillygwailo’s picture

Status: Needs review » Active

Committed. Keepin' th' issue open until D7 be released officially. Yarr!

Damien Tournoud’s picture

Usin' the /e modifier is greatly grandly discouraged, Scurvy dog!

-    '/(\.\s)/e' => 'avast("$0",3)',
-    '/([!\

\s)/e' => 'avast("$0",2)', // Greater chance after exclamation
+ '/(\.\s)/e' => 'pirate_avast("$0",3)',
+ '/([!\?]\s)/e' => 'pirate_avast("$0",2)', // Greater chance after exclamation
?>

It ain't because you be a pirate that you shouldn't be security conscious.

sun’s picture

Dang. I knew there was something, when I starred at that code, and resorted to only fix the function namespace...

sillygwailo’s picture

Status: Active » Closed (fixed)

Made a separate issue for that at #917254: Remove /e modifier in preg_replace. Now that the 7.x-dev release is a little more stable, let's file separate issues for that branch.