introduce theme functions for generating teaser and body

+1. Sounds like a good idea to me, and the code looks sane.

hi ber-
what does this do exactly...?
how would you use it...
any more info on this would be greatly appreciated...

These are theme functions you can override in your theme. See the docs: http://drupal.org/node/11774

+1 on the idea.

I'm wondering though if this patch will work. What is $request set to in

function theme_image_body($node) {
 return image_display($node, $request) . $node->body;
}

? Obviously not to $request = ($_GET['size']) ? $_GET['size'] : 'preview'; as it was before.

Maybe the theming functions need to be:

function theme_image_body($node, $size = 'preview');
function theme_image_teaser($node, $size = 'thumbnail');

The last one just out of consistency.

Tested this patch (of comment #5): applies, works as described, coding style ok.
These theming functions won't hurt anyone.
I propose to commit it to CVS and 4-6.

+1 I don't know how people are getting by without this.

This is ready to be committed. Except for:

-    $result = pager_query(db_rewrite_sql("SELECT n.nid FROM {term_node} t INNER JOIN {node} n ON t.nid=n.nid WHERE n.status=1 AND n.type='image' AND t.tid=%d ORDER BY n.sticky DESC, n.created DESC"), 6, 0, NULL, $tid);
+    $result = pager_query(db_rewrite_sql("SELECT n.nid FROM {term_node} t INNER JOIN {node} n ON t.nid=n.nid WHERE n.status=1 AND n.type='image' AND t.tid=%d ORDER BY n.sticky DESC, n.created DESC"), variable_get('image_images_per_page', 6), 0, NULL, $tid);

That hunk will fail.

BTW: in my "sandbox-image" I've added this too, because I think it is very usefull and probably needed.