I've noticed a couple of bugs in the 5.x module. One is in the "view" case of the epublish_nodeapi function. In page views, the module is supposed to call theme_epublish_navigation, which changes the breadcrumbing and adds some navigation links at the bottom of the page (similar to the navigation links that appear at the bottom of book pages). However, the test being used to determine whether an epublish-related node is being displayed as a page view currently says "if (!$a3)" (which means, "is not a teaser"), rather than "if ($a4)" which is the correct way to test for page view. The result produces some inappropriate changes in breadcrumbing when epublish-related nodes are listed in Drupal search results.
The other bug, in the epublish_form_alter function, makes it impossible to turn the "abstract" feature on or off for individual node types.
I'm attaching a patch that should fix both issues.
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | add.patch | 994 bytes | JHeffner |
| #1 | patch_154.txt | 2.14 KB | sheldon rampton |
| patch_153.txt | 2.18 KB | sheldon rampton |
Comments
Comment #1
sheldon rampton commentedHere's a revised patch. There was a typo in the original.
Comment #2
rconstantine commentedComment #3
JHeffner commentedI know this patch has been ready to be committed for awhile but I have been happily running it. I added an additional access level to your patched version to control who can write an abstract. I found this useful for some internal intranet sites where the content in the site is re-used and/or highlighted in a monthly newsletter. I attached the additional patch on top of your submission below.
Comment #4
gloscon commentedhttp://drupal.org/node/268312
Comment #5
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.