1 2012-06-12T00:08:50 <dreimark> RogerHaase: do you have some better guide than http://wilcofiers.com/2011/03/using-strict-mode-today/ for explaining strict mode?
2 2012-06-12T00:12:30 <ThomasWaldmann> RogerHaase: done
3 2012-06-12T00:12:58 <RogerHaase> dreimark: no, that is as good an explanation as any. jslint complains if "use strict" is missing or put at the beginning of the file.
4 2012-06-12T00:15:23 <dreimark> I just wonder if we have to add it on each function or can do it once on top
5 2012-06-12T00:18:36 <RogerHaase> ThomasWaldmann: delete is a reserved word so you cannot do actionLoading.delete ... jslint complains about actionLoading["destroy"], wants actionLoading.destroy
6 2012-06-12T00:19:23 <ThomasWaldmann> that's a bit inconsistent then
7 2012-06-12T00:20:28 <RogerHaase> dreimark: putting "use strict" at top is allowed, but jslint complains.
8 2012-06-12T00:21:52 *** MaikuMori_
9 2012-06-12T00:23:18 <dreimark> ok, i think then we go with jslint
10 2012-06-12T00:23:50 <bretonium> can I see something like a backtrace, which functions where used to get an item, for example?
11 2012-06-12T00:25:10 <RogerHaase> this is bordering on scope creep again, but an alternate name for "delete" could be used, and perhaps the first 6 functions could be put inside the function beginning at line 106 to reduce teh number of "use strict"s
12 2012-06-12T00:26:16 <dreimark> I have no problem with the amount, it was just that also a new function has this mode if we have it on top
13 2012-06-12T00:27:15 <RogerHaase> and there were some other style changes suggested a few months back on common.js to eliminate a few lines containing only { and }
14 2012-06-12T00:29:05 *** MaikuMori
15 2012-06-12T00:29:24 <dreimark> http://caniuse.com/#feat=use-strict
16 2012-06-12T00:29:58 <dreimark> http://stackoverflow.com/questions/1335851/what-does-use-strict-do-in-javascript-and-what-is-the-reasoning-behind-it
17 2012-06-12T00:30:00 *** kanha
18 2012-06-12T00:30:05 *** kanha_
19 2012-06-12T00:30:58 *** vpv
20 2012-06-12T00:31:15 <RogerHaase> dreimark: iirc, if "use strict" is placed outside a function, then it will apply to every script file loaded thereafter, perhaps leading to unintended errors in old code that cannot run in strict mode.
21 2012-06-12T00:31:17 <bretonium> also, why are content and user profiles stored differently?
22 2012-06-12T00:31:35 *** jaiditya
23 2012-06-12T00:32:09 *** vpv
24 2012-06-12T00:32:53 <dreimark> agreed
25 2012-06-12T00:33:19 <dreimark> bretonium: content has revisions
26 2012-06-12T00:33:33 *** dwcramer
27 2012-06-12T00:34:27 <RogerHaase> gtg
28 2012-06-12T00:34:58 *** RogerHaase
29 2012-06-12T00:35:53 <bretonium> dreimark: ok, and using which code are userprofiles stored then?
30 2012-06-12T00:36:28 *** dwcramer
31 2012-06-12T00:37:36 <bretonium> I meen, where in the code are differences between userprofiles and content begin?
32 2012-06-12T00:37:46 <bretonium> *do differences
33 2012-06-12T00:46:35 <ThomasWaldmann> user profiles are dealt with in user.py
34 2012-06-12T00:46:38 * dreimark is too tired
35 2012-06-12T00:46:39 <dreimark> gn
36 2012-06-12T01:32:28 *** spy
37 2012-06-12T02:35:44 *** jaiditya
38 2012-06-12T03:35:57 *** sudo_dirk
39 2012-06-12T03:37:16 *** sudo_dirk1
40 2012-06-12T06:35:10 <bretonium> ThomasWaldmann: commented http://etherpad.osuosl.org/moin2-bms, 151
41 2012-06-12T08:11:22 *** MattMaker
42 2012-06-12T08:45:33 <dreimark> moin
43 2012-06-12T09:32:58 *** MaikuMori_
44 2012-06-12T09:59:58 <ThomasWaldmann> bretonium: replied
45 2012-06-12T10:48:00 *** Trip__
46 2012-06-12T10:48:25 *** MattMaker
47 2012-06-12T10:48:39 *** MattMaker
48 2012-06-12T11:56:49 <bretonium> also, what is the sence in having separate backends, why aren't they merged with stores?
49 2012-06-12T12:02:36 *** Trip__
50 2012-06-12T12:05:41 <bretonium> I mean, why would anyone want to implement another backend?
51 2012-06-12T12:05:54 *** MattMaker
52 2012-06-12T12:10:41 <bretonium> ThomasWaldmann: http://etherpad.osuosl.org/moin2-bms
53 2012-06-12T12:12:42 <ThomasWaldmann> bretonium: backend and store are 2 different levels
54 2012-06-12T12:13:15 <ThomasWaldmann> you see it when you look at the fileserver backend, that one is NOT using stores
55 2012-06-12T12:16:20 <bretonium> oh, ok. Is it used anywhere now?
56 2012-06-12T12:16:35 <ThomasWaldmann> if you configure it in your config....
57 2012-06-12T12:18:21 <ThomasWaldmann> 151+ still makes no sense btw
58 2012-06-12T12:21:48 <ThomasWaldmann> btw, did you disconnect from EP?
59 2012-06-12T12:21:51 <ThomasWaldmann> bretonium: ^^
60 2012-06-12T12:25:48 <ThomasWaldmann> wake me up if you continue...
61 2012-06-12T12:47:48 <bretonium> yes. I am in the university now, some free minutes appeared.
62 2012-06-12T12:58:13 <ThomasWaldmann> ah, ok :)
63 2012-06-12T12:59:17 <bretonium> back there for the next 20-30 minutes.
64 2012-06-12T13:14:34 *** spy
65 2012-06-12T14:37:49 *** dave_largo
66 2012-06-12T15:10:43 *** dwcramer
67 2012-06-12T15:20:39 <xiaq> https://bitbucket.org/thomaswaldmann/moin-2.0/src/452b341325eb/MoinMoin/items/__init__.py#cl-697
68 2012-06-12T15:20:49 <xiaq> Does the comment still apply?
69 2012-06-12T15:21:18 <xiaq> _do_modify_show_templates() is already implemented: https://bitbucket.org/thomaswaldmann/moin-2.0/src/452b341325eb/MoinMoin/items/__init__.py#cl-304
70 2012-06-12T15:25:43 <xiaq> uh oh, _do_modify_show_templates *is* implemented but only used in Text.do_modify...
71 2012-06-12T15:50:17 *** kanha_
72 2012-06-12T15:50:55 *** kanha
73 2012-06-12T16:19:21 <ThomasWaldmann> xiaq: seems a bit like the comment lines there are outdated
74 2012-06-12T16:19:45 *** dwcramer
75 2012-06-12T16:29:42 <xiaq> ThomasWaldmann: also there are much duplication in the several do_modify
76 2012-06-12T16:30:53 <ThomasWaldmann> that might be
77 2012-06-12T17:52:13 *** RogerHaase
78 2012-06-12T18:04:05 <spy> I have a problem with whoosh sorting by PTIME field. This is my code: http://codereview.appspot.com/6307082
79 2012-06-12T18:04:40 <spy> in fact, it doesn't sort at all, what have I missed?
80 2012-06-12T18:24:13 <spy> ThomasWaldmann: ^^ can help?
81 2012-06-12T19:00:37 <ThomasWaldmann> hi spy
82 2012-06-12T19:06:57 <ThomasWaldmann> spy: review done (whoosh part mainly)
83 2012-06-12T19:11:09 <ThomasWaldmann> gtg/brb
84 2012-06-12T19:24:22 *** spy_
85 2012-06-12T19:28:30 *** spy_
86 2012-06-12T19:37:40 *** dave_largo
87 2012-06-12T19:55:31 *** spy_
88 2012-06-12T19:59:45 *** spy_
89 2012-06-12T20:26:33 *** Trip__
90 2012-06-12T20:27:09 * dreimark we need abort for openid if a user did not know the password of his existing account
91 2012-06-12T20:51:24 *** MattMaker
92 2012-06-12T20:57:42 *** spy_
93 2012-06-12T21:01:50 *** spy_
94 2012-06-12T21:18:39 *** spy_
95 2012-06-12T21:23:05 *** spy_
96 2012-06-12T21:23:21 <RogerHaase> dreimark: link to xstatic multiDownload HOMEPAGE = 'http://biesiad.rushbean.com/multidownload/ is dead; current link may be https://github.com/biesiad/multiDownload
97 2012-06-12T21:24:38 <dreimark> RogerHaase: ok, thx
98 2012-06-12T21:26:25 <dreimark> i fix now the url and look tom. if i can do an upgrade (or have to)
99 2012-06-12T21:47:58 *** spy_
100 2012-06-12T22:22:20 <spy> ThomasWaldmann: replied on http://codereview.appspot.com/6307082/diff/1/MoinMoin/apps/frontend/views.py#newcode484
101 2012-06-12T22:29:02 <spy> dreimark: do you have any ideas on http://codereview.appspot.com/6307082/diff/1/MoinMoin/apps/frontend/views.py#newcode484 ?
102 2012-06-12T23:57:05 *** dwcramer
103 2012-06-12T23:57:48 *** Trip__
104 2012-06-12T23:59:02 *** MattMaker
105
