[PATCH] Add capability for co-maintainers to disown packages

Lukas Fleischer lfleischer at archlinux.org
Fri Feb 23 05:46:05 UTC 2018


On Tue, 06 Feb 2018 at 03:54:56, Mark Weiman wrote:
> Implements FS#53832
> 
> Signed-off-by: Mark Weiman <mark.weiman at markzz.com>
> ---
>  web/html/pkgbase.php             |  3 +++
>  web/html/pkgdisown.php           | 13 ++++++++++---
>  web/lib/pkgbasefuncs.inc.php     | 12 ++++++++++--
>  web/template/pkgbase_actions.php |  2 +-
>  4 files changed, 24 insertions(+), 6 deletions(-)
> 
> [...]
> @@ -23,7 +26,11 @@ if (has_credential(CRED_PKGBASE_DISOWN, $maintainer_uids)): ?>
>                 <?php endforeach; ?>
>         </ul>
>         <p>
> -               <?php if (count($comaintainers) > 0 && !has_credential(CRED_PKGBASE_DISOWN)): ?>
> +
> +               <?php if (in_array(uid_from_sid($_COOKIE["AURSID"]), $comaintainer_uids) && !has_credential(CRED_PKGBASE_DISOWN)):
> +                       $action = "do_DisownComaintainer"; ?>
> +                       <?= __("By selecting the checkbox, you confirm that you want to no longer be a package co-maintainer.") ?>
> +               <?php elseif (count($comaintainers) > 0 && !has_credential(CRED_PKGBASE_DISOWN)): ?>

I am not sure whether it is a good idea to use the same button for
disowning a package as a maintainer or as a co-maintainer? What happens
if a user is both a maintainer and a co-maintainer (and what is the
expected behavior)?

Anyway, merged this into pu as-is for now; we can still replace it
later.

Regards,
Lukas


More information about the aur-dev mailing list