First: nice module, thanks for that!

Currently the pager is shown for a Webform only when the webform content type is used. Since Webforms can be configured for any content type the logic to figure out if a Webform is attached to the current node can be enhanced for any Webform enabled content type.

CommentFileSizeAuthor
#1 webform_pager-support_any_content_type-1947552-1.patch1.12 KBfuerst

Comments

fuerst’s picture

fuerst commented
Status: Active » Needs review
StatusFileSize
new webform_pager-support_any_content_type-1947552-1.patch1.12 KB

The attached patch fixes that. I additionally changed the following in webform_pager_form_alter():

  • removed the unnecessary node_load() although it probably would use a cached node object.
  • no need for calling node_type_get_type() because the node type is available in $node->type
strakkie’s picture

strakkie commented
Status: Needs review » Closed (fixed)

Good find fuerst. Thnx for your patch and the reviewing of the module.

I have applied your patch (and fixed a bug with the primary key in the db table) and created a new release (7.x-1.5). Available for download right now.

Thnx again.

fuerst’s picture

fuerst commented

That's fast - thanks a lot!