Path hook api docs are missing function bodies

Subscribing, this was pointed out in an issue that I've been working on and has been pushed to D8. This should still make it in.

Adding tag....

I think we just may have to do the same thing, like we do with hook_node_insert() and hook_node_update(). Been trying to think of something useful that can be understood with minimal context and am failing.

+++ modules/path/path.api.php	3 Oct 2010 05:57:21 -0000
@@ -13,30 +13,49 @@
+  $query = db_insert('mytable')
+    ->fields(array(
+      'foo' => 'bar',
+      'pid' => $path['pid'],
+    ))

Let's remove $query = here. We should probably also use something like $path['alias'] rather than 'foo' => 'bar'.

+++ modules/path/path.api.php	3 Oct 2010 05:57:21 -0000
@@ -13,30 +13,49 @@
- *   The path array.
+ *   The path thaat has been updated.

thaat = that

Powered by Dreditor.

One more small typo.

+++ modules/path/path.api.php	3 Oct 2010 17:28:18 -0000
@@ -13,30 +13,49 @@
+  db_update('mytable')
+    ->fields(array('alias' => $ptah['alias']))
+    ->condition('pid', $path['pid'])
+    ->execute();
 }

$path <> $ptah

Powered by Dreditor.

I would give a little bit more info about the $path param and what the array contains. Seems like the definition in path_save() would be good:

 * @param $path
 *   An associative array containing the following keys:
 *   - source: The internal system path.
 *   - alias: The URL alias.
 *   - pid: (optional) Unique path alias identifier.
 *   - language: (optional) The language of the alias.

Added full $path parameter definitions from #15. Should be good to go now.

Looks reasonable to me! Thanks juellj and hejrocker.

Looks great.

Committed to HEAD. Thanks!