MoinMoin: MoinMoinChat/Logs/moin-dev/2007-03-19

2007-03-19T22:47:13  <dreimark> may.admin and may.write there could be conflicts
2007-03-19T22:47:51  <dreimark> may.write does not resolve dependencies of may.admin
2007-03-19T22:48:40  <dreimark> if you try with a user test and a page with #acl test:
2007-03-19T22:48:40  <ThomasWaldmann> ?
2007-03-19T22:49:03  <dreimark> may.write this page tells true
2007-03-19T22:49:24  <dreimark> and it crashes because test has no admin rights
2007-03-19T22:51:27  <ThomasWaldmann> current 1.6?
2007-03-19T22:51:42  <dreimark> yeah
2007-03-19T22:51:49  <ThomasWaldmann> how to reproduce?
2007-03-19T22:52:06  <xorAxAx> check in a test :)
2007-03-19T22:52:50  <dreimark> I came to this testing a newer CopyPage
2007-03-19T22:53:14  <dreimark>  if request.user.may.write(newpagename) and request.user.may.admin(newpagename): is needed
2007-03-19T22:53:37  <dreimark> before I have had only request.user.may.write(newpagename)
2007-03-19T22:53:50  <ThomasWaldmann> why admin?
2007-03-19T22:55:24  <dreimark> because the user is not able to save a comment by saveText if he is not able to write to this page
2007-03-19T22:56:27  <dreimark> I found it testing CopyPage
2007-03-19T22:56:33  <ThomasWaldmann> that is write, why admin?
2007-03-19T22:58:41  <dreimark> I destinguish not possible to write to use an other method to write only a log entry by packages.edit_logfile_append
2007-03-19T23:00:01  <dreimark> so RC knows
2007-03-19T23:00:32  <dreimark> xorAxAx:1.7 will have more unittests from me too :)
2007-03-19T23:02:20  <dreimark> as comment if saveText could be used ## page was copied from NewExample
2007-03-19T23:05:53  <ThomasWaldmann> sorry, i dont get what you mean. file a bug, commit a test, ...
2007-03-19T23:07:24  <dreimark> ok
2007-03-19T23:50:22  <dreimark> seems not easy to find a test to reproduce it