summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorcanyonknight <canyonknight@gmail.com>2012-05-23 13:47:27 -0400
committerLukas Fleischer <archlinux@cryptocrack.de>2012-07-06 11:26:23 +0200
commitd8b2eb4b628e8927b0bed1b254c996520de95b83 (patch)
tree38d9030a747f05b7144d7931812bec8d7b235ea5
parent941f310068be4eac7d8a8679944a04495f67461e (diff)
downloadaurweb-d8b2eb4b628e8927b0bed1b254c996520de95b83.tar.xz
pkg_comments.php: Overhaul to match archweb
* Limit PHP echoing XHTML as much as possible, and use pure XHTML * Switch to alternative syntax in control structures for better readability of inter-mixed XHTML * Remove box for every comment and switch to archweb news post style Signed-off-by: canyonknight <canyonknight@gmail.com> Signed-off-by: Lukas Fleischer <archlinux@cryptocrack.de>
-rw-r--r--web/template/pkg_comments.php77
1 files changed, 37 insertions, 40 deletions
diff --git a/web/template/pkg_comments.php b/web/template/pkg_comments.php
index d834651..4e9dfa3 100644
--- a/web/template/pkg_comments.php
+++ b/web/template/pkg_comments.php
@@ -1,45 +1,42 @@
-<div class="pgbox">
<?php
$uid = uid_from_sid($SID);
-while (list($indx, $carr) = each($comments)) { ?>
- <div class="comment-header"><?php
-
- if ($SID) {
- $carr['UserName'] = "<a href=\"account.php?Action=AccountInfo&amp;ID={$carr['UsersID']}\">{$carr['UserName']}</a>";
- }
-
- $commentHeader = '<p style="display:inline;">' . __('Comment by: %s on %s', $carr['UserName'], gmdate('Y-m-d H:i', $carr['CommentTS'])) . '</p>';
-
- if (canDeleteCommentArray($carr, $atype, $uid)) {
- $durl = '<form method="post" action="packages.php?ID='.$row['ID'].'">';
- $durl.= '<fieldset style="display:inline;">';
- $durl.= '<input type="hidden" name="action" value="do_DeleteComment" />';
- $durl.= '<input type="hidden" name="comment_id" value="'.$carr['ID'].'" />';
- $durl.= '<input type="image" src="images/x.png" ';
- $durl.= ' alt="'.__("Delete comment").'" name="submit" value="1" ';
- $durl.= ' />&nbsp;';
- $durl.= '</fieldset>';
-
- $commentHeader = $durl.$commentHeader."</form>";
- }
-
- echo $commentHeader;
-?></div>
- <blockquote class="comment-body">
- <div>
-<?php echo parse_comment($carr['Comments']) ?>
- </div>
- </blockquote>
-<?php
-}
+$count = package_comments_count($_GET['ID']);
?>
+<div id="news">
+ <h3>
+ <a href="<?php echo htmlentities($_SERVER['REQUEST_URI'], ENT_QUOTES) ?>&amp;comments=all" title="<?php echo __('View all %s comments' , $count) ?>"><?php echo __('Latest Comments') ?></a>
+ <span class="arrow"></span>
+ </h3>
+
+ <?php while (list($indx, $row) = each($comments)): ?>
+ <?php if ($SID):
+ $row['UserName'] = "<a href=\"account.php?Action=AccountInfo&amp;ID={$row['UsersID']}\">{$row['UserName']}</a>";
+ endif; ?>
+ <h4>
+ <?php if (canDeleteCommentArray($row, $atype, $uid)): ?>
+ <form method="post" action="packages.php?ID=<?php echo $row['ID'] ?>">
+ <fieldset style="display:inline;">
+ <input type="hidden" name="action" value="do_DeleteComment" />
+ <input type="hidden" name="comment_id" value="<?php echo $row['ID'] ?>" />
+ <input type="image" src="images/x.png" alt="<?php echo __('Delete comment') ?> name="submit" value="1" />
+ </fieldset>
+ </form>
+ <?php endif; ?>
+ <?php echo __('Comment by %s', $row['UserName']) ?>
+ </h4>
+ <p class="timestamp"><?php echo gmdate('Y-m-d H:i', $row['CommentTS']) ?></p>
+ <div class="article-content">
+ <p>
+ <?php echo parse_comment($row['Comments']) ?>
+ </p>
+ </div>
+ <?php endwhile; ?>
</div>
-<?php
-$count = package_comments_count(isset($_GET['ID']) ? $_GET['ID'] : pkgid_from_name($_GET['N']));
-if ($count > 10 && !isset($_GET['comments'])) {
- echo '<div class="pgbox">';
- echo '<a href="'. $_SERVER['REQUEST_URI'] . '&amp;comments=all">'. __('Show all %s comments', $count) . '</a>';
- echo '</div>';
-}
-?>
+<?php if ($count > 10 && !isset($_GET['comments'])): ?>
+<div id="news">
+ <h3>
+ <a href="<?php echo $_SERVER['REQUEST_URI'] ?>&amp;comments=all" title="<?php echo __('View all %s comments', $count) ?>"><?php echo __('All comments', $count) ?></a>
+ </h3>
+</div>
+<?php endif; ?>