An item from the undocumented hooks meta-issue

#675046: Make sure all hooks in D7 have documentation

A pretty straightforward patch, had a hard time coming up with a code example that wasn't offensive and / or stupid. The other option was adding 'WE' => 'Webchickestan' which I'm more than happy to do if someone deems it more appropriate. :P

CommentFileSizeAuthor
#3 hook_countries_alter.patch832 bytesgdd
hook_countries_alter.patch831 bytesgdd
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jhodgdon’s picture

Status: Needs review » Reviewed & tested by the community

Rock on heyrocker!

webchick’s picture

Status: Reviewed & tested by the community » Needs work

LMAO. :)

+++ ./modules/system/system.api.php	18 Feb 2010 00:39:47 -0000
@@ -3197,5 +3197,18 @@ function hook_token_info_alter(&$data) {
+  // Quebec has seceded from Canada. Add to country list

Missing period on "list".

+++ ./modules/system/system.api.php	18 Feb 2010 00:39:47 -0000
@@ -3197,5 +3197,18 @@ function hook_token_info_alter(&$data) {
+  $countries['QU'] = 'Quebec';

QC. :P

I'd fix before commit but I'm having a very bad day and am bound to screw it up. Quick re-roll?

Powered by Dreditor.

gdd’s picture

Status: Needs work » Reviewed & tested by the community
FileSize
832 bytes

I would totally emigrate to webchickestan. What are your policies on accepting aging punk rock nerds with questionable background?

webchick’s picture

Status: Reviewed & tested by the community » Fixed

I'm pretty sure you just described the entire population of webchickestan. You'll feel right at home! :)

(Also, one of my friends is totally going to come in here and re-open this to correct it as "PQ" ;))

Committed to HEAD, thanks! :D

aaron’s picture

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

denix’s picture

Status: Closed (fixed) » Active

It is still undocumented.

jhodgdon’s picture

Status: Active » Closed (fixed)