We added a good deal of @todos to the API docs in #1461840: Clean up Views API documentation for api.drupal.org, this is a start on filling them in.

CommentFileSizeAuthor
#3 1493256.patch6.29 KBdawehner
#1 views-1493256-1.patch35.53 KBtim.plunkett
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

tim.plunkett’s picture

tim.plunkett
he/him
Primary language English
Location Philadelphia
CreditAttribution: tim.plunkett commented
Status: Active » Needs review
FileSize
views-1493256-1.patch35.53 KB

There are still 10 to do, 4 of which are @defgroups.

grep -nr "^ \* @todo\." * | wc -l

Here's the rest.

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented
Status: Needs review » Fixed

So committed that patch. Thanks! (as always).

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented
Status: Fixed » Needs review
FileSize
1493256.patch6.29 KB

Let's open it again.

It doesn't really improve the documentation of the base handlers, though it describes the long lifetime
of views. Do you think it's detailed enough?

tim.plunkett’s picture

tim.plunkett
he/him
Primary language English
Location Philadelphia
CreditAttribution: tim.plunkett commented
Status: Needs review » Needs work 
+++ b/plugins/views_plugin_cache.incundefined
@@ -8,7 +8,15 @@
+ * Cache plugins take care about how views are cached so less
+ * work has to be done.

The first line can only be 80 characters, which is another reason I put this off. Hard to sum up.

After the first line there can be a blank line and then a full paragraph.

Not using dreditor to paste all of them in, because its almost all of them.