Login  Register

Re: Formatting controllers

Posted by Jacques Le Roux on Nov 20, 2009; 10:32pm
URL: http://ofbiz.116.s1.nabble.com/Formatting-controllers-tp623986p628224.html

Label Manager, yes ! I will have a look

Thanks Adrian,

Jacques

From: "Adrian Crum" <[hidden email]>

> Yes. Look at the Label Manager code - it outputs well-formed XML.
>
> Some day this will be easy to do in mini-language, but it's not quite there. You can read and write XML files, but there is no way
> to add the Apache license header in mini-language.
>
> -Adrian
>
> --- On Fri, 11/20/09, Jacques Le Roux <[hidden email]> wrote:
>
>> From: Jacques Le Roux <[hidden email]>
>> Subject: Re: Formatting controllers
>> To: [hidden email]
>> Date: Friday, November 20, 2009, 1:36 PM
>> Do you mean I could even easily
>> format with this code ?
>> Like what I did in http://svn.apache.org/viewvc?rev=882536&view=rev
>>
>> Jacques
>>
>> From: "Adrian Crum" <[hidden email]>
>> > Don't forget the framework includes code to read and
>> write XML files, and you can specify the indentation level.
>> >
>> > -Adrian
>> >
>> > Jacques Le Roux wrote:
>> >> Thanks David, Tim,
>> >>
>> >> Yes, actually I'd like to have them consistent.
>> >> Not for the sake of consistency but because it's
>> easier to read when there is a sole pattern used.
>> >> Your eyes don't have to adapt from a line to the
>> other, less eyeballs work.
>> >> And I find it easier to not have all on one line.
>> >>
>> >> This said, I was expecting to use Oxygen in
>> Eclipse to format blocks, but this seems not possible, I
>> found only Ctrl+Shft+F which formats all the file
>> >> Using regexp is then the solution but much more
>> harder and risky...
>> >>
>> >> I will see, I have some issues with Oxygen and
>> it's hard to get them fixed.
>> >> You have to make a lot of exchange and to explain
>> all things wich much much details.
>> >> To such an extent that the work is almost done...
>> >>
>> >> I'm good to find frustration :/
>> >>
>> >> Jacques
>> >>
>> >> From: "Tim Ruppert" <[hidden email]>
>> >>> "consistent enough" is tough when it slips
>> thru the crack and it's not always consistent. I
>> have to admit to liking it being explicit - but it's
>> really trivial enough to go either way. Some people
>> put the bracket on the same line - some the
>> next. You really just have to pick a guideline
>> and stick to it as far as I'm concerned.
>> >>>
>> >>> Cheers,
>> >>> Ruppert
>> >>>
>> >>> On Nov 19, 2009, at 11:44 PM, David E Jones
>> wrote:
>> >>>
>> >>>>
>> >>>> On Nov 19, 2009, at 6:46 AM, Ashish
>> Vijaywargiya wrote:
>> >>>>
>> >>>>> Hello Jacques,
>> >>>>>
>> >>>>> The following changes are good:
>> >>>>> - <request-map
>> uri="FindSubscriptionResource"><security
>> https="true"
>> >>>>> auth="true"/><response
>> name="success" type="view"
>> >>>>>
>> value="FindSubscriptionResource"/></request-map>
>> >>>>> - <request-map
>> uri="EditSubscriptionResource"><security
>> https="true"
>> >>>>> auth="true"/><response
>> name="success" type="view"
>> >>>>>
>> value="EditSubscriptionResource"/></request-map>
>> >>>>> + <request-map
>> uri="FindSubscriptionResource">
>> >>>>> +
>> <security https="true" auth="true"/>
>> >>>>> +
>> <response name="success" type="view"
>> >>>>> value="FindSubscriptionResource"/>
>> >>>>> + </request-map>
>> >>>>> + <request-map
>> uri="EditSubscriptionResource">
>> >>>>> +
>> <security https="true" auth="true"/>
>> >>>>> +
>> <response name="success" type="view"
>> >>>>> value="EditSubscriptionResource"/>
>> >>>>> + </request-map>
>> >>>>
>> >>>> I disagree. That change turns one line
>> into four, and the pattern for those entries is
>> generally consistent enough that reading beyond the
>> first 50 characters or so it totally redundant information.
>> >>>>
>> >>>> -David
>> >>>>
>> >>>>
>> >>>>
>> >>>>
>> >>>>>
>> >>>>> *But* I don't like the changes for
>> moving "invoke" attribute to the new
>> >>>>> line:
>> >>>>>
>> >>>>> - <event
>> type="java"
>> path="org.ofbiz.product.product.ProductEvents"
>> >>>>>
>> invoke="updateProductQuickAdminShipping"/>
>> >>>>> + <event
>> type="java"
>> path="org.ofbiz.product.product.ProductEvents"
>> >>>>> +
>> invoke="updateProductQuickAdminShipping"/>
>> >>>>>
>> >>>>> And for view map entries - +1.
>> >>>>>
>> >>>>>
>> >>>>> -- Ashish
>> >>>>>
>> >>>>>
>> >>>>> On Thu, Nov 19, 2009 at 6:44 PM,
>> Jacques Le Roux <
>> >>>>> [hidden email]>
>> wrote:
>> >>>>>
>> >>>>>>
>> >>>>>> Hi devs,
>> >>>>>>
>> >>>>>> I often found myself in
>> difficulties when reading controllers files.
>> >>>>>>
>> >>>>>> I would like to format them all as
>> it's done on the attached example
>> >>>>>> http://n4.nabble.com/file/n623986/controller.patch
>> controller.patch , but
>> >>>>>> the view-maps which I would prefer
>> to keep on one line.
>> >>>>>>
>> >>>>>> Opinions ?
>> >>>>>>
>> >>>>>> Thanks
>> >>>>>>
>> >>>>>> Jacques
>> >>>>>> -- View this message in context:
>> >>>>>> http://n4.nabble.com/Formatting-controllers-tp623986p623986.html
>> >>>>>> Sent from the OFBiz - Dev mailing
>> list archive at Nabble.com.
>> >>>>>>
>> >>>>
>> >>>
>> >>>
>> >>
>> >>
>> >>
>> >
>>
>>
>
>
>
>