July 2012 - Aur-dev - lists.archlinux.org

[aur-dev] Introduction and proposal
by Mario Mueller 02 Aug '12

02 Aug '12

Hey folks, my name is Mario Mueller, I'm a web developer from Duesseldorf, Germany. I'm rather new to the Arch Linux community, but I must admit, that I love the distribution already and I want to help the Arch Linux community. I contacted Lukas (cryptocrack) and offered my help by supporting the AUR development with heart and hands. He told me that you guys and gals are willing to relaunch the AUR based on Django - so I started to do so. You can take a look at my proposal at https://bitbucket.org/xenji/aur-django/. I've used the actual PHP code from the AUR git repo as a start. The HTML comes from the archweb repository. I kindly request your feedback on the existing code and requests for the upcoming changes. The test data from the generator script gives a good start for the development, but a dump from the live database, without the Users table, would help me to understand the size of the data. In addition, I kindly ask you for telling me a rough number of visitors / page impression to evaluate the caching mechanism needed for a fast service. Best regards, xenji

5 10

[aur-dev] [PATCH] Move notification logic to separate function
by Nicolas Cornu 28 Jul '12

28 Jul '12

Move notification logic to separate function And optimize database requests in notification --- web/lib/pkgfuncs.inc.php | 58 ++++++++++++++++++++++++++++-------------------- 1 file changed, 34 insertions(+), 24 deletions(-) diff --git a/web/lib/pkgfuncs.inc.php b/web/lib/pkgfuncs.inc.php index 0610617..5420c8f 100644 --- a/web/lib/pkgfuncs.inc.php +++ b/web/lib/pkgfuncs.inc.php @@ -200,24 +200,14 @@ function package_comments($pkgid, $dbh=NULL) { return $comments; } -# Add a comment to a package page and send out appropriate notifications -# TODO: Move notification logic to separate function where it belongs -function add_package_comment($pkgid, $uid, $comment, $dbh=NULL) { +function send_notifications($pkgid, $uid, $subject, $body, $dbh=NULL, $pkg_name="") { if(!$dbh) { $dbh = db_connect(); } - $q = 'INSERT INTO PackageComments '; - $q.= '(PackageID, UsersID, Comments, CommentTS) VALUES ('; - $q.= intval($pkgid) . ', ' . $uid . ', '; - $q.= "'" . db_escape_string($comment) . "', "; - $q.= 'UNIX_TIMESTAMP())'; - db_query($q, $dbh); - # Send email notifications - $q = 'SELECT CommentNotify.*, Users.Email '; - $q.= 'FROM CommentNotify, Users '; - $q.= 'WHERE Users.ID = CommentNotify.UserID '; + $q = 'SELECT Users.Email FROM Users '; + $q.= 'INNER JOIN CommentNotify ON Users.ID=CommentNotify.UserID'; $q.= 'AND CommentNotify.UserID != ' . $uid . ' '; $q.= 'AND CommentNotify.PkgID = ' . intval($pkgid); $result = db_query($q, $dbh); @@ -228,25 +218,45 @@ function add_package_comment($pkgid, $uid, $comment, $dbh=NULL) { array_push($bcc, $row['Email']); } - $q = 'SELECT Packages.* '; - $q.= 'FROM Packages '; - $q.= 'WHERE Packages.ID = ' . intval($pkgid); - $result = db_query($q, $dbh); - $row = mysql_fetch_assoc($result); + if($pkg_name == "") { + $q = 'SELECT Packages.Name '; + $q.= 'FROM Packages '; + $q.= 'WHERE Packages.ID = ' . intval($pkgid); + $result = db_query($q, $dbh); + $row = mysql_fetch_assoc($result); + $pkg_name = $row['Name']; + } # TODO: native language emails for users, based on their prefs # Simply making these strings translatable won't work, users would be # getting emails in the language that the user who posted the comment was in - $body = - 'from ' . $AUR_LOCATION . '/' . get_pkg_uri($row['Name']) . "\n" - . username_from_sid($_COOKIE['AURSID'], $dbh) . " wrote:\n\n" - . $comment - . "\n\n---\nIf you no longer wish to receive notifications about this package, please go the the above package page and click the UnNotify button."; + $body = 'from ' . $AUR_LOCATION . '/' . get_pkg_uri($pkg_name) . "\n" . $body; + $body.= "\n\n---\nIf you no longer wish to receive notifications about this package, please go the the above package page and click the UnNotify button."; $body = wordwrap($body, 70); $bcc = implode(', ', $bcc); $headers = "Bcc: $bcc\nReply-to: nobody(a)archlinux.org\nFrom: aur-notify(a)archlinux.org\nX-Mailer: AUR\n"; - @mail('undisclosed-recipients: ;', "AUR Comment for " . $row['Name'], $body, $headers); + @mail('undisclosed-recipients: ;', $subject . " for " . $pkg_name, $body, $headers); + } +} + +# Add a comment to a package page and send out appropriate notifications +function add_package_comment($pkgid, $uid, $comment, $dbh=NULL) { + if(!$dbh) { + $dbh = db_connect(); } + + $q = 'INSERT INTO PackageComments '; + $q.= '(PackageID, UsersID, Comments, CommentTS) VALUES ('; + $q.= intval($pkgid) . ', ' . $uid . ', '; + $q.= "'" . db_escape_string($comment) . "', "; + $q.= 'UNIX_TIMESTAMP())'; + db_query($q, $dbh); + + # Send notifications + $subject = "AUR Comment"; + $body = username_from_sid($_COOKIE['AURSID'], $dbh) . " wrote:\n\n"; + $body.= $comment; + send_notifications($pkgid, $uid, $subject, $body, $dbh); } # grab package sources -- 1.7.11.3

1 0

[aur-dev] [PATCH 1/2] Optimize database request in the add_package_comment function
by Nicolas Cornu 28 Jul '12

28 Jul '12

--- web/lib/pkgfuncs.inc.php | 11 +++++------ 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/web/lib/pkgfuncs.inc.php b/web/lib/pkgfuncs.inc.php index 0610617..969da11 100644 --- a/web/lib/pkgfuncs.inc.php +++ b/web/lib/pkgfuncs.inc.php @@ -215,11 +215,10 @@ function add_package_comment($pkgid, $uid, $comment, $dbh=NULL) { db_query($q, $dbh); # Send email notifications - $q = 'SELECT CommentNotify.*, Users.Email '; - $q.= 'FROM CommentNotify, Users '; - $q.= 'WHERE Users.ID = CommentNotify.UserID '; - $q.= 'AND CommentNotify.UserID != ' . $uid . ' '; - $q.= 'AND CommentNotify.PkgID = ' . intval($pkgid); + $q = 'SELECT u.Email FROM Users u '; + $q.= 'INNER JOIN CommentNotify cn ON u.ID=cn.UserID'; + $q.= 'AND cn.UserID != ' . $uid . ' '; + $q.= 'AND cn.PkgID = ' . intval($pkgid); $result = db_query($q, $dbh); $bcc = array(); @@ -228,7 +227,7 @@ function add_package_comment($pkgid, $uid, $comment, $dbh=NULL) { array_push($bcc, $row['Email']); } - $q = 'SELECT Packages.* '; + $q = 'SELECT Packages.Name '; $q.= 'FROM Packages '; $q.= 'WHERE Packages.ID = ' . intval($pkgid); $result = db_query($q, $dbh); -- 1.7.11.3

3 5

[aur-dev] [PATCH] Notify users when a package is updated.
by Nicolas Cornu 27 Jul '12

27 Jul '12

Responding to FS #30109 --- web/html/pkgsubmit.php | 33 +++++++++++++++++++++++++++++++++ 1 file changed, 33 insertions(+) diff --git a/web/html/pkgsubmit.php b/web/html/pkgsubmit.php index 84688b4..f7bc430 100644 --- a/web/html/pkgsubmit.php +++ b/web/html/pkgsubmit.php @@ -397,6 +397,39 @@ if ($uid): pkg_notify(account_from_sid($_COOKIE["AURSID"], $dbh), array($packageID), true, $dbh); } + # If it was a previously created package notify users of this update. + if($pdata) { + $uid = uid_from_sid($uid, $dbh); + + # Send email notifications + $q = 'SELECT CommentNotify.*, Users.Email '; + $q.= 'FROM CommentNotify, Users '; + $q.= 'WHERE Users.ID = CommentNotify.UserID '; + $q.= 'AND CommentNotify.UserID != ' . $uid . ' '; + $q.= 'AND CommentNotify.PkgID = ' . intval($packageID); + $result = db_query($q, $dbh); + $bcc = array(); + + if (mysql_num_rows($result)) { + while ($row = mysql_fetch_assoc($result)) { + array_push($bcc, $row['Email']); + } + + # TODO: native language emails for users, based on their prefs + # Simply making these strings translatable won't work, users would be + # getting emails in the language that the user who posted the comment was in + $body = + 'from ' . $AUR_LOCATION . '/' . get_pkg_uri($pkg_name) . "\n" + . username_from_sid($_COOKIE['AURSID'], $dbh) . " update the package to the version: " + . $pkg_version + . "\n\n---\nIf you no longer wish to receive notifications about this package, please go the the above package page and click the UnNotify button."; + $body = wordwrap($body, 70); + $bcc = implode(', ', $bcc); + $headers = "Bcc: $bcc\nReply-to: nobody(a)archlinux.org\nFrom: aur-notify(a)archlinux.org\nX-Mailer: AUR\n"; + @mail('undisclosed-recipients: ;', "AUR Update for " . $pkg_name, $body, $headers); + } + } + # Entire package creation process is atomic end_atomic_commit($dbh); -- 1.7.11.3

2 3

[aur-dev] (no subject)
by Nicolas Cornu 27 Jul '12

27 Jul '12

>From e4ca99407a3efbade5e09a43b4d333ae4daa1e5b Mon Sep 17 00:00:00 2001 From: Nicolas Cornu <nicolac76(a)yahoo.fr> Date: Fri, 27 Jul 2012 22:08:55 +0200 Subject: [PATCH] Notify users when a package is updated. It's a patch to send notify when someone update a package. --- web/html/pkgsubmit.php | 33 +++++++++++++++++++++++++++++++++ 1 file changed, 33 insertions(+) diff --git a/web/html/pkgsubmit.php b/web/html/pkgsubmit.php index 84688b4..f7bc430 100644 --- a/web/html/pkgsubmit.php +++ b/web/html/pkgsubmit.php @@ -397,6 +397,39 @@ if ($uid): pkg_notify(account_from_sid($_COOKIE["AURSID"], $dbh), array($packageID), true, $dbh); } + # If it was a previously created package notify users of this update. + if($pdata) { + $uid = uid_from_sid($uid, $dbh); + + # Send email notifications + $q = 'SELECT CommentNotify.*, Users.Email '; + $q.= 'FROM CommentNotify, Users '; + $q.= 'WHERE Users.ID = CommentNotify.UserID '; + $q.= 'AND CommentNotify.UserID != ' . $uid . ' '; + $q.= 'AND CommentNotify.PkgID = ' . intval($packageID); + $result = db_query($q, $dbh); + $bcc = array(); + + if (mysql_num_rows($result)) { + while ($row = mysql_fetch_assoc($result)) { + array_push($bcc, $row['Email']); + } + + # TODO: native language emails for users, based on their prefs + # Simply making these strings translatable won't work, users would be + # getting emails in the language that the user who posted the comment was in + $body = + 'from ' . $AUR_LOCATION . '/' . get_pkg_uri($pkg_name) . "\n" + . username_from_sid($_COOKIE['AURSID'], $dbh) . " update the package to the version: " + . $pkg_version + . "\n\n---\nIf you no longer wish to receive notifications about this package, please go the the above package page and click the UnNotify button."; + $body = wordwrap($body, 70); + $bcc = implode(', ', $bcc); + $headers = "Bcc: $bcc\nReply-to: nobody(a)archlinux.org\nFrom: aur-notify(a)archlinux.org\nX-Mailer: AUR\n"; + @mail('undisclosed-recipients: ;', "AUR Update for " . $pkg_name, $body, $headers); + } + } + # Entire package creation process is atomic end_atomic_commit($dbh); -- 1.7.11.3

1 0

[aur-dev] [PATCH 0/6] [RFC] Virtual path feature
by Lukas Fleischer 18 Jul '12

18 Jul '12

This implements nicer looking URLs, such as "/packages/" instead of "/packages.php", "/packages/$pkgname/" instead of "/packages.php?ID=$id" etc. Note that these patches are not ready to be merged yet but I'd like to get some comments. We'd probably also have to add rewrite rules to our AUR setup on sigurd or add legacy rules to the routing back end to ensure old URLs still work. Lukas Fleischer (6): Add routing front/back ends Use absolute URIs for style sheets and images Use virtual path feature for links Use virtual paths for package details Redirect to the home page after successful login web/html/pkgsubmit.php: Remove stray "</div>" web/html/addvote.php | 2 +- web/html/home.php | 123 ++++++++++++++++++++++++ web/html/index.php | 160 ++++++++------------------------ web/html/login.php | 4 +- web/html/logout.php | 2 +- web/html/passreset.php | 6 +- web/html/pkgsubmit.php | 6 +- web/html/tu.php | 8 +- web/html/voters.php | 4 +- web/lib/acctfuncs.inc.php | 6 +- web/lib/config.inc.php.proto | 4 + web/lib/pkgfuncs.inc.php | 7 +- web/lib/routing.inc.php | 57 ++++++++++++ web/template/account_details.php | 2 +- web/template/account_edit_form.php | 2 +- web/template/account_search_results.php | 8 +- web/template/actions_form.php | 2 +- web/template/header.php | 26 +++--- web/template/pkg_comments.php | 8 +- web/template/pkg_details.php | 14 +-- web/template/pkg_search_form.php | 2 +- web/template/pkg_search_results.php | 10 +- web/template/search_accounts_form.php | 2 +- web/template/stats/updates_table.php | 2 +- web/template/stats/user_table.php | 4 +- web/template/tu_details.php | 4 +- web/template/tu_list.php | 4 +- 27 files changed, 291 insertions(+), 188 deletions(-) create mode 100644 web/html/home.php create mode 100644 web/lib/routing.inc.php -- 1.7.11.2

3 11

[aur-dev] [PATCH] Use virtual paths for voters page
by Lukas Fleischer 18 Jul '12

18 Jul '12

Link to "/packages/$pkgname/voters/" instead of using "/voters/" and a get parameter to request a specific package by ID. Signed-off-by: Lukas Fleischer <archlinux(a)cryptocrack.de> --- web/html/index.php | 6 ++++++ web/lib/routing.inc.php | 1 - web/template/pkg_details.php | 10 ++++++---- 3 files changed, 12 insertions(+), 5 deletions(-) diff --git a/web/html/index.php b/web/html/index.php index 3539c5a..df0d133 100644 --- a/web/html/index.php +++ b/web/html/index.php @@ -13,6 +13,12 @@ if (isset($tokens[1]) && '/' . $tokens[1] == get_pkg_route()) { $_GET['N'] = $tokens[2]; if (isset($tokens[3])) { + if ($tokens[3] == 'voters') { + $_GET['ID'] = pkgid_from_name($tokens[2]); + include('voters.php'); + return; + } + /* TODO: Remove support for legacy URIs and move these * actions to separate modules. */ switch ($tokens[3]) { diff --git a/web/lib/routing.inc.php b/web/lib/routing.inc.php index 5cffe81..4ca1150 100644 --- a/web/lib/routing.inc.php +++ b/web/lib/routing.inc.php @@ -14,7 +14,6 @@ $ROUTES = array( '/rss' => 'rss.php', '/submit' => 'pkgsubmit.php', '/tu' => 'tu.php', - '/voters' => 'voters.php', '/addvote' => 'addvote.php', ); diff --git a/web/template/pkg_details.php b/web/template/pkg_details.php index 774d670..67cb5d1 100644 --- a/web/template/pkg_details.php +++ b/web/template/pkg_details.php @@ -143,10 +143,12 @@ if ($row["MaintainerUID"]): </tr> <tr> <th><?php echo __('Votes') . ': ' ?></th> -<?php -if ($atype == "Developer" || $atype == "Trusted User"): -?> - <td><a href="<?php echo get_uri('/voters/'); ?>?ID=<?php echo$pkgid ?>"><?php echo $votes ?></a> +<?php if ($atype == "Developer" || $atype == "Trusted User"): ?> +<?php if ($USE_VIRTUAL_URLS): ?> + <td><a href="<?php echo get_pkg_uri($row['Name']); ?>voters/"><?php echo $votes ?></a> +<?php else: ?> + <td><a href="<?php echo get_uri('/voters/'); ?>?ID=<?php echo $pkgid ?>"><?php echo $votes ?></a> +<?php endif; ?> <?php else: ?> <td><?php echo $votes ?></td> <?php endif; ?> -- 1.7.11.2

1 0

[aur-dev] [PATCH 1/2] voters.php: Use "<ul></ul>" for the list of voters
by Lukas Fleischer 18 Jul '12

18 Jul '12

Signed-off-by: Lukas Fleischer <archlinux(a)cryptocrack.de> --- web/html/voters.php | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/web/html/voters.php b/web/html/voters.php index c7c59a2..0ec77ec 100644 --- a/web/html/voters.php +++ b/web/html/voters.php @@ -17,11 +17,11 @@ if ($atype == 'Trusted User' || $atype== 'Developer'): <div class="box"> <h2>Votes for <a href="<?php echo get_pkg_uri(pkgname_from_id($pkgid)); ?>"><?php echo pkgname_from_id($pkgid) ?></a></h2> <div class="boxbody"> - -<?php - while (list($indx, $row) = each($votes)): ?> - <a href="<?php echo get_uri('/account/'); ?>?Action=AccountInfo&ID=<?php echo $row['UsersID'] ?>"><?php echo htmlspecialchars($row['Username']) ?></a><br /> - <?php endwhile; ?> + <ul> + <?php while (list($indx, $row) = each($votes)): ?> + <li><a href="<?php echo get_uri('/account/'); ?>?Action=AccountInfo&ID=<?php echo $row['UsersID'] ?>"><?php echo htmlspecialchars($row['Username']) ?></a></li> + <?php endwhile; ?> + </ul> </div> </div> -- 1.7.11.2

1 1

[aur-dev] [PATCH 1/5] routing.inc.php: Add missing "/account" route
by Lukas Fleischer 18 Jul '12

18 Jul '12

Minor oversight when adding the routing back end in commit eb22bcc7548d1b1025a3832aa4555792729d11b0. Signed-off-by: Lukas Fleischer <archlinux(a)cryptocrack.de> --- web/lib/routing.inc.php | 1 + 1 file changed, 1 insertion(+) diff --git a/web/lib/routing.inc.php b/web/lib/routing.inc.php index bbcb9e2..5cffe81 100644 --- a/web/lib/routing.inc.php +++ b/web/lib/routing.inc.php @@ -5,6 +5,7 @@ $ROUTES = array( '/index.php' => 'home.php', '/packages' => 'packages.php', '/register' => 'account.php', + '/account' => 'account.php', '/accounts' => 'account.php', '/login' => 'login.php', '/logout' => 'logout.php', -- 1.7.11.2

1 4

[aur-dev] [PATCH 1/2] aur.inc.php: Include routing backend
by Lukas Fleischer 17 Jul '12

17 Jul '12

This needs to be available on every page since it is needed for get_uri() and get_pkg_uri() and "web/html/index.php" isn't executed if the virtual path feature is disabled. Reported-by: canyonknight <canyonknight(a)gmail.com> Signed-off-by: Lukas Fleischer <archlinux(a)cryptocrack.de> --- web/html/index.php | 2 -- web/lib/aur.inc.php | 1 + 2 files changed, 1 insertion(+), 2 deletions(-) diff --git a/web/html/index.php b/web/html/index.php index 0af3f2d..1561168 100644 --- a/web/html/index.php +++ b/web/html/index.php @@ -1,8 +1,6 @@ <?php set_include_path(get_include_path() . PATH_SEPARATOR . '../lib'); -include_once("config.inc.php"); -include_once("routing.inc.php"); include_once("aur.inc.php"); include_once("pkgfuncs.inc.php"); diff --git a/web/lib/aur.inc.php b/web/lib/aur.inc.php index 1a6164e..6dcbb34 100644 --- a/web/lib/aur.inc.php +++ b/web/lib/aur.inc.php @@ -11,6 +11,7 @@ include_once('translator.inc.php'); set_lang(); include_once("config.inc.php"); +include_once("routing.inc.php"); include_once("version.inc.php"); include_once("acctfuncs.inc.php"); include_once("cachefuncs.inc.php"); -- 1.7.11.2

1 1

[aur-dev] [PATCH 1/7] Add virtual path support for package actions
by Lukas Fleischer 17 Jul '12

17 Jul '12

This allows for using following URLs: * /package/$pkg_name/flag: Flag a package out-of-date * /package/$pkg_name/unflag: Unflag a package * /package/$pkg_name/notify: Enable comment notifications * /package/$pkg_name/unnotify: Disable comment notifications * /package/$pkg_name/vote: Vote for the package * /package/$pkg_name/unvote: Remove vote Note that this code is very hackish and should be refactored once we drop support for legacy URLs. Signed-off-by: Lukas Fleischer <archlinux(a)cryptocrack.de> --- web/html/index.php | 32 +++++++++++++++++++++++++++++++- 1 file changed, 31 insertions(+), 1 deletion(-) diff --git a/web/html/index.php b/web/html/index.php index de38178..0af3f2d 100644 --- a/web/html/index.php +++ b/web/html/index.php @@ -3,14 +3,44 @@ set_include_path(get_include_path() . PATH_SEPARATOR . '../lib'); include_once("config.inc.php"); include_once("routing.inc.php"); +include_once("aur.inc.php"); +include_once("pkgfuncs.inc.php"); $path = rtrim($_SERVER['PATH_INFO'], '/'); $tokens = explode('/', $path); -if (isset($tokens[1]) &&'/' . $tokens[1] == get_pkg_route()) { +if (isset($tokens[1]) && '/' . $tokens[1] == get_pkg_route()) { if (isset($tokens[2])) { unset($_GET['ID']); $_GET['N'] = $tokens[2]; + + if (isset($tokens[3])) { + /* TODO: Remove support for legacy URIs and move these + * actions to separate modules. */ + switch ($tokens[3]) { + case "vote": + $_POST['do_Vote'] = __('Vote'); + break; + case "unvote": + $_POST['do_UnVote'] = __('UnVote'); + break; + case "notify": + $_POST['do_Notify'] = __('Notify'); + break; + case "unnotify": + $_POST['do_UnNotify'] = __('UnNotify'); + break; + case "flag": + $_POST['do_Flag'] = __('Flag'); + break; + case "unflag": + $_POST['do_UnFlag'] = __('UnFlag'); + break; + } + + $_POST['token'] = $_COOKIE['AURSID']; + $_POST['IDs'] = array(pkgid_from_name($tokens[2]) => '1'); + } } include get_route('/' . $tokens[1]); -- 1.7.11.2

1 6

[aur-dev] [PATCH 1/2] pkg_details.php: Fix link to voters if ID is unset
by Lukas Fleischer 17 Jul '12

17 Jul '12

When a package is requested by name, the "ID" request parameter might be unset. Use the "ID" field of the query result instead. Signed-off-by: Lukas Fleischer <archlinux(a)cryptocrack.de> --- web/template/pkg_details.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/web/template/pkg_details.php b/web/template/pkg_details.php index b4b52d5..6e19b3b 100644 --- a/web/template/pkg_details.php +++ b/web/template/pkg_details.php @@ -2,7 +2,7 @@ $atype = account_from_sid($SID); $uid = uid_from_sid($SID); -$pkgid = intval($_REQUEST['ID']); +$pkgid = intval($row['ID']); $catarr = pkgCategories(); -- 1.7.11.2

1 1

[aur-dev] [PATCH] pkgsubmit.php: Refactor source processing to fix PHP notice
by canyonknight 14 Jul '12

14 Jul '12

A foreach() being run without checking to see if the uploaded PKGBUILD had any sources now no longer causes an undefined index notice when there are no sources. Signed-off-by: canyonknight <canyonknight(a)gmail.com> --- web/html/pkgsubmit.php | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/web/html/pkgsubmit.php b/web/html/pkgsubmit.php index c2a7191..737812e 100644 --- a/web/html/pkgsubmit.php +++ b/web/html/pkgsubmit.php @@ -379,9 +379,9 @@ if ($uid): } # Insert sources - $sources = explode(" ", $new_pkgbuild['source']); - foreach ($sources as $src) { - if ($src != "" ) { + if (!empty($new_pkgbuild['source'])) { + $sources = explode(" ", $new_pkgbuild['source']); + foreach ($sources as $src) { add_pkg_src($packageID, $src, $dbh); } } -- 1.7.11.2

1 0

[aur-dev] [PATCH] aur.inc.php: Use delete_session_id() where appropriate
by canyonknight 14 Jul '12

14 Jul '12

Signed-off-by: canyonknight <canyonknight(a)gmail.com> --- web/lib/aur.inc.php | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/web/lib/aur.inc.php b/web/lib/aur.inc.php index f82a96a..5a70e77 100644 --- a/web/lib/aur.inc.php +++ b/web/lib/aur.inc.php @@ -52,9 +52,7 @@ function check_sid($dbh=NULL) { } elseif ($failed == 2) { # session id timeout was reached and they must login again. # - $q = "DELETE FROM Sessions WHERE SessionID = '"; - $q.= db_escape_string($_COOKIE["AURSID"]) . "'"; - db_query($q, $dbh); + delete_session_id($_COOKIE["AURSID"], $dbh); setcookie("AURSID", "", 1, "/", null, !empty($_SERVER['HTTPS']), true); unset($_COOKIE['AURSID']); -- 1.7.11.2

1 0

[aur-dev] [PATCH] tu.php: Fix PHP notice for "hasvoted" variable
by canyonknight 11 Jul '12

11 Jul '12

Signed-off-by: canyonknight <canyonknight(a)gmail.com> --- web/html/tu.php | 1 + 1 file changed, 1 insertion(+) diff --git a/web/html/tu.php b/web/html/tu.php index 5537d3a..48cd6c1 100644 --- a/web/html/tu.php +++ b/web/html/tu.php @@ -34,6 +34,7 @@ if ($atype == "Trusted User" || $atype == "Developer") { $whovoted = voter_list($row['ID']); $canvote = 1; + $hasvoted = 0; $errorvote = ""; if ($isrunning == 0) { $canvote = 0; -- 1.7.11.1

2 1

[aur-dev] [PATCH] pkgsubmit.php: Remove stray "</div>" tag
by canyonknight 11 Jul '12

11 Jul '12

Signed-off-by: canyonknight <canyonknight(a)gmail.com> --- web/html/pkgsubmit.php | 1 - 1 file changed, 1 deletion(-) diff --git a/web/html/pkgsubmit.php b/web/html/pkgsubmit.php index 2fd21e6..c2a7191 100644 --- a/web/html/pkgsubmit.php +++ b/web/html/pkgsubmit.php @@ -477,7 +477,6 @@ else: endif; ?> - </div> </div> <?php -- 1.7.11.1

2 1

[aur-dev] [PATCH 1/2] pkg_details.php: Fix dependency links
by Lukas Fleischer 09 Jul '12

09 Jul '12

Remove an erroneous quote from the destination URL of links to officially supported package dependencies on archweb. Signed-off-by: Lukas Fleischer <archlinux(a)cryptocrack.de> --- web/template/pkg_details.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/web/template/pkg_details.php b/web/template/pkg_details.php index 05a20fe..67110fa 100644 --- a/web/template/pkg_details.php +++ b/web/template/pkg_details.php @@ -138,7 +138,7 @@ if ($atype == "Developer" || $atype == "Trusted User"): ?> <li><a href="packages.php?ID=<?php echo $darr[2]?>" title="<?php echo __('View packages details for').' '.$darr[0].$darr[1]?>"><?php echo $darr[0].$darr[1]?></a></li> <?php else: ?> - <li><a href="http://www.archlinux.org/packages/?q="<?php echo urlencode($darr[0])?>" title="<?php echo __('View packages details for').' '.$darr[0].$darr[1] ?>"><?php echo $darr[0].$darr[1] ?></a></li> + <li><a href="http://www.archlinux.org/packages/?q=<?php echo urlencode($darr[0])?>" title="<?php echo __('View packages details for').' '.$darr[0].$darr[1] ?>"><?php echo $darr[0].$darr[1] ?></a></li> <?php endif; ?> <?php endwhile; ?> </ul> -- 1.7.11.1

1 1

[aur-dev] [PATCH] Update archweb stylesheet
by Lukas Fleischer 09 Jul '12

09 Jul '12

Pull in the latest changes from the archweb Git repository. Signed-off-by: Lukas Fleischer <archlinux(a)cryptocrack.de> --- web/html/css/archweb.css | 170 +++++++++++++++++++++++++++++++++-------------- 1 file changed, 119 insertions(+), 51 deletions(-) diff --git a/web/html/css/archweb.css b/web/html/css/archweb.css index 303173f..cafae77 100644 --- a/web/html/css/archweb.css +++ b/web/html/css/archweb.css @@ -85,6 +85,10 @@ input[type=submit] { clear: both; } +.hide { + display: none; +} + hr { border: none; border-top: 1px solid #888; @@ -122,6 +126,16 @@ a { color: #e90; } +/* special anchor elements */ +a.headerlink { + visibility: hidden; + padding-left: 0.5em; +} + +h3:hover > a.headerlink { + visibility: visible; +} + /* headings */ h2 { font-size: 1.5em; @@ -258,6 +272,23 @@ table.pretty2 { border: 1px solid #bbb; } + /* additional styles for JS sorting */ + table.pretty2 th.header { + padding-right: 20px; + background-image: url(nosort.gif); + background-repeat: no-repeat; + background-position: center right; + cursor: pointer; + } + + table.pretty2 th.headerSortDown { + background-image: url(desc.gif); + } + + table.pretty2 th.headerSortUp { + background-image: url(asc.gif); + } + table.pretty2 td { padding: 0.35em; border: 1px dotted #bbb; @@ -481,15 +512,11 @@ h3 span.arrow { text-align: right; } - #pkg-updates span.testing, - #pkg-updates span.community-testing, - span.multilib-testing { + #pkg-updates span.testing { font-style: italic; } - #pkg-updates span.staging, - #pkg-updates span.community-staging, - span.multilib-staging { + #pkg-updates span.staging { font-style: italic; color: #ff8040; } @@ -575,25 +602,6 @@ table#download-torrents .cpu-arch { text-align: center; } -table#download-mirrors { - width: auto; - margin-bottom: 1em; -} - - table#download-mirrors td.mirror-country { - padding-top: 1em; - } - - table#download-mirrors td.mirror-server { - padding-right: 1em; - } - - table#download-mirrors a { - display: block; - float: right; - width: 4em; - } - /* pkglists/devlists */ table.results { font-size: 0.846em; @@ -649,8 +657,7 @@ div#pkglist-about { } /* pkglist: results navigation */ -#pkglist-stats-top, -#pkglist-stats-bottom { +.pkglist-stats { font-size: 0.85em; } @@ -668,9 +675,13 @@ div#pkglist-about { } /* search fields and other filter selections */ +.filter-criteria { + margin-bottom: 1em; +} + .filter-criteria h3 { font-size: 1em; - margin-top:0; + margin-top: 0; } .filter-criteria div { @@ -728,6 +739,11 @@ div#pkglist-about { padding: 0.25em 0 0.25em 1.5em; } + #pkgdetails #pkginfo .userdata { + font-size: 0.85em; + padding: 0.5em; + } + /* pkgdetails: flag package */ form#flag-pkg-form label { width: 10em; @@ -795,6 +811,28 @@ form#flag-pkg-form input[type=text] { padding-top: 1em; } + #pkgdetails #pkgfiles li.d { + color: #666; + } + + #pkgdetails #pkgfiles li.f { + } + +/* mirror stuff */ +table td.country { + white-space: normal; +} + +form#list-generator div ul { + list-style: none; + display: inline; + padding-left: 0; +} + + form#list-generator div ul li { + display: inline; + } + /* dev/TU biographies */ div#arch-bio-toc { width: 75%; @@ -802,33 +840,50 @@ div#arch-bio-toc { text-align: center; } -table.arch-bio-entry td.pic { - vertical-align: top; - padding-right: 15px; - padding-top: 10px; -} - - table.arch-bio-entry td.pic img { - padding: 4px; - border: 1px solid #ccc; + div#arch-bio-toc a { + white-space: nowrap; } -table.arch-bio-entry table.bio { - margin-bottom: 2em; +table.arch-bio-entry { + width: 75%; + min-width: 640px; + margin: 0 auto; } - table.arch-bio-entry table.bio th { - text-align: left; - padding-right: 0.5em; + table.arch-bio-entry td.pic { vertical-align: top; - white-space: nowrap; + padding-right: 15px; + padding-top: 2.25em; + } + + table.arch-bio-entry td.pic img { + padding: 4px; + border: 1px solid #ccc; + } + + table.arch-bio-entry td h3 { + border-bottom: 1px dotted #ccc; + margin-bottom: 0.5em; } - table.arch-bio-entry table.bio td { - width: 100%; - padding-bottom: 0.25em; + table.arch-bio-entry table.bio { + margin-bottom: 2em; } + table.arch-bio-entry table.bio th { + color: #666; + font-weight: normal; + text-align: right; + padding-right: 0.5em; + vertical-align: top; + white-space: nowrap; + } + + table.arch-bio-entry table.bio td { + width: 100%; + padding-bottom: 0.25em; + } + /* dev: login/out */ table#dev-login { width: auto; @@ -899,26 +954,26 @@ ul.admin-actions { background: #ffd; } -#dev-signoffs ul { +ul.signoff-list { list-style: none; margin: 0; padding: 0; } -#dev-signoffs .signoff-yes { +.signoff-yes { color: green; font-weight: bold; } -#dev-signoffs .signoff-no { +.signoff-no { color: red; } -#dev-signoffs .signoff-bad { +.signoff-bad { color: darkorange; } -#dev-signoffs .signoff-disabled { +.signoff-disabled { color: gray; } @@ -949,6 +1004,14 @@ ul.admin-actions { color: red; } +#key-status .signed-yes { + color: green; +} + +#key-status .signed-no { + color: red; +} + /* highlight current website in the navbar */ #archnavbar.anb-home ul li#anb-home a, #archnavbar.anb-packages ul li#anb-packages a, @@ -982,3 +1045,8 @@ ul.admin-actions { font-size: 0.85em; line-height: 1em; } + +#visualize-keys svg { + width: 100%; + height: 100%; +} -- 1.7.11.1

1 0

[aur-dev] archweb branch merged into master
by Lukas Fleischer 08 Jul '12

08 Jul '12

Hi, guys! This turned out to take much longer than expected, but the archweb branch (>50 patches, that is) has finally been merged into master. There's a few things that still need some fine tuning but it basically pretty much looks like archweb already. Special thanks fly out to canyonknight, who contributed several patches and helped a lot with merging our work together! Cheers, Lukas

1 0