Thanks Niphlod, I get the trunk with this feature working done now! Christian.
El viernes, 11 de enero de 2013 10:45:55 UTC-3, Niphlod escribió: > > the problem has been fixed in trunk. The issue was *kinda non-existant*for > javascript enabled browsers, because there was an onclick attribute on > the delete button that called the delete url correctly. > > Now there are both the full link to the delete url and the onclick > attribute, so behaviour is consistent both in javascript-enabled and > javascript-disabled browsers. > > Il giorno venerdì 11 gennaio 2013 14:24:46 UTC+1, Christian Espinoza ha > scritto: >> >> This Problem is currently active in 2.3.2 >> >> There is something with the link for delete action at generation of grid >> code: >> >> With 2.2.1 >> >> View->http://127.0.0.1:8000/webapp/admin/users/view/auth_user/6?_signature=d408cbb7bcbce4ec22a9089465dcefc7f1f2a5db >> Edit- >> <http://127.0.0.1:8000/webapp/admin/users/view/auth_user/6?_signature=d408cbb7bcbce4ec22a9089465dcefc7f1f2a5dbEdit->>http://127.0.0.1:8000/webapp/admin/users/edit/auth_user/6?_signature=a688c63924538f53fdf0da2630790d780204403d >> Delete- >> <http://127.0.0.1:8000/webapp/admin/users/edit/auth_user/6?_signature=a688c63924538f53fdf0da2630790d780204403dDelete->>http://127.0.0.1:8000/webapp/admin/users?_signature=ca13c286893cff85c6916ff7723d3f0cd8e24c56#null >> >> With 2.3.2 >> >> View->http://127.0.0.1:8000/webapp/admin/users/view/auth_user/6?_signature=d408cbb7bcbce4ec22a9089465dcefc7f1f2a5db >> >> Edit->http://127.0.0.1:8000/webapp/admin/users/edit/auth_user/6?_signature=a688c63924538f53fdf0da2630790d780204403d >> >> Delete->http://127.0.0.1:8000/avl/admin/users#null >> >> As you can see the Delete option link doesn't point to any function at both >> versions >> >> >> Christian. >> >> >> El miércoles, 26 de diciembre de 2012 20:48:21 UTC-3, Massimo Di Pierro >> escribió: >>> >>> Can you please open a ticket about this? >>> >>> On Wednesday, 26 December 2012 13:40:15 UTC-6, BJ wrote: >>>> >>>> The problem is in sqlhtml.py >>>> >>>> In latest version from line 2293-2297 is following content: >>>> >>>> if deletable and (not callable(deletable) or deletable(row)): >>>> row_buttons.append(gridbutton( >>>> 'buttondelete', 'Delete', >>>> callback=url(args=['delete', tablename, id >>>> ]), >>>> delete='tr')) >>>> >>>> but it should be: >>>> >>>> if deletable and (not callable(deletable) or deletable(row)): >>>> row_buttons.append(gridbutton( >>>> 'buttondelete', 'Delete', >>>> buttonurl=url(args=['delete', tablename, >>>> id]), <--- Missing line to define URL for Delete button >>>> callback=url(args=['delete', tablename, >>>> id]), <--- We need this for warning message (I believe so?) >>>> delete='tr')) >>>> >>>> But we also need to update gridbutton function (line 1806) because we >>>> need button URL and warning. I have tried to add "buttonurl" but now the >>>> record is deleted even if I select "Cancel". >>>> >>>> I don't know how to fix this because I don't how to handle callback. >>>> >>>> Boris >>>> >>>> >>>> Dne sreda, 19. december 2012 17:23:18 UTC+1 je oseba Mandar Vaze >>>> napisala: >>>>> >>>>> I know it is a tough one, but can you guess what "could" be the >>>>> problem ? As I mentioned, "explicitly" adding db.commit() helps "resolve" >>>>> the problem (Whether in gluon/sqlhtml.py or separately in my custom >>>>> ondelete) >>>>> >>>>> Does that give you any hints ? I debugged but (obviously) couldn't >>>>> find the culprit. >>>>> >>>>> Any pointers on how to troubleshoot/debug this ? I think at least one >>>>> more person has similar problem (One that started this thread) >>>>> >>>>> >>>>>> Really not relevant all the modify_grid() part. That doesn't belong >>>>>> to web2py, which you are addressing the bug to. >>>>>> >>>>> >>>>> I know. But it was something different - so I thought it may be >>>>> relevant. >>>>> >>>>> >>>>>> Without the table model, all code is quite useless. We don't need >>>>>> your app. Just *an* app showing the problem. >>>>>> >>>>> >>>>> Ahh, I see. Let me try to create that (app) >>>>> >>>>> -Mandar >>>>> >>>> --
