1 2011-09-19T04:30:40 *** marchael
2 2011-09-19T04:58:42 *** MattMaker
3 2011-09-19T05:37:48 *** marchael
4 2011-09-19T07:27:18 *** MattMaker
5 2011-09-19T09:12:34 *** MattMaker
6 2011-09-19T10:56:28 <ThomasWaldmann> moin
7 2011-09-19T10:59:06 *** mkerrin
8 2011-09-19T11:33:49 *** greg_f
9 2011-09-19T11:44:41 <dreimark> moin
10 2011-09-19T12:25:11 *** raignarok
11 2011-09-19T13:28:33 *** MattMaker
12 2011-09-19T14:26:05 *** raignarok
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
29 2011-09-19T16:53:23 *** raignarok
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
32 2011-09-19T17:58:50 *** raignarok_
33 2011-09-19T18:00:46 *** raignarok
34 2011-09-19T18:12:49 <dreimark> bbl
35 2011-09-19T18:19:19 *** pkumar
36 2011-09-19T18:57:57 *** raignarok_
37 2011-09-19T19:22:57 *** greg_f
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_
63 2011-09-19T20:53:22 *** raignarok_
64 2011-09-19T21:51:31 *** raignarok
65 2011-09-19T22:53:44 *** raignarok
66
MoinMoin: MoinMoinChat/Logs/moin-dev/2011-09-19 (last edited 2011-09-19 02:45:03 by IrcLogImporter)