In the process of upgrading my database calls from D6 to D7, I need to understand the new API. For instance the pager and tablesort handling has been changed. The API page https://api.drupal.org/api/drupal/includes%21database%21database.inc/gro... mentions that I should "consider using the PagerDefault Extender [...], and the Tablesort Extender [...]" but it took some time to find any more info about them.
I propose that those two names are made hyperlinks to https://drupal.org/node/508796 and https://drupal.org/node/1848372 respectively. Thanks in advance.
Comment | File | Size | Author |
---|---|---|---|
#8 | extender_doc-2164179-8.patch | 881 bytes | StephaneQ |
#5 | 2164179-2-5.txt | 993 bytes | StephaneQ |
#5 | extender_doc-2164179-5.patch | 957 bytes | StephaneQ |
#2 | extender_doc-2164179-2.patch | 959 bytes | StephaneQ |
Comments
Comment #1
jhodgdonGood idea... probably we should at least put @see references to PagerDefault and TableSort in this documentation. The 8.x docs need to be fixed up first -- there PagerDefault is turning into a link but not TableSort (missing the full namespaced class name). It still could use an @see.
Comment #2
StephaneQHere is a D8 patch
Comment #3
jhodgdonLooks good! The only thing is that @see lines should be at the bottom of the doc block, and they should be separated from the other documentation by a blank line.
Comment #4
jhodgdonActually... Given that those links apply to this paragraph only, it might make more sense to put them into the text of that paragraph directly rather than an @see (which will go into a See Also section at the bottom of the page, with no context). You can just put the URL in there and it will turn into a link on drupal.org.
Comment #5
StephaneQComment #6
jhodgdonPerfect, thanks!
Comment #7
jhodgdonThanks again! Committed to 8.x. We need something similar for 7.x, but obviously without the namespaces and with the correct 7.x class names.
Comment #8
StephaneQHere is the D7 patch
Comment #9
StephaneQOops, forgot status
Comment #10
jhodgdonThanks! I'll get this in.
Comment #11
jhodgdonThanks again! Committed to 7.x.