[Nitro] Og CRUD naming of methods

Arne Brasseur arne at arnebrasseur.net
Sat Sep 29 09:44:49 EDT 2007


Arne Brasseur schreef:
> Being all in favor of consistent naming could I still say : be careful 
> with this? This is no breaking news but it's surprising how easy one 
> overlooks something when renaming a method. And since the test suite 
> is all but complete...
I mean : far from complete...
>
> Ok, please carry on.
>
> (ab)
>
> George Moschovitis schreef:
>> On second thoughts, If you have the time to go over the source code
>> and apply the naming conventions (while making sure that it doesnt
>> break everything) I could apply the patch in the current version.
>>
>> -g.
>>
>>
>>
>> On 9/29/07, George Moschovitis <george.moschovitis at gmail.com> wrote:
>>   
>>> I more or less agree with consistent naming. But, I am afraid this
>>> will have to wait for a later version.
>>>
>>> thanks for this *useful* post,
>>> -g.
>>>
>>> On 9/29/07, Mark Van De Vyver <mvyver at gmail.com> wrote:
>>>     
>>>> Hi Devs
>>>>
>>>> In [./manager.rb:initialize]
>>>> @store_class.allocate.destroy_db(@options) if
>>>> Og.destroy_schema||@options[:destroy]
>>>>
>>>> and elsewhere there is this:
>>>>
>>>>   alias_method :destroy, :drop_table
>>>>   alias_method :drop_schema, :drop_schema
>>>>
>>>> Despite the typo (in the last alias) it seems there is confusion in
>>>> the code about whether a schema is a db or table...
>>>>
>>>> Is it worth articulating following goal for Og to progress to?
>>>>
>>>> To use C.R.U.D stubs to method names. so prepend:
>>>> create, read update, delete
>>>> to the following
>>>> *_db
>>>> *_table
>>>> *_row
>>>> *_field
>>>>
>>>> I also think the abundance of aliases might hurt in the longer term -
>>>> it's driving me nuts.  Accepting there may be some pain to adopt these
>>>> conventions, but won't it help to have a consistent naming convention
>>>> that you don't have to scramble to the docs to check: 'Is it destroy,
>>>> drop, delete, table, tbl, schema, etc?"
>>>>
>>>> my 2c after a frustrating few days :)
>>>> _______________________________________________
>>>> Nitro-general mailing list
>>>> Nitro-general at rubyforge.org
>>>> http://rubyforge.org/mailman/listinfo/nitro-general
>>>>
>>>>       
>>> --
>>> http://gmosx.me.gr
>>> http://phidz.com
>>> http://blog.gmosx.com
>>> http://cull.gr
>>> http://www.joy.gr
>>> http://nitroproject.org
>>>
>>>     
>>
>>
>>   
>
>
> -- 
> Arne Brasseur
> http://www.arnebrasseur.net
> arne at arnebrasseur.net
> ------------------------------------------------------------------------
>
> _______________________________________________
> Nitro-general mailing list
> Nitro-general at rubyforge.org
> http://rubyforge.org/mailman/listinfo/nitro-general


-- 
Arne Brasseur
http://www.arnebrasseur.net
arne at arnebrasseur.net

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://rubyforge.org/pipermail/nitro-general/attachments/20070929/1d08d192/attachment-0001.html 


More information about the Nitro-general mailing list