diff options
author | canyonknight <canyonknight@gmail.com> | 2012-05-23 19:47:27 +0200 |
---|---|---|
committer | Lukas Fleischer <archlinux@cryptocrack.de> | 2012-07-06 11:26:23 +0200 |
commit | d8b2eb4b628e8927b0bed1b254c996520de95b83 (patch) | |
tree | 38d9030a747f05b7144d7931812bec8d7b235ea5 | |
parent | 941f310068be4eac7d8a8679944a04495f67461e (diff) | |
download | aur-d8b2eb4b628e8927b0bed1b254c996520de95b83.tar.gz aur-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.php | 77 |
1 files changed, 37 insertions, 40 deletions
diff --git a/web/template/pkg_comments.php b/web/template/pkg_comments.php index d8346511..4e9dfa35 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&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.= ' /> '; - $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) ?>&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&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'] . '&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'] ?>&comments=all" title="<?php echo __('View all %s comments', $count) ?>"><?php echo __('All comments', $count) ?></a> + </h3> +</div> +<?php endif; ?> |