I didn't get much out of commenting out those lines. In Chrome I got an unresolvable url.
On Tue, Jan 4, 2011 at 4:47 PM, mdipierro <mdipie...@cs.depaul.edu> wrote: > I think this is now fixed. > > On Jan 4, 1:10 pm, David Bain <pigeonfli...@gmail.com> wrote: > > When I try to access /admin/appadmin in Multiuser/Teaching mode. > > I get an error. > > > > see traceback below: > > > > Traceback (most recent call last): > > File "/home/user/web2pytrunkagain/gluon/restricted.py", line 188, in > > restricted > > exec ccode in environment > > File > "/home/user/web2pytrunkagain/applications/admin/controllers/appadmin.py" > > <https://localhost:8001/admin/default/edit/admin/controllers/appadmin.py > >, > > line 33, in <module> > > if not gluon.fileutils.check_credentials(request): > > File "/home/user/web2pytrunkagain/gluon/fileutils.py", line 300, in > > check_credentials > > return get_session(request, other_application).authorized > > File "/home/user/web2pytrunkagain/gluon/fileutils.py", line 278, in > > get_session > > raise KeyError > > KeyError > > > > On Tue, Jan 4, 2011 at 1:01 PM, mdipierro <mdipie...@cs.depaul.edu> > wrote: > > > > > On Jan 4, 11:07 am, David Bain <pigeonfli...@gmail.com> wrote: > > > > Possible bug: > > > > The teacher account is fine. However I created my first student > account > > > and > > > > was unable to log in with the credentials. > > > > > > Here's my checklist of nice to have features > > > > -------------------------------------------------------------- > > > > > > - Bulk import of users from a csv > > > > > you can do it already via admin/appadmin/ > > > > > > - by default prevent non-teachers from being able to use the > > > application > > > > wizard/ check for upgrades/ new simple application/deply to google > app > > > > engine (I've X'ed out what should not be in the interface by > default) > > > > > check for upgrades should already be disabled. I will look into it. > > > > > > [image: site-1.jpg] > > > > > > - possibly allow teachers to selectively enable features > > > > > We could have more flags in 0.py and corresponding if statements in > > > controllers. > > > > > > On Tue, Jan 4, 2011 at 11:57 AM, mdipierro <mdipie...@cs.depaul.edu> > > > wrote: > > > > > It would be nice to have some documentation. For now even a simple > > > > > blog post that explains purpose, how to, and shows some screen > shots > > > > > will do. > > > > > > > Massimo > > > > > > > On Jan 4, 10:44 am, David Bain <pigeonfli...@gmail.com> wrote: > > > > > > Massimo, > > > > > > Nice! It works... so as long as I register first I'm the teacher > :). > > > > > > I'm just looking at web2py again in the last few weeks. I'm very > > > happy to > > > > > > help with documentation of this feature. Just point me in the > right > > > > > > direction. > > > > > > > > see: Screenshot below: > > > > > > [image: user login.jpg] > > > > > > > > On Tue, Jan 4, 2011 at 10:54 AM, David Bain < > pigeonfli...@gmail.com> > > > > > wrote: > > > > > > > Thanks Massimo, > > > > > > > Checking out trunk now. > > > > > > > > > On Tue, Jan 4, 2011 at 10:49 AM, mdipierro < > > > mdipie...@cs.depaul.edu > > > > > >wrote: > > > > > > > > >> In trunk. > > > > > > > > >> - Deploy the latest admin > > > > > > >> - edit file applications/admin/models/0.py and set > > > > > > >> MULTI_USER_MODE = True > > > > > > >> - Try access admin and it will require registration/login > > > > > > >> - The first user to register is teacher (can see all apps) > > > > > > >> - All other users are students (can only see/edit/create their > own > > > > > > >> apps) > > > > > > > > >> This does not prevent one user from writing dangerous code. > The > > > code, > > > > > > >> whoever creates it, always runs under the same privileged and > has > > > > > > >> access to the entire web2py folder. > > > > > > > > >> At the moment students get access to each other appdmin > > > controllers. > > > > > > > > >> Please help with testing! > > > > > > > > >> On Jan 4, 8:28 am, David Bain <pigeonfli...@gmail.com> wrote: > > > > > > >> > I like the modified admin approach, it would be simpler, for > > > > > teaching > > > > > > >> > purposes it would work, it would need to be modified for > > > production > > > > > > >> style > > > > > > >> > scenarios as it would not be an acceptable risk. > > > > > > > > >> > On Tue, Jan 4, 2011 at 6:50 AM, mdipierro < > > > mdipie...@cs.depaul.edu> > > > > > > >> wrote: > > > > > > >> > > This can be done but it would not prevent one use to write > > > code > > > > > (an > > > > > > >> > > app) that reads or deletes another user app. As long as > this > > > is > > > > > clear, > > > > > > >> > > I coud modify admin for this purpose (or create another > > > admin). > > > > > > > > >> > > On Jan 3, 10:13 pm, VP <vtp2...@gmail.com> wrote: > > > > > > >> > > > I think it'll be convenient for multiple users to work > on > > > one > > > > > web2py > > > > > > >> > > > installation, if there's a layer on top of the admin > app, > > > > > providing > > > > > > >> > > > the following features: > > > > > > > > >> > > > 1. The admin app allows user registration. > > > > > > >> > > > 2. Each user can view/edit only apps that he/she > creates. > > > > > > > > >> > > > This is strictly at the (admin) app level. Everything > is > > > the > > > > > same > > > > > > >> as > > > > > > >> > > > before at the filesystem level. > > > > > > > > >> > > > (This is particularly useful for teaching (I think): one > > > web2py > > > > > app, > > > > > > >> > > > many students). > > > > > > > > user login.jpg > > > > > > 41KViewDownload > > > > > > site-1.jpg > > > > 32KViewDownload > > > > >
