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

2007-09-16T08:08:48  <dreimark> I try to get this working ow why it occures MoinMoinBugs/1.6ErrorWhenFindNoResults
2007-09-16T09:07:04  <TheSheep>     Dependencies = Dependencies
2007-09-16T09:07:05  <TheSheep> lol
2007-09-16T09:07:33  <TheSheep> ain't python sweet?
2007-09-16T15:04:58  <dreimark> moin
2007-09-16T15:05:43  <dreimark> ThomasWaldmann: seems we better should write msg strings without \"
2007-09-16T15:11:47  <dreimark> wait another idea
2007-09-16T15:54:44  <ThomasWaldmann> moin
2007-09-16T15:54:53  <ThomasWaldmann> dreimark: ?
2007-09-16T15:58:45  <dreimark> ThomasWaldmann: I am searching no result yet
2007-09-16T15:59:26  <ThomasWaldmann> btw, test.wikiwikiweb.de has known i18n problems
2007-09-16T15:59:51  <dreimark> ok
2007-09-16T16:00:17  <ThomasWaldmann> (it is because i18n stuff also has links and macros in old syntax and this is not converted yet)
2007-09-16T16:01:23  <ThomasWaldmann> but I hope I can fix some of that today
2007-09-16T16:01:36  <dreimark> :)
2007-09-16T16:09:47  <ThomasWaldmann> yesterday, I found a small ambiguity in [[target|{{image|desc}}|target params]] syntax
2007-09-16T16:09:56  <ThomasWaldmann> but it is fixed now
2007-09-16T16:22:46  <dreimark> watching 99 balloons
2007-09-16T17:09:01  <dreimark> ThomasWaldmann: the problem is caused from WikiNames in the messages
2007-09-16T17:10:15  <dreimark> HelpOnSearching is translated in spain to <a href="/AyudaSobreBAsquedas">AyudaSobreBúsquedas</a>
2007-09-16T17:10:57  <dreimark> <a href="/AyudaSobreB % C3 % BAsquedas">AyudaSobreBúsquedas</a>
2007-09-16T17:13:06  <dreimark> % C  is unknown for substitution
2007-09-16T17:14:35  <dreimark> one solution could be to substitute translated wikiwords too
2007-09-16T17:15:49  <dreimark> e.g.
2007-09-16T17:15:57  <dreimark> text = _( "Your search query {{{\"%s\"}}} didn't return any results. Please change some "
2007-09-16T17:15:57  <dreimark> "terms and refer to %s for more information.%s") % ('ADD', _('HelpOnSearching'), 'blabla')
2007-09-16T17:19:42  <TheSheep> dreimark: maybe they should be just parsed as wiki text?
2007-09-16T17:21:33  <ThomasWaldmann> dreimark: i could understand %C on the left side of % operator making trouble, but in that case it is on the right side!?
2007-09-16T17:22:05  <dreimark> no we have it on the left side
2007-09-16T17:22:14  <ThomasWaldmann> where?
2007-09-16T17:22:36  <dreimark> HelpOnSearching is translated before substitution
2007-09-16T17:22:55  <dreimark> Al buscar <tt>"%s"</tt> no devolvió ningún resultado. P ...
2007-09-16T17:23:28  <dreimark> and resolved to its url
2007-09-16T17:23:37  <ThomasWaldmann> i dont see the problem
2007-09-16T17:24:44  <ThomasWaldmann> where is it on the left side?
2007-09-16T17:25:04  <dreimark> in the url of the parsed  HelpOnSearching
2007-09-16T17:25:30  <dreimark> first translation then substitution
2007-09-16T17:25:46  <ThomasWaldmann> in the above example, HelpOnSearching is on the right side
2007-09-16T17:26:06  <dreimark> ThomasWaldmann: this is the fix
2007-09-16T17:26:42  <dreimark> the po files have it in the text
2007-09-16T17:26:52  <ThomasWaldmann> ah
2007-09-16T17:28:02  <dreimark> have to reboot, bb 5mins
2007-09-16T17:33:22  <dreimark> bb
2007-09-16T17:37:29  <ThomasWaldmann> dreimark: either we have to make very many changes to i18n (like you showed above) or we have to think about another solution
2007-09-16T17:38:12  <dreimark> I do think about another one too, because that makes it difficult for translators
2007-09-16T17:39:32  <dreimark> not knowing the missing words
2007-09-16T17:39:33  * ThomasWaldmann curses logfile module
2007-09-16T17:41:08  <ThomasWaldmann> dreimark: maybe _ function could get another optional argument telling whether it should escape % char for left side of % operator usage
2007-09-16T17:41:08  <dreimark> thats a funny one too
2007-09-16T17:41:44  <dreimark> escaping is a good idea
2007-09-16T17:43:21  <ThomasWaldmann> the % in %s like stuff of course must not get escaped
2007-09-16T17:45:07  <dreimark> if we can escape shouldn't why not in the parser?
2007-09-16T17:45:54  <dreimark> -- shouldn't --
2007-09-16T17:47:11  <ThomasWaldmann> ?
2007-09-16T17:48:10  <ThomasWaldmann> dreimark: btw, running the mig scripts in 1.6 code solves the UserPreferences problem you found recently
2007-09-16T17:48:27  <dreimark> cool
2007-09-16T17:48:41  <ThomasWaldmann> (1.7 code did not convert it because in 1.7 there is no UserPreferences macro any more)
2007-09-16T17:49:01  <TheSheep> I'm thinking of an ugly hack -- the %XX escapes in urls are always upper case, right? ;)
2007-09-16T17:49:37  <dreimark> TheSheep:  seems to me too
2007-09-16T17:49:42  <ThomasWaldmann> except when digits :)
2007-09-16T17:49:58  <ThomasWaldmann> but we don't have %digit, so no problem
2007-09-16T17:50:10  <TheSheep> ThomasWaldmann: you could
2007-09-16T17:50:14  <TheSheep> ThomasWaldmann: %2d
2007-09-16T17:52:15  <ThomasWaldmann> hmm, yes, but %XXd with XX being a bigger number is unlikely
2007-09-16T17:53:02  <ThomasWaldmann> the smallest XX would be in %20 I guess
2007-09-16T17:53:16  <ThomasWaldmann> so if we don't have %20d, it would work
2007-09-16T17:54:10  <ThomasWaldmann> but it is really a ugly hack
2007-09-16T17:54:48  <TheSheep> and getting uglier every minute
2007-09-16T17:55:25  <TheSheep> still tempting :)
2007-09-16T17:55:47  <ThomasWaldmann> pity we don't have upper and lower case for numbers <g>
2007-09-16T17:55:56  <TheSheep> maybe if we checked if it's prime... ;)
2007-09-16T17:56:57  <ThomasWaldmann> hmm, we could run a regex on the result to detect href=".*?"
2007-09-16T17:57:20  <TheSheep> and src="", in case you ever included images
2007-09-16T17:57:40  <TheSheep> and don't forget about %'
2007-09-16T17:57:45  <TheSheep> s appearing naturally
2007-09-16T17:57:49  <TheSheep> in the text itself
2007-09-16T17:58:46  <TheSheep> maybe you could apply the hack to the i18n files instead
2007-09-16T17:59:01  <TheSheep> and turn all %'s that are not %s into %%
2007-09-16T17:59:44  <TheSheep> nah, that's stupid, you still have a problem with urls
2007-09-16T17:59:52  <ThomasWaldmann> and then you're fscked if it is NOT used on the left side of % operator
2007-09-16T18:00:30  <TheSheep> ThomasWaldmann: it's a good idea to always process the strings through the same number of escaping/unescaping oprations
2007-09-16T18:02:53  <dreimark> wikiutil.url_unquote does the job
2007-09-16T18:03:06  * ThomasWaldmann switches moinmaster to read-only
2007-09-16T18:11:55  * ThomasWaldmann will try to convert moinmaster data to 1.6 now
2007-09-16T18:13:43  <dreimark> What other problems did I get if I did wikiutil.url_unquote(translated) in getText?
2007-09-16T18:14:55  <ThomasWaldmann> other than that your url stops working? X)
2007-09-16T18:15:31  <TheSheep> dreimark: if there are any &'s or ?'s, they will stop working after you quote back
2007-09-16T18:15:45  <TheSheep> dreimark: and #'
2007-09-16T18:15:45  <TheSheep> s
2007-09-16T18:15:52  <TheSheep> stupid enter
2007-09-16T18:17:57  * ThomasWaldmann .oO(we need compress / purge policy for next storage backend)
2007-09-16T18:25:44  <grzywacz> xorAxAx, back
2007-09-16T18:28:11  <dreimark> hi grzywacz
2007-09-16T18:28:50  <dreimark> can you have a look on the skipped test in test_events
2007-09-16T19:05:35  <ThomasWaldmann> test.wikiwikiweb.de just got a new set of 1.6-converted moinmaster underlay pages
2007-09-16T20:36:59  <dreimark> need some regex help: [(^\%)]([^s])
2007-09-16T20:37:31  <dreimark> founds  and so on, pretty fine
2007-09-16T20:38:27  <dreimark> % C (without the blank)
2007-09-16T20:38:53  <dreimark> but whats the condition to return only the %
2007-09-16T20:39:38  <dreimark> something like a regex till the signs [A-Z]
2007-09-16T20:51:10  <ThomasWaldmann> dreimark: what's the plan?
2007-09-16T20:51:49  <dreimark> re.sub(regex) to replace the right % to %
2007-09-16T20:52:31  <dreimark> % % ( i should get another irc client)
2007-09-16T20:56:32  <dreimark> ah, let me try a bit longer, its good training
2007-09-16T20:57:59  <ThomasWaldmann> the problem you're trying to solve might not be solveable
2007-09-16T21:00:40  <dreimark> lets see, I keep trying a bit getting the delimiter ;)
2007-09-16T22:24:24  <CIA-27> moin: Reimar Bauer <rb.proj AT googlemail DOT com> default * 2161:ababd6001758 1.6/MoinMoin/i18n/__init__.py: i18n.__init__.getText: new parameter escape_substitution=True to escape % in tranlatation for text substitution
2007-09-16T22:29:46  <ThomasWaldmann> dreimark: that doesn't work for some cases
2007-09-16T22:30:03  <dreimark> which ones
2007-09-16T22:30:24  <ThomasWaldmann> %d, %42d, %20s, ...
2007-09-16T22:31:47  <dreimark> arrgh,  and I didnt like replace to use
2007-09-16T22:32:43  <ThomasWaldmann> and the code is not PEP8, btw :P
2007-09-16T22:33:14  <dreimark> I know already
2007-09-16T22:33:22  <dreimark> py.test should be in 1.6 too
2007-09-16T22:33:59  <ThomasWaldmann> yeah, maybe xorAxAx finds some time
2007-09-16T22:34:18  <xorAxAx> yeah
2007-09-16T22:34:32  <xorAxAx> booking motels will be more important in the next days
2007-09-16T22:35:18  <grzywacz> ;-)
2007-09-16T22:35:52  <dreimark> ThomasWaldmann: dont add test_sourcecode to 1.6 or ?
2007-09-16T22:36:13  <grzywacz> ThomasWaldmann, which of my ssh keys is currently active? :S
2007-09-16T22:37:11  <ThomasWaldmann> dreimark: no, we won't fix pep8 issues in 1.6
2007-09-16T22:37:50  <dreimark> thats good - too much work, but I try to be clean there too :)
2007-09-16T22:38:24  <CIA-27> moin: Karol 'grzywacz' Nowak <grzywacz@sul.uni.lodz.pl> default * 2839:72f1ea5f29b6 1.7/jabberbot/_tests/test_xmppbot.py: Skip tests properly when libxml2 is not installed.
2007-09-16T22:38:29  <grzywacz> ok, this one ;D
2007-09-16T22:39:16  <ThomasWaldmann> grzywacz: it is still all three, but I would like you to reduce this
2007-09-16T22:44:25  <grzywacz> Off to sleep, can't work with my keyboard laying on my feet. ;)
2007-09-16T23:17:10  <dreimark> CIA-27  2162:b94674c8d3271.6/ i18n.__init__.getText: fixed some missing substitution
2007-09-16T23:18:20  <dreimark> I dislike using replace at that point. Any other idea is welcome
2007-09-16T23:18:33  <dreimark> good night
2007-09-16T23:37:30  <CIA-27> moin: Reimar Bauer <rb.proj AT googlemail DOT com> default * 2162:b94674c8d327 1.6/MoinMoin/i18n/__init__.py: i18n.__init__.getText: fixed some missing substitution