From 3aca529725c836e6aa7968f462cff67cb63e0e60 Mon Sep 17 00:00:00 2001 From: Thomas Hochstein Date: Wed, 19 Mar 2014 21:43:48 +0100 Subject: [PATCH 1/6] Add votename to Menu. Display the current voting's description. Also purge some dead code. Picked from upstream. Signed-off-by: Thomas Hochstein --- CHANGES | 3 +++ UVmenu.pm | 19 ++----------------- 2 files changed, 5 insertions(+), 17 deletions(-) diff --git a/CHANGES b/CHANGES index 4afa606..f911feb 100644 --- a/CHANGES +++ b/CHANGES @@ -20,6 +20,9 @@ Version 4.12 (19.01.2014): Wahlschein und einem Musterwahlschein (wie im CfV geposted) zu starten. Dazu muss die Config-Option "sampleballotfile" gesetzt und die Datei vorhanden sein (Standard-Dateiname: ballot.sample) +- im Fehlermenue wird nun der Name der Abstimmung als Ueberschrift + angezeigt. Ausserdem wurde in UVmenu.pm ein redundanter Code-Abschnitt + entfernt Version 4.11 (25.09.2012): - Regular Expression fuer Pruefung des Abstimmungsnamens verbessert. diff --git a/UVmenu.pm b/UVmenu.pm index 486bb2c..07e32c8 100644 --- a/UVmenu.pm +++ b/UVmenu.pm @@ -60,6 +60,7 @@ sub menu { while (1) { system($config{clearcmd}); + print "-> $config{votename} <-\n"; print UVmessage::get("MENU_PROBLEMS") . "\n"; foreach my $error (keys %errors) { @@ -107,7 +108,7 @@ sub menu { open (DIFF, "|$config{diff} - $config{sampleballotfile} | $config{pager}"); print DIFF $$body, "\n"; close (DIFF); - + } elsif ($input eq '1') { system($config{clearcmd}); # ignore SIGPIPE (Bug in more and less) @@ -300,22 +301,6 @@ sub menu { $errors{WrongVoting} = UVmessage::get("MENU_WRONGVOTING"); } - } elsif ($input eq '7') { - my $sel; - do { - print "[a] ", UVmessage::get("MENU_VOTING_CORRECT"), "\n", - "[b] ", UVmessage::get("MENU_VOTING_WRONG"), "\n\n", - UVmessage::get("MENU_PROMPT"); - $sel = ; - } until ($sel =~ /^[ab]$/i); - - if ($sel =~ /^a$/i) { - delete $errors{NoVoting}; - delete $errors{WrongVoting}; - } else { - $errors{WrongVoting} = UVmessage::get("MENU_WRONGVOTING"); - } - } elsif ($input =~ /^i$/i) { my $ignore = UVmessage::get("MENU_IGNORE_STRING"); # Set columns for Text::Wrap From f5828d9fad01f6e4ec78b5a6c00db848b0ba2018 Mon Sep 17 00:00:00 2001 From: Thomas Hochstein Date: Wed, 19 Mar 2014 21:45:08 +0100 Subject: [PATCH 2/6] Enforce strict response for data protection clause. Picked from upstream. Signed-off-by: Thomas Hochstein --- CHANGES | 3 +++ usevote.cfg | 4 ++++ uvvote.pl | 2 +- 3 files changed, 8 insertions(+), 1 deletion(-) diff --git a/CHANGES b/CHANGES index f911feb..ba7460a 100644 --- a/CHANGES +++ b/CHANGES @@ -23,6 +23,9 @@ Version 4.12 (19.01.2014): - im Fehlermenue wird nun der Name der Abstimmung als Ueberschrift angezeigt. Ausserdem wurde in UVmenu.pm ein redundanter Code-Abschnitt entfernt +- BDSG-Klausel muss nun standardmaessig zwingend mit JA beantwortet + werden, DAFUER wird nicht mehr akzeptiert. Dies kann in usevote.cfg + konfiguriert werden Version 4.11 (25.09.2012): - Regular Expression fuer Pruefung des Abstimmungsnamens verbessert. diff --git a/usevote.cfg b/usevote.cfg index 8e0ada3..eb99d33 100644 --- a/usevote.cfg +++ b/usevote.cfg @@ -281,6 +281,10 @@ enth_stimme = (E|E\s*N\s*T\s*H\s*A\s*L\s*T\s*U\s*N\s*G) # Achtung, sollte auch in den Templates im Bestaetigungstext angepasst werden ann_stimme = A\s*N\s*N\s*U\s*L\s*L\s*I\s*E\s*R\s*U\s*N\s*G +# RegExp fuer Bestaetigung der Datenschutzklause (case-insensitive) +# Standardmaessig wird nur JA erkannt +bdsg_confirm = JA + # Template files (these files are in the template directory defined above) tpl_mailheader = "mailheader" # generally used mail header tpl_bouncelist = "bouncelist" # used by uvbounce.pl diff --git a/uvvote.pl b/uvvote.pl index 0e7e2bd..a82874a 100644 --- a/uvvote.pl +++ b/uvvote.pl @@ -319,7 +319,7 @@ sub process_vote { # Should read like this: #a [ STIMME ] Text # (Text is configurable in usevote.cfg) unless ($$body =~ /$bdsg_regexp/s && - $$body =~ /#a\W*?\[\W*?$config{ja_stimme}\W*?\]\W*?$bdsg2_regexp/is) { + $$body =~ /#a\W*?\[\W*?$config{bdsg_confirm}\W*?\]\W*?$bdsg2_regexp/is) { push (@errors, 'InvalidBDSG'); } From 3ab56bde53da7d77b3119e6793059aa9689e0d11 Mon Sep 17 00:00:00 2001 From: Thomas Hochstein Date: Wed, 19 Mar 2014 21:46:32 +0100 Subject: [PATCH 3/6] Correctly find $votename. Picked from upstream. Signed-off-by: Thomas Hochstein --- CHANGES | 2 ++ UVconfig.pm | 1 + uvvote.pl | 4 ++-- 3 files changed, 5 insertions(+), 2 deletions(-) diff --git a/CHANGES b/CHANGES index ba7460a..5067501 100644 --- a/CHANGES +++ b/CHANGES @@ -26,6 +26,8 @@ Version 4.12 (19.01.2014): - BDSG-Klausel muss nun standardmaessig zwingend mit JA beantwortet werden, DAFUER wird nicht mehr akzeptiert. Dies kann in usevote.cfg konfiguriert werden +- Regular Expression fuer Pruefung des Abstimmungsnamens weiter + verbessert Version 4.11 (25.09.2012): - Regular Expression fuer Pruefung des Abstimmungsnamens verbessert. diff --git a/UVconfig.pm b/UVconfig.pm index 571ab69..1c64b14 100644 --- a/UVconfig.pm +++ b/UVconfig.pm @@ -121,6 +121,7 @@ sub read_config { tpl_rule_violated => 'rule-violated', begin_divider => 'Alles vor dieser Zeile bitte loeschen', end_divider => 'Alles nach dieser Zeile bitte loeschen', + ballotintro => 'WAHLSCHEIN fuer', nametext => 'Dein Realname, falls nicht im FROM-Header:', nametext2 => 'Waehlername:', addresstext => 'Waehleradresse:', diff --git a/uvvote.pl b/uvvote.pl index a82874a..c101f1f 100644 --- a/uvvote.pl +++ b/uvvote.pl @@ -231,9 +231,9 @@ sub process_vote { } # correct voting? - if ($$body =~ /\Q$config{ballotintro}\E\s+(.+?)[^\S\n]*\n([>:|]*?[\t ]+(\S+.+)\s*$)?/m) { + if ($$body =~ /\Q$config{ballotintro}\E\s+(.+?)\s*\n([>:|]*?[\t ]+(\S+.+)\s*$)?/m) { $voting = $1; - $voting .= " $3" if defined($3); + $voting .= " $3" if (defined($3) and $3 !~ /\Q$config{nametext}\E/); push (@errors, 'WrongVoting') if ($config{votename} !~ /^\s*\Q$voting\E\s*$/); } else { push (@errors, 'NoVoting'); From 923dc1773ec7aac5b3600730945ba2e0d12e76df Mon Sep 17 00:00:00 2001 From: Thomas Hochstein Date: Wed, 19 Mar 2014 21:48:02 +0100 Subject: [PATCH 4/6] Change quorum default value to 50 "JA" votes. Picked from upstream. Signed-off-by: Thomas Hochstein --- CHANGES | 1 + UVconfig.pm | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/CHANGES b/CHANGES index 5067501..162d6c6 100644 --- a/CHANGES +++ b/CHANGES @@ -28,6 +28,7 @@ Version 4.12 (19.01.2014): konfiguriert werden - Regular Expression fuer Pruefung des Abstimmungsnamens weiter verbessert +- Standardwert fuer Mindest-JA-Stimmenzahl von 60 auf 50 verringert Version 4.11 (25.09.2012): - Regular Expression fuer Pruefung des Abstimmungsnamens verbessert. diff --git a/UVconfig.pm b/UVconfig.pm index 1c64b14..f78e3b9 100644 --- a/UVconfig.pm +++ b/UVconfig.pm @@ -94,7 +94,7 @@ sub read_config { ann_stimme => 'A\s*N\s*N\s*U\s*L\s*L\s*I\s*E\s*R\s*U\s*N\s*G', bdsg_confirm => 'JA', condition1 => '$yes>=2*$no', # twice as many yes as no - condition2 => '$yes>=60', # min 60 yes votes + condition2 => '$yes>=50', # min 60 yes votes prop_formula => '$yes/$no', tpl_ack_mail => 'ack-mail', tpl_bouncelist => 'bouncelist', From 6f88a29957653b1a7719f1ad11c8d48ee98cfc48 Mon Sep 17 00:00:00 2001 From: Thomas Hochstein Date: Wed, 19 Mar 2014 21:48:25 +0100 Subject: [PATCH 5/6] Fix typo in README. Picked from upstream. Signed-off-by: Thomas Hochstein --- README | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/README b/README index 286e1ad..fec87c8 100644 --- a/README +++ b/README @@ -430,7 +430,7 @@ Nun solltest Du zunaechst mit (1) die Mail ansehen und anhand der bemaengelten Punkte entscheiden, ob der Wahlschein gueltig oder ungueltig ist bzw. welche Maengel tatsaechlich zutreffen. Mit (0) kannst Du einen Dateivergleich zu einem vorher hinterlegten -Musterwahlschein ausführen ("diff"), um Aenderungen/Fehler schneller +Musterwahlschein ausfuehren ("diff"), um Aenderungen/Fehler schneller zu sehen. Mit den Optionen (2) bis (7) kannst Du einzelne Eigenschaften From f9a99fd83a49d5e8ea785af3e91fa2679482da44 Mon Sep 17 00:00:00 2001 From: Thomas Hochstein Date: Wed, 19 Mar 2014 21:48:48 +0100 Subject: [PATCH 6/6] Update CHANGES for release. Picked from upstream. Signed-off-by: Thomas Hochstein --- CHANGES | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/CHANGES b/CHANGES index 162d6c6..f434572 100644 --- a/CHANGES +++ b/CHANGES @@ -1,10 +1,10 @@ -UseVoteGer Versionshistorie (aktuelle Version: 4.12, released 19.01.2014) +UseVoteGer Versionshistorie (aktuelle Version: 4.12, released 28.02.2014) ========================================================================= TODO: - aussortieren von Bounces aus Stimmenliste -Version 4.12 (19.01.2014): +Version 4.12 (28.02.2014): - uvbounce.pl und uvballot.pl benutzen jetzt wieder den uebergebenen Dateinamen statt irrtuemlich das Mailboxfile aus usevote.cfg (Bugfix in UVreadmail.pm)