Quick patch for the full teaser formatter. loaded nodes should have the same build_mode as the referencing node.

CommentFileSizeAuthor
cck-6--2-4.patch792 bytesjapanitrat

Comments

yched’s picture

yched commented
Status: Needs review » Needs work

Hm, I don't think so.
D6 makes this a little messy ($node->build_mode vs. $teaser param), but conceptually, 'full node' and 'teaser' *are* build modes. We streamlined this in D7 already.
There's no 'RSS mode in teaser version', there's 'full', 'teaser', 'RSS', etc...

I think the actual feature here would be to have build modes other than 'full' and 'teaser' be available as additional formatters.

japanitrat’s picture

japanitrat commented

awh allright, this makes sense, but as you mentioned, D6 doesn't do it that way.

hanno’s picture

hanno commented
Category: feature » bug
Issue tags: +SimpleNews, +build mode

This patch fixes a problem I have with the simplenews module.

In newsletters I use noderefence fields to include the articles.

Simplenews uses two specific build_modes ("email_plain" or "email_html"). These build modes are not sent to the referenced fields, so I get the normal build view instead.

Hence, it is not possible to get these nodes in the email-format in the newsletter. Also, block edit (and possible other modules) render the nodes as if it is for screen display, resulting in unexpected additional information (#812386: Disallow the edit links to appear in simplenews, rss, preview and print).

Is it possible to commit this patch?