In that case, this should work as well: query = db.auth_user.id == request.args[0]).select() if request.args[0].isdigit() else db.auth_user.username == request.args[0] user_id = db(query).select()[0].id or 0
On Jul 15, 6:54 am, Kenneth Lundström <kenneth.t.lundst...@gmail.com> wrote: > Thanks, > > logic error noted, in this case it woun t be a problem. Can t think of a > fix without changing the idea behind this. > > I find it interesting that some fields expect a specific type. Is ID the > only one? > > user = db(db.auth_user.id == request.args[0] if > request.args[0].isdigit() else 0).select() > > This didn t work, it throws an error saying no table selected. > But instead I did it like this: > if request.args[0].isdigit(): > user = db(db.auth_user.id == str(request.args[0])).select() > if len(user): > user_id = user[0].id > else: > user_id = 0 > else: > user = db(db.auth_user.username == request.args[0]).select() > if len(user): > user_id = user[0].id > else: > user_id = 0 > > Kenneth > > > > > > > > > request.args always contain strings. str(request.args(0)) is > > extraneous > > > There's a logic hole here: What if my username is "1" but my record's > > id is 22. This code will return the record db.auth_user[1] when it's > > supposed to return db.auth_user[22] > > > Anyway, ou get "invalid literal for int() with base 10" error because > > auth_user.id expects integers and you're sending it a string, > > "kenneth". Quick fix: > > user = db(db.auth_user.id == request.args[0] if > > request.args[0].isdigit() else 0).select() > > > That fixes the syntax error but not the logic error I pointed out > > earlier. > > > On Jul 15, 5:06 am, Kenneth Lundstr m<kenneth.t.lundst...@gmail.com> > > wrote: > >> Hello, > > >> why can t I do this: > >> user = db(db.auth_user.id == 'kenneth').select() > > >> I know that this should result in len(user) = 0 but I m sending > >> sometimes a text and sometimes a number and I m trying to do this: > >> user = db(db.auth_user.id == str(request.args[0])).select() > >> if len(user): > >> user_id = user[0].id > >> else: > >> user = db(db.auth_user.username == request.args[0]).select() > >> if len(user): > >> user_id = user[0].id > >> else: > >> user_id = 0 > > >> Error ticket for "kenneths" > > >> Ticket ID > > >> 85.76.66.169.2011-07-15.00-45-45.b6793e0f-2144-4a02-802f-d7b8a45ec8a5 > > >> Version > > >> web2py^(TM) Version 1.97.1 (2011-06-26 19:25:44) > >> Python Python 2.6.5: /usr/bin/python > > >> Traceback > > >> 1. > >> 2. > >> 3. > >> 4. > >> 5. > >> 6. > >> 7. > >> 8. > >> 9. > >> 10. > >> 11. > >> 12. > >> 13. > >> 14. > >> 15. > >> 16. > >> 17. > >> 18. > >> 19. > >> 20. > >> 21. > >> 22. > >> 23. > >> 24. > > >> Traceback(most recent call last): > >> File"/data/domains/web2py/gluon/restricted.py",line192,inrestricted > >> execccodeinenvironment > >> > >> File"/data/domains/web2py/applications/kenneths/controllers/ticket.py"<https://web2py.nudata.fi/admin/default/edit/kenneths/controllers/tick...>,line43,in<module> > >> File"/data/domains/web2py/gluon/globals.py",line137,in<lambda> > >> self._caller=lambdaf:f() > >> > >> File"/data/domains/web2py/applications/kenneths/controllers/ticket.py"<https://web2py.nudata.fi/admin/default/edit/kenneths/controllers/tick...>,line9,innew_ticket > >> user=db(db.auth_user.id=='kenneth').select() > >> File"/data/domains/web2py/gluon/dal.py",line5394,inselect > >> return self.db._adapter.select(self.query,fields,attributes) > >> File"/data/domains/web2py/gluon/dal.py",line1168,inselect > >> sql=self._select(query,fields,attributes) > >> File"/data/domains/web2py/gluon/dal.py",line1078,in_select > >> sql_w=' WHERE '+self.expand(query) > >> File"/data/domains/web2py/gluon/dal.py",line937,inexpand > >> returnexpression.op(expression.first,expression.second) > >> File"/data/domains/web2py/gluon/dal.py",line886,inEQ > >> return'(%s = %s)'% > >> (self.expand(first),self.expand(second,first.type)) > >> File"/data/domains/web2py/gluon/dal.py",line943,inexpand > >> return self.represent(expression,field_type) > >> File"/data/domains/web2py/gluon/dal.py",line1280,inrepresent > >> returnstr(int(obj)) > >> ValueError:invalid literalforint()with base10:'kenneth' > > >> Error snapshot > > >> |<type 'exceptions.ValueError'>(invalid literal for int() with base 10: > >> 'kenneth')|
