IRC logs of #tryton for Monday, 2008-06-16 #tryton log beginning Mon Jun 16 00:00:01 CEST 2008
CIA-52tryton: C?dric Krier <> default * 434:b1af1d5077c0 tryton/tryton/gui/window/ ( Guidelines00:09
CIA-52tryton: C?dric Krier <> default * 613:89bb53449c27 trytond/trytond/osv/ Remove size constraint for Reference field00:09
CIA-52tryton: C?dric Krier <> default * 614:109660fbaa0d trytond/trytond/ir/ Remove size constraint on actions00:09
-!- kultviec1( has left #tryton00:35
-!- yangoon1( has joined #tryton05:20
-!- cedk(n=ced@gentoo/developer/cedk) has joined #tryton08:45
-!- gadaga( has joined #tryton09:14
-!- Timitos(n=Timitos@ has joined #tryton09:26
Timitosgadaga: hi09:34
gadagaTimitos: hi09:34
-!- bechamel( has joined #tryton09:47
cedkWhat do you think about adding a field on ir_rule_group to have the rules applied on all groups by default ?10:42
cedklike global_p but with an OR in the sql query10:43
cedkit will behave like if the rule will be added on all users10:43
cedkbechamel: ping10:45
bechamelcedk: global_p ?10:50
cedkbechamel: global_p make the rule global so it is apply on all with an "AND"10:51
cedkso global rules can not be by pass by a rule put on a group10:53
cedkhere is the example where I'm block with the current rule system10:54
cedkit is for timesheet, we want to allow user to see only her timesheet10:54
cedkand allow user that are in "timesheet admin" group to see all the timesheets10:54
cedkso first I try with a global rule but I can not allow "timesheet admin" users to see all timesheet10:55
cedkone solution is to put all other users in a group that have the default rule10:55
cedkbut it is not very userfriendly because each time you create a user, you must put it in this group10:56
cedkso for me the solution is to create a default rule that is apply on every user but that is not global10:57
bechamelcedk: adding group to user at creation is common, i dont see a problem10:59
Timitosbechamel: +111:00
bechamelcedk: for me a user without group should be restricted to a very minimal access11:00
cedkbechamel: because this rule 'see only my timesheet' is rule that must be satisfied every time for everybody11:01
cedkbechamel: so the rule must be defined on all users but not global because we need to be able to bypass it for some users11:01
cedkthat's why I want to add a default_p field on rule_group to make this11:02
bechamelcedk: the problem is that we need to express OR, which is not implemented, instead of creating an exception i would prefer to develop the feature.11:06
cedkbechamel: I don't think so11:07
cedkbechamel: we have OR11:07
cedkwhat we don't have is to be able to put a rule on every users11:07
cedkbechamel: OR is just create to rule_group11:11
bechamelcedk: what about a group "timesheet user" and a group "timesheet admin" and the timesheet admin group is allowed to see all the timesheet11:14
cedkbechamel: what about user that is in any groups ?11:17
bechamelcedk: these groups are exclusives of course11:19
cedkbechamel: what about user that is in any groups ?11:20
cedkbechamel: how do you make groups exclusives ?11:20
bechamelcedk: my opinion is that the actual rule system is not ok, but i dont think that adding a field will solve the problem11:22
cedkbechamel: for me it is enough11:22
cedkwe need to match how customers think11:23
cedkthey think like this: user can only see her timesheet and admin can see every body11:23
cedkI have made the dev11:31
cedkbechamel: I commit ?11:31
cedkbechamel: It works for the timesheet needs11:31
cedkbechamel: and for you, in fact it is the clause of rules that are not enough11:32
bechamelcedk: yes commit, it will do the trick until we got a better idea :)11:34
-!- Timitos(n=Timitos@ has left #tryton11:34
CIA-52tryton: C?dric Krier <> default * 29:519ef7569ac0 timesheet/line.xml: Add record rules on timesheet lines11:38
CIA-52tryton: C?dric Krier <> default * 615:5df119ab9b78 trytond/trytond/ (ir/ ir/rule.xml res/ res/
CIA-52tryton: Improve record rules:11:38
CIA-52tryton: - Add default_p to allow to put a rule_group on all users11:38
CIA-52tryton: - Fix the "rule_group is True if there is no rule"11:38
CIA-52tryton: - Fix domain on user and group11:38
cedkbechamel: not the rule_group11:40
-!- cedk_( has joined #tryton11:50
-!- cedk_( has joined #tryton11:53
-!- cedk_( has joined #tryton11:54
-!- cedk(n=ced@gentoo/developer/cedk) has joined #tryton11:54
-!- cedk(n=ced@gentoo/developer/cedk) has joined #tryton12:01
bechamelcedk: about the documentation, i just discovered which tool is used by python peoples:
bechamelcedk: it allow syntax highlighting and footnotes12:09
cedkbechamel: the python doc is in tex12:14
bechamelcedk: the old one maybe,
cedkbechamel: why not12:24
-!- yangoon( has joined #tryton12:38
-!- kultviech( has joined #tryton12:45
-!- kultviech( has left #tryton12:45
-!- gadaga( has joined #tryton13:54
-!- cedk(n=ced@gentoo/developer/cedk) has joined #tryton14:11
-!- mr_amit(n=amit@ has joined #tryton14:25
-!- mr_amit(n=amit@ has left #tryton14:25
cedkit was a indian from tiny :-)14:28
bechamelgrep /var/log/httpd/access.log :)14:30
cedkzgrep /var/log/lighttpd/access.log*|wc14:35
cedk240 lines14:35
CIA-52tryton: htgoebel roundup * #130/patch: Added menu some shortcuts.: [new] Added menu some shortcuts.16:06
CIA-52tryton: default * 435:4a9c0d9a155e tryton/share/tryton/ Added menu some shortcuts.16:58
CIA-52tryton: ced roundup * #130/patch: Added menu some shortcuts.: [resolved] Applied16:58
-!- kultviech( has joined #tryton17:47
CIA-52tryton: gadaga roundup * #131/server crash in ubuntu 8.04 : cannot import name xpath: [new] File "./trytond", line 7, in <module> import trytond File "/home/ruijuan/tryton/trytond/trytond/", line 1, in <module> ...18:12
CIA-52tryton: ced roundup * #131/server crash in ubuntu 8.04 : cannot import name xpath: [chatting] did you have install pyxml?18:17
CIA-52tryton: C?dric Krier <> default * 616:b897ab0d0a14 trytond/trytond/tools/ Remove import minidom18:37
udonoanother fork?
cedkI propose to remove pyxml dependency and you lxml instead19:11
cedkpyxml begin to be deprecated on some distribution19:12
-!- udono( has left #tryton19:42
-!- Timitos(n=Timitos@ has joined #tryton19:57
cedkI have already the xpath in inherit view working with lxml20:01
-!- Timitos(n=Timitos@ has left #tryton20:46
cedkbechamel: what do you think about lxml ?21:09
bechameli didn't know that pyxml is deprecated21:11
bechamelcedk: i made a google search with pyxml and lxml and i found: "So in conclusion: lxml kicks ass" :)21:13
cedkbechamel: it is more pytonic21:14
cedkbechamel: on ubuntu they put pyxml in old subfolder21:14
bechamelcedk: i was suprised that there is not a lot of people on the channel, ... but i realised that the germany team play tonight21:20
cedkbechamel: last release of pyxml: November 29, 200421:20
cedkI have almost remove dependency to pyxml21:21
bechamelcedk: ok21:22
cedkbechamel: go to lunch21:23
bechamelcedk: bye21:24
cedkbechamel: I have done the replacement of pyxml with lxml22:59
cedkbechamel: Do I commit ?22:59
bechamelcedk: yes yes23:14
CIA-52tryton: C?dric Krier <> default * 617:3d56b1784786 trytond/trytond/ (ir/ui/ osv/ res/ Remove pyxml dependency, use lxml instead23:16
CIA-52tryton: C?dric Krier <> default * 10:07fa8c553738 analytic_account/ Remove pyxml dependency, use lxml instead23:16
CIA-52tryton: C?dric Krier <> default * 4:b79d95b48f2f analytic_invoice/ Remove pyxml dependenxy, use lxml instead23:16
CIA-52tryton: C?dric Krier <> default * 3:f745ee93f96b analytic_purchase/ Remove pyxml dependency, use lxml instead23:16
CIA-52tryton: C?dric Krier <> default * 12:457f57c5acac gentoo-overlay/app-office/trytond/ (Manifest trytond-9999.ebuild): Remove pyxml dependency, use lxml instead23:19
CIA-52tryton: C?dric Krier <> default * 13:baea4e54caab gentoo-overlay/app-office/tryton/ (Manifest tryton-9999.ebuild): Remove pyxml dependency23:20
-!- yangoon3( has joined #tryton23:20
CIA-52tryton: C?dric Krier <> default * 436:3deb2353e9c3 tryton/ Remove xml dependency in setup.py23:20
CIA-52tryton: C?dric Krier <> default * 618:2f4bb79af7ef trytond/ Add dependency to lxml in setup.py23:21
yangoon3cedk: just observing, that all database tables are created with oids. Is this necessary or could they be omitted?23:22
CIA-52tryton: ced roundup * #131/server crash in ubuntu 8.04 : cannot import name xpath: [resolved] We remove the dependency for pyxml and use lxml instead. Changeset 3d56b178478623:23
cedkyangoon3: I don't see oid ?23:24
yangoon3cedk: all tables were created WITH OIDS;, and they were created by tryton23:26
cedkyangoon3: how do you see that ?23:27
yangoon3par exemple avec pgadmin3, psql...23:28
cedkyangoon3: with pgsql, I do \d table and I don't see oid23:28
bechamelcedk: yangoon3 talks about the id column23:28
cedkyangoon3: do you mean "id" ?23:29
yangoon3cedk: no it's not id, it is really oid23:30
yangoon3cedk: but I have to precise23:30
yangoon3not alltabels, but some tables23:31
yangoon3tables product_23:31
cedkyangoon3: which tables ?23:32
cedkyangoon3: can you copy/paste the output ?23:33
yangoon3all product_, all purchase_, all person_ ...23:35
yangoon3cedk: but only some ir_23:35
cedkyangoon3: do you mean:
yangoon3cedk: exactly23:37
cedkyangoon3: it is not tryton that create it but postgresql23:38
yangoon3cedk: of course, but postgres default is WITHOUT OIDS23:38
cedkyangoon3: tryton doesn't create any oid23:38
yangoon3cedk: so the OIDS have to be explicitely set in some tryton database definitions23:39
yangoon3cedk: this is only the matter to set WITH OIDS while creating the table23:39
cedkyangoon3: I see that table are created with oids23:41
yangoon3cedk: if tryton doesn't need them (hopefully, because poor design), they should be omitted23:41
cedkyangoon3: yes I think we can remove it23:43
yangoon3cedk: I don't think, they are needed by tryton, because tryton uses its own set of id23:43
yangoon3they are probably only a relict of former databes dumps (before postgres8.0)23:44
CIA-52tryton: C?dric Krier <> default * 619:5855c3cedffd trytond/trytond/osv/ Remove "WITH OIDS" on table creation23:47
cedkyangoon3: fixed23:48
yangoon3cedk: you are really fast!23:49
yangoon3cedk: respect23:49
cedkyangoon3: by the way we remove dependency to pyxml and use instead lxml23:50
cedkyangoon3: so if you update you need to install lxml23:50
yangoon3ok, thanks23:50
yangoon3cedk: for tryton or trytond?23:51
cedkyangoon3: for trytond23:53
cedkyangoon3: and in fact tryton doesn't depend on pyxml23:53
CIA-52tryton: C?dric Krier <> default * 620:fddcd062aa4b trytond/trytond/ (ir/ report/ Fix missing import minidom23:54
CIA-52tryton: ced roundup * #70/xml: base validation on Namespaces: We have removed pyxml dependency and use lxml instead. lxml have also validator for relax-ng.23:56

Generated by 2.11.0 by Marius Gedminas - find it at!