| chat.freenode.net #tryton log beginning Tue Jan 19 00:00:02 CET 2010 | ||
| 2010-01-19 00:47 -!- irclog(n=irclog@tycho.b2ck.com) has joined #tryton | ||
| 2010-01-19 00:47 -!- FWiesing(n=FWiesing@194-208-185-012.tele.net) has left #tryton | ||
| 2010-01-19 00:53 -!- tekknokrat(n=gthieleb@dslb-088-075-227-054.pools.arcor-ip.net) has left #tryton | ||
| 2010-01-19 02:28 -!- vengfulsquirrel(n=ian@c-69-181-194-95.hsd1.ca.comcast.net) has joined #tryton | ||
| 2010-01-19 04:25 -!- ikks(n=ikks@190.158.102.38) has joined #tryton | ||
| 2010-01-19 04:47 -!- ikks(n=ikks@190.158.102.38) has joined #tryton | ||
| 2010-01-19 05:18 -!- yangoon(n=mathiasb@p549F7710.dip.t-dialin.net) has joined #tryton | ||
| 2010-01-19 08:24 -!- LucaSub(n=LucaSub@host138-180-dynamic.5-87-r.retail.telecomitalia.it) has joined #tryton | ||
| 2010-01-19 08:38 -!- enlightx(n=enlightx@static-217-133-61-144.clienti.tiscali.it) has joined #tryton | ||
| 2010-01-19 09:06 -!- vengfulsquirrel(n=ian@c-69-181-194-95.hsd1.ca.comcast.net) has joined #tryton | ||
| 2010-01-19 09:08 -!- paepke(n=paepke@p4FEB098A.dip0.t-ipconnect.de) has joined #tryton | ||
| 2010-01-19 09:10 -!- cedk(n=ced@gentoo/developer/cedk) has joined #tryton | ||
| 2010-01-19 09:38 -!- bechamel(n=user@host-85-201-159-186.brutele.be) has joined #tryton | ||
| 2010-01-19 10:12 <CIA-5> reihle roundup * #1385/Problem with CalDAV and Lightning 1.0: As Timitos wrote in his post on Bugzilla, I connected with Evolution, and the calendar entries looked as they should. That makes it clear that the ... | ||
| 2010-01-19 10:12 <CIA-5> http://bugs.tryton.org/roundup/issue1385 | ||
| 2010-01-19 10:13 -!- LucaSub(n=LucaSub@host138-180-dynamic.5-87-r.retail.telecomitalia.it) has left #tryton | ||
| 2010-01-19 11:12 <CIA-5> matb roundup * #1386/Module company doesn't install together with party_bank_de: | ||
| 2010-01-19 11:12 <CIA-5> [new] Following two modules install without problems on 1.4 | ||
| 2010-01-19 11:12 <CIA-5> http://mercurial.intuxication.org/hg/party_bank | ||
| 2010-01-19 11:12 <CIA-5> http://mercurial.intuxication.org/hg ... | ||
| 2010-01-19 11:12 <CIA-5> http://bugs.tryton.org/roundup/issue1386 | ||
| 2010-01-19 11:23 -!- paepke_(n=paepke@p4FEB0908.dip0.t-ipconnect.de) has joined #tryton | ||
| 2010-01-19 12:11 -!- essich_(n=essich@p4FCF94A2.dip0.t-ipconnect.de) has joined #tryton | ||
| 2010-01-19 12:29 <CIA-5> C?dric Krier <ced@b2ck.com> default * 1490:9f1d4df6387c tryton/tryton/gui/window/view_form/model/field.py: | ||
| 2010-01-19 12:29 <CIA-5> Send signal record-changed to the wrong parent | ||
| 2010-01-19 12:29 <CIA-5> It must be send to the ModelRecord of the field and not to the | ||
| 2010-01-19 12:29 <CIA-5> ModelRecordGroup | ||
| 2010-01-19 12:29 <CIA-5> http://hg.tryton.org/tryton/rev/9f1d4df6387c | ||
| 2010-01-19 12:40 <CIA-5> Bertrand Chenal <bch@b2ck.com> default * 2143:0334cc7b7b46 trytond/trytond/ (ir/sequence.py res/ir.py res/ir.xml): Add rules on sequences | ||
| 2010-01-19 12:40 <CIA-5> http://hg.tryton.org/trytond/rev/0334cc7b7b46 | ||
| 2010-01-19 12:40 <CIA-5> Bertrand Chenal <bch@b2ck.com> default * 2144:8b79ad8ee8a3 trytond/trytond/model/modelstorage.py: Merge | ||
| 2010-01-19 12:40 <CIA-5> http://hg.tryton.org/trytond/rev/8b79ad8ee8a3 | ||
| 2010-01-19 12:40 <CIA-5> Bertrand Chenal <bch@b2ck.com> default * 307:ec7bf061fcc0 party/party.xml: Add groups on sequence type | ||
| 2010-01-19 12:40 <CIA-5> http://hg.tryton.org/modules/party/rev/ec7bf061fcc0 | ||
| 2010-01-19 12:41 <CIA-5> Bertrand Chenal <bch@b2ck.com> default * 548:2b285d38426d account/ (journal.xml move.xml): Add groups on sequence types | ||
| 2010-01-19 12:41 <CIA-5> http://hg.tryton.org/modules/account/rev/2b285d38426d | ||
| 2010-01-19 12:41 <CIA-5> Bertrand Chenal <bch@b2ck.com> default * 400:c40e7ed7a84b account_invoice/invoice.xml: Add groups on sequence type | ||
| 2010-01-19 12:41 <CIA-5> http://hg.tryton.org/modules/account_invoice/rev/c40e7ed7a84b | ||
| 2010-01-19 12:41 <CIA-5> Bertrand Chenal <bch@b2ck.com> default * 229:92d1ebcc025a sale/sale.xml: Add Sale Administrator group. Add groups on sequence type | ||
| 2010-01-19 12:41 <CIA-5> http://hg.tryton.org/modules/sale/rev/92d1ebcc025a | ||
| 2010-01-19 12:41 <CIA-5> Bertrand Chenal <bch@b2ck.com> default * 358:1590eba6f1a1 purchase/purchase.xml: Add groups on sequence type | ||
| 2010-01-19 12:41 <CIA-5> http://hg.tryton.org/modules/purchase/rev/1590eba6f1a1 | ||
| 2010-01-19 12:42 <CIA-5> Bertrand Chenal <bch@b2ck.com> default * 632:9912452bb68e stock/shipment.xml: Add groups on sequence types | ||
| 2010-01-19 12:42 <CIA-5> http://hg.tryton.org/modules/stock/rev/9912452bb68e | ||
| 2010-01-19 12:42 <CIA-5> Bertrand Chenal <bch@b2ck.com> default * 633:d4786180e68d stock/shipment.py: Merge | ||
| 2010-01-19 12:42 <CIA-5> http://hg.tryton.org/modules/stock/rev/d4786180e68d | ||
| 2010-01-19 12:48 <CIA-5> htgoebel roundup * #1387/"Create Database" does not reconnect to server: | ||
| 2010-01-19 12:48 <CIA-5> [new] * Start Server | ||
| 2010-01-19 12:48 <CIA-5> * Start Client, but do _not_ connect to a database | ||
| 2010-01-19 12:48 <CIA-5> * stop server | ||
| 2010-01-19 12:48 <CIA-5> * In the Client: | ||
| 2010-01-19 12:48 <CIA-5> - select "Create Database" | ||
| 2010-01-19 12:48 <CIA-5> - can n ... | ||
| 2010-01-19 12:48 <CIA-5> http://bugs.tryton.org/roundup/issue1387 | ||
| 2010-01-19 12:53 <CIA-5> Bertrand Chenal <bch@b2ck.com> default * 2145:be03fd18a19b trytond/CHANGELOG: Updated CHANGELOG | ||
| 2010-01-19 12:53 <CIA-5> http://hg.tryton.org/trytond/rev/be03fd18a19b | ||
| 2010-01-19 12:57 -!- ikks(n=ikks@190.158.102.38) has joined #tryton | ||
| 2010-01-19 13:23 <CIA-5> htgoebel roundup * #1357/ERROR:dispatcher:Exception calling method execute on wizard ir.module.module.config: [invalid] can't reproduce it :-( | ||
| 2010-01-19 13:23 <CIA-5> http://bugs.tryton.org/roundup/issue1357 | ||
| 2010-01-19 13:23 -!- yangoon(n=mathiasb@p549F71F1.dip.t-dialin.net) has joined #tryton | ||
| 2010-01-19 13:40 -!- woakas(n=woakas@190.144.69.234) has joined #tryton | ||
| 2010-01-19 14:13 <CIA-5> C?dric Krier <ced@b2ck.com> default * 2146:f7d3fd252538 trytond/trytond/model/modelstorage.py: | ||
| 2010-01-19 14:13 <CIA-5> Fix test on _search_domain_active for XML-RPC for issue1380 | ||
| 2010-01-19 14:13 <CIA-5> Tuple in XML-RPC are list | ||
| 2010-01-19 14:13 <CIA-5> http://hg.tryton.org/trytond/rev/f7d3fd252538 | ||
| 2010-01-19 14:13 <CIA-5> C?dric Krier <ced@b2ck.com> default * 2147:e68d1e920d6d trytond/CHANGELOG: merge | ||
| 2010-01-19 14:13 <CIA-5> http://hg.tryton.org/trytond/rev/e68d1e920d6d | ||
| 2010-01-19 14:17 <CIA-5> C?dric Krier <ced@b2ck.com> default * 634:552208701792 stock/move.py: | ||
| 2010-01-19 14:17 <CIA-5> Fix test on to_location_warehouse for XML-RPC for issue1380 | ||
| 2010-01-19 14:17 <CIA-5> Tuple in XML-RPC are list | ||
| 2010-01-19 14:17 <CIA-5> http://hg.tryton.org/modules/stock/rev/552208701792 | ||
| 2010-01-19 14:19 <CIA-5> ced roundup * #1380/xml-rpc search_read anormal behavior with [('active', '=', False)]: [resolved] Fix with changeset f7d3fd252538 and changeset 552208701792 | ||
| 2010-01-19 14:19 <CIA-5> http://bugs.tryton.org/roundup/issue1380 | ||
| 2010-01-19 15:25 -!- juanfer(n=juanfer@190.144.69.234) has joined #tryton | ||
| 2010-01-19 15:34 -!- sharoon(n=sharoont@78.150.22.90) has joined #tryton | ||
| 2010-01-19 16:28 -!- rednul_(n=rednul@host-98-127-54-98.bln-mt.client.bresnan.net) has joined #tryton | ||
| 2010-01-19 16:41 <CIA-5> ced roundup * #1386/Module company doesn't install together with party_bank_de: | ||
| 2010-01-19 16:41 <CIA-5> [invalid] It works here. | ||
| 2010-01-19 16:41 <CIA-5> It seems that some custom module (even if not installed) add a domain on a field in the wrong way. | ||
| 2010-01-19 16:41 <CIA-5> http://bugs.tryton.org/roundup/issue1386 | ||
| 2010-01-19 17:37 <CIA-5> C?dric Krier <ced@b2ck.com> default * 1491:565b82b46c33 tryton/tryton/gui/window/view_form/screen/screen.py: | ||
| 2010-01-19 17:37 <CIA-5> Test range before use it | ||
| 2010-01-19 17:37 <CIA-5> get_visible_range can resturn None | ||
| 2010-01-19 17:37 <CIA-5> http://hg.tryton.org/tryton/rev/565b82b46c33 | ||
| 2010-01-19 17:41 -!- fil_(n=phil@blue.hands.com) has joined #tryton | ||
| 2010-01-19 17:42 -!- johbo_(n=joh@85.16.72.173) has joined #tryton | ||
| 2010-01-19 17:55 <CIA-5> C?dric Krier <ced@b2ck.com> default * 1492:927d887a8ba4 tryton/tryton/gui/window/view_form/view/list.py: Scroll before setting the cursor | ||
| 2010-01-19 17:55 <CIA-5> http://hg.tryton.org/tryton/rev/927d887a8ba4 | ||
| 2010-01-19 17:55 <CIA-5> C?dric Krier <ced@b2ck.com> default * 1493:9ea37f6d2ddf tryton/tryton/gui/window/form.py: Set the cursor after reload | ||
| 2010-01-19 17:55 <CIA-5> http://hg.tryton.org/tryton/rev/9ea37f6d2ddf | ||
| 2010-01-19 18:23 <cedk> essich_: will you work on 181154? | ||
| 2010-01-19 18:25 <cedk> essich_: or do you let me integret the patch? | ||
| 2010-01-19 19:27 -!- pepeu(n=manuel@201.155.193.192) has joined #tryton | ||
| 2010-01-19 19:32 -!- vengfulsquirrel(n=ian@c-69-181-194-95.hsd1.ca.comcast.net) has joined #tryton | ||
| 2010-01-19 19:36 <essich_> cedk: did we agree on the coding still yet? | ||
| 2010-01-19 19:38 <essich_> cedk: esp. usage of __all__? | ||
| 2010-01-19 19:46 <cedk> essich_: no | ||
| 2010-01-19 19:48 <essich_> cedk: so my work was for nothing. going to delete the reivew-set. | ||
| 2010-01-19 19:55 <essich_> cedk: if you change your mind, let me know | ||
| 2010-01-19 19:56 <panthera> as of know, we have all of tryton1.4 in squeeze, yay :) | ||
| 2010-01-19 19:56 <panthera> s/know/now/ | ||
| 2010-01-19 19:59 <cedk> essich_: it is sad that you don't want to contribute because we avoid the usage of "import *" | ||
| 2010-01-19 19:59 <cedk> essich_: and we are not the only one to do so: http://google-styleguide.googlecode.com/svn/trunk/pyguide.html | ||
| 2010-01-19 19:59 <cedk> panthera: great | ||
| 2010-01-19 20:00 <cedk> panthera: do you want to make a news on the website? | ||
| 2010-01-19 20:10 <essich_> cedk: Tis styleguid is not relevant for Tryton, see http://code.google.com/p/tryton/wiki/CodingGuidelines | ||
| 2010-01-19 20:11 <essich_> cedk: you must not set up new rule every few weeks just because you had a new idea | ||
| 2010-01-19 20:12 <cedk> essich_: I don't say that ! | ||
| 2010-01-19 20:13 <essich_> cedk: if "from M import *" is the only matter, we can agree. | ||
| 2010-01-19 20:14 <essich_> cedk: but you alos dislike __all__ | ||
| 2010-01-19 20:14 <essich_> s/alos/also/ | ||
| 2010-01-19 20:15 <cedk> essich_: it is unuseful if you don't import * | ||
| 2010-01-19 20:15 -!- LucaSub1(n=LucaSub@host138-180-dynamic.5-87-r.retail.telecomitalia.it) has joined #tryton | ||
| 2010-01-19 20:15 -!- LucaSub1(n=LucaSub@host138-180-dynamic.5-87-r.retail.telecomitalia.it) has left #tryton | ||
| 2010-01-19 20:15 <cedk> essich_: and it is revelant only for library not inside software | ||
| 2010-01-19 20:16 <essich_> cedk: __all__ documnts the interface of the module. and documentation is always good. | ||
| 2010-01-19 20:17 <essich_> cedk: __all__ is only a plus, no negative effects. | ||
| 2010-01-19 20:17 <essich_> cedk: __all__ is recomenced by PEP8 (which says the underscore prefix is the old way) | ||
| 2010-01-19 20:18 <cedk> essich_: we don't care | ||
| 2010-01-19 20:18 <essich_> cedk: *you* do not care. I do | ||
| 2010-01-19 20:18 <cedk> essich_: we don't use it in Tryton because it is not a library | ||
| 2010-01-19 20:18 <essich_> cedk: this is q question of code quality | ||
| 2010-01-19 20:18 <cedk> essich_: no | ||
| 2010-01-19 20:18 <essich_> cedk: why no? | ||
| 2010-01-19 20:18 <cedk> ther is no use | ||
| 2010-01-19 20:19 <essich_> cedk: it is documenting the interface, this is a use | ||
| 2010-01-19 20:19 <cedk> essich_: there is no interface as it is not a library | ||
| 2010-01-19 20:19 <essich_> cedk: every module is an interface, per definitionem | ||
| 2010-01-19 20:20 <essich_> s/is/has/ | ||
| 2010-01-19 20:20 <cedk> essich_: no | ||
| 2010-01-19 20:20 <cedk> essich_: we don't use it like that | ||
| 2010-01-19 20:21 <essich_> cedk: it does not matter how *you* are using it. it's still true | ||
| 2010-01-19 20:21 <vengfulsquirrel> Ha wow, this is an unfruitful argument. | ||
| 2010-01-19 20:22 <cedk> essich_: we will not include different codeing style in Tryton | ||
| 2010-01-19 20:22 <essich_> cedk: so I can not contribute any more? :-(( | ||
| 2010-01-19 20:22 <vengfulsquirrel> I think tryton does need more documentation but I don't think __all__ is going to help the parts that are the problem. | ||
| 2010-01-19 20:23 <cedk> __all__ doesn't give any information | ||
| 2010-01-19 20:23 <cedk> it is just to follow version change | ||
| 2010-01-19 20:23 <vengfulsquirrel> Well it gives some, but it seems like if the module/interface is really important it would be better documented with a .rst file. | ||
| 2010-01-19 20:23 <cedk> it is useful for base module but not for specific software like Tryton | ||
| 2010-01-19 20:24 <cedk> vengfulsquirrel: __all__ just gives what will be imported by import *, but as we don't use it, it gives any information | ||
| 2010-01-19 20:25 <essich_> cedk: I'm not going to wast my time discussion teh usefuillness of __all__ | ||
| 2010-01-19 20:26 <essich_> cedk: it seams as if you are still thinking of a small, irrelevante framework. | ||
| 2010-01-19 20:26 <cedk> essich_: absolutly not | ||
| 2010-01-19 20:26 <essich_> cedk: but if tryton gains momentum, there will be a lot of developers which need the information | ||
| 2010-01-19 20:27 <vengfulsquirrel> cedk: I just meant it COULD be used to document which parts of a module are meant for public usage but I agree/think that in tryton's case rarely is that kind of documentation needed. | ||
| 2010-01-19 20:27 <essich_> cedk: so you arguments about "not an interface" are irrelevant | ||
| 2010-01-19 20:27 <cedk> essich_: we use the "old way" | ||
| 2010-01-19 20:27 <vengfulsquirrel> Tryton needs overview documentation and usage documentation, there are not that many interfaces that need to be documented for the developer. | ||
| 2010-01-19 20:27 <cedk> vengfulsquirrel: +1 | ||
| 2010-01-19 20:28 <essich_> cedk: you are using the "old way" but I should chance to "while true"? You are so labile | ||
| 2010-01-19 20:28 <cedk> and once again __all__ gives just a list, it doesn't document more | ||
| 2010-01-19 20:29 <essich_> cedk: cedk: __all__ documents the public interface | ||
| 2010-01-19 20:29 <vengfulsquirrel> PEP8 does not say you should use __all__ it just says IF you use import * then you should use __all__ over the old way of prefixing. Where "old way" means the old way of supporting import *. I don't think tryton uses import * right? | ||
| 2010-01-19 20:29 <cedk> essich_: without _ is public with _ is not | ||
| 2010-01-19 20:30 <cedk> vengfulsquirrel: right | ||
| 2010-01-19 20:30 <vengfulsquirrel> ha we could have documented something already | ||
| 2010-01-19 20:30 <essich_> cedk: this is old style, PEP 8 recomends __all__ | ||
| 2010-01-19 20:30 <bechamel> 2all: the python doc says : "Package authors may also decide not to support it, if they don’t see a use for importing * from their package", and everywhere we do import like "from tryton.model import modelsql" so for me __all __ is not needed | ||
| 2010-01-19 20:30 <vengfulsquirrel> essich_: it only recomends __all__ IF you use import * | ||
| 2010-01-19 20:30 <cedk> essich_: why is it old way ? I don't follow rule without explaination | ||
| 2010-01-19 20:30 <essich_> vengfulsquirrel: this is the result of cedk's nit-picking :-( | ||
| 2010-01-19 20:31 <cedk> essich_: import * was always forbiden | ||
| 2010-01-19 20:31 <essich_> cedk: you forbate in on January, 13th. only 6 days ago. | ||
| 2010-01-19 20:32 <cedk> essich_: It was missing, | ||
| 2010-01-19 20:33 <essich_> bechamel: if we are not using "forom M import *", there is no need for prefix underscores, too. But tryton uses pefix underscore widespread. | ||
| 2010-01-19 20:33 <cedk> essich_: look at the code there was no import * | ||
| 2010-01-19 20:33 <cedk> essich_: it is useful even without import * | ||
| 2010-01-19 20:33 <cedk> essich_: it is the "documentation" | ||
| 2010-01-19 20:34 <bechamel> essich_: it just help people to know what is important and what is not | ||
| 2010-01-19 20:34 <essich_> trytond/trytond/backend/sqlite/__init__.py | ||
| 2010-01-19 20:34 <essich_> trytond/trytond/ir/__init__.py | ||
| 2010-01-19 20:34 <essich_> trytond/trytond/ir/module/__init__.py | ||
| 2010-01-19 20:34 <essich_> trytond/trytond/ir/ui/__init__.py | ||
| 2010-01-19 20:34 <essich_> trytond/trytond/model/__init__.py | ||
| 2010-01-19 20:34 <essich_> trytond/trytond/model/fields/__init__.py | ||
| 2010-01-19 20:34 <essich_> trytond/trytond/report/__init__.py | ||
| 2010-01-19 20:34 <essich_> trytond/trytond/res/__init__.py | ||
| 2010-01-19 20:34 <essich_> trytond/trytond/tests/__init__.py | ||
| 2010-01-19 20:34 <cedk> essich_: STOP | ||
| 2010-01-19 20:34 <essich_> trytond/trytond/tools/__init__.py | ||
| 2010-01-19 20:34 <essich_> trytond/trytond/webdav/__init__.py | ||
| 2010-01-19 20:34 <essich_> trytond/trytond/wizard/__init__.py | ||
| 2010-01-19 20:34 <essich_> trytond/trytond/workflow/__init__.py | ||
| 2010-01-19 20:35 <essich_> cedk: as you said: there is no from M import * | ||
| 2010-01-19 20:35 <bechamel> actually I never saw myself using import * | ||
| 2010-01-19 20:35 <cedk> essich_: except in __init__.py | ||
| 2010-01-19 20:35 <essich_> cedk: don't be afraid, there are all ;-) | ||
| 2010-01-19 20:36 <essich_> bechamel: shouldn't we help developers? or should we make their live harder as it is necessary? | ||
| 2010-01-19 20:37 <essich_> cedk: report/report.py is designed for "from M import *" | ||
| 2010-01-19 20:38 <cedk> essich_: why do you say that? | ||
| 2010-01-19 20:39 <essich_> cedk: report/__init__.py has "from report import *", thus report/report.py has to be designe to be used this way. | ||
| 2010-01-19 20:40 <cedk> essich_: it is design to be use like this: from trytond.report import Report | ||
| 2010-01-19 20:40 <essich_> cedk: you do not use it this way. See report/__init__.py | ||
| 2010-01-19 20:41 <essich_> cedk: *you* are using "from M import *", but I must not design a moduel to be used liek this? | ||
| 2010-01-19 20:41 <essich_> cedk: you are contraticting yoursefl | ||
| 2010-01-19 20:41 <cedk> essich_: this is the last time I said: "Not for __init__.py" | ||
| 2010-01-19 20:42 <cedk> I stop there, I don'T want to spend more time on this | ||
| 2010-01-19 20:42 <essich_> cedk: we are not taking about __init__.py. We are talking abour report.py | ||
| 2010-01-19 20:42 <essich_> cedk: report.py is the one which gets imported be 'from report import *' | ||
| 2010-01-19 20:42 <essich_> cedk: report.py is the one thar should define __all__ | ||
| 2010-01-19 20:44 <vengfulsquirrel> essich_: I see what you mean but its still not worth changing that since developers are not ever going to or be expected to use import *. | ||
| 2010-01-19 20:44 <vengfulsquirrel> Let's talk about how to do .search() on a many2many which I think is more productive and I still have not resolved. | ||
| 2010-01-19 20:45 <essich_> vengfulsquirrel: depend on where developers are coming from | ||
| 2010-01-19 20:45 -!- essich_(n=essich@p4FCF94A2.dip0.t-ipconnect.de) has left #tryton | ||
| 2010-01-19 20:45 -!- essich_(n=essich@p4FCF94A2.dip0.t-ipconnect.de) has joined #tryton | ||
| 2010-01-19 20:46 <bechamel> for everybody information: essich removed all his issues on codereview, so he only succeed to waist our (and his) time :( | ||
| 2010-01-19 20:46 <vengfulsquirrel> essich_: The only case where import * would be used would be in __init__.py files. Any other case should use absolute imports. | ||
| 2010-01-19 20:46 <essich_> vengfulsquirrel: experienced python programmers are quite used to this | ||
| 2010-01-19 20:47 <essich_> bechamel: Thei are only "private" | ||
| 2010-01-19 20:47 <vengfulsquirrel> essich_: Quite used to what? | ||
| 2010-01-19 20:47 <essich_> vengfulsquirrel: maybe. But this is no argument not *not* using __all__ | ||
| 2010-01-19 20:48 <essich_> vengfulsquirrel: maybe. quite used to look at the interface (as defined in __all__) or to use "from M import *" | ||
| 2010-01-19 20:49 <essich_> vengfulsquirrel: why sould * be allowed for absolut imports? | ||
| 2010-01-19 20:49 <vengfulsquirrel> essich_: It just isn't worth the effort to change it, I tried using __all__ in the past and I found it hard to maintain. Especially since I never use import *. Experienced python developers do not use import *, as far as I know it is always frowned upon. | ||
| 2010-01-19 20:50 <essich_> bechamel: this fruitless discussion about nit-picking issues and removing stuff which helps is wasting my time :-( | ||
| 2010-01-19 20:51 <essich_> vengfulsquirrel: I use it in all my modules (except for quick'n'dirty) | ||
| 2010-01-19 20:52 <essich_> vengfulsquirrel: And I seldom use "from M import *", but I design for others to use it that way if they like. | ||
| 2010-01-19 20:53 <vengfulsquirrel> essich_: Real documentation is better and more easily maintained than __all__, I guess that is the point I tried to make earlier. The maintenance required to keep __all__ relevant makes it not worth it. | ||
| 2010-01-19 20:54 <vengfulsquirrel> essich_: Experienced python programmers would expect free text documentation that explains how to use the interface and the important components(that would probably span many python modules). They would never open the file and check whats in __all__. | ||
| 2010-01-19 20:55 <essich_> vengfulsquirrel: Documentation on a seperate place will not stay current, even in the short run. This is why Python has doc-strings | ||
| 2010-01-19 20:55 -!- essich_(n=essich@p4FCF94A2.dip0.t-ipconnect.de) has left #tryton | ||
| 2010-01-19 20:55 -!- essich_(n=essich@p4FCF94A2.dip0.t-ipconnect.de) has joined #tryton | ||
| 2010-01-19 20:56 <essich_> vengfulsquirrell: I expcet usinf pydoc <module> | ||
| 2010-01-19 20:57 <vengfulsquirrel> essich_: __all__ is just hard to maintain because no one uses it, and you change a function/class way in a file and forgot to update __all__ , documentation in a separate place will at least be seen by more people that can tell that it has become outdated and they will try to update it | ||
| 2010-01-19 20:58 <vengfulsquirrel> essich_: And tryton's real documentation problem now is that more usage documentation is needed that __all__ would not help with. | ||
| 2010-01-19 20:58 <vengfulsquirrel> I almost always know what function/field/model/class I need I just don't know how I should use it to get what I want done. | ||
| 2010-01-19 20:59 <vengfulsquirrel> essich_: I think instead of spending time running around change _function to function and putting function in __all__ we should focus that energy on more usage documentation because that is what developers will expect to find. | ||
| 2010-01-19 21:00 <essich_> vengfulsquirrell: I'm not changing module, just to all __all__. I've written now modules whre I've put in _all__ | ||
| 2010-01-19 21:00 <essich_> vengfulsquirrell: I'm not changing modules, just to add __all__ everywhere. I've written now modules whre I've put in _all__ | ||
| 2010-01-19 21:01 <essich_> vengfulsquirrell: but cedk does not like it. I should remove it. | ||
| 2010-01-19 21:01 <vengfulsquirrel> Well to maintain consistency with the rest of the project __all__ must be removed from your new modules or added to all the tryton modules. | ||
| 2010-01-19 21:01 <vengfulsquirrel> So I don't think its worth it to add __all__ to the entire project, when it doesn't solve the/a problem that tryton is having it only adds more maintenance. | ||
| 2010-01-19 21:01 <essich_> vengfulsquirrell: not necessary. The rule could be to add it whenever a module interfac changes. | ||
| 2010-01-19 21:02 <vengfulsquirrel> No I think it must be all or nothing in order to maintain consistency. | ||
| 2010-01-19 21:02 <essich_> vengfulsquirrell: leaving it away does not help, too. And discussion it only wastes time | ||
| 2010-01-19 21:06 <essich_> vengfulsquirrell: regarding documentation: a basic problem is that cedk is not even adding docstrings to new code | ||
| 2010-01-19 21:08 <vengfulsquirrel> We only need docstrings in certain places, I think the bigger problem is usage documentation. | ||
| 2010-01-19 21:14 <CIA-5> htgoebel roundup * #1388/search_rec_name fails for ir.lang: | ||
| 2010-01-19 21:14 <CIA-5> [new] Cédric Krier schrieb: | ||
| 2010-01-19 21:14 <CIA-5> > > This will work as search_rec_name search first for the code: | ||
| 2010-01-19 21:14 <CIA-5> > > | ||
| 2010-01-19 21:14 <CIA-5> > > "Name","lang" | ||
| 2010-01-19 21:14 <CIA-5> > > "Gustav Gast","en_U ... | ||
| 2010-01-19 21:14 <CIA-5> http://bugs.tryton.org/roundup/issue1388 | ||
| 2010-01-19 21:15 <essich_> vengfulsquirrell: writing a complete documentation is much, much work. Docstrings are much easier to write and halp other to understand the code. | ||
| 2010-01-19 21:15 <essich_> vengfulsquirrell: Nevertheless, if you are taking about documentation of concepts, I fully agree :-) | ||
| 2010-01-19 21:17 <bechamel> essich_: to be clear: cedk is nit-picking since day one, and yes it's painful, and I'm/was the first to suffer from it, but at the end the codebase is really homogeneous and it's a big plus IMO | ||
| 2010-01-19 21:19 <bechamel> essich_: and for me it's such an advantage that I'm ready to put my ego in my pocket. I know from experience that if I was right and cedk didn't agree, it's a matter of time for the issue to get back in our discussion and get changed my way. I only need to be patient | ||
| 2010-01-19 21:34 <essich_> bechamel: a homogeneous code and it's a big plus. shure. | ||
| 2010-01-19 21:35 <essich_> bechamel: but cedk's nit-picking is *de-motivating*, because he is not honoring what ha be done with not even a word. HGe is just nit-picking on unimportand stuff. | ||
| 2010-01-19 21:36 <essich_> bechamel: and his coding style is odd in some places (like the header comments in each file) | ||
| 2010-01-19 21:36 <bechamel> essich_: it works both ways: if it's unimportant, why you wanna fight so hard for it ? | ||
| 2010-01-19 21:39 <essich_> bechamel: because it's unimportant :-) Why is cedk riding about it instead of just taking it? | ||
| 2010-01-19 21:39 <essich_> bechamel: even writing a comment "remove ..." in rietveld is a wast of time | ||
| 2010-01-19 21:40 <essich_> bechamel: (I know I'm iding on it hre, too) | ||
| 2010-01-19 21:40 <essich_> s/hre/here/ | ||
| 2010-01-19 21:40 <vengfulsquirrel> Well probably because of the inconsistent code base. | ||
| 2010-01-19 21:41 <bechamel> essich_: I agree that a "nice work" or something like that on reitveld comment would be better, but now you know it, comments (even negatives) means that people are interested by your work | ||
| 2010-01-19 21:49 <vengfulsquirrel> Can someone explain to me how to search a many2many like this: http://paste.pocoo.org/show/167263/ | ||
| 2010-01-19 21:50 <cedk> vengfulsquirrel: the best is to search on res.user-res.group | ||
| 2010-01-19 21:51 <cedk> vengfulsquirrel: but ('groups', =, group_id) should work | ||
| 2010-01-19 21:52 <cedk> vengfulsquirrel: or ('groups', 'in', [group_id]) | ||
| 2010-01-19 21:53 <CIA-5> htgoebel roundup * #1389/wrong message if importing many2many fails: [new] When importing a many2many field, the filed get split at the comma. If the lookup fails, the error/warning message shows the cell content, n ... | ||
| 2010-01-19 21:53 <CIA-5> http://bugs.tryton.org/roundup/issue1389 | ||
| 2010-01-19 22:03 <essich_> bechamel: in your team you ar obviously the one responsibe for social competence ;-) | ||
| 2010-01-19 22:21 -!- FWiesing(n=FWiesing@194-208-185-012.tele.net) has joined #tryton | ||
| 2010-01-19 22:30 <CIA-5> vengfulsquirrel roundup * #1390/Cannot receive customer return shipment.: [new] This traceback occurs when I am creating a manual return shipment with one entry and have clicked the "Received" button. Should return ship ... | ||
| 2010-01-19 22:30 <CIA-5> http://bugs.tryton.org/roundup/issue1390 | ||
| 2010-01-19 22:30 <CIA-5> vengfulsquirrel roundup * #1390/Cannot receive customer return shipment.: [chatting] Oops, forgot to add people other than myself. | ||
| 2010-01-19 22:30 <CIA-5> http://bugs.tryton.org/roundup/issue1390 | ||
| 2010-01-19 23:02 -!- paepke(n=paepke@p5B32F083.dip.t-dialin.net) has joined #tryton | ||
| 2010-01-19 23:50 <CIA-5> ced roundup * #1388/search_rec_name fails for ir.lang: [invalid] There is no search_rec_name on ir.lang | ||
| 2010-01-19 23:50 <CIA-5> http://bugs.tryton.org/roundup/issue1388 | ||
| 2010-01-19 23:53 <CIA-5> ced roundup * #1390/Cannot receive customer return shipment.: [closed] Duplicate issue1379 | ||
| 2010-01-19 23:53 <CIA-5> http://bugs.tryton.org/roundup/issue1390 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!