- Aur-dev - lists.archlinux.org

[aur-dev] [PATCH] Make external links in comments clickable (FS#20137).
by Lukas Fleischer 25 Jan '11

25 Jan '11

This is a bit hacky patch to make links in AUR comments clickable (fixes FS#20137 [1]). Huge parts of this code are ripped from the DokuWiki plugin that is also used in Flyspray. I didn't have any time to test it extensively so I'd suggest to do some more tests if this will be commited. [1] https://bugs.archlinux.org/task/20137 --- web/lib/aur.inc | 45 +++++++++++++++++++++++++++++++++++++++++ web/template/pkg_comments.php | 2 +- 2 files changed, 46 insertions(+), 1 deletions(-) diff --git a/web/lib/aur.inc b/web/lib/aur.inc index bd69c4c..b0cfdc8 100644 --- a/web/lib/aur.inc +++ b/web/lib/aur.inc @@ -494,3 +494,48 @@ function salted_hash($passwd, $salt) } return md5($salt . $passwd); } + +function parse_link($matches) +{ + $name = $url = $matches[0]; + + if(substr($url, 0, 3) == 'ftp' && (substr($url, 0, 6) != 'ftp://')) { + $url = 'ftp://'.$url; + } + elseif (substr($url, 0, 3) == 'www') { + $url = 'http://'.$url; + } + + $url = str_replace('&', '&', $url); + $url = str_replace('&amp;', '&', $url); + $url = strtr($url, array('>' => '%3E', '<' => '%3C', '"' => '%22')); + + return '<a href="' . $url . '">' . $name . '</a>'; +} + +function parse_comment($comment) +{ + $schemes = array('http', 'https', 'ftp'); + $ltrs = '\w'; + $gunk = '\/\#~:.?+=&%@!\-'; + $punc = '.:?\-;,'; + $host = $ltrs . $punc; + $any = $ltrs . $gunk . $punc; + + $patterns = array(); + + foreach ($schemes as $scheme) { + $patterns[] = '(\b(?i)' . $scheme . '(?-i):\/\/[' . $any . ']+?(?=[' . $punc . ']*[^' . $any . ']))'; + } + + $patterns[] = '(\b(?i)www?(?-i)\.[' . $host . ']+?\.[' . $host . ']+?[' . $any . ']+?(?=[' . $punc . ']*[^' . $any . ']))'; + $patterns[] = '(\b(?i)ftp?(?-i)\.['. $host . ']+?\.[' . $host . ']+?[' . $any . ']+?(?=[' . $punc . ']*[^' . $any . ']))'; + + $regex = '/' . implode('|', $patterns) . '/msS'; + + $comment = htmlspecialchars($comment); + $comment = preg_replace_callback($regex, parse_link, $comment . "\n"); + $comment = nl2br($comment); + + return $comment; +} diff --git a/web/template/pkg_comments.php b/web/template/pkg_comments.php index 02171a0..2ca9bf0 100644 --- a/web/template/pkg_comments.php +++ b/web/template/pkg_comments.php @@ -20,7 +20,7 @@ while (list($indx, $carr) = each($comments)) { ?> ?></div> <blockquote class="comment-body"> <div> -<?php echo nl2br(htmlspecialchars($carr['Comments'])) ?> +<?php echo parse_comment($carr['Comments']) ?> </div> </blockquote> <?php -- 1.7.3

4 15

Re: [aur-dev] Request to drop keditbookmarks-svn
by Konor John 04 Jan '11

04 Jan '11

keditbookmarks is too small and not active developed application, so i think that keditbookmarks-svn isn't needed in aur. Additionally, PKGBUILD code is too ugly 04.01.2011, 14:24, "Andrea Scarpino" <andrea(a)archlinux.org>: > On Tuesday 04 January 2011 21:18:21 Joker-jar wrote: > >> Today i recieve mail: >>> AUR Comment for keditbookmarks >>> >>> duplicate of kdebase-keditbookmarks. deleted >> So, delete keditbookmarks-svn also. I'm author of package. Thanks > > keditbookmarks-svn is keditbookmarks from SVN not the stable version thus I > didn't delete it. > Are you sure you want to delete it? > > PS. Moving to aur-general which is the right place for these things. > > -- > Andrea Scarpino > Arch Linux Developer

1 0

[aur-dev] Request to drop keditbookmarks-svn
by Joker-jar 04 Jan '11

04 Jan '11

Today i recieve mail: > AUR Comment for keditbookmarks > > duplicate of kdebase-keditbookmarks. deleted So, delete keditbookmarks-svn also. I'm author of package. Thanks

2 1

Re: [aur-dev] Checking packages names against official repos in the AUR when uploading
by Loui Chang 18 Dec '10

18 Dec '10

On Thu 16 Dec 2010 10:35 +0100, Lukas Fleischer wrote: > Hi, Loui! > > I'm recently having a look at FS#12902 [1]. It's about comparing names > to the official repo databases when uploading a package. I think that > would be really easy to implement if we have an additional directory > where we put repo databases and keep them in sync using a `pacman -Sy` > cronjob or something similar. Parsing the databases is very easy using > Archive_Tar (which we already use anyways) as well. Only thing I don't > like about that is the delay caused by mirrors and the cronjob which > will be a problem if we decide to move a package from the official repos > to [unsupported]. > > What do you think about that? Should we implement this at all? :) > > [1] https://bugs.archlinux.org/task/12902 Hello Mr Fleischer! Yeah I think it should be implemented. We can check the local sync database (or point it to any arbitrary pacman database in config.inc) and create dummy entries for packages that are already in the binary repos. To get around the problem of moving packages to [unsupported] we can make the upload restriction exempt for Trusted Users, since they'll probably be the ones moving the packages and should be trusted to check the binary repos before submitting to the AUR. You should send this to the mailing list too! CCd Cheers.

1 0

[aur-dev] [PATCH] Do not display current votes in All Votes
by Andrea Scarpino 05 Dec '10

05 Dec '10

Subject says everything. -- Andrea Scarpino Arch Linux Developer

3 4

[aur-dev] genpopo, translation_tool python2
by jesse jaara 04 Dec '10

04 Dec '10

Make genpopo and translation_tool scripts to use python2 instead of now default python3. Hopely someone will convert these to python3 compliant form. -- (\_ /) copy the bunny to your profile (0.o ) to help him achieve world domination. (> <) come join the dark side. /_|_\ (we have cookies.)

2 1

[aur-dev] [PATCH] Add timestamp when a package is flagged out-of-date (FS#20848).
by Lukas Fleischer 04 Dec '10

04 Dec '10

--- UPGRADING | 4 ++++ support/schema/aur-schema.sql | 1 + web/lang/cs.po | 2 +- web/lang/de.po | 2 +- web/lang/el_GR.po | 2 +- web/lang/fr.po | 2 +- web/lang/he.po | 2 +- web/lang/hr.po | 2 +- web/lang/hu.po | 2 +- web/lang/it.po | 2 +- web/lang/nb_NO.po | 2 +- web/lang/pl.po | 2 +- web/lang/ro.po | 2 +- web/lang/ru.po | 2 +- web/lang/sr.po | 2 +- web/lang/tr.po | 2 +- web/lang/uk.po | 2 +- web/lang/zh_CN.po | 2 +- web/lib/pkgfuncs.inc | 3 ++- web/template/pkg_details.php | 3 ++- 20 files changed, 25 insertions(+), 18 deletions(-) diff --git a/UPGRADING b/UPGRADING index 743f404..99c5d1a 100644 --- a/UPGRADING +++ b/UPGRADING @@ -1,6 +1,10 @@ Upgrading ========= +From 1.7.0 +---------- +ALTER TABLE Packages ADD OutOfDateTS BIGINT UNSIGNED NOT NULL; + From 1.6.0 to 1.7.0 ------------------- ALTER TABLE Users ADD Salt CHAR(32) NOT NULL DEFAULT ''; diff --git a/support/schema/aur-schema.sql b/support/schema/aur-schema.sql index 250d405..15705a3 100644 --- a/support/schema/aur-schema.sql +++ b/support/schema/aur-schema.sql @@ -120,6 +120,7 @@ CREATE TABLE Packages ( LocationID TINYINT UNSIGNED NOT NULL DEFAULT 1, NumVotes INTEGER UNSIGNED NOT NULL DEFAULT 0, OutOfDate TINYINT UNSIGNED DEFAULT 0, + OutOfDateTS BIGINT UNSIGNED NOT NULL, SubmittedTS BIGINT UNSIGNED NOT NULL, ModifiedTS BIGINT UNSIGNED NOT NULL, SubmitterUID INTEGER UNSIGNED NOT NULL DEFAULT 0, -- who submitted it? diff --git a/web/lang/cs.po b/web/lang/cs.po index e430d05..214df00 100644 --- a/web/lang/cs.po +++ b/web/lang/cs.po @@ -15,7 +15,7 @@ global $_t; $_t["Search by"] = "Vyhledat dle"; -$_t["This package has been flagged out of date."] = "BalÃÄek byl oznaÄen jako zastaralÃ½."; +$_t["This package has been flagged out of date"] = "BalÃÄek byl oznaÄen jako zastaralÃ½"; $_t["Voted"] = "HlasovÃ¡no"; diff --git a/web/lang/de.po b/web/lang/de.po index 929e6f8..bbaa16b 100644 --- a/web/lang/de.po +++ b/web/lang/de.po @@ -390,7 +390,7 @@ $_t["You are not allowed to access this area."] = "Es ist Dir nicht erlaubt, auf $_t["Unsafe"] = "Unsicher"; -$_t["This package has been flagged out of date."] = "Dieses Paket wurde als veraltet markiert."; +$_t["This package has been flagged out of date"] = "Dieses Paket wurde als veraltet markiert"; $_t["The above files have been verified (by %h%s%h) and are safe to use."] = "Die obigen Dateien wurden (von %h%s%h) als sicher markiert."; diff --git a/web/lang/el_GR.po b/web/lang/el_GR.po index 5f5c96c..4118c18 100644 --- a/web/lang/el_GR.po +++ b/web/lang/el_GR.po @@ -86,7 +86,7 @@ $_t["Unsupported packages are user produced content. Any use of the provided fil $_t["Search by"] = "ÎÎ½Î±Î¶Î®ÏÎ·ÏÎ· ÎºÎ±ÏÎ¬"; -$_t["This package has been flagged out of date."] = "ÎÏÏÏ ÏÎ¿ ÏÎ±ÎºÎÏÎ¿ ÎÏÎµÎ¹ ÏÎ±ÏÎ±ÎºÏÎ·ÏÎ¹ÏÏÎµÎ¯ ÏÎ±ÏÏÏÎ·Î¼ÎÎ½Î¿."; +$_t["This package has been flagged out of date"] = "ÎÏÏÏ ÏÎ¿ ÏÎ±ÎºÎÏÎ¿ ÎÏÎµÎ¹ ÏÎ±ÏÎ±ÎºÏÎ·ÏÎ¹ÏÏÎµÎ¯ ÏÎ±ÏÏÏÎ·Î¼ÎÎ½Î¿"; $_t["Voted"] = "Î¨Î·ÏÎ¹ÏÎ¼ÎÎ½Î¿"; diff --git a/web/lang/fr.po b/web/lang/fr.po index 596f4af..1ce1896 100644 --- a/web/lang/fr.po +++ b/web/lang/fr.po @@ -405,7 +405,7 @@ $_t["Out of Date"] = "PÃ©rimÃ©"; $_t["Required by"] = "Requis par"; -$_t["This package has been flagged out of date."] = "Ce paquet a Ã©tÃ© marquÃ© comme pÃ©rimÃ©."; +$_t["This package has been flagged out of date"] = "Ce paquet a Ã©tÃ© marquÃ© comme pÃ©rimÃ©"; $_t["Toggle Notify"] = "Suivre/Ne plus suivre"; diff --git a/web/lang/he.po b/web/lang/he.po index d5c19b7..ad0c558 100644 --- a/web/lang/he.po +++ b/web/lang/he.po @@ -14,7 +14,7 @@ global $_t; $_t["Confirm your e-mail address:"] = "×××©××¨ ××ª×××ª ×××××´× ×©××"; -$_t["This package has been flagged out of date."] = "×××××× ××¡××× ×ª ××× ×¢××× ××ª."; +$_t["This package has been flagged out of date"] = "×××××× ××¡××× ×ª ××× ×¢××× ××ª"; $_t["Missing a required field."] = "×©×× ×××¨×× ××¡×¨."; diff --git a/web/lang/hr.po b/web/lang/hr.po index 93028b7..4d46fec 100644 --- a/web/lang/hr.po +++ b/web/lang/hr.po @@ -162,7 +162,7 @@ $_t["Description"] = "Opis"; $_t["unknown"] = "nepoznato"; -$_t["This package has been flagged out of date."] = "Paket je obiljeÅ¾en kao zastarijeli."; +$_t["This package has been flagged out of date"] = "Paket je obiljeÅ¾en kao zastarijeli"; $_t["No New Comment Notification"] = "Nemoj me viÅ¡e obavjeÅ¡tavati o novim komentarima"; diff --git a/web/lang/hu.po b/web/lang/hu.po index e198ac3..54f3a12 100644 --- a/web/lang/hu.po +++ b/web/lang/hu.po @@ -6,7 +6,7 @@ global $_t; $_t["Search by"] = "KeresÃ©s eszerint"; -$_t["This package has been flagged out of date."] = "Ez a csomag elavultnak van jelÃ¶lve."; +$_t["This package has been flagged out of date"] = "Ez a csomag elavultnak van jelÃ¶lve"; $_t["Missing a required field."] = "Egy kÃ¶telezÅ mezÅ megadÃ¡sa hiÃ¡nyzik."; diff --git a/web/lang/it.po b/web/lang/it.po index 4bd0727..5876c45 100644 --- a/web/lang/it.po +++ b/web/lang/it.po @@ -411,7 +411,7 @@ $_t["Required by"] = "Richiesto da"; $_t["The above files have been verified (by %h%s%h) and are safe to use."] = "I file sono stati verificati (da %h%s%h) ed il loro utilizzo Ã¨ da considerarsi sicuro."; -$_t["This package has been flagged out of date."] = "Questo pacchetto Ã¨ stato contrassegnato come non aggiornato."; +$_t["This package has been flagged out of date"] = "Questo pacchetto Ã¨ stato contrassegnato come non aggiornato"; $_t["Toggle Notify"] = "Rimuovi la notifica"; diff --git a/web/lang/nb_NO.po b/web/lang/nb_NO.po index 687e639..1b8b20e 100644 --- a/web/lang/nb_NO.po +++ b/web/lang/nb_NO.po @@ -14,7 +14,7 @@ global $_t; $_t["Search by"] = "SÃ¸k etter"; -$_t["This package has been flagged out of date."] = "Denne pakken har blitt merket som utdatert."; +$_t["This package has been flagged out of date"] = "Denne pakken har blitt merket som utdatert"; $_t["Voted"] = "Stemmet"; diff --git a/web/lang/pl.po b/web/lang/pl.po index 967ab36..e337bb1 100644 --- a/web/lang/pl.po +++ b/web/lang/pl.po @@ -367,7 +367,7 @@ $_t["unknown"] = "nieznana"; $_t["Out of Date"] = "Nieaktualny"; -$_t["This package has been flagged out of date."] = "Ten pakiet zostaÅ oznaczony jako nieaktualny."; +$_t["This package has been flagged out of date"] = "Ten pakiet zostaÅ oznaczony jako nieaktualny"; $_t["Required by"] = "Wymagane przez"; diff --git a/web/lang/ro.po b/web/lang/ro.po index 0118f24..3a41477 100644 --- a/web/lang/ro.po +++ b/web/lang/ro.po @@ -14,7 +14,7 @@ global $_t; $_t["Search by"] = "CautÄ dupÄ"; -$_t["This package has been flagged out of date."] = "Acest pachet a fost marcat ca Neactualizat."; +$_t["This package has been flagged out of date"] = "Acest pachet a fost marcat ca Neactualizat"; $_t["Voted"] = "Votat"; diff --git a/web/lang/ru.po b/web/lang/ru.po index 2382b35..020f230 100644 --- a/web/lang/ru.po +++ b/web/lang/ru.po @@ -389,7 +389,7 @@ $_t["You are not allowed to access this area."] = "ÐÐ¾ÑÑÑÐ¿ ÑÑÐ´Ð° Ð²Ð°Ð¼ $_t["Unsafe"] = "ÐÐµÐ±ÐµÐ·Ð¾Ð¿Ð°Ð½ÑÐ¹"; -$_t["This package has been flagged out of date."] = "ÐÐ°ÐºÐµÑ Ð¾ÑÐ¼ÐµÑÐµÐ½ ÐºÐ°Ðº ÑÑÑÐ°ÑÐµÐ²ÑÐ¸Ð¹."; +$_t["This package has been flagged out of date"] = "ÐÐ°ÐºÐµÑ Ð¾ÑÐ¼ÐµÑÐµÐ½ ÐºÐ°Ðº ÑÑÑÐ°ÑÐµÐ²ÑÐ¸Ð¹"; $_t["The above files have been verified (by %h%s%h) and are safe to use."] = "Ð¤Ð°Ð¹Ð»Ñ Ð¿ÑÐ¾Ð²ÐµÑÐµÐ½Ñ %h%s%h Ð¸ Ð¼Ð¾Ð³ÑÑ Ð±ÑÑÑ Ð±ÐµÐ·Ð¾Ð¿Ð°ÑÐ½Ð¾ Ð¸ÑÐ¿Ð¾Ð»ÑÐ·Ð¾Ð²Ð°Ð½Ñ."; diff --git a/web/lang/sr.po b/web/lang/sr.po index ddd54da..c743728 100644 --- a/web/lang/sr.po +++ b/web/lang/sr.po @@ -14,7 +14,7 @@ global $_t; $_t["Search by"] = "PretraÅ¾uje se"; -$_t["This package has been flagged out of date."] = "Paket je oznaÄen kao zastareo"; +$_t["This package has been flagged out of date"] = "Paket je oznaÄen kao zastareo"; $_t["Missing a required field."] = "Nedostaje neophodno polje."; diff --git a/web/lang/tr.po b/web/lang/tr.po index 5d41840..df608ac 100644 --- a/web/lang/tr.po +++ b/web/lang/tr.po @@ -400,7 +400,7 @@ $_t["Required by"] = "Ä°htiyaÃ§ duyan"; $_t["The above files have been verified (by %h%s%h) and are safe to use."] = "YukarÄ±daki dosyalar doÄrulandÄ± (%h%s%h tarafÄ±ndan) ve kullanÄ±m iÃ§in gÃ¼venli olarak iÅaretlendi."; -$_t["This package has been flagged out of date."] = "Bu paket gÃ¼ncel deÄil olarak iÅaretlendi."; +$_t["This package has been flagged out of date"] = "Bu paket gÃ¼ncel deÄil olarak iÅaretlendi"; $_t["Toggle Notify"] = "Toggle Notify"; diff --git a/web/lang/uk.po b/web/lang/uk.po index 007c30e..3023f78 100644 --- a/web/lang/uk.po +++ b/web/lang/uk.po @@ -170,7 +170,7 @@ $_t["Description"] = "ÐÐ¿Ð¸Ñ"; $_t["unknown"] = "Ð½ÐµÐ²ÑÐ´Ð¾Ð¼Ð¾"; -$_t["This package has been flagged out of date."] = "Ð¦ÑÐ¾Ð¼Ñ Ð¿Ð°ÐºÑÐ½ÐºÑ Ð¿ÑÐ¸Ð·Ð½Ð°ÑÐµÐ½Ð¾ Ð¼ÑÑÐºÑ ÐÐ°ÑÑÐ°ÑÑÐ»Ð¸Ð¹"; +$_t["This package has been flagged out of date"] = "Ð¦ÑÐ¾Ð¼Ñ Ð¿Ð°ÐºÑÐ½ÐºÑ Ð¿ÑÐ¸Ð·Ð½Ð°ÑÐµÐ½Ð¾ Ð¼ÑÑÐºÑ ÐÐ°ÑÑÐ°ÑÑÐ»Ð¸Ð¹"; $_t["No New Comment Notification"] = "ÐÐµÐ¼Ð°Ñ Ð½Ð¾Ð²Ð¸Ñ ÑÐ¿Ð¾Ð²ÑÑÐµÐ½Ñ Ð¿ÑÐ¾ ÐºÐ¾Ð¼ÐµÐ½ÑÐ°ÑÑ"; diff --git a/web/lang/zh_CN.po b/web/lang/zh_CN.po index 19cc4f6..42d0f09 100644 --- a/web/lang/zh_CN.po +++ b/web/lang/zh_CN.po @@ -17,7 +17,7 @@ global $_t; $_t["Search by"] = "æç´¢"; -$_t["This package has been flagged out of date."] = "æ¤åå·²è¢«æ è®°ä¸ºè¿æã"; +$_t["This package has been flagged out of date"] = "æ¤åå·²è¢«æ è®°ä¸ºè¿æã"; $_t["Missing a required field."] = "ç¼ºå°å¿å¡«é¡¹ã"; diff --git a/web/lib/pkgfuncs.inc b/web/lib/pkgfuncs.inc index 5605daa..02c9c1c 100644 --- a/web/lib/pkgfuncs.inc +++ b/web/lib/pkgfuncs.inc @@ -643,7 +643,8 @@ function pkg_flag ($atype, $ids, $action = True) { } $ood = $action ? 1 : 0; - $q = "UPDATE Packages SET OutOfDate = " . $ood; + $q = "UPDATE Packages SET OutOfDate = " . $ood . ","; + $q.= " OutOfDateTS = UNIX_TIMESTAMP()"; $q.= " WHERE ID IN (" . $flag . ")"; db_query($q, $dbh); diff --git a/web/template/pkg_details.php b/web/template/pkg_details.php index 9c76ab3..a0996b9 100644 --- a/web/template/pkg_details.php +++ b/web/template/pkg_details.php @@ -35,6 +35,7 @@ $license = empty($row['License']) ? $msg : $row['License']; # Print the timestamps for last updates $updated_time = ($row["ModifiedTS"] == 0) ? "(unknown)" : gmdate("r", intval($row["ModifiedTS"])); $submitted_time = ($row["SubmittedTS"] == 0) ? "(unknown)" : gmdate("r", intval($row["SubmittedTS"])); +$out_of_date_time = ($row["OutOfDateTS"] == 0) ? "(unknown)" : gmdate("r", intval($row["OutOfDateTS"])); ?> <div class="pgbox"> @@ -70,7 +71,7 @@ $submitted_time = ($row["SubmittedTS"] == 0) ? "(unknown)" : gmdate("r", intval( } if ($row["OutOfDate"] == 1) { - echo "<br /><span class='f6'>".__("This package has been flagged out of date.")."</span>"; + echo "<br /><span class='f6'>".__("This package has been flagged out of date")." (".$out_of_date_time.")</span>"; } ?> </p> -- 1.7.3

4 17

[aur-dev] [PATCH] Bold links in the archnavbar header.
by Lukas Fleischer 21 Nov '10

21 Nov '10

--- web/html/css/archnavbar/archnavbar.css | 4 ++++ 1 files changed, 4 insertions(+), 0 deletions(-) diff --git a/web/html/css/archnavbar/archnavbar.css b/web/html/css/archnavbar/archnavbar.css index 8d1d785..e8ade28 100644 --- a/web/html/css/archnavbar/archnavbar.css +++ b/web/html/css/archnavbar/archnavbar.css @@ -39,3 +39,7 @@ div#archdev-navbar li a { padding: 0 0.5em; color: #07b; } /* highlight current website */ #archnavbar.anb-aur ul li#anb-aur a { color: white !important; } + +/* style the links */ +#archnavbar ul#archnavbarlist li a { color: #999; font-weight: bold !important; text-decoration: none !important; } +#archnavbar ul li a:hover { color: white !important; text-decoration: underline !important; } -- 1.7.3

2 1

[aur-dev] [PATCH] FS#3542 - auto redirect from confirmation screens
by Dan Vratil 21 Nov '10

21 Nov '10

Hi, this patch is adding meta refresh tag and a link back to the package page from the screens that confirm comment deletion and change of package category. The link is displayed always, but the autoredirection is done only when the action was sucessfull. This allows user to read or note down a possible error output in case of failure. The delay for refresh is set to 5 seconds, but that can be discussed. Dan -- --------------------------------- Dan Vrátil vratil(a)progdansoft.com ICQ 249163429 Jabber progdan(a)jabber.cz Tel. +420 732 326 870 Tento email neobsahuje žádné viry, protože odesílatel nepoužívá Windows. / This email does not contain any viruses because the sender does not use Windows.

3 9

[aur-dev] RFC: create link for account registration and account information next to the login.
by Justin Davis 17 Nov '10

17 Nov '10

Hello aur-dev! I have checked out the AUR sources from the code.toofishes.net git repository and started playing with it. While I was messing around I remembered a slight annoyance I had with the design of the website. The accounts "toolbar" link is a tad confusing. I see now that TU are able to view and edit accounts through that link but for the average user it is a misnomer. I did not expect that the "Accounts" link was what I had to click in order to register an account on the AUR. After I registered successfully, let's say I am looking for my own account information. "Accounts" is not a word I associate with my account. This is because I do not have more than one account. My suggestion, which I am willing to write a patch for, is to create a registration link near the login/password input boxes. When logged in this registration link is instead a link for viewing/editing account information. These links would be next to where the "[Forgot Password]" link is now. Or where the "[Logout]" link is when logged in. When an unprivileged user is logged in there is no "Accounts" link in AUR's little toolbar on the left. Only TUs and Devs have that link in order to edit accounts. This change of location creates a greater contrast between the normal AUR toolbar links and user account links. Often links that manipulate a users account are grouped closer together and away from links that basically get work done and are used more commonly. This also removes the generally confusing "Accounts" link from people who do not realize that it serves a dual purpose for account administration as well as personal account editing... oh and account registration. The only downside is that new translations would be needed for the registration link and for the account information viewing/editing link. As I side-note I would also like to move the account form printing and processing code out of acctfuncs.inc and into a template/controller. I would also make a new template/ file for the account info/edit form which would then be used from the account info and registration, and the TU's account admin. This would leave the low-level account login, password, etc code inside lib/acctfuncs.inc, move the html printing code into a template/ file, and the form checking code... somewhere else. Anyways, what do people think about those simple changes in design? -- -Justin

3 4