[prev in list] [next in list] [prev in thread] [next in thread] 

List:       listar-dev
Subject:    [EDev] Another patch for LSG2
From:       Tomas Ogren <stric () ing ! umu ! se>
Date:       2003-02-10 16:44:10
[Download RAW message or body]

Hello.
This patch contains 3 chunks, the first two just make it more usable IMO
(esp. with our ~300 lists.. showing them all in a listbox isn't very
usable)

The last chunk makes non-admins (but still subscribers) view userlists
of private lists.

/Tomas
-- 
Tomas Ögren, stric@ing.umu.se, http://www.ing.umu.se/~stric/
|- Student at Computing Science, University of Umeå
`- Sysadmin at {cs,ing,acc}.umu.se


-- Attached file included as plaintext by Ecartis --

--- site/src/modules/lsg2/lsg2-func.c	Sat May 25 08:45:13 2002
+++ sunos58.new/src/modules/lsg2-func.c	Mon Feb 10 17:32:19 2003
@@ -124,7 +124,7 @@
   char dname[BIG_BUF];
   struct list_user user;
 
-   printf("<select name=\"lcgi-list\" size=1>\n");
+   printf("<select name=\"lcgi-list\">\n");
 
    if (!(status = LMAPI->walk_lists(&dname[0]))) {
        printf("<option value=\".uhoh.\">-- Unable to access lists --\n");
@@ -168,7 +168,7 @@
       }
    }
 
-   printf("<select name=\"lcgi-list\" size=%d>\n", count);
+   printf("<select name=\"lcgi-list\">\n");
 
    if (!(status = LMAPI->walk_lists(&dname[0]))) {
        printf("<option value=\".uhoh.\">-- Unable to access lists --\n");
@@ -366,7 +366,7 @@
 
    if (LMAPI->user_hasflag(&user,"ADMIN")) isadmin = 1; else isadmin = 0;
 
-   if ((strcmp(LMAPI->get_string("who-status"),"admin") != 0) && !isadmin) {
+   if ((strcmp(LMAPI->get_string("who-status"),"admin") == 0) && !isadmin) {
       printf("<B>Permission denied.</b>");
       return 1;
    }



[prev in list] [next in list] [prev in thread] [next in thread]