MoinMoin: MoinMoinChat/Logs/moin-dev/2011-09-19

   1 2011-09-19T04:30:40  *** marchael has left #moin-dev
   2 2011-09-19T04:58:42  *** MattMaker has quit IRC
   3 2011-09-19T05:37:48  *** marchael has joined #moin-dev
   4 2011-09-19T07:27:18  *** MattMaker has joined #moin-dev
   5 2011-09-19T09:12:34  *** MattMaker has quit IRC
   6 2011-09-19T10:56:28  <ThomasWaldmann> moin
   7 2011-09-19T10:59:06  *** mkerrin has joined #moin-dev
   8 2011-09-19T11:33:49  *** greg_f has joined #moin-dev
   9 2011-09-19T11:44:41  <dreimark> moin
  10 2011-09-19T12:25:11  *** raignarok has joined #moin-dev
  11 2011-09-19T13:28:33  *** MattMaker has joined #moin-dev
  12 2011-09-19T14:26:05  *** raignarok has quit IRC
  13 2011-09-19T14:48:13  <ThomasWaldmann> ronny: check the st-ng repo, acl middleware :)
  14 2011-09-19T14:49:55  <ronny> not right now, will get at it later
  15 2011-09-19T14:51:49  <ronny> ThomasWaldmann: did you push?
  16 2011-09-19T14:52:05  <ronny> whops
  17 2011-09-19T14:52:09  <ronny> was in the normal moin repo
  18 2011-09-19T14:54:07  <ronny> ThomasWaldmann: why does protecteditem allow setting the itemid?
  19 2011-09-19T14:54:59  <ronny> iter_revs is missing a check
  20 2011-09-19T14:56:03  <ronny> ThomasWaldmann: am i understanding correct that revision reading is based on permissions on the latest revision?
  21 2011-09-19T14:58:31  <ThomasWaldmann> ronny: not sure if it is needed, change as you deem fit :)
  22 2011-09-19T14:58:44  <ThomasWaldmann> yes, we check latest rev
  23 2011-09-19T14:58:51  <ThomasWaldmann> that is moin 1.9 compatible behaviour
  24 2011-09-19T14:59:16  <ThomasWaldmann> and as we have the acl in latest-rev index and usually load the whoosh doc from there anyway, it is no problem
  25 2011-09-19T15:06:01  <ronny> ThomasWaldmann: then there is no need for e xtra checks
  26 2011-09-19T15:06:50  <ThomasWaldmann> if we want, we could even check if the user has also permissions according to THAT revisions' acl
  27 2011-09-19T15:07:39  <ThomasWaldmann> brb
  28 2011-09-19T16:07:49  *** pkumar has joined #moin-dev
  29 2011-09-19T16:53:23  *** raignarok has joined #moin-dev
  30 2011-09-19T17:40:11  <CIA-65> Reimar Bauer <rb.proj AT googlemail DOT com> default * 566:ca006bf7dae8 extensions/data/plugin/script/account/create_many.py: script.account.create_many: script to create many users given from a comma separated list of a file e.g. Username, user@email.de
  31 2011-09-19T17:40:18  *** mkerrin has quit IRC
  32 2011-09-19T17:58:50  *** raignarok_ has joined #moin-dev
  33 2011-09-19T18:00:46  *** raignarok has quit IRC
  34 2011-09-19T18:12:49  <dreimark> bbl
  35 2011-09-19T18:19:19  *** pkumar has quit IRC
  36 2011-09-19T18:57:57  *** raignarok_ has quit IRC
  37 2011-09-19T19:22:57  *** greg_f has quit IRC
  38 2011-09-19T19:56:54  <ThomasWaldmann> re
  39 2011-09-19T20:06:03  <ThomasWaldmann> ronny: ok, added a "require read" for iter_revs
  40 2011-09-19T20:06:42  <ThomasWaldmann> about setting .itemid in ProtectedItem: needed by new items, they assign a new itemid internally
  41 2011-09-19T20:12:00  <ronny> ThomasWaldmann: dont they create one on first rev creation?
  42 2011-09-19T20:13:22  <ThomasWaldmann> store_revision, yes
  43 2011-09-19T20:14:26  <ronny> ThomasWaldmann: so why does the wrapper need to be a able to set it?
  44 2011-09-19T20:15:49  <ThomasWaldmann> because the instance is ProtectedItem, so accessing .itemid = ... goes through it
  45 2011-09-19T20:18:10  <ronny> ThomasWaldmann: but the id is assigned in the indexer, not the protector
  46 2011-09-19T20:18:24  <ThomasWaldmann> but maybe we do not need to set .itemid in the indexer
  47 2011-09-19T20:19:03  <ThomasWaldmann> ah, right,we proxy, we do not subclass
  48 2011-09-19T20:19:07  <ronny> ThomasWaldmann: unless there is a real usecase for proxying that write access, it should be readonly
  49 2011-09-19T20:19:09  * ThomasWaldmann was confused
  50 2011-09-19T20:19:20  <ThomasWaldmann> yes
  51 2011-09-19T20:19:36  <ThomasWaldmann> what i was describing was for subclassing/mixin
  52 2011-09-19T20:20:17  <ThomasWaldmann> i'll fix it
  53 2011-09-19T20:20:20  <ThomasWaldmann> anything else
  54 2011-09-19T20:20:22  <ThomasWaldmann> ?
  55 2011-09-19T20:25:43  <ronny> ThomasWaldmann: the rest looks ok atm
  56 2011-09-19T20:27:48  <ThomasWaldmann> hmm, looks like i have to do some non-moin work :|
  57 2011-09-19T20:28:26  <ThomasWaldmann> waldi: xorAxAx: please comment on issues you find in https://bitbucket.org/thomaswaldmann/storage-ng
  58 2011-09-19T20:28:46  <ThomasWaldmann> (or anyone else interested :)
  59 2011-09-19T20:29:07  <ThomasWaldmann> I want to kill the current storage subsystem of moin2 and replace it with that asap.
  60 2011-09-19T20:30:13  <ThomasWaldmann> (storage+router+indexing+acl layer and serialization, to be precise, in a separate branch at the beginning)
  61 2011-09-19T20:30:26  <ThomasWaldmann> brb
  62 2011-09-19T20:45:07  *** raignarok_ has joined #moin-dev
  63 2011-09-19T20:53:22  *** raignarok_ has quit IRC
  64 2011-09-19T21:51:31  *** raignarok has joined #moin-dev
  65 2011-09-19T22:53:44  *** raignarok has quit IRC
  66