Login | Register
My pages Projects Community openCollabNet

propel
Reply to message

* = Required fields
* Subject
* Body
Attachments
Send reply to
Topic
Author (directly in email)
Please type the letters in the image above.

Original message

Author hlellelid
Full name Hans Lellelid
Date 2006-07-24 06:20:06 PDT
Message Ok, a patch would be great :)

Thanks -
Hans

Roel Vanhout wrote:
> I'll look into making a patch, but I'd rather not commit it myself - I
> got yelled at once in another project a couple of years ago, now I'm a
> bit hesitant to commit to codebases I'm not very familiar with :) I'll
> let the list know. Thanks.
>
> cheers,
>
> roel
>
>
> On 7/24/06, Hans Lellelid <hans at velum dot net> wrote:
>> Hi Roel,
>>
>> I think we just need to change that behavior. Perhaps we should make a
>> PHPOMBuilder subclass that holds any PHP specific stuff. I'll see if I
>> can find a few minutes to look at that today. (Or, if you'd like to
>> refactor that class, you can send me an email w/ username request for
>> SVN and I'll make you an account.)
>>
>> Cheers,
>> Hans
>>
>> Roel Vanhout wrote:
>> > Hello Soenke,
>> >
>> > Thanks for your answer, this works for all classes that are in the
>> > propel\generator\c​lasses\propel\engi​ne\builder\om\php​5 directory but
>> > there is one file (OMBuilder.php) that is one directory higher. I
>> > don't see it mentioned in the default.properties list of builders. In
>> > this file, which defines the OMBuilder class, the <?php is added to
>> > every file that is written out and it sets .cpp as extention for files
>> > that are written, both of which I'd like to change. I don't see
>> > however where I can indicate that my custom class should be used.
>> >
>> > cheers,
>> >
>> > roel
>> >
>> >
>> > On 7/24/06, Soenke Ruempler <ruempler@topconc​epts.com> wrote:
>> >> Roel Vanhout <mailto:roel.vanh​out at gmail dot com> wrote on Monday, July
>> 24,
>> >> 2006 12:50 PM:
>> >>
>> >> > I'm not sure if that was directed at the PECL thing or the pure C++
>> >> > that I'm trying to accomplish :) , but either way: is there
>> anyone who
>> >> > knows the answer to my original question whether it is possible
>> to use
>> >> > another OMBuilder class? Thanks.
>> >>
>> >> Of course, in build.properties we have it like this:
>> >>
>> >> propel.builder.mapbuilder.class = tcc.Propel.Engine.Builder.Map
>> >> propel.builder.object.class =tcc.Propel.Engine.B​uilder.ComplexObject​
>> >> propel.builder.peer.class =tcs.Propel.Engine.B​uilder.ComplexPeer
>> >>
>> >> -soenke
>> >>
>> >> --------------------​--------------------​--------------------​---------
>> >> To unsubscribe, e-mail: dev-unsubscribe@prop​el.tigris.org
>> >> For additional commands, e-mail: dev-help at propel dot tigris dot org
>> >>
>> >>
>> >
>> > --------------------​--------------------​--------------------​---------
>> > To unsubscribe, e-mail: dev-unsubscribe@prop​el.tigris.org
>> > For additional commands, e-mail: dev-help at propel dot tigris dot org
>> >
>>
>> --------------------​--------------------​--------------------​---------
>> To unsubscribe, e-mail: dev-unsubscribe@prop​el.tigris.org
>> For additional commands, e-mail: dev-help at propel dot tigris dot org
>>
>>
>
> --------------------​--------------------​--------------------​---------
> To unsubscribe, e-mail: dev-unsubscribe@prop​el.tigris.org
> For additional commands, e-mail: dev-help at propel dot tigris dot org
>