1 2013-06-20T02:36:30 *** bilal
2 2013-06-20T02:57:12 *** bilal
3 2013-06-20T03:43:42 <ReimarBauer> moin
4 2013-06-20T03:49:59 <ReimarBauer> spy_: I miss your diary entries, on what do you work currently
5 2013-06-20T07:17:03 *** ashutosh1461
6 2013-06-20T07:18:58 <ThomasWaldmann> moin
7 2013-06-20T07:27:35 <ashutosh1461> moin
8 2013-06-20T07:28:53 <ashutosh1461> ThomasWaldmann, what is the new problem ?
9 2013-06-20T07:31:37 <ThomasWaldmann> I just found it was not finished/ready yet, waldi also contributed some ideas, see etherpad
10 2013-06-20T08:16:37 <ana_ananasova> ReimarBauer: comment to your suggestion https://codereview.appspot.com/10386044/
11 2013-06-20T09:02:53 <ashutosh1461> ThomasWaldmann, should not http://server/[NAMESPACE/][[@FIELD/]NAME][/+VIEW] be http://server/[+VIEW/][NAMESPACE/][[@FIELD/]NAME] ?
12 2013-06-20T09:13:36 *** sharky93
13 2013-06-20T09:13:55 <sharky93> moin
14 2013-06-20T09:14:43 <sharky93> TheSheep, about the onclick @ https://bitbucket.org/thomaswaldmann/moin-2.0/src/ad303592c64c1743ac218d42835493cb9d97440b/MoinMoin/templates/modify.html?at=default#cl-26
15 2013-06-20T09:14:52 <TheSheep> sharky93: http://api.jquery.com/data/#data-html5
16 2013-06-20T09:18:41 <waldi> ThomasWaldmann: do we really want to support regex subscriptions? they are O(n^2) and it is not trivial to check for this. and for most people recursive subscriptions would suffice
17 2013-06-20T09:18:59 <ThomasWaldmann> ashutosh1461: REST usually has the view at the end
18 2013-06-20T09:19:14 <waldi> n is the length of the string to be checked, so the pagename
19 2013-06-20T09:19:25 <ThomasWaldmann> (and while we are restructuring the urls anyway...)
20 2013-06-20T09:22:05 <ThomasWaldmann> waldi: some people subscribed to .*
21 2013-06-20T09:23:14 <waldi> ThomasWaldmann: well. we can have a root (/) item
22 2013-06-20T09:24:02 <ThomasWaldmann> you mean name == ''?
23 2013-06-20T09:24:16 <waldi> something like that
24 2013-06-20T09:25:23 <ThomasWaldmann> well, but the other toplevel pages then would be siblings, not childs
25 2013-06-20T09:25:38 <TheSheep> children
26 2013-06-20T09:25:45 <waldi> no, they would be children
27 2013-06-20T09:25:47 <ThomasWaldmann> yeah :)
28 2013-06-20T09:26:37 <ThomasWaldmann> /(empty) and /foo are on same level
29 2013-06-20T09:27:03 <waldi> depends on the definition. /foo is a child of /
30 2013-06-20T09:28:09 <ThomasWaldmann> if / is a directory ... - but we just have items
31 2013-06-20T09:28:38 <TheSheep> just have a special case for "all"
32 2013-06-20T09:28:54 <TheSheep> maybe use glob instead of regex? :)
33 2013-06-20T09:29:10 <ThomasWaldmann> doesn't make it better
34 2013-06-20T09:31:03 <TheSheep> just kidding
35 2013-06-20T09:31:16 <ThomasWaldmann> we also should consider namespaces
36 2013-06-20T09:31:19 <TheSheep> but you could just make the empty string a special case for all pages
37 2013-06-20T09:31:57 <ThomasWaldmann> (and then you have to see whether someone entered an empty string in a form field or not :)
38 2013-06-20T09:32:35 <ana_ananasova> make a checkbox for that in the settings
39 2013-06-20T09:33:28 <ThomasWaldmann> waldi: use same pattern as discussed for urls on this etherpad? http://etherpad.osuosl.org/moin-itemid line 20 minus view / server part?
40 2013-06-20T09:34:04 <ThomasWaldmann> one could subscribe to tags then
41 2013-06-20T09:34:32 <ThomasWaldmann> (still expensive...)
42 2013-06-20T09:36:07 <ThomasWaldmann> maybe we need to do this totally different
43 2013-06-20T09:36:30 <ThomasWaldmann> precise subscriptions are no problem, quick
44 2013-06-20T09:37:34 <ThomasWaldmann> but anything that involves matching is slow as it needs to iterate over all users, match n patterns found in the profile against m pages
45 2013-06-20T09:37:56 <ThomasWaldmann> editing an item is more frequent than name changes (creation, rename, deletion)
46 2013-06-20T09:38:41 <ThomasWaldmann> so at least for name matching, it would be faster to update stuff when these actions happen
47 2013-06-20T09:39:18 <ThomasWaldmann> we'ld somehow need triggers for name/tags/... meta changes updating precise subscriptions...
48 2013-06-20T09:40:39 <ana_ananasova> one item creation involves iterating over all user profiles and matching every regexp. also slow
49 2013-06-20T09:40:43 <ThomasWaldmann> so the email sending would only work with ID subscriptions (-> fast), but these ID subscriptions would be updated by rules
50 2013-06-20T09:41:04 <ThomasWaldmann> yes, but less frequent than a edit
51 2013-06-20T09:42:21 <ThomasWaldmann> maybe it could be even done in the background / queuing system / cron job
52 2013-06-20T09:43:44 <ana_ananasova> what if a modify will happen before all subscription lists will be updated...
53 2013-06-20T09:44:23 <ThomasWaldmann> then someone would not get notified...
54 2013-06-20T09:45:09 <ThomasWaldmann> (not nice)
55 2013-06-20T09:45:53 <ThomasWaldmann> but performance is really an issue here, people have done hacks for 1.9 to tune it (but likely introduced new issues by them)
56 2013-06-20T09:46:50 <ThomasWaldmann> and iirc i also got that "subscribe to tag" feature request multiple times
57 2013-06-20T09:48:13 <ana_ananasova> shall we implement that one too?
58 2013-06-20T09:48:37 <ThomasWaldmann> well, if one uses whoosh, it's not much different
59 2013-06-20T09:49:30 <ThomasWaldmann> but we need to solve the performance issue in general
60 2013-06-20T09:51:20 <ThomasWaldmann> hmm, what if we index all patterns. then we could iterate over whoosh's term list ("dictionary") and we would not need to iterate over all userprofiles.
61 2013-06-20T09:51:49 <ThomasWaldmann> each term has a pointer to the documents (userprofiles) where it was encountered.
62 2013-06-20T09:52:17 <ThomasWaldmann> so we could do the match and if it succeeds, know the userprofiles...
63 2013-06-20T09:52:55 <ana_ananasova> see here http://etherpad.osuosl.org/moin-mail
64 2013-06-20T09:53:31 <ana_ananasova> but it describes the process on item modify
65 2013-06-20T10:54:00 <sharky93> TheSheep, so .data() renders the value as a string each time, so instead of Home/test for ex.. we get "Home/test".
66 2013-06-20T10:56:03 *** greg_f
67 2013-06-20T10:58:56 <TheSheep> sharky93: I don't understand
68 2013-06-20T10:59:26 <TheSheep> sharky93: .data('foo') gives you the content of the data-foo html attribute
69 2013-06-20T11:04:36 <sharky93> TheSheep, it automatically pads quotes the content, which was not the case with onclick=InsertName({{ fullname }})
70 2013-06-20T11:05:58 <TheSheep> sharky93: .data() doesn't quote anything, just returns a string
71 2013-06-20T11:07:28 <TheSheep> sharky93: you mean that you need to remove {% set fullname = fullname|json_dumps %} ?
72 2013-06-20T11:08:45 * TheSheep is reading http://alistapart.com/article/writing-testable-javascript
73 2013-06-20T11:09:26 <sharky93> TheSheep, not sure about that but the .data() returns me a " "string" " ... rather than just "string".
74 2013-06-20T11:10:28 <TheSheep> sharky93: and what did you put in the data-foo attribute in the html?
75 2013-06-20T11:10:37 <TheSheep> sharky93: view page's source
76 2013-06-20T11:20:20 <sharky93> TheSheep, hmm, the data-attr has the value as data-attr = ""somestring""
77 2013-06-20T11:20:37 <sharky93> But the template has data-href = {{ fullname }}
78 2013-06-20T11:21:49 <TheSheep> sharky93: look at the line before it in the template
79 2013-06-20T11:22:18 <TheSheep> sharky93: and guess what it does
80 2013-06-20T11:23:06 <TheSheep> sharky93: https://bitbucket.org/thomaswaldmann/moin-2.0/src/ad303592c64c1743ac218d42835493cb9d97440b/MoinMoin/templates/modify.html?at=default#cl-25
81 2013-06-20T11:23:41 <sharky93> ok :)
82 2013-06-20T11:24:42 <sharky93> i see :D
83 2013-06-20T11:24:52 <sharky93> JSON representations ..
84 2013-06-20T11:24:54 <sharky93> hmm
85 2013-06-20T11:27:11 <TheSheep> json is basically javascript
86 2013-06-20T11:27:37 <TheSheep> so that simply quotes the string so that it can be included in the javascript code
87 2013-06-20T11:27:58 <TheSheep> it adds the "" and also escapes any " inside the string
88 2013-06-20T11:28:17 <TheSheep> you have to do that for the translations strings in dictionary.js, btw
89 2013-06-20T11:40:16 <sharky93> hmm, maybe in a new patchset.
90 2013-06-20T11:42:03 * sharky93 runs the linter again
91 2013-06-20T11:43:06 <sharky93> TheSheep, should we use one "use strict" for the whole ready function?
92 2013-06-20T12:00:16 <TheSheep> sharky93: as long as it works
93 2013-06-20T12:18:05 <sharky93> TheSheep, is it a bad practice to declare functions within if { .. } blocks ? .. the linter suggests that this is not advisable in strict mode.
94 2013-06-20T12:19:08 <TheSheep> follow the linter :)
95 2013-06-20T12:35:04 <xiaq> ThomasWaldmann: https://codereview.appspot.com/10374049/
96 2013-06-20T12:35:07 <xiaq> eSyr: ^^
97 2013-06-20T12:37:16 <sharky93> TheSheep, why does the linter want me to declare all variables with a single var. That is so clumsy :(
98 2013-06-20T12:37:51 <TheSheep> sharky93: complain to Crockford
99 2013-06-20T12:37:58 <ReimarBauer> btw. have you all the recent version ?
100 2013-06-20T12:38:35 <xiaq> sharky93: are we still looking for a suitable framework? just discovered 2 more... added to http://etherpad.osuosl.org/moin-ui
101 2013-06-20T12:38:47 <sharky93> TheSheep, ^
102 2013-06-20T12:39:20 <xiaq> Topcoat and AUI
103 2013-06-20T12:39:34 <xiaq> topcoat seems to be all css
104 2013-06-20T12:39:39 <xiaq> AUI is used by bitbucket
105 2013-06-20T12:40:03 <TheSheep> they have benchmarks instead of documentation
106 2013-06-20T12:41:03 <xiaq> TheSheep: ah, topcoat is "brand new", which reads "no doc" :)
107 2013-06-20T12:42:08 <TheSheep> xiaq: AUI is under Apache license, not compatible with gpl2
108 2013-06-20T12:42:52 <xiaq> ah, so both are out :(
109 2013-06-20T12:42:57 <TheSheep> (if we could use Apache2 licnse, I would prefer Bootstrap anyways)
110 2013-06-20T12:43:14 <sharky93> even xiaq would agree to that :)
111 2013-06-20T12:43:37 <TheSheep> checking on topcoat
112 2013-06-20T12:43:45 <sharky93> TheSheep, so we follow whatever the linter cries about :|
113 2013-06-20T12:44:04 <TheSheep> topcoat is also Apache2
114 2013-06-20T12:44:35 <TheSheep> sharky93: yes, the linter is written in such a way, as to force you to follow the JavaScript: The Good Parts book
115 2013-06-20T12:45:14 <TheSheep> of course, that doesn't free you from thinking
116 2013-06-20T12:45:34 <TheSheep> so, if the linter complains about something, it has to be corrected
117 2013-06-20T12:45:49 <TheSheep> but if the linter doesn't complain about something, it doesn't mean that it is good
118 2013-06-20T12:46:53 <sharky93> TheSheep, there are options for tolerating some stuff, those have to be at default settings? like there is an option for many vars in a function.
119 2013-06-20T12:54:39 <sharky93> TheSheep, should we use that directive?
120 2013-06-20T12:56:27 <TheSheep> sharky93: I would rather prefer to use those directives only when we have to
121 2013-06-20T12:56:31 <TheSheep> sharky93: like with the _()
122 2013-06-20T12:56:54 <TheSheep> sharky93: you will soon get used to the singl var :)
123 2013-06-20T13:14:55 <sharky93> hopefully :)
124 2013-06-20T13:15:26 <sharky93> TheSheep, do we lose the unused vars or maybe just comment them, we may have to check the changesets which put them in there in the first place.
125 2013-06-20T13:15:56 <TheSheep> sharky93: you have version control system, so you don't need to comment out code
126 2013-06-20T13:16:06 <TheSheep> sharky93: you can always go back to the previous version and check
127 2013-06-20T13:18:27 * sharky93 testing the JS after the linter terrified him
128 2013-06-20T13:25:41 * ThomasWaldmann still thinks "what would happen if we use jquery-mobile?" would anybody complain? would we run into issues? how about search engine crawlers?
129 2013-06-20T13:35:40 <sharky93> TheSheep, patchset2 @ https://codereview.appspot.com/10427043/ .. too many chunks :)
130 2013-06-20T13:36:09 <sharky93> i tested, everything works as expected.
131 2013-06-20T13:46:42 <TheSheep> sharky93: I will look at it in about 2-3 hours
132 2013-06-20T13:47:27 <sharky93> TheSheep, ok proceeding with the image-resizing.
133 2013-06-20T13:58:33 <ThomasWaldmann> sharky93: did you do the first review of the patchset yourself?
134 2013-06-20T13:59:29 <sharky93> yes
135 2013-06-20T14:00:01 <ThomasWaldmann> you inserted a blank line without special reason. you should find and fix such stuff yourself before asking for review.
136 2013-06-20T14:00:35 <ThomasWaldmann> (same for typos or other obvious issues, including obvious coding issues)
137 2013-06-20T14:01:46 <ThomasWaldmann> mentor time is precious, use it wisely
138 2013-06-20T14:24:24 *** brunomartin
139 2013-06-20T14:52:10 *** bilal
140 2013-06-20T15:20:46 *** bilal
141 2013-06-20T16:14:26 *** ashutosh_singla1
142 2013-06-20T16:14:57 *** ashutosh1461
143 2013-06-20T16:34:25 <xiaq> ThomasWaldmann: is j-m also for desktops?
144 2013-06-20T16:49:35 *** greg_f
145 2013-06-20T17:05:14 *** greg_f
146 2013-06-20T17:15:40 *** greg_f
147 2013-06-20T17:18:14 *** greg_f
148 2013-06-20T17:18:59 <TheSheep> sharky93: now I see what you meant by the multiple vars
149 2013-06-20T17:19:16 <TheSheep> sharky93: you were right, I rest my case
150 2013-06-20T17:19:23 <TheSheep> sharky93: use multiple vars please
151 2013-06-20T17:21:15 <sharky93> hmm :P .. rest everything is ok?
152 2013-06-20T17:22:00 <TheSheep> sharky93: didn't get there yet
153 2013-06-20T17:22:34 <TheSheep> sharky93: ideally those should all be separate functions, as they were, only not global
154 2013-06-20T17:23:17 <TheSheep> sharky93: but I can't think of a nice way of doing it -- maybe by putting them all on a single MoinMoin object?
155 2013-06-20T17:23:30 <TheSheep> sharky93: what do you think?
156 2013-06-20T17:24:10 <sharky93> TheSheep, just tolerate multiple vars?
157 2013-06-20T17:26:56 <TheSheep> sharky93: the problem is, if we do it all in a single large function, it's difficult to debug and test
158 2013-06-20T17:31:16 <TheSheep> sharky93: actually, you could leave them all as function, degfined inside the $(function () { ... });, and call them at the end of that scope
159 2013-06-20T17:31:56 <TheSheep> sharky93: sorry I'm not being clear from the start, but I have to see the code to get a feel of how good it looks :)
160 2013-06-20T17:32:01 <ashutosh_singla1> ThomasWaldmann, https://bitbucket.org/thomaswaldmann/moin-2.0/src/ad303592c64c1743ac218d42835493cb9d97440b/MoinMoin/util/_tests/test_interwiki.py?at=default#cl-42 line 42
161 2013-06-20T17:32:17 <ashutosh_singla1> should not it be '/%2Bmodify/SomePage' ?
162 2013-06-20T17:33:29 <sharky93> TheSheep, hmm, i understand .. more reviews perhaps :) smaller changesets :)
163 2013-06-20T17:34:58 <TheSheep> sharky93: then you can have a single var per function and still have it nicely divided into separate functions
164 2013-06-20T17:36:59 <sharky93> TheSheep, why make it into one document.ready ?
165 2013-06-20T17:37:56 <TheSheep> sharky93: actually the best organization would be like this:
166 2013-06-20T17:48:02 <sharky93> TheSheep, like?
167 2013-06-20T17:52:13 <TheSheep> http://paste.openstack.org/show/39046/
168 2013-06-20T17:52:21 <TheSheep> sorry, got carried off experimenting
169 2013-06-20T17:53:24 <TheSheep> so basically all the initialization functions would be methods on the MoinMoin object, and you would just call them all in order in a single document.ready
170 2013-06-20T18:01:18 <TheSheep> sharky93: one document.ready, so that you can see in one place what is happening
171 2013-06-20T18:02:00 <TheSheep> sharky93: it would be even bettter to have different objects for different functionalities, but that's too much for now, I think
172 2013-06-20T18:17:52 <sharky93> TheSheep, yep that is a nice way
173 2013-06-20T18:17:58 <ana_ananasova> waldi: need your advice here (lines 103:134) http://etherpad.osuosl.org/moin-mail
174 2013-06-20T18:18:14 <sharky93> TheSheep, i dont understand 'prototype' well.
175 2013-06-20T18:19:13 * sharky93 googling
176 2013-06-20T18:20:43 <sharky93> ok i understand now
177 2013-06-20T18:21:06 <ana_ananasova> ohh there's a great article about objects, delegation and prototyping in js -> http://davidwalsh.name/javascript-objects-deconstruction
178 2013-06-20T18:22:14 <ana_ananasova> sharky93: read it (if you wish of course) when you feel like wanting to know more :)
179 2013-06-20T18:22:43 <sharky93> ana_ananasova, thank you :)
180 2013-06-20T18:39:12 *** greg_f
181 2013-06-20T18:50:50 *** bilal
182 2013-06-20T18:54:49 *** bilal
183 2013-06-20T18:55:50 *** bilal
184 2013-06-20T19:00:59 <TheSheep> sharky93: if you don't know about prototype, then I recommend watching Crocford's course in js
185 2013-06-20T19:01:24 <TheSheep> sharky93: https://www.youtube.com/watch?v=v2ifWcnQs6M
186 2013-06-20T19:01:31 <TheSheep> sharky93: it's about 3h total, but worth it
187 2013-06-20T19:08:16 <TheSheep> sharky93: sorry, 1.5h
188 2013-06-20T19:14:09 <TheSheep> sharky93: ugh, the click call in that pastebin of mine was wrong
189 2013-06-20T19:14:39 <TheSheep> sharky93: there should be no second argument, and the first time should be wrapped in a function
190 2013-06-20T19:21:41 <sharky93> TheSheep, ah, i see
191 2013-06-20T19:22:07 <sharky93> TheSheep, iirc wrapping is only necessary if im passing arguments to the function
192 2013-06-20T19:22:49 <sharky93> TheSheep, i had googled about prototypes, so have a basic idea now .. but yeah, i'll watch that soon :)
193 2013-06-20T19:23:18 * sharky93 testing the JS
194 2013-06-20T19:23:47 <TheSheep> sharky93: wrapping is necessary if you use 'this' inside of the method
195 2013-06-20T19:24:13 <TheSheep> sharky93: it's also explained in that video
196 2013-06-20T20:11:26 <sharky93> TheSheep, patch set3 @ https://codereview.appspot.com/10427043/
197 2013-06-20T20:12:16 * sharky93 used the same name :)
198 2013-06-20T20:14:36 <ThomasWaldmann> re
199 2013-06-20T20:19:39 <TheSheep> sharky93: just a couple of small things
200 2013-06-20T20:19:51 <TheSheep> sharky93: otherwise I think it's good
201 2013-06-20T20:23:11 <sharky93> TheSheep, im all ears :D
202 2013-06-20T20:24:52 <TheSheep> sharky93: I commented it
203 2013-06-20T20:24:57 <TheSheep> sharky93: just read the comments :)
204 2013-06-20T20:25:21 <TheSheep> oh, sorry, didn't click send
205 2013-06-20T20:25:24 <sharky93> TheSheep, i can't see any. I guess you havent published
206 2013-06-20T20:25:28 <sharky93> yeah. :P
207 2013-06-20T20:30:21 <TheSheep> try now
208 2013-06-20T20:30:26 <sharky93> yeah, can see
209 2013-06-20T20:40:00 <ThomasWaldmann> ashutosh_singla1: no, since recent werkzeug/flask, it is + again (not %2B)
210 2013-06-20T20:42:30 <sharky93> TheSheep, I think changing the prefixing the class name with "moin" should be the part of a different changeset since we would also need to fix the css for the broken icon, the "plus sign" for expansion.
211 2013-06-20T20:42:51 <TheSheep> sharky93: yes
212 2013-06-20T20:43:01 <sharky93> TheSheep, removing "shortname" causes an internal error, so that as well.
213 2013-06-20T20:43:02 <TheSheep> sharky93: good point
214 2013-06-20T20:43:17 <TheSheep> hmm, it'
215 2013-06-20T20:43:32 <TheSheep> hmm, that's quaint, it's probably used somewhere further
216 2013-06-20T20:45:21 <sharky93> there's some issue with argument matching. I grep'ed, has not been used anywhere else apart from the sphinx package. not sure if that should cause an issue. but neways
217 2013-06-20T20:47:24 <sharky93> TheSheep, final look i guess @ https://codereview.appspot.com/10427043
218 2013-06-20T20:50:31 <TheSheep> sharky93: it's good
219 2013-06-20T20:52:55 <ReimarBauer> gn
220 2013-06-20T20:54:08 <ashutosh_singla1> ThomasWaldmann, is there a way to update my environment with re-installing ?
221 2013-06-20T20:54:27 *** ashutosh_singla1
222 2013-06-20T20:58:09 * sharky93 pushes commit to local repo
223 2013-06-20T20:58:55 <sharky93> TheSheep, have you looked up the resizing stuff?
224 2013-06-20T20:59:50 * sharky93 wonders about the amount of time we should plan for that
225 2013-06-20T21:00:46 <ThomasWaldmann> pip install --upgrade -e . (or so)
226 2013-06-20T21:01:25 <TheSheep> sharky93: why do you need to know how long it will take?
227 2013-06-20T21:01:50 <sharky93> TheSheep, i think i work better under some bound time frame :)
228 2013-06-20T21:05:29 <TheSheep> sharky93: so, I suppose it's a day for the argument parser, maybe two if there are problems, then a day for the default html output and tests, then a day for other markups that have syntax for it (in the mean time research on how that markup looks like)
229 2013-06-20T21:06:06 <TheSheep> then final touches and documentation
230 2013-06-20T21:06:18 <TheSheep> so a week on the outside
231 2013-06-20T21:06:57 <sharky93> ah, nice :)
232 2013-06-20T21:07:22 <sharky93> TheSheep, ThomasWaldmann anything else on the frameworks?
233 2013-06-20T21:15:31 <ana_ananasova> waldi, ThomasWaldmann feedback needed on lines 103:134 - http://etherpad.osuosl.org/moin-mail
234 2013-06-20T21:27:31 <ashutosh1461> ThomasWaldmann, and there is no facility for @, we have to use %40 there.
235 2013-06-20T21:31:26 *** ana-balica
236 2013-06-20T21:34:33 *** ana_ananasova
237 2013-06-20T21:37:23 *** ana_ananasova
238 2013-06-20T21:51:43 <ashutosh1461> ThomasWaldmann, interwiki module patch https://codereview.appspot.com/10439044
239 2013-06-20T21:54:25 *** ana_ananasova
240 2013-06-20T22:12:05 *** ashutosh1461
241 2013-06-20T22:34:17 <sharky93> gn
242 2013-06-20T22:35:24 *** sharky93
243 2013-06-20T22:35:39 *** sharky93
244 2013-06-20T22:36:46 *** sharky93
245 2013-06-20T22:37:00 *** sharky93
246 2013-06-20T23:11:20 *** brunomartin
247 2013-06-20T23:25:08 *** bilal
248