2008-08-12T02:02:07 <CIA-54> Christopher Denter <moin GUESSWHAT the DASH space DASH station ROUNDTHING com> default * 4758:290f9cdd28b5 1.8-storage-cdenter/MoinMoin/action/AttachFile.py: storage: action/AttachFile: Make move basically work (by renaming the item in which the attachment resides) for now
2008-08-12T07:42:48 <xorAxAx> dennda: i dont really understand what the problem of that approach is
2008-08-12T07:42:56 <xorAxAx> it looks correct to call rename
2008-08-12T08:58:01 <ThomasWaldmann> moin
2008-08-12T12:07:27 <ThomasWaldmann> dennda: + if not backend.has_item(new_pagename): # XXX Is this a proper way to do the check?
2008-08-12T12:08:18 <ThomasWaldmann> (in the attachment renaming code)
2008-08-12T12:09:09 <ThomasWaldmann> i would say "no" because attachments are items on their own now. we also allow creation of subpages without parent page, so there is no point in disallowing it for attachments
2008-08-12T12:09:48 <ThomasWaldmann> also, the error msg you emit there is talking about "not enough rights", but your code has no ACL check at all)
2008-08-12T12:15:32 <ThomasWaldmann> ThomasWaldmann: the fs17 converter needs to copy parent-page acls to attachments
2008-08-12T12:19:57 <ThomasWaldmann> ThomasWaldmann: except when acl_hierarchic is True
2008-08-12T12:42:15 <xorAxAx> we will have a meeting tomorrow, right?
2008-08-12T12:42:39 <xorAxAx> then we should ping everybody again because thomas started the last one a bit ambiguously
2008-08-12T12:45:38 <ThomasWaldmann> sure we have a meeting, like every week
2008-08-12T12:46:14 <xorAxAx> good.
2008-08-12T12:46:42 <xorAxAx> byeongweon, dennda, dreimark, mitsuhiko, PawelPacana, TheSheep, waldi, zenhase: meeting like always, dont forget :)
2008-08-12T12:46:56 <byeongweon> xorAxAx: ok.
2008-08-12T12:49:44 <ThomasWaldmann> http://moinmo.in/GoogleSoc2008 btw i didn't see anyone starting the "what you can do in the final days" checklist
2008-08-12T13:11:44 <CIA-54> Bastian Blank <bblank@thinkmo.de> default * 48:547347034e65 EmeraldTree/emeraldtree/ElementTree.py: ElementTree - Change handling of undefined namespace
2008-08-12T13:11:44 <CIA-54> Bastian Blank <bblank@thinkmo.de> default * 49:78ef2b77ca4a EmeraldTree/emeraldtree/tests/test_ElementTree.py: Tests for ElementTree - Fix tests for default namespace, fix types
2008-08-12T13:18:19 <CIA-54> Bastian Blank <bblank@thinkmo.de> default * 4337:6ed8536696ac 1.8-dom-bblank/MoinMoin/converter2/moinwiki_in.py: Moin Wiki input converter - Parse strike
2008-08-12T13:34:51 <CIA-54> Thomas Waldmann <tw AT waldmann-edv DOT de> default * 4759:05e3dbd1f86c 1.8-storage-cdenter/MoinMoin/storage/backends/fs17.py: fs17 storage backend: added some TODO for attachment revision metadata
2008-08-12T13:43:50 <CIA-54> Bastian Blank <bblank@thinkmo.de> default * 4338:6bfcfedf80a4 1.8-dom-bblank/MoinMoin/converter2/html_out.py: HTML output converter - Support text-decoration attribute
2008-08-12T13:43:51 <CIA-54> Bastian Blank <bblank@thinkmo.de> default * 4339:c2aba4a5b858 1.8-dom-bblank/MoinMoin/converter2/ (_tests/test_moinwiki_in.py moinwiki_in.py): Moin Wiki input converter - Support strike and underline
2008-08-12T13:43:53 <CIA-54> Bastian Blank <bblank@thinkmo.de> default * 4340:c9fed0867dd9 1.8-dom-bblank/MoinMoin/converter2/macro.py: Macro converter - Report caller of request.write
2008-08-12T13:58:12 <PawelPacana> xorAxAx: ack
2008-08-12T16:21:16 <dreimark> xorAxAx: ok
2008-08-12T16:32:06 <dreimark> bbl
2008-08-12T18:18:30 <ThomasWaldmann> bbl
2008-08-12T19:45:02 <ThomasWaldmann> re
2008-08-12T19:58:34 <ThomasWaldmann> waldi: the roman numbered stuff emits currently a bullet list
2008-08-12T19:59:10 <ThomasWaldmann> waldi: definition lists don't work
2008-08-12T20:00:39 <ThomasWaldmann> waldi: table attrs
2008-08-12T20:01:02 <ThomasWaldmann> waldi: indented blocks
2008-08-12T20:03:35 <CIA-54> Christopher Denter <moin GUESSWHAT the DASH space DASH station ROUNDTHING com> default * 4762:3696ff0bdd97 1.8-storage-cdenter/MoinMoin/action/ (AttachFile.py __init__.py): storage: action/attachfile: Allow directly adressing attachments via mywiki.de/page/attachment
2008-08-12T20:04:04 <ThomasWaldmann> dennda: did you read what I told you?
2008-08-12T20:05:04 <dennda> ThomasWaldmann: that the backend.has_item() check is not that good?
2008-08-12T20:05:25 <dennda> or what are you talking of?
2008-08-12T20:05:51 <ThomasWaldmann> yes
2008-08-12T20:06:03 <dennda> If only CIA wasn't such a lazy bastard
2008-08-12T20:06:24 <dennda> it would have told you of the following changeset: http://hg.moinmo.in/moin/1.8-storage-cdenter/rev/fe20cb99dcd3
2008-08-12T20:06:31 <ThomasWaldmann> seen your changeset
2008-08-12T20:07:27 <ThomasWaldmann> see 12:09
2008-08-12T20:11:33 <dennda> What's the problem? You want attachments for pages that don't exist yet?
2008-08-12T20:12:14 <ThomasWaldmann> see 12:09
2008-08-12T20:12:53 <dennda> I have seen 12:09
2008-08-12T20:13:05 <dennda> See 20:11 :)
2008-08-12T20:13:17 <dennda> and probably ignore the first sentence
2008-08-12T20:13:30 <ThomasWaldmann> so you have noted the similarity between attachments as items and pages as items?
2008-08-12T20:14:22 <ThomasWaldmann> or rather SUBpages
2008-08-12T20:14:52 <dennda> well sure
2008-08-12T20:16:03 <dennda> But I am still not completely sure what you want me to do, especially because you didn't agree on my last question :)
2008-08-12T20:16:19 <ThomasWaldmann> you could just drop that code
2008-08-12T20:19:10 <dennda> Ah so the answer to my question is "yes"
2008-08-12T20:19:16 <dennda> ok well, if you want it that way, I am fine with it
2008-08-12T20:19:41 <dennda> the problem then is, that the user has no buttons or something to get to the attachments of a non existant page
2008-08-12T20:19:50 <dennda> there probably is some url-parameter
2008-08-12T20:20:48 <ThomasWaldmann> dennda: try it with 1.7, i guess you can also create attachments for nonexist pages there
2008-08-12T20:21:13 <dennda> ok, I have no objections
2008-08-12T20:21:49 <ThomasWaldmann> ok, so, on what else did you work today?
2008-08-12T20:25:18 <dennda> mainly the changeset at 20:03 and some talking with my mentor on other things like mimetype handling and editor conflicts that need to be solved
2008-08-12T20:25:36 <dennda> after those remaining things are done I will likely work on the TODO list
2008-08-12T20:26:17 <dreimark> dennda: do we meet tommorrow johill ?
2008-08-12T20:26:27 <dennda> arg, wanted to ask him. forgot
2008-08-12T20:26:43 <ThomasWaldmann> btw, you can also work on both in parallel (maybe the not so demanding stuff after wakeup and before going to bed :)
2008-08-12T20:26:56 <ThomasWaldmann> ok, gtg, bbl
2008-08-12T20:27:13 <dennda> hm good idea
2008-08-12T20:28:47 <dennda> dreimark: I sent a message. Dunno if he'll read it in time though
2008-08-12T20:29:08 <dreimark> :)
2008-08-12T20:35:45 <dennda> 13:00 I assume
2008-08-12T20:35:53 <dreimark> yeah
2008-08-12T20:35:57 <dennda> Midnight, so to say :(
2008-08-12T20:36:08 <dreimark> hehe
2008-08-12T20:42:46 <dreimark> dennda: acls will get an new ui for attachments or how can we set acls for them?
2008-08-12T20:43:10 <dennda> dreimark: I was ordered to "just forget about ACLs completely"
2008-08-12T20:43:32 <dreimark> ok
2008-08-12T20:44:18 <ThomasWaldmann> there's no time left for that anyway
2008-08-12T20:44:41 <dreimark> byeongweon: please try a copy of HelpOnMoinWikiSyntax with your changes. save it first with the text edirot and then with the gui and look at the changes
2008-08-12T20:44:43 <ThomasWaldmann> and what we will likely do about acls is to have some storage api compliant middleware
2008-08-12T20:45:22 <dreimark> byeongweon: it removes currently every bulletlist
2008-08-12T20:45:26 <dreimark> xorAxAx: ^^
2008-08-12T20:45:35 <ThomasWaldmann> oops
2008-08-12T20:46:18 <dreimark> byeongweon: and it deletes indenting blanks
2008-08-12T20:48:22 <xorAxAx> dreimark: please put that also onto the wikipage, thanks
2008-08-12T20:48:34 <dreimark> doing currently
2008-08-12T20:51:48 <dreimark> byeongweon: http://moinmo.in/HelpOnMoinWikiSyntax?action=raw
2008-08-12T20:51:57 <dreimark> don't copy the acl settings
2008-08-12T20:52:33 <dreimark> bbl
2008-08-12T21:12:13 <CIA-54> Bastian Blank <bblank@thinkmo.de> default * 4341:0982b3df678d 1.8-dom-bblank/MoinMoin/converter2/html_out.py: HTML output converter - Set list attributes
2008-08-12T21:12:14 <CIA-54> Bastian Blank <bblank@thinkmo.de> default * 4342:82bb07b9774d 1.8-dom-bblank/MoinMoin/converter2/moinwiki_in.py: Moin Wiki input converter - Set list styles
2008-08-12T22:00:25 <dreimark> PawelPac1na: http://moinmo.in/PawelPacana/MercurialBackend?action=AttachFile&do=view&target=traceback.html
2008-08-12T22:00:55 <dreimark> could not find MARK ?
2008-08-12T22:00:59 <dreimark> hi fpletz
2008-08-12T22:02:04 <dreimark> TheSheep: PawelPac1na probably I do something fundemental wrong
2008-08-12T22:02:32 <xorAxAx> dreimark: yes, you are reimar and not mark :)
2008-08-12T22:02:43 <dreimark> he
2008-08-12T22:05:35 <TheSheep> dreimark: at some point there was a bug in the metadata stroing of the backend
2008-08-12T22:05:40 <TheSheep> dreimark: storing
2008-08-12T22:05:50 <dreimark> TheSheep: what can I do?
2008-08-12T22:06:13 <TheSheep> dreimark: so if you saved some revisions then, they have broken metadata, and you should use a new repository
2008-08-12T22:06:26 <TheSheep> dreimark: sorry about that
2008-08-12T22:06:39 <TheSheep> dreimark: unless this is a fresh repository?
2008-08-12T22:07:12 <dreimark> TheSheep: no big problem (until we are not in production), I try a new one ;)
2008-08-12T22:08:27 <dreimark> TheSheep: works now
2008-08-12T22:08:35 <TheSheep> dreimark: great, thanks for testing
2008-08-12T22:08:54 <dreimark> TheSheep: how can I access underlay ?
2008-08-12T22:09:35 <dennda> dreimark: With the new storage layer?
2008-08-12T22:09:54 <TheSheep> dreimark: I think it will have a separate storage, maybe even that read-only fs17
2008-08-12T22:10:19 <TheSheep> dreimark: I'm not sure
2008-08-12T22:10:20 <dreimark> example please if there is one.
2008-08-12T22:10:48 <dennda> http://paste.pocoo.org/show/82020/
2008-08-12T22:10:54 <dennda> If that is what you mean
2008-08-12T22:11:44 <dennda> That just uses the clone method to transfer all items from the old backend to the new backend
2008-08-12T22:12:24 <TheSheep> dennda: there won't be any underlay?
2008-08-12T22:12:44 <TheSheep> dennda: or should there be a seaparate backend, and some sort of 'merging' wrapper around them?
2008-08-12T22:13:04 <dennda> What makes you think there won't be any underlay?
2008-08-12T22:13:13 <TheSheep> dennda: there is none now
2008-08-12T22:13:44 <dennda> With "underlay" we are talking of the stock-pages that Moin ships, correct?
2008-08-12T22:13:46 <TheSheep> dennda: not really important, I'm just curious
2008-08-12T22:13:52 <TheSheep> dennda: yes
2008-08-12T22:13:53 <dennda> dreimark: johill will be there tomorrow if he can manage it
2008-08-12T22:14:03 <dreimark> dennda: very good
2008-08-12T22:14:33 <dreimark> [connverting FSPageBackend to MercurialBackend]: .....................
2008-08-12T22:14:42 <dreimark> is running currently
2008-08-12T22:15:09 <dennda> TheSheep: Why is there none? You could go several approaches. You could either copy the data via that fs17 backend into your new backend and have the same effect as before, or you could use a seperate backend that only contains the underlay and use some code that, if an item is not in your usual backend, checks if it exists in the underlay backend
2008-08-12T22:15:18 <dennda> Which basically is what you just described
2008-08-12T22:15:43 <dennda> dreimark: That's the verbose output the clone method has. You can disable it if you don't want it to be that verbose.
2008-08-12T22:15:53 <TheSheep> dennda: shouldn't that be build in in moin?
2008-08-12T22:16:46 <dennda> If you go for the latter approach (which has not been decided, to my knowledge), yes, probably
2008-08-12T22:17:24 <TheSheep> ok, thanks, that's what I meant by 'there is no underlay now' :)
2008-08-12T22:17:51 <dennda> not out of the box, that is
2008-08-12T22:18:22 <TheSheep> sure, one has to just implement it ;)
2008-08-12T22:18:46 <dennda> "one" is a busy guy, apparently :)
2008-08-12T22:19:25 <TheSheep> if one desires to use a bleeding edge version of moin, one has what he asked for ;)
2008-08-12T22:19:40 <TheSheep> s/he/one
2008-08-12T22:19:41 <dennda> indeed
2008-08-12T22:20:12 <dreimark> dennda: how much failing pages do you have seen?
2008-08-12T22:20:24 <dennda> shouldn't be too hard. a simple wrapper should be enough
2008-08-12T22:20:45 <dennda> dreimark: There are quite a few. I don't know for sure why they fail. I guess PawelPac1na can tell you more about the reason for the failures
2008-08-12T22:21:11 <dreimark> ok, will ask
2008-08-12T22:25:55 <dennda> What was the difference between edit_log_addr and edit_log_hostname?
2008-08-12T22:33:55 <dreimark> PawelPac1na: MoinMoin/security/__init__.py:78: UnicodeWarning: Unicode equal comparison failed to convert both arguments to Unicode - interpreting them as being unequal
2008-08-12T22:34:42 <dreimark> TheSheep: Can I expect to have a RC page ?
2008-08-12T22:36:24 <dreimark> dennda: I think I should disable the clone line after I have once cloned
2008-08-12T22:38:57 <TheSheep> dreimark: if you migrated the underlay, then yes, if not, you have to create it
2008-08-12T22:39:28 * dennda deletes his answer since TheSheep was faster
2008-08-12T22:39:35 <dreimark> I have had a page created and deleted it
2008-08-12T22:40:15 <dreimark> the new question is now does the backend know to send me now the underlay page
2008-08-12T22:40:22 <TheSheep> dennda: sorry
2008-08-12T22:40:30 <dreimark> and not the non existing page dialog
2008-08-12T22:40:40 <TheSheep> dreimark: that part is still missing
2008-08-12T22:41:05 <TheSheep> dreimark: basically, there is no "double" storage yet
2008-08-12T22:41:59 <TheSheep> dreimark: what if you wanted to delete the page also from underlay?
2008-08-12T22:44:28 <dreimark> in principle that is possible by ReplaceUnderlay|filename|pagename from a package file
2008-08-12T22:44:36 <dreimark> http://moinmo.in/HelpOnPackageInstaller
2008-08-12T22:47:26 <dreimark> TheSheep: the german RC page works,
2008-08-12T22:47:37 <dreimark> now I get a mpatchError or a new page
2008-08-12T22:47:53 <TheSheep> mpatch?
2008-08-12T22:48:14 <dreimark> that was the title, patch cannot be decoded
2008-08-12T22:48:42 * dreimark adds a traceback
2008-08-12T22:50:39 <dreimark> TheSheep: PawelPac1na http://moinmo.in/PawelPacana/MercurialBackend?action=AttachFile&do=view&target=traceback2.html
2008-08-12T22:51:24 <dreimark> hmm may be TestSeite already exists
2008-08-12T22:51:49 <dreimark> moment I guess my fault
2008-08-12T22:51:56 * dreimark tries again
2008-08-12T22:54:37 <dreimark> TheSheep PawelPac1na I'd prefer also an ascii name_mapping file
2008-08-12T22:55:12 <xorAxAx> dreimark: ?
2008-08-12T22:59:32 <dreimark> I have a 0 byte file in rev and I want to know which page name it is
2008-08-12T23:00:02 <dreimark> so i have currently to look for this.name
2008-08-12T23:00:12 <dreimark> that way it is easy
2008-08-12T23:01:23 <dreimark> so I don't need that mapping file
2008-08-12T23:02:22 <dreimark> the RC page in my wiki has 0 bytes
2008-08-12T23:04:13 <dreimark> TheSheep: PawelPac1na are there some profiling timetests available?
2008-08-12T23:04:28 <TheSheep> no
2008-08-12T23:05:52 <dreimark> it looks to me that string to edit a page is a bit slower as on other instances
2008-08-12T23:06:13 <dreimark> s/string/starting/
2008-08-12T23:06:47 <dreimark> the lower part of the markup descriptions takes very long
2008-08-12T23:07:00 <ThomasWaldmann> re
2008-08-12T23:07:08 * ThomasWaldmann works on fs17
2008-08-12T23:08:07 <TheSheep> dreimark: yes, it's not very optimized, we have several ideas on how to make it at least as fast as the fs backend, but for now we want to just have somthing working
2008-08-12T23:08:30 <dreimark> ok
2008-08-12T23:09:18 <dreimark> TheSheep: my problem is I don't know what in work is or what is for whatever reason is broken
2008-08-12T23:10:02 <dreimark> so I think I can just report
2008-08-12T23:10:25 <TheSheep> dreimark: we are grateful for all the problems you find, they all need fixing after all
2008-08-12T23:12:33 <dreimark> I guess it is known that all macros handling pagename of the old rules needs some adjustments, e.g. LikePages, EditTemplates. So I go more to look into the edit / saving etc. direction
2008-08-12T23:12:47 <dreimark> for now
2008-08-12T23:14:58 <CIA-54> Christopher Denter <moin GUESSWHAT the DASH space DASH station ROUNDTHING com> default * 4763:5b61380171d7 1.8-storage-cdenter/MoinMoin/action/AttachFile.py: storage: action/attachfile: allow creation of attachments on pages that have not been created yet
2008-08-12T23:14:58 <CIA-54> Christopher Denter <moin GUESSWHAT the DASH space DASH station ROUNDTHING com> default * 4764:c1e69d878f5e 1.8-storage-cdenter/MoinMoin/action/info.py: storage: fix info action on attachments. don't fail if certain metadata is not available
2008-08-12T23:22:02 * ThomasWaldmann slaps CIA-54
2008-08-12T23:25:16 <PawelPacana> re
2008-08-12T23:26:26 <PawelPacana> dreimark: migration is slow, and working on underlay is slow because hg is throwing some exceptions ('compression') i didnt figure out yet :o
2008-08-12T23:27:40 <dreimark> .oO can one explain why this can be the result of an traceback and can be used to create a page ?
2008-08-12T23:27:49 <dreimark> http://moinmo.in/PawelPacana/MercurialBackend/BugReports?action=AttachFile&do=view&target=traceback3.html
2008-08-12T23:28:21 <dreimark> ThomasWaldmann: xorAxAx ^^
2008-08-12T23:29:54 <dreimark> PawelPacana: ok, i have started to write some reports of my findings at http://moinmo.in/PawelPacana/MercurialBackend/BugReports
2008-08-12T23:59:07 <ThomasWaldmann> zenhase: 2 days no commit? MoinMoin: MoinMoinChat/Logs/moin-dev/2008-08-12 (last edited 2008-08-12 00:15:02 by IrcLogImporter)
