On 8/30/07, Adam Gomaa <[EMAIL PROTECTED]> wrote:

> Writing the patches ended up being pretty straightforward. Included are
> the following:
>
>  * A patch to elixir/entity.py (assignment_syntax.patch) that adds a
> process_attrs() method to EntityMeta, and calls it in __init__().
>
>  * A patch to docs/tutorial.rst (tutorial_assignment_syntax.patch) which
> updates the code samples to use class assignment rather than
> with_fields.
>
>  * The test_attr_syntax.py file, which is mostly a copy of
> test_movies.py, just with the class attribute syntax.
>
> All the patches are against SVN revision 193.
>
> All tests still run successfully with SQLAlchemy 0.4. I did have to
> make an unrelated fix on elixir/ext/associable.py to get it to work
> with 0.4, but that was the case before or after these patches. I don't
> have access to a machine with 0.3 right now, but I will tomorrow.

Care to explain what was that fix? (or maybe we already corrected that
in the mean time).

> Let me know how this is.

These patches look good to me. For consistency's sake, I'd appreciate
if you'd implement the same for relationships. This shouldn't be
difficult though. It'll only be a tiny bit more work than for fields,
as you'd have to make Relationships be initializable without knowing
their name (as was already the case for fields).

-- 
Gaƫtan de Menten
http://openhex.org

--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups 
"SQLElixir" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at 
http://groups.google.com/group/sqlelixir?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to