Add PySON to Tryton server

[yangoon1, 2010-01-17 19:03:17]

http://codereview.appspot.com/183067/diff/4001/2016
File trytond/model/fields/float.py (right):

http://codereview.appspot.com/183067/diff/4001/2016#newcode12
trytond/model/fields/float.py:12: 'digits must be tuple of intergers or PYSON'
typo: integers

[yangoon1, 2010-01-17 19:06:17]

Would also be nice to have updated docs.

[bch, 2010-01-18 12:58:49]

http://codereview.appspot.com/183067/diff/2058/2061
File doc/views.rst (right):

http://codereview.appspot.com/183067/diff/2058/2061#newcode381
doc/views.rst:381: * ``domain``: a PySON string that is evaluate with the object
value as
"which is evaluated with" would be better than "that is evaluate with"

http://codereview.appspot.com/183067/diff/2058/2070
File trytond/model/fields/field.py (right):

http://codereview.appspot.com/183067/diff/2058/2070#newcode24
trytond/model/fields/field.py:24: #assert test_domain(value), 'invalid domain'
Why this part is kept in comments ?

[yangoon1, 2010-01-18 14:13:39]

Marked also some wordings not related to this patch, so if you are fixing
anyway...

OTOH there are also number of wordings to correct in models.rst etc., so it
could be worth another patch.

http://codereview.appspot.com/183067/diff/4074/4077
File doc/views.rst (right):

http://codereview.appspot.com/183067/diff/4074/4077#newcode9
doc/views.rst:9: In tryton, objects can have several views, it is the action,
that opens the window, that tells which views must be used.
In Tryton objects can have several views. It is the action,  that opens the
window and that tells which view to use.

http://codereview.appspot.com/183067/diff/4074/4077#newcode288
doc/views.rst:288: * ``on_write``: The name of a function that will be called
when a record is saved.  The function must have this form ``on_write(self,
cursor, user, ids, context=None)``.
The function must have this form -> Syntax:

http://codereview.appspot.com/183067/diff/4074/4077#newcode290
doc/views.rst:290: * ``editable``: If it is set to ``top`` or ``bottom``, the
list become editable and the new record will be add on ``top`` or ``bottom`` of
the list.
If it is set to ``top`` or ``bottom``, the list becomes editable and the new
record will be added on ``top`` or ``bottom`` of the list.

http://codereview.appspot.com/183067/diff/4074/4077#newcode292
doc/views.rst:292: * ``sequence``: The name of the field that is used for
sorting.  So this field must be an interger and it will be updated to match the
new sort when the user use the ``Drag and Drop`` between rows of the list.
This field must be an integer and it will be updated to match the new sort order
when the user uses ``Drag and Drop`` on list rows.

http://codereview.appspot.com/183067/diff/4074/4077#newcode294
doc/views.rst:294: * ``colors``: A PySON string that will be evaluated for each
records and will return a string containing the name of the color.
A PySON string that will be evaluated for each record. A string containing the
name of the color will be returned.

http://codereview.appspot.com/183067/diff/4074/4077#newcode296
doc/views.rst:296: * ``fill``: A boolean to specify if the last column must fill
the remain free space in the view.
remaining

http://codereview.appspot.com/183067/diff/4074/4077#newcode298
doc/views.rst:298: * ``toolbar``: A boolean to specify on tree if there is a
toolbar on the left that take the first elements of the tree (like for the
menu).
takes

http://codereview.appspot.com/183067/diff/4074/4077#newcode298
doc/views.rst:298: * ``toolbar``: A boolean to specify on tree if there is a
toolbar on the left that take the first elements of the tree (like for the
menu).
tree view, if

http://codereview.appspot.com/183067/diff/4074/4077#newcode378
doc/views.rst:378: * ``key``: can be used to distinguish fields with the same
name but
name, but with different domains.

http://codereview.appspot.com/183067/diff/4074/4077#newcode382
doc/views.rst:382: context. If the result is true the field value is added to
the graph
graph. otherwise not -> not necessary

http://codereview.appspot.com/183067/diff/4074/4077#newcode385
doc/views.rst:385: * ``fill``: defined if the graph must be fill
shall be filled.

http://codereview.appspot.com/183067/diff/4074/4077#newcode387
doc/views.rst:387: * ``empty``: defined if the line graph must put a point for
missing
for a missing

http://codereview.appspot.com/183067/diff/4074/4077#newcode411
doc/views.rst:411: Inherited a view means that the original view will be
modified by a set of rules that are defined with XML.
Inherit a view means, that

http://codereview.appspot.com/183067/diff/4074/4077#newcode413
doc/views.rst:413: For this purpose, the inheritance engine use some xpath
expressions.
uses xpath expressions.

http://codereview.appspot.com/183067/diff/4074/4077#newcode417
doc/views.rst:417: If the field ``domain`` is not set or evaluated to True, the
inheritance will be proceed.
proceeded.

http://codereview.appspot.com/183067/diff/4074/4077#newcode426
doc/views.rst:426: Each inherit view must start with this tag.
inherited

http://codereview.appspot.com/183067/diff/4074/4077#newcode433
doc/views.rst:433: * ``position``: Define the position from the finded node, it
can be ``before``, ``after``, ``replace``, ``inside`` or ``replace_attributes``
which will change the attributes.
Define the position in relation to the node found. It can be ...

[ced, 2010-01-18 14:22:11]

I only fix typo related to PySON other must be in an other patch.

http://codereview.appspot.com/183067/diff/4074/4077
File doc/views.rst (right):

http://codereview.appspot.com/183067/diff/4074/4077#newcode382
doc/views.rst:382: context. If the result is true the field value is added to
the graph
On 2010/01/18 14:13:39, yangoon wrote:
> graph. otherwise not -> not necessary

I don't understand

[yangoon1, 2010-01-19 10:33:23]

http://codereview.appspot.com/183067/diff/4074/4077
File doc/views.rst (right):

http://codereview.appspot.com/183067/diff/4074/4077#newcode382
doc/views.rst:382: context. If the result is true the field value is added to
the graph
On 2010/01/18 14:22:12, ced wrote:
> On 2010/01/18 14:13:39, yangoon wrote:
> > graph. otherwise not -> not necessary
> 
> I don't understand

If the result is true the field value is added to the graph.

should be sufficient.