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

Comments

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Status: Needs review » Reviewed & tested by the community

Rock on heyrocker!

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
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

gdd
he/him
Primary language English
Location Portland, OR
commented
Status: Needs work » Reviewed & tested by the community
StatusFileSize
new hook_countries_alter.patch832 bytes

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

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
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

aaron commented

http://en.wikipedia.org/wiki/Guam#Name_change might be more topical :P

Status: Fixed » Closed (fixed)

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

denix’s picture

denix commented
Status: Closed (fixed) » Active

It is still undocumented.

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Status: Active » Closed (fixed)

Huh? It is documented:
https://api.drupal.org/api/drupal/modules!system!system.api.php/function...