1 2009-11-26T00:19:47 *** dainbrain
2 2009-11-26T00:29:58 *** grzywacz
3 2009-11-26T00:30:12 *** JosefMeier
4 2009-11-26T01:43:56 *** dainbrain
5 2009-11-26T03:42:30 *** xorAxAx
6 2009-11-26T04:15:33 *** xorAxAx
7 2009-11-26T04:38:22 *** ronny
8 2009-11-26T04:38:22 *** waldi
9 2009-11-26T04:38:22 *** dreimark
10 2009-11-26T04:38:22 *** vpv
11 2009-11-26T04:38:22 *** dennda
12 2009-11-26T04:38:22 *** tpfennig
13 2009-11-26T04:38:22 *** ser
14 2009-11-26T04:40:33 *** tpfennig
15 2009-11-26T04:40:33 *** ser
16 2009-11-26T04:40:33 *** ronny
17 2009-11-26T04:40:33 *** dennda
18 2009-11-26T04:40:33 *** dreimark
19 2009-11-26T04:40:33 *** waldi
20 2009-11-26T04:40:33 *** vpv
21 2009-11-26T09:36:10 <ThomasWaldmann> moin
22 2009-11-26T09:36:22 * ThomasWaldmann looks at josef's code
23 2009-11-26T09:39:05 <ThomasWaldmann> hmm, usage of request.getScriptname() should fill the logs with deprecation warnings
24 2009-11-26T09:41:50 <ThomasWaldmann> l68..71 can be easily done in a single line, using x = y or z
25 2009-11-26T09:46:02 <ThomasWaldmann> l73..75 look like they are not needed (and even wrong for the somewhat strange, but existing usecase of a deleted page that has attachments)
26 2009-11-26T09:48:42 <ThomasWaldmann> l83 has to be checked for XSS / missing escaping
27 2009-11-26T09:53:21 <ThomasWaldmann> l149 onkeyup="OnPagenameChange();" onchange="OnPagenameChange();" is that both needed?
28 2009-11-26T09:54:02 <ThomasWaldmann> l159 same q.
29 2009-11-26T10:05:34 <ThomasWaldmann> l1.144 + 149 hardcoded colours
30 2009-11-26T10:07:49 <ThomasWaldmann> l1.192 doesn't that already exist???
31 2009-11-26T10:15:07 <ThomasWaldmann> l1.241..244 has confusing names, maybe rather use attachmentName on the left side
32 2009-11-26T10:19:11 <ThomasWaldmann> l1.278+ wtf?
33 2009-11-26T10:25:34 <ThomasWaldmann> line numbers refer to:
34 2009-11-26T10:25:37 <ThomasWaldmann> http://s15358202.onlinehome-server.info/hg/moin-19-modified-clean-changesets/rev/bec06fa7f2aa
35 2009-11-26T10:32:16 <ThomasWaldmann> that "LanguageString()" (l1.278+) stuff has to be removed completely. it is just used at 2 places and those 2 places should be done differently.
36 2009-11-26T10:34:06 <ThomasWaldmann> if you don't want that empty pagename or empty attachname is submitted, I suggest you just do not enable OK submit button if those fields are empty.
37 2009-11-26T10:42:02 <ThomasWaldmann> <a href="test.pdf" title="attachment:test.pdf">test.pdf</a>
38 2009-11-26T10:43:06 <ThomasWaldmann> that href is not really useful. it looks like this works by ignoring href in the converter and just using what title contains. :)
39 2009-11-26T10:45:31 <ThomasWaldmann> of one edits the attachment name, it doesn't even change the href
40 2009-11-26T11:18:31 <dreimark> moin
41 2009-11-26T11:36:11 <ThomasWaldmann> hi dreimark
42 2009-11-26T12:56:03 *** vinci_
43 2009-11-26T13:12:58 *** tpfennig
44 2009-11-26T15:18:39 *** vinci_
45 2009-11-26T16:26:50 *** vinci_
46 2009-11-26T19:42:53 *** vinci__
47 2009-11-26T19:58:04 *** vinci_
48 2009-11-26T20:28:06 *** vinci__
49 2009-11-26T20:44:12 *** grzywacz
50 2009-11-26T22:10:49 <dreimark> re
51 2009-11-26T22:11:49 <ThomasWaldmann> hi dreimark
52 2009-11-26T22:13:49 <dreimark> lots of work because of hardware defects (normally that happens in may)
53 2009-11-26T22:15:01 <ThomasWaldmann> disks?
54 2009-11-26T22:15:17 <dreimark> yeah and graphic cards
55 2009-11-26T22:53:21 *** JosefMeier
56 2009-11-26T23:14:09 *** JosefMeier
57 2009-11-26T23:52:42 *** JosefMeier
58
MoinMoin: MoinMoinChat/Logs/moin-dev/2009-11-26 (last edited 2009-11-25 23:30:02 by IrcLogImporter)