Skip to content

Commit

Permalink
enable turning off votes in alerts
Browse files Browse the repository at this point in the history 
This also tidies up editing MP alerts so it's mostly buttons rather than
having to go through a form.
  • Loading branch information
@struan
struan committed Dec 9, 2024
1 parent 649e4d8 commit 73d06c5
Show file tree
Hide file tree
Showing 8 changed files with 103 additions and 26 deletions.
24 changes: 23 additions & 1 deletion classes/AlertView/Standard.php
View file
Original file line number Diff line number Diff line change
Expand Up @@ -158,18 +158,24 @@ private function getBasicData() {
$this->data['pid'] = trim(get_http_var("pid"));
$this->data['pc'] = get_http_var('pc');
$this->data['submitted'] = get_http_var('submitted') || $this->data['pid'] || $this->data['keyword'] || $this->data['step'];
$this->data['ignore_speaker_votes'] = get_http_var('ignore_speaker_votes');

if ($this->data['addword'] || $this->data['step']) {
$alert = $this->alert->check_token($this->data['token']);

$criteria = '';
$alert_ignore_speaker_votes = 0;
if ($alert) {
$criteria = $alert['criteria'];
$alert_ignore_speaker_votes = $alert['ignore_speaker_votes'];
}

$ignore_speaker_votes = get_http_var('ignore_speaker_votes', $alert_ignore_speaker_votes);
$this->data['ignore_speaker_votes'] = ($ignore_speaker_votes == 'on' || $ignore_speaker_votes == 1);

$this->data['alert'] = $alert;

$this->data['alert_parts'] = \MySociety\TheyWorkForYou\Utility\Alert::prettifyCriteria($criteria, $alert['ignore_speaker_votes'], true);
$this->data['alert_parts'] = \MySociety\TheyWorkForYou\Utility\Alert::prettifyCriteria($criteria, $alert_ignore_speaker_votes, true);

$existing_rep = '';
if (isset($this->data['alert_parts']['spokenby'])) {
Expand Down Expand Up @@ -237,6 +243,22 @@ private function getBasicData() {
$this->data['results'] = '';

$this->getSearchSections();
} else if ($this->data['mp_step'] == 'mp_alert') {
$alert = $this->alert->check_token($this->data['token']);
if ($alert) {
$ignore_speaker_votes = get_http_var('ignore_speaker_votes', $alert['ignore_speaker_votes']);
$this->data['ignore_speaker_votes'] = ($ignore_speaker_votes == 'on' || $ignore_speaker_votes == 1);

$existing_rep = '';
if (isset($alert['criteria'])) {
$alert_parts = \MySociety\TheyWorkForYou\Utility\Alert::prettifyCriteria($alert['criteria'], $alert['ignore_speaker_votes'], true);
$existing_rep = $alert_parts['spokenby'][0];
$this->data['pid'] = $alert_parts['pid'];
}
$this->data['keyword'] = get_http_var('mp_search', $existing_rep);
} else {
$this->data['ignore_speaker_votes'] = get_http_var('ignore_speaker_votes');
}
} # XXX probably should do something here if $alertsearch is set

$this->data['sign'] = get_http_var('sign');
Expand Down
11 changes: 9 additions & 2 deletions classes/Utility/Alert.php
View file
Original file line number Diff line number Diff line change
Expand Up @@ -74,6 +74,7 @@ public static function forUser($email) {
'status' => $status,
'criteria' => $criteria,
'raw' => $row['criteria'],
'ignore_speaker_votes' => $row['ignore_speaker_votes'],
'keywords' => [],
'exclusions' => [],
'sections' => [],
Expand All @@ -89,7 +90,7 @@ public static function forUser($email) {

public static function prettifyCriteria($alert_criteria, $ignore_speaker_votes = false, $as_parts = false) {
$text = '';
$parts = ['words' => [], 'sections' => [], 'exclusions' => [], 'match_all' => true];
$parts = ['words' => [], 'sections' => [], 'exclusions' => [], 'match_all' => true, 'pid' => false];
if ($alert_criteria) {
# check for phrases
if (strpos($alert_criteria, ' OR ') !== false) {
Expand All @@ -116,7 +117,13 @@ public static function prettifyCriteria($alert_criteria, $ignore_speaker_votes =
$exclusions = [];
$sections = [];
$sections_verbose = [];
$spokenby = array_values(\MySociety\TheyWorkForYou\Utility\Search::speakerNamesForIDs($alert_criteria));
$speaker_parts = \MySociety\TheyWorkForYou\Utility\Search::speakerNamesForIDs($alert_criteria);
$pids = array_keys($speaker_parts);
$spokenby = array_values($speaker_parts);

if (count($pids) == 1) {
$parts['pid'] = $pids[0];
}

foreach ($criteria as $c) {
if (preg_match('#^section:(\w+)#', $c, $m)) {
Expand Down
2 changes: 1 addition & 1 deletion www/includes/easyparliament/alert.php
View file
Original file line number Diff line number Diff line change
Expand Up @@ -167,7 +167,7 @@ public function add($details, $confirmation_email = false, $instantly_confirm =
':criteria' => $criteria,
])->first();
if ($q) {
if ($q['deleted']) {
if ($q['deleted'] || $q['ignore_speaker_votes'] != $ignore_speaker_votes) {
$this->db->query("UPDATE alerts SET deleted=0, ignore_speaker_votes=:ignore_speaker_votes
WHERE email = :email
AND criteria = :criteria
Expand Down
4 changes: 4 additions & 0 deletions www/includes/easyparliament/templates/html/alert/_alert_form.php
View file
Original file line number Diff line number Diff line change
Expand Up @@ -105,6 +105,10 @@
<p><?= gettext("Or edit the name") ?></p>
<?php } ?>
<input type="text" id="representative" name="representative" value="<?= _htmlspecialchars($representative) ?>" aria-required="true">
<div class="checkbox-wrapper">
<input type="checkbox" id="ignore_speaker_votes" name="ignore_speaker_votes"<?= $ignore_speaker_votes ? ' checked' : ''?>>
<label for="ignore_speaker_votes"><?= gettext('Do not include votes') ?></label>
</div>
</div>


Expand Down
56 changes: 37 additions & 19 deletions www/includes/easyparliament/templates/html/alert/_list_accordian.php
View file
Original file line number Diff line number Diff line change
Expand Up @@ -186,35 +186,53 @@
<span><?= gettext('Confirm alert') ?></span>
</button>
<?php } elseif ($alert['status'] == 'suspended') { ?>
<button type="submit" class="button button-outline small" name="action" value="Resume">
<span><?= gettext('Resume alert') ?></span>
<i aria-hidden="true" class="fi-play"></i>
</button>
<button type="submit" class="button button-outline small" name="action" value="Resume">
<span><?= gettext('Resume alert') ?></span>
<i aria-hidden="true" class="fi-play"></i>
</button>
<?php } else { ?>
<button type="submit" class="button button--outline small" name="action" value="Suspend">
<i aria-hidden="true" class="fi-pause"></i>
<span><?= gettext('Suspend alert') ?></span>
</button>
<button type="submit" class="button small button--outline-red" name="action" value="Delete">
<i aria-hidden="true" class="fi-trash"></i>
<span><?= gettext('Delete alert') ?></span>
</button>
<button type="submit" class="button button--outline small" name="action" value="Suspend">
<i aria-hidden="true" class="fi-pause"></i>
<span><?= gettext('Suspend alert') ?></span>
</button>
<button type="submit" class="button small button--outline-red" name="action" value="Delete">
<i aria-hidden="true" class="fi-trash"></i>
<span><?= gettext('Delete alert') ?></span>
</button>
<?php } ?>
</form>
<form action="<?= $actionurl ?>" method="POST">
<input type="hidden" name="step" value="define">
<input type="hidden" name="t" value="<?= _htmlspecialchars($alert['token']) ?>">
<button type="submit" class="button button--outline small" value="Edit">
<i aria-hidden="true" class="fi-page-edit"></i>
<span><?= gettext('Edit alert') ?></span>
</button>
<?php if (count($alert['words'])) { ?>
<input type="hidden" name="step" value="define">
<input type="hidden" name="shown_related" value="1">
<button type="submit" class="button button--outline small" value="Edit">
<i aria-hidden="true" class="fi-page-edit"></i>
<span><?= gettext('Edit alert') ?></span>
</button>
<?php } else { ?>
<input type="hidden" name="mp_step" value="mp_confirm">
<input type="hidden" name="pid" value="<?= $alert['pid'] ?>">
<?php if ($alert['ignore_speaker_votes'] == 1) { ?>
<input type="hidden" name="ignore_speaker_votes" value="0">
<button type="submit" class="button button--outline small" value="Edit">
<i aria-hidden="true" class="fi-page-edit"></i>
<span><?= gettext('Include votes') ?></span>
</button>
<?php } else { ?>
<input type="hidden" name="ignore_speaker_votes" value="1">
<button type="submit" class="button button--outline small" value="Edit">
<i aria-hidden="true" class="fi-page-edit"></i>
<span><?= gettext('Ignore votes') ?></span>
</button>
<?php } ?>
<?php } ?>
</form>
</div>
<?php } ?>
<?php if (!in_array(implode('', $person_alerts[0]['spokenby']), $all_keywords)) { ?>
<p class="alert-page-subsection--subtitle">Alert when <?= _htmlspecialchars(implode(', ', $person_alerts[0]['spokenby'])) ?> is <strong>mentioned</strong></p>
<form action="<?= $actionurl ?>" method="post">
<input type="hidden" name="keyword" value="<?= _htmlentities(implode('', $person_alerts[0]['spokenby'])) ?>">
<input type="hidden" name="words[]" value="<?= _htmlentities(implode('', $person_alerts[0]['spokenby'])) ?>">
<button type="submit" class="button small" name="action" value="Subscribe">
<?= gettext('Create new alert') ?>
<i aria-hidden="true" role="img" class="fi-megaphone"></i>
Expand Down
7 changes: 6 additions & 1 deletion www/includes/easyparliament/templates/html/alert/_mp_alert_form.php
View file
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@
<input type="hidden" name="t" value="<?= _htmlspecialchars($token) ?>">
<input type="hidden" name="email" value="<?= _htmlspecialchars($email) ?>">
<input type="hidden" name="pid" value="<?= $member->person_id() ?>">
<input type="hidden" name="ignore_speaker_votes" value="<?= $ignore_speaker_votes ?>">
<?= $member->full_name() ?>
(<?= _htmlspecialchars($constituency) ?>) in <?= _htmlspecialchars($member->house_text($member->house_disp)) ?>
<input type="submit" class="button small" value="<?= gettext('Subscribe') ?>"></form>
Expand All @@ -25,12 +26,16 @@
</p>
<?php } ?>

<?php if ($token) { ?>
<input type="hidden" name="t" value="<?= _htmlspecialchars($token) ?>">
<?php } ?>

<p>
<?php if ($pid) { ?>
<input type="text" class="form-control" name="mp_search" id="mp_search" disabled="disabled"
value="<?= $pid_member->full_name() ?><?php if ($pid_member->constituency()) { ?> (<?= _htmlspecialchars($pid_member->constituency()) ?>)<?php } ?>">
<?php } elseif ($keyword) { ?>
<input type="text" class="form-control" name="mp_search" id="mp_search" disabled="disabled" value="<?= _htmlspecialchars($display_keyword) ?>">
<input type="text" class="form-control" name="mp_search" id="mp_search" value="<?= _htmlspecialchars($keyword) ?>">
<?php } else { ?>
<label for="mp-postcode">Search postcode, or MP name</label>
<input id="mp-postcode" type="text" class="form-control" placeholder="<?= gettext('e.g. ‘B2 4QA’ or ‘John Doe’') ?>" name="mp_search" id="mp_search" value="<?= _htmlentities($search_text) ?>" style="min-width:300px;">
Expand Down
24 changes: 22 additions & 2 deletions www/includes/easyparliament/templates/html/alert/_own_mp_alerts.php
View file
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@
<div class="alert-page-alert-controls">
<form action="<?= $actionurl ?>" method="POST">
<input type="hidden" name="t" value="<?= _htmlspecialchars($alert['token']) ?>">
<input type="hidden" name="pid" value="<?= _htmlspecialchars($alert['pid']) ?>">
<?php if ($alert['status'] == 'unconfirmed') { ?>
<button type="submit" class="button button--outline small" name="action" value="Confirm">
<i aria-hidden="true" class="fi-save"></i>
Expand All @@ -25,13 +26,31 @@
</button>
</form>
<form action="<?= $actionurl ?>" method="POST">
<input type="hidden" name="t" value="<?= _htmlspecialchars($alert['token']) ?>">
<?php if (count($alert['words'])) { ?>
<input type="hidden" name="step" value="define">
<input type="hidden" name="shown_related" value="1">
<input type="hidden" name="t" value="<?= _htmlspecialchars($alert['token']) ?>">
<button type="submit" class="button button--outline small" value="Edit">
<i aria-hidden="true" class="fi-page-edit"></i>
<span><?= gettext('Edit alert') ?></span>
</button>
<?php } else { ?>
<input type="hidden" name="mp_step" value="mp_confirm">
<input type="hidden" name="pid" value="<?= $alert['pid'] ?>">
<?php if ($alert['ignore_speaker_votes'] == 1) { ?>
<input type="hidden" name="ignore_speaker_votes" value="0">
<button type="submit" class="button button--outline small" value="Edit">
<i aria-hidden="true" class="fi-page-edit"></i>
<span><?= gettext('Include votes') ?></span>
</button>
<?php } else { ?>
<input type="hidden" name="ignore_speaker_votes" value="1">
<button type="submit" class="button button--outline small" value="Edit">
<i aria-hidden="true" class="fi-page-edit"></i>
<span><?= gettext('Ignore votes') ?></span>
</button>
<?php } ?>
<?php } ?>
</form>
<?php } ?>
</div>
Expand All @@ -40,7 +59,8 @@
<?php if (!in_array($own_mp_criteria, $all_keywords)) { ?>
<p class="alert-page-subsection--subtitle">Alert when <?= _htmlspecialchars($own_mp_criteria) ?> is <strong>mentioned</strong></p>
<form action="<?= $actionurl ?>" method="post">
<input type="hidden" name="keyword" value="<?= _htmlentities($own_mp_criteria) ?>">
<input type="hidden" name="step" value="confirm">
<input type="hidden" name="words[]" value="<?= _htmlentities($own_mp_criteria) ?>">
<button type="submit" class="button button--outline small" name="action" value="Subscribe">
<?= gettext('Create new alert') ?>
<i aria-hidden="true" role="img" class="fi-megaphone"></i>
Expand Down
1 change: 1 addition & 0 deletions www/includes/easyparliament/templates/html/alert/index.php
View file
Original file line number Diff line number Diff line change
Expand Up @@ -155,6 +155,7 @@
<input type="hidden" name="t" value="<?= _htmlspecialchars($token) ?>">
<input type="hidden" name="email" value="<?= _htmlspecialchars($email) ?>">
<input type="hidden" name="pid" value="<?= $row['person_id'] ?>">
<input type="hidden" name="ignore_speaker_votes" value="<?= $ignore_speaker_votes ?>">
<?php
$name = member_full_name($row['house'], $row['title'], $row['given_name'], $row['family_name'], $row['lordofname']);
if ($row['constituency']) {
Expand Down

0 comments on commit 73d06c5

Please sign in to comment.