I've included the previous patch and now I've also added per node type enabling of the link.
To be fine and dandy the checkbox element that sets/unsets the link displaying in the teaser should
be done with an AHAH form. But in D5 there's no such form element. It can be implemented using a contrib module but that introduces a dependency for the module.
Hopefully soon I will have a patch for the D6 version of the module. There is easier to handle dynamic form elements.
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | addtoany_drupal5_module_node_typeV3.patch | 24.83 KB | perusio |
| #2 | addtoany_drupal5_module_node_typeV2.patch | 23.64 KB | perusio |
| addtoany_drupal5_module_node_type.patch | 23.65 KB | perusio |
Comments
Comment #1
micropat commentedUpon saving, even if one of the buttons are checked (and not using an uploaded image), "Missing file for upload" appears.
Good note on the checkbox/AHAH. We'll stay away from external dependencies, but dynamic form elements in D6 would be great.
Comment #2
perusio commentedOops. Fixed, I hope. tested both cases, i.e., with and without custom button.
Comment #3
perusio commentedMessed up in the handling of the per node link enabling checkbox. I didn't notice it at first due to the row in the variable table being already filled by the original module in my devel site. All broke when I installed the patched module in another site.
Anyway is fixed now. Try it out and port it when you can.
Soon I will start to work in patching the 6.x version.
Thank you
Comment #4
xjmThese features are include in the 6.x release, so closing this issue.
Comment #5
xjm