Changeset 84

Timestamp:

03/07/07 12:19:47 (6 years ago)

Author:

ged

Message:

- Fixed documentation for belongs_to relationships (the arguemnt is "required",

not "nullable").

- Fixed typo which broke the use_alter argument on belongs_to relationships.
- reordered CHANGES file: feature changes/addition then bugfixes then cosmectic/cleanup

Location:

elixir/trunk

Files:

• CHANGES (modified) (2 diffs)
• elixir/relationships.py (modified) (2 diffs)

elixir/trunk/CHANGES

@@ -10 +10 @@
   autoloaded entities. It is only required to specify it if you have several
   belongs_to relationships between two entities/tables.
-- Fixed bug preventing having entities without any statement.
 - Applied patch from "Wavy" so that columns of a table are in the same order
   as they were declared.
-- Made some PEP8 tweaks in many places. Used the pep8 script provided with 
-  Cheesecake.
 - Foreign key names generated by belongs_to relationships use column names 
   instead of relation names in case we have a relation with the same name
@@ -20 +17 @@
   table inheritance (and we set a custom column name in one of them to avoid
   a column-name conflict).
+- Using invalid options on entities will now raise an exception
+- Fixed bug preventing having entities without any statement.
+- Fixed documentation for belongs_to relationships (the arguemnt is "required",
+  not "nullable").
+- Fixed typo which broke the use_alter argument on belongs_to relationships.
 - Actually make the code python 2.3 compatible (Robin's patch was based on
   0.1.0 while I had introduced more decorators in the trunk in the mean time).
-- Using invalid options on entities will now raise an exception
+- Made some PEP8 tweaks in many places. Used the pep8 script provided with 
+  Cheesecake.
 - Some cleanup/useless code removal
 

elixir/trunk/elixir/relationships.py

@@ -70 +70 @@
 | ``colname``          | Specify a custom column name.                        |
 +----------------------+------------------------------------------------------+
-| ``nullable``         | True if this column should allow nulls. Defaults to  |
-|                      | True unless this column is a primary key column.     |
+| ``required``         | Specify whether or not this field can be set to None |
+|                      | (left without a value). Defaults to ``False``,       |
+|                      | unless the field is a primary key.                   |
 +----------------------+------------------------------------------------------+
 | ``column_kwargs``    | A dictionary holding any other keyword argument you  |
@@ -321 +322 @@
         self.constraint_kwargs = kwargs.pop('constraint_kwargs', {})
         if 'use_alter' in kwargs:
-            self.contraint_kwargs['use_alter'] = kwargs.pop('use_alter')
+            self.constraint_kwargs['use_alter'] = kwargs.pop('use_alter')
 
         self.foreign_key = list()