Anytime a vote is opened, closed, scheduled, or the schedule changed, this should be logged with watchdog().

CommentFileSizeAuthor
#5 election-2626714-5.patch4.77 KBpjcdawkins
#4 election-2626714-4.patch3.03 KBpjcdawkins
#3 election-2626714-3.patch2.99 KBpjcdawkins
#2 election-log_vote_open_close-2626714-2-D7.patch1.58 KBLiam Morland
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Liam Morland created an issue. See original summary.

Liam Morland’s picture

Liam Morland
Primary language English
Location Ontario, CA 🇨🇦
CreditAttribution: Liam Morland at University of Waterloo commented
Assigned: Liam Morland » Unassigned
Status: Active » Needs review
FileSize
election-log_vote_open_close-2626714-2-D7.patch1.58 KB
pjcdawkins’s picture

pjcdawkins CreditAttribution: pjcdawkins commented
FileSize
election-2626714-3.patch2.99 KB

This should be in hooks, to accommodate when an election might be updated outside of that form.

pjcdawkins’s picture

pjcdawkins CreditAttribution: pjcdawkins commented
FileSize
election-2626714-4.patch3.03 KB

Comment tweak

pjcdawkins’s picture

pjcdawkins CreditAttribution: pjcdawkins commented
FileSize
election-2626714-5.patch4.77 KB

Clearer schedule summary and moving the key part of the message a bit earlier (so it's visible on the admin/reports/dblog overview)

pjcdawkins’s picture

pjcdawkins CreditAttribution: pjcdawkins commented
Status: Needs review » Fixed

Committed

  • pjcdawkins committed 7a89c73 on 7.x-1.x 
    Issue #2626714 by Liam Morland: Call watchdog() when opening or closing...
Liam Morland’s picture

Liam Morland
Primary language English
Location Ontario, CA 🇨🇦
CreditAttribution: Liam Morland at University of Waterloo commented

Thanks very much.

pjcdawkins’s picture

pjcdawkins CreditAttribution: pjcdawkins commented

I should mention that you can do this sort of thing via hook_entity_update() in a custom module, or via Rules.

Still, this seemed like a valuable security feature.

Status: Fixed » Needs work

The last submitted patch, 5: election-2626714-5.patch, failed testing.

pjcdawkins’s picture

pjcdawkins CreditAttribution: pjcdawkins commented
Status: Needs work » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.