IRC log of #schooltool for Thursday, 2006-06-08

*** pcardune has quit IRC00:01
*** jinty has quit IRC02:03
*** tiredbones has quit IRC02:42
*** tiredbones has joined #schooltool02:42
*** tristil has joined #schooltool05:02
*** tristil has quit IRC06:09
*** tristil has joined #schooltool06:41
tristilI'm getting a mysterious traceback relating to the new securitypolicy meta.zcml after the recent merge:
*** tristil has quit IRC07:44
*** gintas has joined #schooltool10:26
*** ignas has joined #schooltool11:01
povbot/svn/commits: * gintas committed revision 6184:11:18
povbot/svn/commits: Removed trailing whitespace.11:18
povbot/svn/commits: * gintas committed revision 6185:11:20
povbot/svn/commits: More trailing whitespace.11:20
povbot/svn/commits: * gintas committed revision 6186:11:26
povbot/svn/commits: Whitespace nazism.11:26
*** vidasp has joined #schooltool11:30
*** thisfred has joined #schooltool11:31
*** faassen has joined #schooltool11:32
povbot/svn/commits: * gintas committed revision 6187:11:47
povbot/svn/commits: Added a setting for public access to the person list, uncommented related test in demographics.11:47
povbot/svn/commits: * vidas committed revision 6188:11:51
povbot/svn/commits: Evolution script - remove all local grants.11:51
povbot/svn/commits: * gintas committed revision 6189:11:55
povbot/svn/commits: Removed obsolete branch.11:55
povbot/svn/commits: * gintas committed revision 6190:12:17
povbot/svn/commits: Allow only the administration to edit access control.12:17
povbot/svn/commits: * ignas committed revision 6191:12:26
povbot/svn/commits: Replaced OpenOffice writer odt with an Microsoft Excel 97 xls.12:26
povbot/svn/commits: XLS because both OpenOffice and Gnumeric support it flawlessly.12:26
povbot/svn/commits: * gintas committed revision 6192:12:27
povbot/svn/commits: Security views are now longer available to everyone.12:27
*** vidasp has quit IRC12:31
faassenI'm trying to evolve a sample data database of a february schooltool checkout12:41
faassento the current trunk12:42
faassenbut fails.12:42
faassenbecause the application object for some reason cannot be adapted to ITimeTables12:42
faassenanyone have any clue?12:42
ignascan i see a traceback ?12:45
faassenTraceback (most recent call last):12:45
faassen  File "/home/faassen/working/schooltool/schooltool/Zope3/src/zope/app/generations/", line 404, in evolve12:45
faassen    manager.evolve(context, generation)12:45
faassen  File "/home/faassen/working/schooltool/schooltool/Zope3/src/zope/app/generations/", line 133, in evolve12:45
faassen    evolver.evolve(context)12:45
faassen  File "/home/faassen/working/schooltool/schooltool/src/schooltool/generations/", line 45, in evolve12:45
faassen    for tt in ITimetables(app).timetables.values():12:45
faassenTypeError: ('Could not adapt', < object at 0xb73ef7ac>, <InterfaceClass schooltool.timetable.interfaces.ITimetables>)12:45
faassenTraceback (most recent call last):12:45
faassenso, it tries to do ITimeTables on a schooltool application object and fails.12:45
ignasmaybe IHaveTimetables was removed from schooltool App12:47
ignasi think it was12:47
faassenso what now?12:49
ignasa sec12:49
povbot/svn/commits: * gintas committed revision 6193:12:49
povbot/svn/commits: Small cleanups.12:50
povbot/svn/commits: * gintas committed revision 6194:12:50
povbot/svn/commits: Changed wording.12:51
*** mgedmin has joined #schooltool12:51
ignasit would be nice if you would make a copy of that Data.fs12:52
ignasand for now as not to hinder your progress i think using TimetablesAdapter directly on the application should do the trick12:53
faassenwhat do you mean, make a copy of the Data.fs for what?12:54
faassenI'm testing my own evolution scripts.12:54
faassenI have a copy of that Data.fs obviously.12:54
faassenI just created it for this purpose.12:54
faassenso I could import TimeTablesAdapter and then apply it directly, not doing the adapter lookup?12:54
faassenI should check that in, right?12:54
faassenI just want it to evolve up to the point where it breaks for reasons of my own wrong-doing. :)12:55
ignasdoing an adapter lookup in an evolution scriptis a bad idea12:55
povbot/svn/commits: * gintas committed revision 6195:12:56
povbot/svn/commits: Allowed leaders to view and edit groups.12:56
ignasi would even go as far as to not use the adapter but twiddle with "real" attributes of the object12:56
ignasas the code in the adapter might change in the future12:56
ignasand evolution scripts should not "evolve" themselves12:56
faassenthough in case of IAnnotations and such I might want to still use an adapter. :)12:56
faassenbut yeah, messing with data directly seems to be the way to go. it's nto going to make evolution easier.12:57
ignasyes, but evolution scripts  should be 1. stable 2. stable 3. stable12:57
ignascan't see easy to write in the list :(12:58
ignasand yes i hate it12:58
faassenyeah, they're awful. they're definitely not easy to write.12:58
ignashave you noticed the new security policy ?13:01
faassennow generation11 fails.13:01
faassenI haven't looked at it yet.13:01
faassenI just want to get the evolution script for demographics working.13:01
faassenI mean, I briefly glanced over it.13:01
ignasi mean from the UI perspective13:01
faassenTraceback (most recent call last):13:01
faassen  File "/home/faassen/working/schooltool/schooltool/Zope3/src/zope/app/generations/", line 404, in evolve13:01
faassen    manager.evolve(context, generation)13:01
faassen  File "/home/faassen/working/schooltool/schooltool/Zope3/src/zope/app/generations/", line 133, in evolve13:01
faassen    evolver.evolve(context)13:01
faassen  File "/home/faassen/working/schooltool/schooltool/src/schooltool/generations/", line 39, in evolve13:01
faassen    roles = IPrincipalRoleManager(app)13:01
faassenTypeError: ('Could not adapt', < object at 0xb74487ac>, <InterfaceClass>)13:02
faassenhaven't even booted the UI yet. :)13:02
faassensince this is security related, I don't know what to do now.13:02
ignasi'd even say dump it (we don't have permissions that should get migrated anymore)13:02
ignasthough i am not sure it is correct approach13:03
povbot/svn/commits: * gintas committed revision 6196:13:03
povbot/svn/commits: Shuffled security declarations around.13:03
faassendump the whole evolution script?13:03
faassenit only seems to be tweaking with roles and permissions.13:03
ignasthese roles do not exist anymore13:04
faassenI put a return on top, see which evolution it fails on now :)13:04
faassenI expected this.13:04
faassenI mean, I expected the evolutions wouldn't work and I needed to spend a lot of time fixing existing ones. :)13:04
faasseni had a horrible time writing the one evolution I did.13:04
faassenreplacing objects with instances of another class, yuck.13:05
ignasyou should not base your opinion on the "VERY WORST CASE i could imagine" ;)13:05
faassenof course I should! :)13:06
ignascan i say "we told you should use annotations" now ? :)13:07
faassenwell, I realized that drawback a montha go when I wrote that script. :)13:08
faassenit's a question of tradeoffs.13:08
faassenand evolution scripts are horrible to write anyway.13:08
ignasmost of the time, yes13:09
ignasas there aren't many unit tests for evolve14, is the __annotations missing trailing "__" intentionally in "new_person.__annotations = person.__annotations__" ?13:11
faassenick, I imagine not.13:12
faassenI'll extend the test for that.13:13
faassengood catch!13:14
*** vidasp has joined #schooltool13:26
povbot/svn/commits: * gintas committed revision 6197:13:27
povbot/svn/commits: Added settings for group view permission.13:27
povbot/svn/commits: * vidas committed revision 6198:13:41
povbot/svn/commits: Restore manager user with new permissions (manager group).13:41
povbot/svn/commits: * gintas committed revision 6199:13:42
povbot/svn/commits: Added a setting for members editing group calendars.13:42
povbot/svn/commits: * gintas committed revision 6200:13:46
povbot/svn/commits: Everyone can view courses.13:46
*** Aiste has quit IRC13:56
faassenI really have no idea what I'm doing with evolve10.py14:07
faassenI changed it so that evolution doesn't crash14:07
faassenbut the tests fail now14:07
faassenand I have no idea what the tests are even intended to test.14:07
faassenI mean, the tests for evolve10 fail.14:08
faassenand I don't know what's going on.14:08
ignasfaassen: the adapter was being stubbed and now as you are not using adaptation it breaks14:10
faassenwhere does it provide the stubbed adapter?14:10
faassenI'm trying to figure out where this stubbing takes place.14:10
faassenI'd happily use a stubbed adapter.14:11
ignasoh sorry14:11
ignasit's implements(ITimetables)14:11
ignasnot nice14:11
ignasyou will probably have to add fake annotations14:11
faassenbut where is it stubbing an adapter? oh, you mean these things implement the interface themselves.14:11
faasseninstead of going through..14:11
faassenI already did that.14:11
faassenit still breaks.14:11
ignasfaassen: send me a diff for that evolution script14:12
ignasand i'll look into it14:12
faassenI'll just send you the whole two files.14:12
ignasok14:12, right?14:12
faassenI'm sending you the code I have now.14:12
faassenwhich adds the annotations (in the tests)14:13
faassenand uses the TimetablesAdapter directly.14:13
faassenthough of course that won't be correct.14:13
faassenas these things pretend to implement their own adapter.14:13
faassensorto f. :)14:13
faassenon their way.14:16
*** Aiste has joined #schooltool14:21
*** alga has joined #SchoolTool14:26
povbot/svn/commits: * gintas committed revision 6201:14:27
povbot/svn/commits: Fixed permissions on resources.14:27
*** vidasp has quit IRC14:38
povbot/svn/commits: * gintas committed revision 6202:14:39
povbot/svn/commits: Fixed permissions for terms.14:39
povbot/svn/commits: * gintas committed revision 6203:14:51
povbot/svn/commits: Oops.14:51
povbot/svn/commits: * gintas committed revision 6204:15:09
povbot/svn/commits: Added a setting to disallow a user to change preferences.15:09
povbot/svn/commits: Currently although access control actually works, there is no good way to hide the preferences view from the user.15:09
povbot/svn/commits: * ignas committed revision 6205:15:11
povbot/svn/commits: Not very elegant but working fix for an evolution script that relied on adapterst that might not exist anymore in the future.15:11
ignasfaassen: fixed the script15:11
faassenignas: cool15:14
faassenignas: thanks15:14
povbot/svn/commits: * gintas committed revision 6206:15:19
povbot/svn/commits: Cosmetic fixes.15:19
povbot/svn/commits: * gintas committed revision 6207:15:19
povbot/svn/commits: Tabs!15:19
povbot/svn/commits: * faassen committed revision 6208:15:24
povbot/svn/commits: evolve11 used to set up roles and permissions, but since the new security policy is now in place this has become obsolete (and in fact breaks evolution).15:24
povbot/svn/commits: * faassen committed revision 6209:15:25
povbot/svn/commits: There was a bug in the carrying over of annotations. Fixed and extended the tests to catch this issue.15:25
povbot/svn/commits: * gintas committed revision 6210:15:40
povbot/svn/commits: Ouch, our security settings were not persistent ;(15:40
povbot/svn/commits: I this that developer sandboxes may also be polluted with nonpersistent dicts...15:40
*** tiredbones has quit IRC15:42
*** tiredbones has joined #schooltool15:43
povbot/svn/commits: * gintas committed revision 6211:15:47
povbot/svn/commits: The preference view is now registered on IPersonPreferences, as it should be.  The view can now be properly secured.  However, the external menu item can not be hidden automatically as it is.15:47
povbot/svn/commits: * vidas committed revision 6212:16:01
povbot/svn/commits: Evolution script - restore all manager users (permissions, not passwords).16:01
*** pcardune has joined #schooltool16:03
pcardunesrichter: ayt?16:05
srichterpcardune: yeah, but for long ;-)16:05
pcardunefor long? or not for long?16:05
srichterpcardune: yeah, but not for long ;-)16:06
pcarduneWe have an impenetrable infinite recursion bug which I can't figure out.  I can get it to happen through a python prompt, but only through the web interface16:06
pcarduneI'm going to set up an instance for you16:06
pcardunewith the bug in place16:07
pcarduneand I would love it if you could take a look at it at some point16:07
pcardune(doesn't have to be right now of course)16:07
pcarduneok, thanks16:07
srichtereven better would be to tell me how I can reproduce it on my local machine16:07
*** gintas has quit IRC16:08
pcarduneOk, it is kind of complicated, so I'll try and write it down somewhere16:08
povbot/svn/commits: * faassen committed revision 6213:16:15
povbot/svn/commits: Added some more documentation.16:15
*** pcardune_ has joined #schooltool16:16
ignassrichter: maybe you know some easy way of adding an zmi "action" that would be visible when looking at person while at the same time it's permissions would be determined based on person.preferences object ?16:16
mgedmina custom menu item perhaps?16:17
srichterI agree with MArius16:18
srichterin another project I actually started to not use menus at all anymore16:19
srichterwe just use viewlets, like I did for some parts of ST16:19
srichterok, I am out of here; time to go teaching16:19
*** pcardune has quit IRC16:29
pcardune_does anyone know what the deal is with the dashboard package?  Is it staying around?  I don't see it attached anywhere in the UI16:34
*** Aiste_ has joined #schooltool16:40
*** Aiste has quit IRC16:42
pcardune_ignas: do you know if the dashboard is sticking around? and is the original Person index.html available anywhere?16:51
ignasno don't really know16:51
ignasthere was a  story for that in some contract but it got postponed i think16:51
pcardune_Maybe I should ask th1a16:52
ignasprobably yes16:52
povbot/svn/commits: * ignas committed revision 6214:16:53
povbot/svn/commits: Fix a failing unit test.16:53
pcardune_th1a: ayt?17:18
*** ignas has quit IRC17:39
faassenpcardune_: concerning the Person index.html, I removed it as it was not necessary anymore.17:42
faassenpcardune_: of course your use case is valid, it's just that we keep going between "Person should be usable independently and all customizations should be on the outside" and "we can't really test that at this stage and we need to get features done" positions.17:43
faassenif I see stuff like this..17:43
faassenComponentLookupError: (<persistent broken schooltool.timetable.term.TermContainer instance '\x00\x00\x00\x00\x00\x00\x00\x05'>, <InterfaceClass schooltool.securitypolicy.interfaces.ICrowd>, 'schooltool.view')17:43
faassenafter an upgrade.17:44
faassenthen that's not good, right? :)17:44
pcardune_faassen: that doesn't look good...17:50
*** pcardune_ is now known as pcardune17:50
pcardunefaassen: I had registered some viewlets that were used on that Person page template, and i'm wondering if I should move them elsewhere?17:51
pcardunewhere do you think stuff like that should Go17:51
mgedminpersistent broken -- the class moved, and no one added a BBB link?17:52
faassenpcardune: some of those viewlets moved to the ..18:00
faassenmgedmin: I've no idea what this is all about. :)18:00
faassenpcardune: the schooldata part of the demographics screens18:00
faassenpcardune: but it was admittedly difficult to find a place for them. I tried to give it all a place somewhere, but it's quite likely things are missing.18:01
mgedminfaassen: the 'schooltool.timetable.term' module does not have a TermContainer class, but Persistent objects stored in the ZODB want to import that module, and instantiate TermContainers18:01
faassenpcardune: if you have suggestions on how it should go..18:01
faassenmgedmin: okay, this is all not good. :)18:01
faassenmgedmin: it seems I'm the guy actually ending up testing the upgrade. :)18:01
faassenmgedmin: inadventantly, I only want to get the demographics stuff working. :)18:01
faassenpcardune: if you have suggestions, please write them up in an email to schooltool-dev and I'll see what I can do.18:02
pcardunefaassen: Well, I think I'll wait until we have some sort of scheme for a UI refactor... that is what it really depends on18:06
faassenUI refactoring? :)18:07
faassenthe changes to the way persons are displayed already led to so much difficulty changing the tests, we'd need a good strategy there definitely.18:08
*** th1a has quit IRC18:24
*** th1a_desk has joined #schooltool18:24
*** Aiste_ is now known as Aiste18:45
povbot/svn/commits: * vidas committed revision 6215:18:59
povbot/svn/commits: Fix broken unit test.18:59
*** th1a_desk is now known as th1a19:01
povbot/svn/commits: * vidas committed revision 6216:19:25
povbot/svn/commits: Added new person preferences option to make calendar public.19:25
pcardunefaassen: do you know what a "crowd" is?19:43
faassenpcardune: not yet, it's a security thing.19:59
pcarduneIt looks pretty cool20:04
*** thisfred has quit IRC20:10
*** faassen has quit IRC20:26
pcardunemgedmin: could you explain to me something about relationships?21:20
mgedminask away21:20
pcardunesections have a members=relationshipProperty(URIMembership, URIGroup, URIMember)...21:21
pcardunemgedmin: when i try to do this with another content component, it says "InvalidRelationship: Groups must provide IGroup." ... but section doesn't provide IGroup?21:22
mgedminclass ISection(IGroup):21:22
pcardunei thought I had looked at that already21:23
*** mgedmin has quit IRC22:37
*** tristil has joined #schooltool23:01
*** jinty has joined #schooltool23:03
tristilI get a mysterious ImportError on startup with zope 3.3 beta 1 and most recent svn, relating to the new securitypolicy code.
povbot/svn/commits: * pcardune committed revision 6217:23:32
povbot/svn/commits: Added IBaseGroup to distinguish from IGroup.  IBaseGroup is only functionality that is used in other components (cando), while IGroup is for the actual Group content component.23:32
pcardunetristil: did you perform a make update?23:34
tristilpcardune, it's just the schooltool directory from /src in a zope 3.3 instance. I know that's not a support issue, but the error is completely opaque to me, since it seems to be contradicting what's actually the case, and I have no idea how to debug it.23:37
pcarduneIt might have to do with zope package includes23:39
tristilDrives me crazy :)23:39
jintyth1a: ayt? it's a difficult today, mind if we chat next week about the new servr?23:39
pcarduneyou need to make sure there is a slug for the securitypolicy package in the right place23:39
tristilpcardune, I included the site.zcml at the end of the traceback.23:40
pcarduneI think "make update" will handle all that23:40
pcardunetristil: hmmm, I'm not really sure where the problem is23:41
th1ajinty: Oh, I'm here.23:49
th1aThat's fine.23:49
*** alga has quit IRC23:55
th1apcardune: I need to see if my Skype actually works on my new laptop.23:55
th1aCan I try calling you?23:55

Generated by 2.15.1 by Marius Gedminas - find it at!