Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
The links for login and register to post new comments does not refer to the actual form upon login. It just points to the top of the node. This is due to the "#form_comment" instead of "#form-comment". I have attached the needed patch.
A simple thing, but it was bugging our project manager, so I thought I would fix and submit.
Greg
Comment | File | Size | Author |
---|---|---|---|
#2 | comment.module_70.patch | 590 bytes | Gman |
comment.module_69.patch | 596 bytes | Gman | |
Comments
Comment #1
joemoraca CreditAttribution: joemoraca commentedpatch would not apply line numbers were way off...
Comment #2
Gman CreditAttribution: Gman commentedSorry about that. I don't know how that happened. Take from the current version of HEAD.
-Greg
Comment #3
ChrisKennedy CreditAttribution: ChrisKennedy commentedComments were recently fixed in head (see http://drupal.org/node/124742), which is probably why the line numbers were off.
Anyway, when you roll a patch a) please roll it from the root drupal directory so that it can easily be applied b) use -up as options to your diff so that the context can be quickly seen. More info at http://drupal.org/patch
Comment #4
Dries CreditAttribution: Dries commentedThe patch did not apply but I re-rolled it for you and committed it to CVS HEAD and DRUPAL-5. Good catch, and kudos to your project manager. ;-)
Comment #5
Gman CreditAttribution: Gman commentedThanks. I will make sure to adjust my patching notes to conform to the above link.
First patch to core! Hope to contribute in bigger ways in the future. :)
Comment #6
(not verified) CreditAttribution: commented