On the function anonymizer_form_submit, the code for anonymize and unanonymize nodes must work only when the node is being anonymized or were anonymized previously in order to unanonymize.
I think this is critical.
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | anonymizer.module-889374-2.patch | 1.93 KB | gaele |
| #3 | anonymizer.module-889374-1.diff | 2.14 KB | dannymontalvo |
Comments
Comment #1
dannymontalvo commentedHello,
I am assuming that you're familiar with the module, and configured correctly to work with some content types.
I tested the module to anonymize a node, and unanonymize nodes(revert back to original author), changing the checkbox option on the node_form for anonymizing and saving, what i am looking is a wrong functionality, something is bad on the function anonymizer_form_submit.
I hope that explain in a more detail the problem.
Cheers
Comment #2
dannymontalvo commentedTake a look to the code for the function anonymizer_form_submit
I can't understand what is done on the first condition on the function, I think that is equivalent to the code of the section for "Anonymize post". maybe that can be the source of the bug.
Cheers
Comment #3
dannymontalvo commentedAnyway, I have a patch that seems to correct the bug, and were tested. I think that could be better but for now seems to be ok.
Comment #4
gaele commentedUpdated patch
Comment #5
gaele commentedComment #6
develcuy commented@gaele thanks for the patch but you should not RTBC your own code ;)
Comment #7
develcuy commented@dannymontalvo solid patch
Comment #8
develcuy commented@gaele, just seen that you fixed a typo, next time please comment your changes. Thanks for your contribution to you both!