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

CommentFileSizeAuthor
#3 piratized.modules.png16.81 KBsun
#2 piratized.png5.13 KBsun
#1 pirate-DRUPAL-7--1.render.1.patch5.25 KBsun

Comments

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Priority: Normal » Critical
Status: Active » Needs review
StatusFileSize
new pirate-DRUPAL-7--1.render.1.patch5.25 KB

See #310812: Pirates in Drupal

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
StatusFileSize
new piratized.png5.13 KB

piratized.png

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
StatusFileSize
new piratized.modules.png16.81 KB

piratized.modules.png

sillygwailo’s picture

sillygwailo
Location Toronto, ON
commented
Status: Needs review » Active

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

damien tournoud’s picture

damien tournoud commented

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

sun
Primary language German
Location Karlsruhe
commented

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

sillygwailo’s picture

sillygwailo
Location Toronto, ON
commented
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.