The 2026 Drupal Business Survey is open. Your response will help shape the future of the Drupal business ecosystem.I was trying to get the weblink module working so I could do some testing on the admin links form but it doesn't seem to work right. I made a couple of changes:
* remove the old 4.6 node type hooks
* replaced the hook_nodeapi with a hook_node_form
* changed the value returned by links_weblink_node_info to reflect that all the methods are in 'links_weblink' not 'weblink'
for some reason it still doesn't present a form to add/remove links. perhaps i'm missing something. if the patch makes no sense let me know.
| Comment | File | Size | Author |
|---|---|---|---|
| links_weblink.module.patch | 2.16 KB | drewish |
Comments
Comment #1
drewish commentedComment #2
syscrusher commentedI'm sorry, drewish...I didn't see this one until now, else I would have reviewed and committed it. How does this relate to your latest comment in response to 39877? Is this patch now obsoleted by your update to that issue? Or should I still review it tonight?
Scott
Comment #3
drewish commentedi think it's still good to go. All my changes to http://drupal.org/node/39877 (i think you transposed two digits) were aimed at the links_related module, and based on Eaton's comment on that thread, this patch is inclusive of his.
Comment #4
drewish commentedi think it's still good to go. All my changes to http://drupal.org/node/39877 (i think you transposed two digits) were aimed at the links_related module, and based on Eaton's comment on that thread, this patch is inclusive of his.
Comment #5
syscrusher commentedPatch committed to CVS HEAD. Please take a look, if you have time, and make sure I got everything right. Also note the addition of TODO.txt, which at the moment lists a couple of known bugs and missing features. Thanks for the patches!
Scott
Comment #6
drewish commentedthanks, i think you got it
Comment #7
syscrusher commentedThanks, drewish.
Comment #8
(not verified) commented