[aur-dev] [PATCH 1/2] Split out code to generate action links
Lukas Fleischer
lfleischer at archlinux.org
Fri Jun 26 10:41:44 UTC 2015
Add (and use) two new helper functions html_account_link() and
html_account_form() to generate the links in the package actions box.
Signed-off-by: Lukas Fleischer <lfleischer at archlinux.org>
---
web/lib/aur.inc.php | 37 ++++++++++++++++++
web/template/pkg_details.php | 74 ++++++++++--------------------------
web/template/pkgbase_details.php | 82 ++++++++++++++--------------------------
3 files changed, 86 insertions(+), 107 deletions(-)
diff --git a/web/lib/aur.inc.php b/web/lib/aur.inc.php
index 7f923d7..95f72ce 100644
--- a/web/lib/aur.inc.php
+++ b/web/lib/aur.inc.php
@@ -222,6 +222,43 @@ function html_format_maintainers($maintainer, $comaintainers) {
}
/**
+ * Format a link in the package actions box
+ *
+ * @param string $uri The link target
+ * @param string $desc The link label
+ *
+ * @return string The generated HTML code for the action link
+ */
+function html_action_link($uri, $desc) {
+ $code = '<a href="' . htmlspecialchars($uri, ENT_QUOTES) . '">';
+ $code .= htmlspecialchars($desc) . '</a>';
+
+ return $code;
+}
+
+/**
+ * Format a form in the package actions box
+ *
+ * @param string $uri The link target
+ * @param string $action The action name (passed as HTTP POST parameter)
+ * @param string $desc The link label
+ *
+ * @return string The generated HTML code for the action link
+ */
+function html_action_form($uri, $action, $desc) {
+ $code = '<form action="' . htmlspecialchars($uri, ENT_QUOTES) . '" ';
+ $code .= 'method="post">';
+ $code .= '<input type="hidden" name="token" value="';
+ $code .= htmlspecialchars($_COOKIE['AURSID'], ENT_QUOTES) . '" />';
+ $code .= '<input type="submit" class="button text-button" name="';
+ $code .= htmlspecialchars($action, ENT_QUOTES) . '" ';
+ $code .= 'value="' . htmlspecialchars($desc, ENT_QUOTES) . '" />';
+ $code .= '</form>';
+
+ return $code;
+}
+
+/**
* Determine the user's e-mail address in the database using a session ID
*
* @param string $sid User's session ID
diff --git a/web/template/pkg_details.php b/web/template/pkg_details.php
index 6864431..e894c07 100644
--- a/web/template/pkg_details.php
+++ b/web/template/pkg_details.php
@@ -83,6 +83,9 @@ foreach ($rels as $rel) {
# $sources[0] = 'src';
$sources = pkg_sources($row["ID"]);
+
+$base_uri = get_pkgbase_uri($row['BaseName']);
+
?>
<div id="pkgdetails" class="box">
<h2><?= __('Package Details') . ': ' . htmlspecialchars($row['Name']) . ' ' . htmlspecialchars($row['Version']) ?></h2>
@@ -99,75 +102,40 @@ $sources = pkg_sources($row["ID"]);
<li><span class="flagged"><?php if ($row["OutOfDateTS"] !== NULL) { echo __('Flagged out-of-date')." (${out_of_date_time})"; } ?></span></li>
<?php if ($uid): ?>
<?php if ($row["OutOfDateTS"] === NULL): ?>
- <li>
- <form action="<?= get_pkgbase_uri($row['BaseName']) . 'flag/'; ?>" method="post">
- <input type="hidden" name="token" value="<?= htmlspecialchars($_COOKIE['AURSID']) ?>" />
- <input type="submit" class="button text-button" name="do_Flag" value="<?= __('Flag package out-of-date') ?>" />
- </form>
- </li>
+ <li><?= html_action_form($base_uri . 'flag/', "do_Flag", __('Flag package out-of-date')) ?></li>
<?php elseif (($row["OutOfDateTS"] !== NULL) && has_credential(CRED_PKGBASE_UNFLAG, $maintainers)): ?>
- <li>
- <form action="<?= get_pkgbase_uri($row['BaseName']) . 'unflag/'; ?>" method="post">
- <input type="hidden" name="token" value="<?= htmlspecialchars($_COOKIE['AURSID']) ?>" />
- <input type="submit" class="button text-button" name="do_UnFlag" value="<?= __('Unflag package') ?>" />
- </form>
- </li>
+ <li><?= html_action_form($base_uri . 'unflag/', "do_UnFlag", __('Unflag package')) ?></li>
<?php endif; ?>
+
<?php if (pkgbase_user_voted($uid, $base_id)): ?>
- <li>
- <form action="<?= get_pkgbase_uri($row['BaseName']) . 'unvote/'; ?>" method="post">
- <input type="hidden" name="token" value="<?= htmlspecialchars($_COOKIE['AURSID']) ?>" />
- <input type="submit" class="button text-button" name="do_UnVote" value="<?= __('Remove vote') ?>" />
- </form>
- </li>
+ <li><?= html_action_form($base_uri . 'unvote/', "do_UnVote", __('Remove vote')) ?></li>
<?php else: ?>
- <li>
- <form action="<?= get_pkgbase_uri($row['BaseName']) . 'vote/'; ?>" method="post">
- <input type="hidden" name="token" value="<?= htmlspecialchars($_COOKIE['AURSID']) ?>" />
- <input type="submit" class="button text-button" name="do_Vote" value="<?= __('Vote for this package') ?>" />
- </form>
- </li>
+ <li><?= html_action_form($base_uri . 'vote/', "do_Vote", __('Vote for this package')) ?></li>
<?php endif; ?>
+
<?php if (pkgbase_user_notify($uid, $base_id)): ?>
- <li>
- <form action="<?= get_pkgbase_uri($row['BaseName']) . 'unnotify/'; ?>" method="post">
- <input type="hidden" name="token" value="<?= htmlspecialchars($_COOKIE['AURSID']) ?>" />
- <input type="submit" class="button text-button" name="do_UnNotify" value="<?= __('Disable notifications') ?>" />
- </form>
- </li>
+ <li><?= html_action_form($base_uri . 'unnotify/', "do_UnNotify", __('Disable notifications')) ?></li>
<?php else: ?>
- <li>
- <form action="<?= get_pkgbase_uri($row['BaseName']) . 'notify/'; ?>" method="post">
- <input type="hidden" name="token" value="<?= htmlspecialchars($_COOKIE['AURSID']) ?>" />
- <input type="submit" class="button text-button" name="do_Notify" value="<?= __('Notify of new comments') ?>" />
- </form>
- </li>
+ <li><?= html_action_form($base_uri . 'notify/', "do_Notify", __('Notify of new comments')) ?></li>
<?php endif; ?>
+
<?php if (has_credential(CRED_PKGBASE_EDIT_COMAINTAINERS, array($row["MaintainerUID"]))): ?>
- <li><a href="<?= get_pkgbase_uri($row['BaseName']) . 'comaintainers/'; ?>"><?= __('Manage Co-Maintainers'); ?></a></li>
+ <li><?= html_action_link($base_uri . 'comaintainers/', __('Manage Co-Maintainers')) ?></a></li>
<?php endif; ?>
+
<li><span class="flagged"><?php if ($row["RequestCount"] > 0) { echo _n('%d pending request', '%d pending requests', $row["RequestCount"]); } ?></span></li>
- <li><a href="<?= get_pkgbase_uri($row['BaseName']) . 'request/'; ?>"><?= __('File Request'); ?></a></li>
+ <li><?= html_action_link($base_uri . 'request/', __('File Request')) ?></a></li>
+
<?php if (has_credential(CRED_PKGBASE_DELETE)): ?>
- <li><a href="<?= get_pkgbase_uri($row['BaseName']) . 'delete/'; ?>"><?= __('Delete Package'); ?></a></li>
- <li><a href="<?= get_pkgbase_uri($row['BaseName']) . 'merge/'; ?>"><?= __('Merge Package'); ?></a></li>
- <?php endif; ?>
+ <li><?= html_action_link($base_uri . 'delete/', __('Delete Package')) ?></a></li>
+ <li><?= html_action_link($base_uri . 'merge/', __('Merge Package')) ?></a></li>
<?php endif; ?>
<?php if ($uid && $row["MaintainerUID"] === NULL): ?>
- <li>
- <form action="<?= get_pkgbase_uri($row['BaseName']) . 'adopt/'; ?>" method="post">
- <input type="hidden" name="token" value="<?= htmlspecialchars($_COOKIE['AURSID']) ?>" />
- <input type="submit" class="button text-button" name="do_Adopt" value="<?= __('Adopt Package') ?>" />
- </form>
- </li>
+ <li><?= html_action_form($base_uri . 'adopt/', "do_Adopt", __('Adopt Package')) ?></li>
<?php elseif (has_credential(CRED_PKGBASE_DISOWN, array($row["MaintainerUID"]))): ?>
- <li>
- <form action="<?= get_pkgbase_uri($row['BaseName']) . 'disown/'; ?>" method="post">
- <input type="hidden" name="token" value="<?= htmlspecialchars($_COOKIE['AURSID']) ?>" />
- <input type="submit" class="button text-button" name="do_Disown" value="<?= __('Disown Package') ?>" />
- </form>
- </li>
+ <li><?= html_action_form($base_uri . 'disown/', "do_Disown", __('Disown Package')) ?></li>
+ <?php endif; ?>
<?php endif; ?>
</ul>
</div>
diff --git a/web/template/pkgbase_details.php b/web/template/pkgbase_details.php
index dc30e48..f6d8071 100644
--- a/web/template/pkgbase_details.php
+++ b/web/template/pkgbase_details.php
@@ -17,6 +17,12 @@ $maintainer = username_from_id($row["MaintainerUID"]);
$comaintainers = pkgbase_get_comaintainers($base_id);
$packager = username_from_id($row["PackagerUID"]);
+if ($row["MaintainerUID"] !== NULL) {
+ $maintainers = array_merge(array($row["MaintainerUID"]), pkgbase_get_comaintainer_uids(array($base_id)));
+} else {
+ $maintainers = NULL;
+}
+
$votes = $row['NumVotes'];
# In case of wanting to put a custom message
@@ -28,6 +34,9 @@ $submitted_time = ($row["SubmittedTS"] == 0) ? $msg : gmdate("Y-m-d H:i", intval
$out_of_date_time = ($row["OutOfDateTS"] == 0) ? $msg : gmdate("Y-m-d", intval($row["OutOfDateTS"]));
$pkgs = pkgbase_get_pkgnames($base_id);
+
+$base_uri = get_pkgbase_uri($row['Name']);
+
?>
<div id="pkgdetails" class="box">
<h2><?= __('Package Base Details') . ': ' . htmlspecialchars($row['Name']) ?></h2>
@@ -44,75 +53,40 @@ $pkgs = pkgbase_get_pkgnames($base_id);
<li><span class="flagged"><?php if ($row["OutOfDateTS"] !== NULL) { echo __('Flagged out-of-date')." (${out_of_date_time})"; } ?></span></li>
<?php if ($uid): ?>
<?php if ($row["OutOfDateTS"] === NULL): ?>
- <li>
- <form action="<?= get_pkgbase_uri($row['Name']) . 'flag/'; ?>" method="post">
- <input type="hidden" name="token" value="<?= htmlspecialchars($_COOKIE['AURSID']) ?>" />
- <input type="submit" class="button text-button" name="do_Flag" value="<?= __('Flag package out-of-date') ?>" />
- </form>
- </li>
- <?php elseif (($row["OutOfDateTS"] !== NULL) && has_credential(CRED_PKGBASE_UNFLAG, array($row["MaintainerUID"]))): ?>
- <li>
- <form action="<?= get_pkgbase_uri($row['Name']) . 'unflag/'; ?>" method="post">
- <input type="hidden" name="token" value="<?= htmlspecialchars($_COOKIE['AURSID']) ?>" />
- <input type="submit" class="button text-button" name="do_UnFlag" value="<?= __('Unflag package') ?>" />
- </form>
- </li>
+ <li><?= html_action_form($base_uri . 'flag/', "do_Flag", __('Flag package out-of-date')) ?></li>
+ <?php elseif (($row["OutOfDateTS"] !== NULL) && has_credential(CRED_PKGBASE_UNFLAG, $maintainers)): ?>
+ <li><?= html_action_form($base_uri . 'unflag/', "do_UnFlag", __('Unflag package')) ?></li>
<?php endif; ?>
+
<?php if (pkgbase_user_voted($uid, $base_id)): ?>
- <li>
- <form action="<?= get_pkgbase_uri($row['Name']) . 'unvote/'; ?>" method="post">
- <input type="hidden" name="token" value="<?= htmlspecialchars($_COOKIE['AURSID']) ?>" />
- <input type="submit" class="button text-button" name="do_UnVote" value="<?= __('Remove vote') ?>" />
- </form>
- </li>
+ <li><?= html_action_form($base_uri . 'unvote/', "do_UnVote", __('Remove vote')) ?></li>
<?php else: ?>
- <li>
- <form action="<?= get_pkgbase_uri($row['Name']) . 'vote/'; ?>" method="post">
- <input type="hidden" name="token" value="<?= htmlspecialchars($_COOKIE['AURSID']) ?>" />
- <input type="submit" class="button text-button" name="do_Vote" value="<?= __('Vote for this package') ?>" />
- </form>
- </li>
+ <li><?= html_action_form($base_uri . 'vote/', "do_Vote", __('Vote for this package')) ?></li>
<?php endif; ?>
+
<?php if (pkgbase_user_notify($uid, $base_id)): ?>
- <li>
- <form action="<?= get_pkgbase_uri($row['Name']) . 'unnotify/'; ?>" method="post">
- <input type="hidden" name="token" value="<?= htmlspecialchars($_COOKIE['AURSID']) ?>" />
- <input type="submit" class="button text-button" name="do_UnNotify" value="<?= __('Disable notifications') ?>" />
- </form>
- </li>
+ <li><?= html_action_form($base_uri . 'unnotify/', "do_UnNotify", __('Disable notifications')) ?></li>
<?php else: ?>
- <li>
- <form action="<?= get_pkgbase_uri($row['Name']) . 'notify/'; ?>" method="post">
- <input type="hidden" name="token" value="<?= htmlspecialchars($_COOKIE['AURSID']) ?>" />
- <input type="submit" class="button text-button" name="do_Notify" value="<?= __('Notify of new comments') ?>" />
- </form>
- </li>
+ <li><?= html_action_form($base_uri . 'notify/', "do_Notify", __('Notify of new comments')) ?></li>
<?php endif; ?>
+
<?php if (has_credential(CRED_PKGBASE_EDIT_COMAINTAINERS, array($row["MaintainerUID"]))): ?>
- <li><a href="<?= get_pkgbase_uri($row['Name']) . 'comaintainers/'; ?>"><?= __('Manage Co-Maintainers'); ?></a></li>
+ <li><?= html_action_link($base_uri . 'comaintainers/', __('Manage Co-Maintainers')) ?></a></li>
<?php endif; ?>
+
<li><span class="flagged"><?php if ($row["RequestCount"] > 0) { echo _n('%d pending request', '%d pending requests', $row["RequestCount"]); } ?></span></li>
- <li><a href="<?= get_pkgbase_uri($row['Name']) . 'request/'; ?>"><?= __('File Request'); ?></a></li>
+ <li><?= html_action_link($base_uri . 'request/', __('File Request')) ?></a></li>
+
<?php if (has_credential(CRED_PKGBASE_DELETE)): ?>
- <li><a href="<?= get_pkgbase_uri($row['Name']) . 'delete/'; ?>"><?= __('Delete Package'); ?></a></li>
- <li><a href="<?= get_pkgbase_uri($row['Name']) . 'merge/'; ?>"><?= __('Merge Package'); ?></a></li>
- <?php endif; ?>
+ <li><?= html_action_link($base_uri . 'delete/', __('Delete Package')) ?></a></li>
+ <li><?= html_action_link($base_uri . 'merge/', __('Merge Package')) ?></a></li>
<?php endif; ?>
<?php if ($uid && $row["MaintainerUID"] === NULL): ?>
- <li>
- <form action="<?= get_pkgbase_uri($row['Name']) . 'adopt/'; ?>" method="post">
- <input type="hidden" name="token" value="<?= htmlspecialchars($_COOKIE['AURSID']) ?>" />
- <input type="submit" class="button text-button" name="do_Adopt" value="<?= __('Adopt Package') ?>" />
- </form>
- </li>
+ <li><?= html_action_form($base_uri . 'adopt/', "do_Adopt", __('Adopt Package')) ?></li>
<?php elseif (has_credential(CRED_PKGBASE_DISOWN, array($row["MaintainerUID"]))): ?>
- <li>
- <form action="<?= get_pkgbase_uri($row['Name']) . 'disown/'; ?>" method="post">
- <input type="hidden" name="token" value="<?= htmlspecialchars($_COOKIE['AURSID']) ?>" />
- <input type="submit" class="button text-button" name="do_Disown" value="<?= __('Disown Package') ?>" />
- </form>
- </li>
+ <li><?= html_action_form($base_uri . 'disown/', "do_Disown", __('Disown Package')) ?></li>
+ <?php endif; ?>
<?php endif; ?>
</ul>
</div>
--
2.4.4
More information about the aur-dev
mailing list