MoinMoin: MoinMoinChat/Logs/moin-dev/2009-07-30

2009-07-30T01:09:21  *** grzywacz has quit IRC
2009-07-30T01:54:20  *** cdyson37 has quit IRC
2009-07-30T06:14:12  *** PawelPac1na has joined #moin-dev
2009-07-30T06:15:07  *** PawelPacana has quit IRC
2009-07-30T08:12:50  <ThomasWaldmann> moin
2009-07-30T09:33:05  <_peck_> ThomasWaldmann: no it's not related to wrapping but to sharing binds
2009-07-30T09:33:45  <_peck_> either i share bind parameters with the connexion class and it can't do password substitutions or i don't and i have to duplicate configuration
2009-07-30T09:41:02  <ThomasWaldmann> yeah. what i meant is you don't have the problem THERE then (but you still would have it in general).
2009-07-30T09:52:16  <_peck_> in fact that functionnality (bind_dn='%(xxx)s') is mutualy exclusive with groups
2009-07-30T10:03:24  <ThomasWaldmann> you mean if the bind is done as login user?
2009-07-30T10:06:10  <_peck_> yes
2009-07-30T10:06:46  <_peck_> we don't have the login/password after the first page
2009-07-30T10:07:01  <_peck_> so we can't bind anymore with them
2009-07-30T10:26:55  <ThomasWaldmann> ehrm, right. so I guess we need a configured bind user/pw
2009-07-30T10:45:11  <CIA-43> Bastian Blank <bblank@thinkmo.de> default * 6629:a3b7d1e17a35 2.0-storage-dom-bblank/MoinMoin/converter2/_tests/ (test_creole_in.py test_html_out.py test_moinwiki_in.py): Tests for converters - Use seperate style attribute
2009-07-30T10:45:14  <CIA-43> Bastian Blank <bblank@thinkmo.de> default * 6630:ac47290f635f 2.0-storage-dom-bblank/MoinMoin/converter2/html_out.py: HTML output converter - Use style attribute, remove attribute to style conversion
2009-07-30T10:45:16  <CIA-43> Bastian Blank <bblank@thinkmo.de> default * 6631:9efb2bc785e6 2.0-storage-dom-bblank/MoinMoin/converter2/creole_in.py: Creole input converter - Remove page URI handling
2009-07-30T10:45:17  <CIA-43> Bastian Blank <bblank@thinkmo.de> default * 6632:13c307215e42 2.0-storage-dom-bblank/MoinMoin/converter2/ (_tests/test_creole_in.py creole_in.py): Creole input converter - Support style argument, remove old test
2009-07-30T10:45:19  <CIA-43> Bastian Blank <bblank@thinkmo.de> default * 6633:f9929be6ca06 2.0-storage-dom-bblank/MoinMoin/converter2/ (_tests/test_moinwiki_in.py moinwiki_in.py): Moin Wiki input converter - Remove page URI handling
2009-07-30T10:45:20  <CIA-43> Bastian Blank <bblank@thinkmo.de> default * 6634:68017b688486 2.0-storage-dom-bblank/MoinMoin/converter2/moinwiki_in.py: Moin Wiki input converter - Support style argument
2009-07-30T11:40:07  <_peck_> and be able to override them from ldap_auth
2009-07-30T12:05:22  <waldi> okay, I need to extend the argument parse for the table styles. it needs to have more knowledge about the short attributes like |2
2009-07-30T12:08:12  <waldi> *gna*
2009-07-30T12:20:48  <CIA-43> Bastian Blank <bblank@thinkmo.de> default * 6635:e62f57869180 2.0-storage-dom-bblank/MoinMoin/converter2/_tests/ (test_creole_in.py test_moinwiki_in.py): Tests for Creole input converter, Tests for Moin Wiki input converter - Use new Element write support
2009-07-30T12:20:49  <CIA-43> Bastian Blank <bblank@thinkmo.de> default * 6636:d98652b3cbdb 2.0-storage-dom-bblank/MoinMoin/converter2/ (_tests/test_html_out.py html_out.py): HTML output converter - Support columns and row spans
2009-07-30T12:20:50  <CIA-43> Bastian Blank <bblank@thinkmo.de> default * 6637:5394352c84d4 2.0-storage-dom-bblank/MoinMoin/converter2/ (_tests/test_moinwiki_in.py moinwiki_in.py): Moin Wiki input converter - Support some table attributes
2009-07-30T12:20:52  <CIA-43> Bastian Blank <bblank@thinkmo.de> default * 6638:5ae0cc51db58 2.0-storage-dom-bblank/MoinMoin/converter2/moinwiki_in.py: Moin Wiki input converter - Support simple column span
2009-07-30T12:23:33  <waldi> or make them more lazy ...
2009-07-30T12:46:05  <dennda> Ok guys, it's finally over... for now...
2009-07-30T13:20:33  <ThomasWaldmann> dennda: :)
2009-07-30T13:20:45  <ThomasWaldmann> dennda: how was it?
2009-07-30T13:24:02  <dennda> Could have been a 1.0 if I would have had more time
2009-07-30T13:24:11  <dennda> we'll see what the outcome is
2009-07-30T13:25:15  <dennda> Having a coffee and a red bull certainly helps after only 4 hours of sleep
2009-07-30T13:25:46  <dennda> (so if I send strange character sequences that means my head hit the keyboard, but I'm fine atm)
2009-07-30T13:29:33  <dennda> getting up half past 6 is fun. especially if you have NO WARM WATER since HALF A WEEK
2009-07-30T13:30:06  <dennda> So, where were we again...
2009-07-30T13:30:25  <dennda> ThomasWaldmann: Are you still undecided wrt trashbin? :D
2009-07-30T13:32:37  <dennda> ThomasWaldmann: I was unable to reproduce the py.test creation of instance folders, btw
2009-07-30T13:37:40  <ThomasWaldmann> cold water helps :P
2009-07-30T13:38:20  <ThomasWaldmann> maybe first solve all the other todos, not deletion/renaming
2009-07-30T13:41:12  <dennda> ok, where to start... hm...
2009-07-30T13:42:30  <ThomasWaldmann> tests? working underlay migration path (for $NORMALUSER)?
2009-07-30T13:43:16  <ThomasWaldmann> or add a "create" capability?
2009-07-30T13:43:31  <dennda> just thinking aloud, but thanks anyway :-)
2009-07-30T13:44:35  <ThomasWaldmann> update diary for last week? :)
2009-07-30T13:56:53  <_peck_> ThomasWaldmann: what do you think of an api like this in LDAPConnection :
2009-07-30T13:56:57  <_peck_> def new_connection(self):  # store and return ldap.intialize
2009-07-30T13:56:57  <_peck_> def default_bind(self): # use default binddn and bindpw
2009-07-30T13:56:57  <_peck_> LDAPAuth would use new_connection and its own bind
2009-07-30T13:56:58  <_peck_> LDAPGroups would use new_connection and default_bind
2009-07-30T14:22:31  <ThomasWaldmann> does it make sense to use 2 different bind methods?
2009-07-30T14:23:19  <ThomasWaldmann> _peck_:
2009-07-30T14:23:27  <ThomasWaldmann> waldi:
2009-07-30T14:25:14  <_peck_> yes it makes sense because LDAPAuth use its own bind with user's dn and pw or with %(username)s replacement
2009-07-30T14:26:16  <ThomasWaldmann> that's how it is now, yes
2009-07-30T14:26:45  <_peck_> i think that feature should stay
2009-07-30T14:26:47  <ThomasWaldmann> but if we need a non-user-login based binddn/pw anyway for ldap groups, what do we win by not using that for both
2009-07-30T14:27:14  <ThomasWaldmann> (and I think users wanting to use ldap for auth likely will also use ldap groups, when available)
2009-07-30T14:27:46  <waldi> ldapauth needs to use two different binds. one to lookup the user and one to check it
2009-07-30T14:28:29  <ThomasWaldmann> waldi: the user lookup bind can be done with user-login-name and user-login-pw right now
2009-07-30T14:28:53  <dennda> hm I wonder what broke the acl tests
2009-07-30T14:29:42  <ThomasWaldmann> _peck_: if we really need both, having a ldapconnection method bind(self, name=None, password=None) would make it a bit more consistent
2009-07-30T14:29:50  <waldi> ThomasWaldmann: maybe I missunderstand that, but that looks like a chicken/egg-problem
2009-07-30T14:30:00  <_peck_> yes all in one why not
2009-07-30T14:30:38  <ThomasWaldmann> waldi: it sometimes works :)
2009-07-30T14:43:50  <_peck_> i think this class would be better called LDAPConnectionManager or LDAPConnectionFactory and the method somethong like get_connection
2009-07-30T14:44:41  <ThomasWaldmann> well, especially if it is not a wrapper after it has made the connection
2009-07-30T14:46:32  <_peck_> it is not really, in fact it is a factory with a utility function that could be put anywhere else
2009-07-30T14:55:35  <_peck_> mybe i should put search_single in ldap_groups
2009-07-30T15:16:25  <dennda> shouldn't adding a right to All: in acl_rights_before be sufficient to set it to True for all users?
2009-07-30T15:16:35  <dennda> for some reason it still is false
2009-07-30T15:17:03  <ThomasWaldmann> yes, except there is something else even before it
2009-07-30T15:19:02  <dennda> ThomasWaldmann: "something else before it"?
2009-07-30T15:19:11  <ThomasWaldmann> left of it
2009-07-30T15:25:29  <dennda> ThomasWaldmann: "something else" being...?
2009-07-30T15:25:35  <dennda> this is what I got:
2009-07-30T15:26:08  <dennda> (Pdb) cfg.acl_rights_valid
2009-07-30T15:26:10  <dennda> ['read', 'write', 'delete', 'admin', 'destroy', 'create']
2009-07-30T15:26:57  <dennda> and
2009-07-30T15:28:20  <dennda> .oO(where is that bloody line again)
2009-07-30T15:31:14  <dennda> heh
2009-07-30T15:31:27  <dennda> (Pdb) cfg.acl_rights_default
2009-07-30T15:31:28  <dennda> u'All:read,write,delete,revert,admin'
2009-07-30T15:31:34  <dennda> although I added it to default rights
2009-07-30T15:33:01  <dennda> oh
2009-07-30T15:37:06  <ThomasWaldmann> didn't we kill revert?
2009-07-30T15:37:24  <dennda> yeah wikiconfig.py is obsolete
2009-07-30T15:37:27  <dennda> just edited it
2009-07-30T15:39:55  <dennda> that just cost me 20-30 mins to realize
2009-07-30T15:40:34  <dennda> (and some other strange bug wrt importing)
2009-07-30T15:43:09  <CIA-43> Christopher Denter <moin GUESSWHAT the DASH space DASH station ROUNDTHING com> default * 6299:57234ad89561 2.0-storage/ (4 files in 4 dirs): storage: Introduce 'create' ACL privilege. This allows to have some pages that users may freely modify, but not create pages on their own.
2009-07-30T15:43:57  <dennda> ThomasWaldmann: The problem I see with create is that there's no mechanism yet that makes it possible for users to add attachments to a page they may edit
2009-07-30T15:44:45  <ThomasWaldmann> ah, that was fast :)
2009-07-30T15:45:10  <ThomasWaldmann> that's also create, sure.
2009-07-30T15:45:19  <ThomasWaldmann> == expected feature
2009-07-30T15:47:07  <dennda> was there a bit of irony in your voice? :P
2009-07-30T15:47:48  <ThomasWaldmann> no :)
2009-07-30T15:47:57  <dennda> ThomasWaldmann: well and how do you want to check that properly?
2009-07-30T15:48:39  <ThomasWaldmann> on create_item maybe? :)
2009-07-30T15:50:43  <dennda> Did we already decide how we link pages and their attachments on the item level?
2009-07-30T15:51:01  <ThomasWaldmann> [[target]]
2009-07-30T15:51:37  <dennda> .oO(I'm a bit surprised that I still know my surname. My brain's obviously fail atm)
2009-07-30T15:55:15  <dennda> ThomasWaldmann: that's the syntax I assume. I rather meant what I need to check in create_item, if given a name for an attachment
2009-07-30T15:56:25  <ThomasWaldmann> there is no such thing as "an item name for an attachment"
2009-07-30T15:56:41  <ThomasWaldmann> you just have items with names and they have revisions
2009-07-30T15:57:49  <ThomasWaldmann> (and they have a mimetype and other metadata, but we do not have ACLs behaving differently depending on the mimetype)
2009-07-30T15:57:52  <dennda> if you add an attachment, an item is created for that attachment. so if you don't have the create privilege, you can't add attachments.
2009-07-30T15:58:08  <ThomasWaldmann> yes, sure
2009-07-30T15:58:57  <ThomasWaldmann> if you don't have water, you can't have a shower.
2009-07-30T15:59:31  <dennda> so everything is as it should be?
2009-07-30T15:59:52  <ThomasWaldmann> i didn't test it, but I hope so. :)
2009-07-30T16:00:23  <dennda> no extra specialcasing for allowing to add attachments on writable items without create privilege?
2009-07-30T16:00:23  <ThomasWaldmann> maybe you misunderstood what i said at 15:45/47
2009-07-30T16:00:23  <dennda> I hope so
2009-07-30T16:00:26  <ThomasWaldmann> no extras
2009-07-30T16:00:30  <dennda> ok
2009-07-30T16:00:32  <dennda> very good
2009-07-30T16:00:52  * ThomasWaldmann doesn't want a pony
2009-07-30T16:00:57  <dennda> (though that may be a bit odd. but having a wiki where you can't create new pages may be considered odd in the first place)
2009-07-30T16:01:23  <dennda> ok. /me adds tests
2009-07-30T16:01:50  <ThomasWaldmann> that's for rather restrictive environments. I won't restrict that except if spammers begin to really annoy me.
2009-07-30T16:02:29  <ThomasWaldmann> btw, get rid of that revert thing if you encounter it in somewhere
2009-07-30T16:02:53  <dennda> yes
2009-07-30T16:10:02  <ThomasWaldmann> btw, more consistent would be to give it only to Known:, but not to All:
2009-07-30T16:10:15  <dennda> should I change that?
2009-07-30T16:10:19  <ThomasWaldmann> because Known is able to delete (thus he should also have rights to create)
2009-07-30T16:10:46  <dennda> don't we need it in both places then due to first match?
2009-07-30T16:11:04  <dennda> otherwise by default (modulo wikiconfig settings) not all users can create pages
2009-07-30T16:13:10  <ThomasWaldmann> otoh All: may not delete, so maybe he shouldn't create either
2009-07-30T16:13:48  <ThomasWaldmann> yes, that would mean a change of behaviour for better consistency
2009-07-30T16:15:23  <ThomasWaldmann> so maybe Known:read,write,create,delete All:read,write for now
2009-07-30T16:15:55  <ThomasWaldmann> we'll see with the beta whether we get complaints :)
2009-07-30T16:17:03  <ThomasWaldmann> btw, please move CREATE to the left of DELETE in rights_valid
2009-07-30T16:24:30  <_peck_> ThomasWaldmann: i uploaded a version with a patch to ldap_login
2009-07-30T16:26:20  <dennda> ThomasWaldmann: i'm sure as hell there'll be complains "why can't my users create pages anymore"
2009-07-30T16:39:55  <ThomasWaldmann> the default answer will be "because acl_rights_default tells so" :)
2009-07-30T16:40:35  <ThomasWaldmann> btw, add a HINT: ... to docs/CHANGES, please
2009-07-30T16:42:47  <ThomasWaldmann> dennda: on #moin, someone is asking for history capability :)
2009-07-30T17:22:08  <dennda> And I'll tell them it's your fault that it's telling so :D
2009-07-30T17:44:01  <CIA-43> Thomas Waldmann <tw AT waldmann-edv DOT de> default * 4925:c37ea194d626 1.9/MoinMoin/web/flup_frontend.py: flup_frontend: __import__ compatibility fix for python 2.4
2009-07-30T17:47:48  <ThomasWaldmann> so much for developers not using the minimum required python version :P
2009-07-30T17:54:28  <dimazest> ThomasWaldmann: i use 2.4
2009-07-30T17:54:55  <dimazest> does test exist for that?
2009-07-30T17:55:39  <dimazest> ThomasWaldmann: search code is kind of working with xappy, but i need to polish it
2009-07-30T17:57:18  <dennda> ThomasWaldmann: should trusted also have create? (since they also have delete)
2009-07-30T17:58:36  <ThomasWaldmann> dennda: yes
2009-07-30T17:58:48  <ThomasWaldmann> Trusted is better than Known
2009-07-30T17:58:58  <dennda> only trusted?
2009-07-30T18:07:20  <dennda> ThomasWaldmann: ^
2009-07-30T18:07:38  <ThomasWaldmann> ?
2009-07-30T18:08:04  <dennda> only add it to trusted or both trusted and known?
2009-07-30T18:08:07  <dennda> (since both have delete)
2009-07-30T18:08:12  <ThomasWaldmann> both
2009-07-30T18:08:35  <dennda> ok
2009-07-30T18:10:13  <_peck_> youhou
2009-07-30T18:15:12  <ThomasWaldmann> 2u2
2009-07-30T18:20:35  <_peck_> EWRONGCHAN
2009-07-30T18:45:52  <dennda> wtf what is broken here
2009-07-30T18:49:20  <dennda> (Pdb) AclWrappedRevision(self._item.create_revision(0), self)
2009-07-30T18:49:20  <dennda> {}
2009-07-30T18:49:30  <dennda> today is the day of strange python bugs for me
2009-07-30T18:55:29  <dennda> (Pdb) rev
2009-07-30T18:55:29  <dennda> {}
2009-07-30T18:55:29  <dennda> (Pdb) rev.timestamp
2009-07-30T18:55:30  <dennda> 1248972882L
2009-07-30T18:55:41  <dennda> getting closer :-)
2009-07-30T19:05:58  <dennda> ThomasWaldmann: did you change something with respect to test sandboxing lately? (i.e., every testcase having its own, clean backend)
2009-07-30T19:06:59  <ThomasWaldmann> i killed some non-working code a month ago iirc
2009-07-30T19:07:42  <ThomasWaldmann> that cloned some empty directory into the memory backend
2009-07-30T19:16:26  <dennda> aha
2009-07-30T19:16:47  <dennda> some tests are broken that relied on getting an empty backend
2009-07-30T19:16:53  <dennda> perhaps due to something else though
2009-07-30T19:27:08  <ThomasWaldmann> well, i don't think it makes a difference to clone something empty or not clone :P
2009-07-30T19:27:31  <ThomasWaldmann> and some tests rely on empty backend, some want some specific pages
2009-07-30T19:27:50  <dennda> at least the thing I have seen here is nonsense anyway
2009-07-30T19:28:20  <dennda> ok seems it was my fault
2009-07-30T19:28:53  <dennda> i had that test/ directory I removed lately copied over to some other place to see if that works and forgot to remove it, and then somehow that seems to have turned up in the test backends
2009-07-30T19:41:41  <ThomasWaldmann> try that xml stuff :)
2009-07-30T19:41:53  * ThomasWaldmann already had his first try
2009-07-30T19:42:53  <dennda> you don't happen to be a apple vendor by the way, do you? :)
2009-07-30T19:43:04  * dennda likely needs a new laptop
2009-07-30T19:52:14  <ThomasWaldmann> sometimes :)
2009-07-30T19:52:48  <ThomasWaldmann> but, as you are a student, iirc you likely get it cheaper with their student rebate
2009-07-30T19:52:56  <dennda> yes
2009-07-30T19:53:15  <ThomasWaldmann> brb
2009-07-30T20:19:54  * dennda tries to adapt language_setup.py
2009-07-30T20:34:59  <dennda> sigh... when did all this stuff break...
2009-07-30T20:36:44  <_peck_> ThomasWaldmann: i think my current patch is in a good shape, do you think i should change something ?
2009-07-30T20:37:28  <ThomasWaldmann> _peck_: i'll have a look in 10 mins
2009-07-30T20:39:05  <dennda> MoinMoin.datastruct is some dimazest-thing, correct?
2009-07-30T20:40:16  <dennda> because I can't log in with a user anymore and datastruct is involved
2009-07-30T20:40:40  <dimazest> dennda: yes that's mine
2009-07-30T20:40:53  <dennda> dimazest: can you check that?
2009-07-30T20:41:16  <dimazest> dennda: what should i do?
2009-07-30T20:41:46  <dennda> try to log in
2009-07-30T20:41:48  <dennda> with a user
2009-07-30T20:41:52  <dennda> and see if that works
2009-07-30T20:41:59  <dennda> that is broken for me in 2.0-storage
2009-07-30T20:42:15  <dimazest> let me clone it
2009-07-30T20:43:07  <dennda> no idea what that code does. difficult to tell for me whether it behaves correctly or not
2009-07-30T20:54:46  <dimazest> dennda: http://paste.pocoo.org/show/131776/
2009-07-30T20:55:30  <dennda> what is that now again...
2009-07-30T20:55:35  <dennda> i've not seen that bug
2009-07-30T20:57:25  <dennda> dimazest: can't reproduce. doesn't it even start underlay conversion?
2009-07-30T20:57:43  <dimazest> wait, i need to untar it
2009-07-30T20:59:06  *** amartani has quit IRC
2009-07-30T21:00:59  <dennda> actually you shouldn't need to
2009-07-30T21:02:13  <CIA-43> Thomas Waldmann <tw AT waldmann-edv DOT de> default * 4352:de0f87166f1c 1.8/MoinMoin/config/__init__.py: add gopher protocol to url_schemas
2009-07-30T21:02:14  <CIA-43> Thomas Waldmann <tw AT waldmann-edv DOT de> default * 4353:4c142c453f11 1.8/MoinMoin/auth/ldap_login.py: merge main
2009-07-30T21:05:25  * ThomasWaldmann looks at ldap group stuff
2009-07-30T21:09:24  <ThomasWaldmann> _peck_: indenting is 4 blanks per level (PEP8)
2009-07-30T21:09:46  <dennda> dimazest: any progress?
2009-07-30T21:09:53  <ThomasWaldmann> line 129/130 in the 3rd patch
2009-07-30T21:12:11  <ThomasWaldmann> _peck_: you didn't fix all what I told when I reviewed it 1st and 2nd time
2009-07-30T21:12:43  <ThomasWaldmann> (line 205, first term)
2009-07-30T21:13:52  <ThomasWaldmann> 221 pep8 failure - please run py.test
2009-07-30T21:15:04  <ThomasWaldmann> _peck_: do you have an irc log to read the old stuff again?
2009-07-30T21:15:22  <ThomasWaldmann> otherwise do it on the moin wiki
2009-07-30T21:16:48  <dennda> how do I become a superuser?
2009-07-30T21:18:02  <dennda> assuming for a moment that I was able to log in... :-)
2009-07-30T21:19:04  <ThomasWaldmann> wikiconfig superuser = ['loginname']
2009-07-30T21:19:20  <dennda> thanks
2009-07-30T21:20:48  <dennda> dimazest: what is MoinMoin.datastruct.WikiDicts supposed to be?
2009-07-30T21:24:12  <dennda> http://paste.pocoo.org/show/131782/
2009-07-30T21:24:16  <dennda> this is the traceback I get
2009-07-30T21:26:24  <dennda> dimazest: that only happens if I try to log in with a user account that doesn't exist
2009-07-30T21:26:42  <dennda> eeks
2009-07-30T21:26:55  <dennda> why is that datastruct code called like a bizillion times?
2009-07-30T21:27:13  * dennda added a print to it somewhere and now the print got executed *very* often
2009-07-30T21:31:27  <dennda> dreimark: around?
2009-07-30T21:33:16  <ThomasWaldmann> dennda: try browser language := english
2009-07-30T21:35:42  <ThomasWaldmann> dennda: that's a chicken and egg problem
2009-07-30T21:36:25  <ThomasWaldmann> auth wants i18n, i18n wants dicts, dicts want storage (and storage maybe needs a user first)
2009-07-30T21:37:19  <dennda> what problem exactly do you think that solves?
2009-07-30T21:37:35  <ThomasWaldmann> that was the problem, not the solution
2009-07-30T21:38:02  <dennda> i meant setting browser language to english. that sounded like a workaround
2009-07-30T21:39:04  <ThomasWaldmann> maybe it doesnt help, as it will maybe also try EnglishDict
2009-07-30T21:39:32  <dennda> 'EnglishDict' was what was printed that often, yes
2009-07-30T21:39:46  <dennda> my system already is english
2009-07-30T21:39:46  <ThomasWaldmann> let me look
2009-07-30T21:41:56  <ThomasWaldmann> dennda: is that msg not in your po files?
2009-07-30T21:42:21  <dennda> hu?
2009-07-30T21:43:10  <dennda> What I'm trying to do, btw, is get a very simple UI for syspages upgrade up and running so I can finish the syspages upgrade path for normal users
2009-07-30T21:43:53  <ThomasWaldmann> well, it only tries the dicts if it can't find it in po
2009-07-30T21:44:11  <dennda> I took language_setup.py and threw out stuff until it doesn't cause moin to raise an internal server error anymore. unfortunately I have no idea though how to add a "browse files" widget (i guess that's DataBrowserWidget) so that a user can upload an xml file
2009-07-30T21:44:52  <ThomasWaldmann> to work around that i18n problem us _ = lambda x: x
2009-07-30T21:45:18  <dennda> can you help me with that gui stuff?
2009-07-30T21:45:59  <dennda> ok
2009-07-30T21:53:51  <dimazest> i still get ImportError: No module named autoadmin
2009-07-30T21:54:11  <dennda> I don't know where you got that from, dimazest
2009-07-30T21:54:21  <dennda> I can run a fresh clone perfectly fine here
2009-07-30T21:54:34  <dennda> it starts underlay conversion automatically and all is well
2009-07-30T21:55:31  * ThomasWaldmann does some sysadmining
2009-07-30T21:56:04  <dennda> dimazest: to be precise, I don't have that module either
2009-07-30T21:56:43  <dennda> dimazest: actually, where did you get that wikiconfig_local.py from?
2009-07-30T21:56:47  <ThomasWaldmann> what do you need autoadmin for?
2009-07-30T21:57:02  <dennda> delete/rename it.
2009-07-30T21:59:03  <dimazest> ok it's my fault, i think i was playing with openid and autoadmin
2009-07-30T22:00:20  <_peck_> ThomasWaldmann: py.test doesn't give me anything, which parameters should i give it ?
2009-07-30T22:00:43  <dennda> _peck_: none. just run it inside a folder that contains the tests
2009-07-30T22:01:03  <dennda> specifically, you can run py.test MoinMoin/_tests/test_sourcecode.py (or similar)
2009-07-30T22:02:02  <_peck_> ah ok it's better
2009-07-30T22:02:33  <dennda> if your file is accessible from the test it should check your code for styleguide violations and output them
2009-07-30T22:03:53  <dimazest> now i get another error http://paste.pocoo.org/show/131788/
2009-07-30T22:04:16  <dennda> ThomasWaldmann: What do I need to do so that a user can upload an xml file which is then stored somewhere temporarily?
2009-07-30T22:05:43  <dennda> oh
2009-07-30T22:05:45  <ThomasWaldmann> what is that good for?
2009-07-30T22:06:04  <dennda> ThomasWaldmann: Didn't we agree that'd be the way for users to upgrade sys pages?
2009-07-30T22:06:19  <dennda> can it be that the fs backend doesn't work on mac?
2009-07-30T22:07:12  <ThomasWaldmann> the packages we deliver are currently (1.9) stored as page attachments
2009-07-30T22:07:13  <dimazest> dennda: probaly, tomorrow i'll try on linux
2009-07-30T22:07:31  <dennda> dimazest: please change backend_uri = 'fs:instance' in wikiconfig.py to backend_uri = 'memory:'
2009-07-30T22:08:25  <dennda> ThomasWaldmann: well I just need some way to get both the xml file and the request object at the same time. I don't really care what the actual gui looks like
2009-07-30T22:11:35  <ThomasWaldmann> well, the 1:1 approach would be to have the xml data in some items
2009-07-30T22:11:43  <ThomasWaldmann> like LanguageSetup/de
2009-07-30T22:12:27  <dimazest> dennda: i use
2009-07-30T22:12:43  <dimazest> class Config(LocalConfig): backend_uri = 'memory:'
2009-07-30T22:12:54  <dimazest> but it still uses fs backend
2009-07-30T22:13:14  <dimazest> i need to go now, see you tomorrow
2009-07-30T22:13:21  <dennda> dimazest: touch .success_creating_dev_wiki
2009-07-30T22:13:26  <dennda> in 2.0-storage/
2009-07-30T22:14:42  <dennda> ThomasWaldmann: ok. still I need a way for a user to specify and send that so I can get the item, the xml from it and perform the upgrade
2009-07-30T22:17:51  <ThomasWaldmann> use a url arg
2009-07-30T22:18:53  <dimazest> dennda: i was able to login
2009-07-30T22:19:40  <dennda> dimazest: with a user that did not yet exist?
2009-07-30T22:20:35  <dimazest> no with not existing user i get Internal Server Error
2009-07-30T22:22:16  <dennda> that's what I'm talking about
2009-07-30T22:22:34  <CIA-43> Christopher Denter <moin GUESSWHAT the DASH space DASH station ROUNDTHING com> default * 6300:6460d8b5874e 2.0-storage/MoinMoin/config/multiconfig.py: storage: -All:create, +Trusted:create, +Known:create
2009-07-30T22:22:36  <CIA-43> Christopher Denter <moin GUESSWHAT the DASH space DASH station ROUNDTHING com> default * 6301:9adb0712581e 2.0-storage/MoinMoin/storage/_tests/test_middleware_acl.py: storage: add acl test for 'create'
2009-07-30T22:22:37  <CIA-43> Christopher Denter <moin GUESSWHAT the DASH space DASH station ROUNDTHING com> default * 6302:acf8030954d9 2.0-storage/MoinMoin/storage/backends/memory.py: storage: make membackend return storedrevisions in history
2009-07-30T22:22:39  <CIA-43> Christopher Denter <moin GUESSWHAT the DASH space DASH station ROUNDTHING com> default * 6303:e9d093c1bf01 2.0-storage/MoinMoin/storage/_tests/test_backends.py: storage: remove stupid test clause
2009-07-30T22:22:41  <CIA-43> Christopher Denter <moin GUESSWHAT the DASH space DASH station ROUNDTHING com> default * 6304:f2a09484cd3b 2.0-storage/MoinMoin/storage/_tests/test_middleware_acl.py: storage: fix test
2009-07-30T22:25:49  <dimazest> dennda: it checks Page(self.request, u'EnglishDict').exists()
2009-07-30T22:26:37  <dimazest> ok, i need to go to sleep, good night
2009-07-30T22:31:11  <dennda> well yeah I see that
2009-07-30T22:31:13  <dennda> good night
2009-07-30T22:59:29  <dennda> ThomasWaldmann: Hmmm. Do I need to allow my new action somewhere? I get "excluded or unknown action"
2009-07-30T22:59:55  *** amartani has joined #moin-dev
2009-07-30T22:59:59  <ThomasWaldmann> dennda: no
2009-07-30T23:00:17  <dennda> hm
2009-07-30T23:01:09  <dennda> ThomasWaldmann: What I did was create an item "foo", fill in the test syspages data you gave me as data, save that item (or rather the revision) and pass it to unserialize
2009-07-30T23:04:36  <ThomasWaldmann> yes, and?
2009-07-30T23:05:13  <dennda> ok, nothing. so that seems to sound reasonable :-)
2009-07-30T23:05:22  <dennda> now I just need to find out why it fails
2009-07-30T23:14:31  <dennda> yay! :-)
2009-07-30T23:17:35  <ThomasWaldmann> :)
2009-07-30T23:18:50  <dennda> i'll committ a proof of concept just not to lose anything when fiddling around further
2009-07-30T23:19:33  * dennda <3 ducktyping
2009-07-30T23:41:11  <ThomasWaldmann> looks like create acl works, some tests fail because of missing permissions
2009-07-30T23:41:31  <dennda> which tests exactly?
2009-07-30T23:45:56  <ThomasWaldmann> just run them :)
2009-07-30T23:49:32  *** amartani has quit IRC
2009-07-30T23:50:24  <CIA-43> Christopher Denter <moin GUESSWHAT the DASH space DASH station ROUNDTHING com> default * 6305:a9aaf432ac02 2.0-storage/MoinMoin/ (3 files in 2 dirs): storage: Proof of concept for syspage upgrade via webnterface by providing the name of an item containing the syspage xml data. There's much that needs to be done...
2009-07-30T23:54:36  <dennda> anyway I'm off for today. that was a loooooong day for me :-)
2009-07-30T23:55:00  <dennda> (surprised I havn't fallen asleep until now although my eyes are heavy since 5 hours at least)
2009-07-30T23:57:17  <ThomasWaldmann> gn
2009-07-30T23:57:21  <dennda> n8