Since installing beta10, my Drupal log shows the following message when people visit nodes of a CCK node type that I have defined:
Missing argument 3 for subscriptions_ui_link() in /home/chelmsf/public_html/sites/default/modules/subscriptions/subscriptions_ui.module on line 39.
There is no subscribe link visible, and I don't think there ever has been on these nodes.
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | subscriptions_ui.hook_link.patch | 692 bytes | salvis |
Comments
Comment #1
salvisThanks for reporting this. Change
function subscriptions_ui_link($type, $node, $teaser)to
function subscriptions_ui_link($type, $node = NULL, $teaser = NULL)and just below
!teasertoempty(teaser).Do you get the Subscribe link now?
Comment #2
reikiman commentedI was seeing this issue - applied the above change - and was able to see the subscribe link.
But your change to '!teaser' I don't understand the description.
Comment #3
salvisHere's the patch.
Comment #4
rallycivic commentedThanks, that has stopped the messages I had in the log.
Comment #5
rhinokitty commentedFor newbies like me:
Applying a patch is easy using the Linux command line utility "Patch". Type into the command line "Man patch" and it will show you information on how to use it.
How I did it:
Get all of the files in one place (desktop works) and navigate there with the command line (cd /whatever/directory/it/is). Type into the command line:
patch subscriptions_ui.module subscriptions_ui.hook_link.patchWhy? The second bit "subscriptions_ui.module" is what needs to be patched. The third bit "subscriptions_ui.hook_link.patch" is the patch that you can download from the above commend (right click save as). The first part "patch" simply tells the computer which program to use. I am on Ubuntu Gnu/Linux and this was easy to do, just a little unnerving because the command line can be intimidating. Now I feel confident in patching files! Cheers!
Comment #6
salvisComment #7
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.
Comment #8
rallycivic commentedThis message has come back with beta12, but function subscriptions_ui_link is now on line 66.
I have changed line 66 to end
$teaser = NULL) rather than just $teaser)
Comment #9
salvisAh, yes, that fix was inadvertently reverted. Thanks!
It should be
Comment #10
salvisComment #11
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.