1 2010-06-02T00:00:44 <dreimark> gn
2 2010-06-02T00:11:03 *** RogerHaase
3 2010-06-02T03:35:35 <diofeher> good night, any1 here?
4 2010-06-02T04:08:58 *** DmitryAndreev_
5 2010-06-02T04:12:22 *** DmitryAndreev
6 2010-06-02T04:46:42 <ei-grad> Moin
7 2010-06-02T04:59:32 <diofeher> it is morning in russia? :P moin ei-grad
8 2010-06-02T05:22:13 <ei-grad> 9:22
9 2010-06-02T05:22:40 <ei-grad> )
10 2010-06-02T06:18:50 *** kursor
11 2010-06-02T06:34:54 <CIA-47> Diogenes Augusto <diofeher@gmail.com> default * 8235:31b9c590fba3 2.0-dev-jinja2-plugins-dfeher/MoinMoin/ (templates/footer.html theme/jinja.py): [templating] pageinfo function converted
12 2010-06-02T06:34:55 *** kursor
13 2010-06-02T06:34:56 <CIA-47> Diogenes Augusto <diofeher@gmail.com> default * 8236:e2f17026b51a 2.0-dev-jinja2-plugins-dfeher/MoinMoin/theme/jinja.py: [templating] added some documentation to ThemeBase.render function
14 2010-06-02T06:44:52 <ei-grad> ThomasWaldmann: about .dn in user object... did i understand correctly, that LDAPGroups should check the user belongs to the group, even it was not authenticated by LDAPAuth?
15 2010-06-02T07:27:07 <CIA-47> Diogenes Augusto <diofeher@gmail.com> default * 8237:a1751e8cf67a 2.0-dev-jinja2-plugins-dfeher/MoinMoin/ (templates/head.html theme/jinja.py): [templating] deleted headscript function and moved directly to template head.html
16 2010-06-02T07:27:07 <CIA-47> Diogenes Augusto <diofeher@gmail.com> default * 8238:bf5c5430270d 2.0-dev-jinja2-plugins-dfeher/MoinMoin/ (templates/head.html theme/jinja.py): [templating] removed some duplicated variables and converted more method from html_head
17 2010-06-02T07:31:01 <diofeher> going to sleep, good night or good morning, don't know :)
18 2010-06-02T07:31:08 <ei-grad> good night
19 2010-06-02T07:32:18 <TheSheep> good morning
20 2010-06-02T07:32:19 <TheSheep> :)
21 2010-06-02T07:36:14 <ei-grad> TheSheep: moin
22 2010-06-02T08:48:29 *** gwork
23 2010-06-02T09:20:55 <eSyr> ThomasWaldmann: looks like next several days Dmitry would have no time for working on his GSoC project due to preparing to passing graduation exam.
24 2010-06-02T09:22:09 <eSyr> ThomasWaldmann: he said exam would be 4th June, and until that time there likely would be no progress from him.
25 2010-06-02T09:32:46 *** diofeher
26 2010-06-02T09:32:47 *** eSyr
27 2010-06-02T09:32:48 *** elmo
28 2010-06-02T09:32:48 *** dennda
29 2010-06-02T09:32:48 *** aa_
30 2010-06-02T09:32:48 *** gwork
31 2010-06-02T09:32:49 *** TheSheep
32 2010-06-02T09:32:49 *** CIA-47
33 2010-06-02T09:32:49 *** xjjk
34 2010-06-02T09:32:50 *** ThomasWaldmann
35 2010-06-02T09:32:51 *** waldi
36 2010-06-02T09:32:51 *** zenhase
37 2010-06-02T09:32:51 *** DmitryAndreev_
38 2010-06-02T09:32:51 *** mivert
39 2010-06-02T09:32:53 *** xorAxAx
40 2010-06-02T09:32:54 *** franklin
41 2010-06-02T09:32:54 *** valeuf
42 2010-06-02T09:32:55 *** vpv
43 2010-06-02T09:32:56 *** ei-grad
44 2010-06-02T09:32:56 *** ChanServ
45 2010-06-02T09:32:57 *** ronny_
46 2010-06-02T09:32:57 *** dreimark
47 2010-06-02T09:32:57 *** blathijs
48 2010-06-02T09:39:18 *** gwork
49 2010-06-02T09:39:18 *** DmitryAndreev_
50 2010-06-02T09:39:18 *** ei-grad
51 2010-06-02T09:39:18 *** diofeher
52 2010-06-02T09:39:18 *** xorAxAx
53 2010-06-02T09:39:18 *** TheSheep
54 2010-06-02T09:39:18 *** valeuf
55 2010-06-02T09:39:18 *** CIA-47
56 2010-06-02T09:39:18 *** franklin
57 2010-06-02T09:39:18 *** mivert
58 2010-06-02T09:39:18 *** vpv
59 2010-06-02T09:39:18 *** xjjk
60 2010-06-02T09:39:18 *** eSyr
61 2010-06-02T09:39:18 *** ThomasWaldmann
62 2010-06-02T09:39:18 *** elmo
63 2010-06-02T09:39:18 *** dennda
64 2010-06-02T09:39:18 *** aa_
65 2010-06-02T09:39:18 *** ronny_
66 2010-06-02T09:39:18 *** blathijs
67 2010-06-02T09:39:18 *** waldi
68 2010-06-02T09:39:18 *** dreimark
69 2010-06-02T09:39:18 *** zenhase
70 2010-06-02T09:39:18 *** ChanServ
71 2010-06-02T09:39:18 *** gibson.freenode.net
72 2010-06-02T09:55:31 *** ei-grad
73 2010-06-02T09:56:51 *** ei-grad
74 2010-06-02T10:21:26 <eSyr> ThomasWaldmann: during working with moin/dom converter Dmitry has found some lack of features in moin_in converter. Should they be fixed or something?
75 2010-06-02T10:33:00 <dreimark> ei-grad: yes that should be possible
76 2010-06-02T10:34:57 <dreimark> it should be possible that one can use ldap only for group managment and for auth e.g. moin_auth
77 2010-06-02T10:36:41 <dreimark> or a different ldap
78 2010-06-02T10:37:02 <dreimark> moin
79 2010-06-02T10:37:35 <dreimark> eSyr: can we see a wiki page about the missing features - a bit of documentation
80 2010-06-02T10:38:20 <dreimark> and Dmitry should tell in the diary that he is blocked by the exam
81 2010-06-02T10:38:46 <dreimark> then it is later by reviewing easier for us, why we have no entries there
82 2010-06-02T10:39:08 <eSyr> oops.
83 2010-06-02T10:39:53 <dreimark> it is just good to have a summary page because that is relevant for other parts too
84 2010-06-02T10:52:57 <dreimark> ThomasWaldmann: http://paste.pocoo.org/show/220941/ Me wants to remove recoverpassword link for ldap from SystemAdmin
85 2010-06-02T10:53:41 <dreimark> or in general for auth which work similar
86 2010-06-02T10:53:53 <dreimark> if request.user.auth_method in request.cfg.auth_can_logout: is used in the theme
87 2010-06-02T10:54:06 <dreimark> is that the correct check?
88 2010-06-02T11:11:30 <ThomasWaldmann> eSyr: yes, missing features should be fixed (best would be in main repo if it is not related to student's project)
89 2010-06-02T11:12:46 <ThomasWaldmann> ei-grad: if you authenticate by Kerberos (for example), you currently do not use ldap_login. so yes, it would be good if groups stuff worked without ldap_login.
90 2010-06-02T11:13:42 <ThomasWaldmann> eSyr: DmitryAndreev_: just put some entries about exam (preparation) into diary
91 2010-06-02T11:16:31 <DmitryAndreev_> ThomasWaldmann, ok, i'll do it. about fixes in main repo, it would be harder to test moinwiki_in without mine moinwiki_out.
92 2010-06-02T11:18:40 <ThomasWaldmann> yes, that might be, but there shouldn't just be roundtrip tests
93 2010-06-02T11:19:22 * ThomasWaldmann has seen working roundtrip testing just because both functions just did nothing.
94 2010-06-02T11:21:27 <ThomasWaldmann> recoverpass_link = unicode(unicode(ret)) wtf?
95 2010-06-02T11:22:04 <waldi> someone wants to be sure it is really a unicode object
96 2010-06-02T11:23:57 <waldi> (or ret contains a broken __unicode__ method)
97 2010-06-02T11:25:18 <ThomasWaldmann> dreimark: the test whether recover password makes sense is rather indirect
98 2010-06-02T11:25:58 <ThomasWaldmann> so, maybe rather test for the auth method name and find out for with auth method names we can support that
99 2010-06-02T11:45:50 *** ei-grad
100 2010-06-02T11:45:52 *** ei-grad
101 2010-06-02T12:21:03 <valeuf> Same for me, there is missing part in html_out converter,
102 2010-06-02T12:21:22 <valeuf> I can easily fix it (like underline, sub, super, and so ...)
103 2010-06-02T12:22:08 <valeuf> it can be also interesting to write equivalences pages, it is good to have an overview about the conversion between two elements
104 2010-06-02T12:25:49 <ThomasWaldmann> valeuf: shall I add the moin/2.0-dev repo to your key?
105 2010-06-02T12:26:21 <ThomasWaldmann> (you could fix there what you like and merge it afterwards into your repo)
106 2010-06-02T12:28:30 <valeuf> ThomasWaldmann: as you want, it is a big deal if I do it in my repos ?
107 2010-06-02T12:32:24 <valeuf> ThomasWaldmann: maybe we should ask waldi too ?
108 2010-06-02T12:32:46 <valeuf> waldi: what if I add support for some missing stuffs in HTML_out (especially what you have marked TODO)
109 2010-06-02T12:33:03 <valeuf> also, should I change the tests ? to use XPATH ?
110 2010-06-02T12:35:25 <waldi> valeuf: please do patches. i'm not sure this elements already have a proper definition
111 2010-06-02T12:38:20 <ThomasWaldmann> maybe a wiki page about missing stuff in the converters to collect issues and patches?
112 2010-06-02T12:39:10 <waldi> yep
113 2010-06-02T12:39:25 <waldi> several of the definitions are already wrong
114 2010-06-02T12:40:21 <waldi> valeuf: where does lxml come from?
115 2010-06-02T12:41:58 <waldi> if you are going the xpath route then please use it completely. xpatch can also match attributes and values
116 2010-06-02T12:43:48 <waldi> valeuf: however it can't check for non-existance. how do you do negative checks that the converter don't put unwanted things in?
117 2010-06-02T12:50:45 <waldi> and I miss any namespace definition. xpath themself is not really namespace away and there is no sign of the therefor necessary implementation specific namespace handling
118 2010-06-02T12:56:14 <dreimark> ThomasWaldmann: ok, thats better
119 2010-06-02T12:58:48 <valeuf> ThomasWaldmann: wiki page for missing stuffs looks fine for me
120 2010-06-02T12:59:10 <valeuf> waldi: it was a discussion about test with XPATH, after different experience, xorAxAx and me agree an using lxml
121 2010-06-02T12:59:27 <valeuf> waldi: so if you want to run tests, you need ta have python-lxml on your machine
122 2010-06-02T13:00:00 <valeuf> waldi: I do not check namespaces, I proposed a solution with namespaces checking and ET, but it has been rejected
123 2010-06-02T13:00:46 <valeuf> waldi: for negative checks, it is a good idea, the only place I should check is the child of the final element
124 2010-06-02T13:01:01 <valeuf> waldi: I can easily add a test to check the element does not have child
125 2010-06-02T13:01:16 <ThomasWaldmann> valeuf: if your tests require lxml, they should check for it and use py.skip("skipping tests as required lxml is not present") or so
126 2010-06-02T13:01:46 <valeuf> valeuf: I match attributes and values with Xpath already
127 2010-06-02T13:01:58 <valeuf> ThomasWaldmann: yep, you are right, I will add it
128 2010-06-02T13:02:27 <ThomasWaldmann> (try: import lxml except ImportError: py.skip(...)
129 2010-06-02T13:02:37 <waldi> valeuf: no, you don't. it misses to much "[" for that
130 2010-06-02T13:04:14 <valeuf> waldi: aah do you mean something like /body/div/span[@text-baseline-shift="sub"] ??
131 2010-06-02T13:10:46 <valeuf> waldi: I will try to improve my tests so :) thanks for the advice
132 2010-06-02T14:33:51 *** kursor
133 2010-06-02T15:13:06 <diofeher> moin
134 2010-06-02T15:31:46 <CIA-47> Diogenes Augusto <diofeher@gmail.com> default * 8239:41a9b8501f68 2.0-dev-jinja2-plugins-dfeher/MoinMoin/ (templates/header.html theme/jinja.py): [templating] removed unnecessary strings and converted ThemeBase.msg() a little
135 2010-06-02T15:31:49 <diofeher> TheSheep: hey, can you review my last patches?
136 2010-06-02T15:35:30 <ThomasWaldmann> diofeher: add a blank after #
137 2010-06-02T15:37:58 <ThomasWaldmann> also, you may want to think about how to better pass data into the template rendering call
138 2010-06-02T15:39:20 <ThomasWaldmann> maybe for some stuff, you could just pass page, user, cfg object and then pick the stuff you need from that in the template
139 2010-06-02T15:40:15 <ThomasWaldmann> check your spacing, it is only partly pep8
140 2010-06-02T15:41:21 <ThomasWaldmann> btw, matter of taste, but for easy typing I sometimes prefer dict(foo=bar, bar=baz) notation instead of {'foo': bar, 'bar': baz} - at least on german keyboards, the first one is much easier to type
141 2010-06-02T15:42:27 <ThomasWaldmann> please also check about i18n usage from template
142 2010-06-02T15:43:00 <ThomasWaldmann> (but be aware that our i18n depends on current user's language!)
143 2010-06-02T15:46:03 <ThomasWaldmann> you also wanted to work on "what links to here" magic removal
144 2010-06-02T15:47:40 <diofeher> ok thomas, thanks for the review
145 2010-06-02T15:47:51 <diofeher> i'm gonna this "what links to here" now
146 2010-06-02T15:48:03 <diofeher> and then fix pep8 errors
147 2010-06-02T15:48:34 <ThomasWaldmann> when fixing pep8 errors, you could also normaliz docstrings to:
148 2010-06-02T15:48:37 <ThomasWaldmann> """
149 2010-06-02T15:48:41 <ThomasWaldmann> short desc
150 2010-06-02T15:48:43 <ThomasWaldmann>
151 2010-06-02T15:48:47 <ThomasWaldmann> long desc ...
152 2010-06-02T15:48:49 <ThomasWaldmann> ...
153 2010-06-02T15:48:56 <ThomasWaldmann> params desc ...
154 2010-06-02T15:48:57 <ThomasWaldmann> ...
155 2010-06-02T15:48:58 <ThomasWaldmann> """
156 2010-06-02T15:56:11 <diofeher> ok thomas, thanks :)
157 2010-06-02T15:56:22 <diofeher> a tip received from ssbr_ in #python : http://pypi.python.org/pypi/pep8/
158 2010-06-02T15:56:28 <diofeher> to check pep8 errors =)
159 2010-06-02T15:58:31 <diofeher> loved this so much :)
160 2010-06-02T16:08:56 <eSyr> diofeher: http://moinmo.in/CodingStyle?action=AttachFile&do=view&target=python.vim
161 2010-06-02T16:09:07 <eSyr> diofeher: http://moinmo.in/CodingStyle
162 2010-06-02T16:09:26 <eSyr> diofeher: and py.test source
163 2010-06-02T16:09:56 <eSyr> (also you can add py.test in pre-commit hook)
164 2010-06-02T16:12:45 <diofeher> MoinMoin/theme/jinja.py:863:33: W601 .has_key() is deprecated, use 'in'
165 2010-06-02T16:12:51 <diofeher> should i change has_key to in?
166 2010-06-02T16:13:18 <diofeher> eSyr: thanks, but i use textmate :/
167 2010-06-02T16:13:33 <diofeher> and I've read Coding Style... But sometimes I forget to put in the right way :P
168 2010-06-02T16:19:49 <diofeher> i think it will be a good idea we make a pep8 command-line checker according to moinmoin style
169 2010-06-02T16:25:24 <diofeher> W291 trailing whitespace
170 2010-06-02T16:25:24 <diofeher> W291 trailing whitespace
171 2010-06-02T16:25:24 <diofeher> W291 trailing whitespace
172 2010-06-02T16:25:24 <diofeher> W291 trailing whitespace
173 2010-06-02T16:25:27 <diofeher> ops
174 2010-06-02T16:25:38 <diofeher> sorry.. but, what is this error?
175 2010-06-02T16:26:08 <eSyr> diofeher: you have whitspace characters between last non-whitespace character on line and end of line.
176 2010-06-02T16:26:32 <eSyr> (or have lines which have only whitespace characters)
177 2010-06-02T16:26:34 <diofeher> thanks eSyr =)
178 2010-06-02T16:27:02 <diofeher> first option ^^
179 2010-06-02T16:32:54 <CIA-47> Diogenes Augusto <diofeher@gmail.com> default * 8240:db31daa78fdf 2.0-dev-jinja2-plugins-dfeher/MoinMoin/ (templates/head.html theme/jinja.py): [templating] normalized docstrings and fix pep8 errors using pep8 checker
180 2010-06-02T16:36:43 <diofeher> guys, what ide do you use to code? for who use ide, of course
181 2010-06-02T16:40:20 <eSyr> diofeher: vim, lol
182 2010-06-02T16:40:56 <eSyr> diofeher: http://www.google.com/images?q=vim+ide
183 2010-06-02T16:46:48 <diofeher> wow, never saw these images :O
184 2010-06-02T16:52:47 <diofeher> eSyr: but you use GUI Vim. right?
185 2010-06-02T17:03:44 <aa_> if you want to try a complete vim IDE, look at pida.co.uk
186 2010-06-02T17:03:51 <aa_> (it embeds vim)
187 2010-06-02T17:04:27 <aa_> http://pida.co.uk/files/screenshots/pida_0-5-2_17.png
188 2010-06-02T17:07:33 <diofeher> nice... tried to install but giving me error :/
189 2010-06-02T17:08:23 <diofeher> gonna try later... thanks for the suggestion aa_
190 2010-06-02T17:08:26 <diofeher> bye
191 2010-06-02T17:20:43 *** gwork
192 2010-06-02T18:39:13 *** DmitryAndreev__
193 2010-06-02T18:39:15 *** DmitryAndreev_
194 2010-06-02T18:40:40 *** DmitryAndreev__
195 2010-06-02T18:45:22 *** DmitryAndreev
196 2010-06-02T18:49:40 *** DmitryAndreev
197 2010-06-02T19:34:34 *** DmitryAndreev
198 2010-06-02T19:47:10 *** DmitryAndreev
199 2010-06-02T19:54:22 *** DmitryAndreev
200 2010-06-02T20:09:32 *** DmitryAndreev
201 2010-06-02T20:56:35 *** kursor
202 2010-06-02T21:29:50 *** mivert_
203 2010-06-02T21:33:17 *** mivert
204 2010-06-02T21:57:49 <diofeher> TheSheep: are you there? i've some doubts about actions (i'm trying to kill backling magic and make a new action)
205 2010-06-02T21:59:43 <ThomasWaldmann> diofeher: for a minimal thing, you just need to redirect to correct url
206 2010-06-02T22:00:19 <diofeher> ThomasWaldmann, yes, exactly
207 2010-06-02T22:01:12 <diofeher> but i cannot get the correct link... I'm looking in Page functions
208 2010-06-02T22:01:18 <diofeher> Page.url() right?
209 2010-06-02T22:23:08 <diofeher> {'do': 'fullsearch', 'value': u'linkto:LanguageSetup', 'context': '180'}
210 2010-06-02T22:23:15 <diofeher> /LanguageSetup?do=fullsearch&value=linkto%3ALanguageSetup&context=180
211 2010-06-02T22:23:26 <diofeher> is converting " to %3A
212 2010-06-02T22:23:42 <diofeher> when i call ThemeBase.url
213 2010-06-02T22:23:49 <diofeher> ops, Page.url
214 2010-06-02T22:27:40 <CIA-47> Diogenes Augusto <diofeher@gmail.com> default * 8241:5d5ea6c65643 2.0-dev-jinja2-plugins-dfeher/MoinMoin/ (theme/jinja.py action/backlink.py): [templating] adding new action (backlink) with one bug and adding to theme
215 2010-06-02T22:27:40 <diofeher> i'm gonna leave now... back in 4 hours
216 2010-06-02T22:41:23 <ThomasWaldmann> diofeher: you have the correct url creation code within the code you have to remove from theme
217 2010-06-02T22:44:47 <dreimark> ei-grad: where is the promised double work?
