Create an update for signatures
merlinofchaos - May 26, 2009 - 05:44
| Project: | Node comments |
| Version: | 6.x-2.x-dev |
| Component: | Code |
| Category: | task |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | active |
| Issue tags: | DruBB |
Jump to:
Description
We need an update for D6 comment.module style signatures. There are a couple of choices we have to make:
Do we want to leave the whole signature issue alone and just let the node signatures module take care of it?
Where we do update, how do we deal with older posts that will have the signature embedded into the code?
#6
#7
Ha ha. I just got a request for signatures on the branch with this version of nodecomment for signatures. Can the old school way of grabbing signatures and prepopulating the node form, and ignoring that if signatures for forums is enabled.
#8
Man, that'll get ugly, I think. It may be possible, but...
#9
We definitely have an argument that the preprocess functions patch wasn't fully thought through, in terms of how it would affect DX. Perhaps this is my fault, as I was so busy on other stuff that as the only person with real objections to that patch, I didn't feel like standing up and being the guy blocking the patch, and so I put fairly minimal effort into it.
That said, the DX WTF here is mighty, I think, so either a rollback should happen, or this. The problem is, 'this' can only happen if there is a reasonable likelihood of it making it in. The actual patch is likely to be tedious to create.
#10
Sigh. #9 on wrong issue. I hate how I do that.