On 10/25/07, John Mettraux <[EMAIL PROTECTED]> wrote:
>
>
> On 10/25/07, Tomaso Tosolini <[EMAIL PROTECTED]> wrote:
> > Ok,
> > here is the archive
> >
> > I would be grateful you could let me know about any addition and/or
> > change needed
>
> Hi Tomaso,
>
> you've done lots of work, congratulations.
>
>
> Some points :
>
> - could you tell your text editor to treat tabs as 4 whitespaces ?
> Indentation is very random on my side, tabs are bad


Sorry, I didn't notice this... correcting immediately


- why naming the database table engine_db_storage, why not naming it
> "expressions" (and "errors" or "process_errors") ? It follows the
> ActiveRecord style where a table of customers is named "customers"


Feel free to choose the name you like best. I'm used to set manually the
table names in my ActiveRecord subclasses
because my databases sometimes have many tables and is not very easy for me
follow AR style, expecially when
having join tables or other things. No problem for me to use a different
name.
Now Expressions go into expressions table and errors to process_errors


- serialization to YAML is automatic, no need to call dump or load by
> yourself, in the ActiveRecord base extension, for example :
>
>     class Thing < ActiveRecord::Base
>
>         belongs_to :workitem
>         serialize :my_column
>     end
>
> http://api.rubyonrails.org/classes/ActiveRecord/Base.html#M001020


You are right, of course ... correcting...

 Hope it is better...

Bye
Tom

Wdyt ?
>
>
> Best regards, thanks,
>
> --
> John Mettraux   -///-   http://jmettraux.openwfe.org
>
> >
>

--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups 
"OpenWFEru dev" 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/openwferu-dev?hl=en
-~----------~----~----~----~------~----~------~--~---

Attachment: db_persited_engine.tar.gz
Description: GNU Zip compressed data

Reply via email to