MoinMoin: MoinMoinChat/Logs/moin-dev/2007-09-09

2007-09-09T00:15:01  <dreimark> my test case py.test _tests/test_Page.py  macro/_tests/test_GetVal.py
2007-09-09T00:22:17  <dreimark> gn
2007-09-09T00:23:25  <ThomasWaldmann> {{http://www.google.de/intl/de_de/images/logo.gif|Google Logo|height="100", width="1000"}}
2007-09-09T00:25:45  <ThomasWaldmann> gn dreimark
2007-09-09T00:29:03  <xorAxAx> ThomasWaldmann: | is a safe char, right?
2007-09-09T00:29:11  <xorAxAx> so that will break with valid urls with | in them
2007-09-09T00:31:34  <ThomasWaldmann> yes
2007-09-09T00:31:51  <ThomasWaldmann> but that's how creole is specified
2007-09-09T00:35:57  <xorAxAx> well, openxml is specified somehow as well :)
2007-09-09T02:20:06  <CIA-27> moin: Thomas Waldmann <tw AT waldmann-edv DOT de> * 2812:6ac9e14a0241 1.7/MoinMoin/parser/text_moin_wiki.py: wiki parser: link/transclusion parameter parsing
2007-09-09T08:37:18  <grzywacz> moin
2007-09-09T10:26:07  <vpv> moin, here's a suggestion for improving error messages from the fedora-docs list: https://www.redhat.com/archives/fedora-docs-list/2007-September/msg00010.html
2007-09-09T10:26:44  <vpv> personally, I'm a bit suprised that someone doesn't allow cookies by default, but I still thought to pass that to you...
2007-09-09T11:06:05  <dreimark> strange that he does not open an usueablity bug on MM
2007-09-09T11:06:58  <dreimark> " and are somewhat non-trivial to ask of MoinMoin"
2007-09-09T11:10:45  <dreimark> bbl
2007-09-09T11:52:23  <ThomasWaldmann> moin
2007-09-09T13:04:45  <CIA-27> moin: Thomas Waldmann <tw AT waldmann-edv DOT de> * 2813:82a39e6078d7 1.7/MoinMoin/parser/text_moin_wiki.py: wiki_parser: refined acceptable_keys
2007-09-09T19:44:37  <dreimark> moin
2007-09-09T19:45:07  <dreimark> ThomasWaldmann: if ImageLink gots removed calls need to be converted too
2007-09-09T20:01:51  <dreimark> in that merge text should be the explanaition added why those pages needs to be checked for renaming.
2007-09-09T20:02:12  <dreimark> because of changes to  _ and ' ' ...
2007-09-09T20:18:22  <dreimark> data.pre160 as backup is a good idea, if someone missed to do it himself. We should inform about that change because you need some more space ;)
2007-09-09T20:20:34  <dreimark> further checking of that migrated wiki I do tomorrow, currently it looks well done :)
2007-09-09T20:22:26  <dreimark> bbl
2007-09-09T20:43:12  <ThomasWaldmann> moin
2007-09-09T21:37:41  * ThomasWaldmann writes ImageLink converter
2007-09-09T22:19:52  <CIA-27> moin: Thomas Waldmann <tw AT waldmann-edv DOT de> * 2814:7bde33d15531 1.7/MoinMoin/script/migration/ (_conv160_wiki.py _tests/test_conv160_wiki.py): 1.6 converter: convert ImageLink to new link/transclusion syntax
2007-09-09T22:20:36  <ThomasWaldmann> dreimark: please check if the test cases cover everything needed
2007-09-09T22:36:45  <ThomasWaldmann> dreimark: btw, that ImageLink to new syntax conversion will be useful later also, when we convert from attachments to items (less cases to consider, more explicit)
2007-09-09T22:37:14  <dreimark> :)
2007-09-09T22:38:58  <ThomasWaldmann> and the converter can be even made to track renames (not implemented yet)
2007-09-09T22:38:58  <dreimark> ThomasWaldmann: I think I add some more tests
2007-09-09T22:39:51  <dreimark> and do fix the comments copy left over
2007-09-09T22:40:12  <dreimark> ThomasWaldmann: track renames would be fine too
2007-09-09T22:42:13  <ThomasWaldmann> (22:39) <  dreimark> and do fix the comments copy left over
2007-09-09T22:42:18  <ThomasWaldmann> what does that mean?
2007-09-09T22:42:52  <dreimark> I have copied parts of that test from the Hits test and on some places I have not exchanged the comment
2007-09-09T22:46:16  <dreimark> ThomasWaldmann:
2007-09-09T22:47:17  <dreimark> a test with an uploaded image is missing
2007-09-09T22:47:50  <dreimark> and a test to link to itself
2007-09-09T23:01:04  <ThomasWaldmann> [[ImageLink(test.png)]] this is a link to itself, isnt it?
2007-09-09T23:01:18  <dreimark> yeah
2007-09-09T23:01:32  <dreimark> it works
2007-09-09T23:01:47  <dreimark> with an url but that
2007-09-09T23:02:37  <dreimark> with an url its easy to test but that is differnt from local storage
2007-09-09T23:04:36  <ThomasWaldmann> just add the missing test, i guess that is easier than explaining :)
2007-09-09T23:05:09  <dreimark> doing
2007-09-09T23:32:08  <CIA-27> moin: Reimar Bauer <rb.proj AT googlemail DOT com> * 2815:104590a28e54 1.7/MoinMoin/macro/_tests/test_ImageLink.py: test_ImageLink: added two new tests, fixed class name
2007-09-09T23:39:27  <ThomasWaldmann> dreimark: if we kill ImageLink, we don't need tests for it
2007-09-09T23:40:20  <dreimark> I know but then the parser has to be tested.
2007-09-09T23:40:30  <ThomasWaldmann> in case it was unclear, I meant the MoinMoin/script/migration/_tests/ tests
2007-09-09T23:41:35  <dreimark> oops, yes that was unclear
2007-09-09T23:46:47  <dreimark> I do check that tomorrow
2007-09-09T23:48:50  <dreimark> gn