Guido,

Why not, please create a jira issue an attach your patch. Please if someone see 
a problem fell free to express yourself.

Thanks

Jacques

----- Original Message ----- 
From: "Guido Amarilla" <[EMAIL PROTECTED]>
To: <dev@ofbiz.apache.org>
Sent: Saturday, January 13, 2007 4:47 PM
Subject: Re: Quote screens issues


> 2007/1/13, Jacques Le Roux <[EMAIL PROTECTED]>:
> > Guido,
> >
> > From: "Guido Amarilla" <[EMAIL PROTECTED]>
> > > I just wanted to confirm that these are acceptable issues, before
> > > creating them in JIRA:
> > >
> > > 1) In Quote edit screens the title is made by combining Quote ID +
> > > Quote Description. As Description is a multiline text, and could be
> > > long, this really should be Quote Id + Quote Name. Isn´t it?
> >
> > Description is *not* a multiline text (or I don't know how to put 
> > multi-lines in, I did not look at code yet). So it seems not a
> > problem to me.
> >
>
> Yes, it`s true. it is not a multiline text. I needed some field to put
> a description for the quote. I understand that a description is a
> field that I can write longer sentences to describe the quote. If that
> is true, then that should be changed to show the quote name as a
> header, and this is a proposed patch.
>
> Index: ofbiz/trunk/applications/order/widget/ordermgr/CommonScreens.xml
> ===================================================================
> --- ofbiz/trunk/applications/order/widget/ordermgr/CommonScreens.xml (revision
> 495129)
> +++ ofbiz/trunk/applications/order/widget/ordermgr/CommonScreens.xml (working
> copy)
> @@ -68,7 +68,7 @@
>                                      <html><html-template
> location="component://order/webapp/ordermgr/quote/QuoteTabBar.ftl"/></html>
>                                  </platform-specific>
>                                  <container>
> -                                    <label
> style="head1">[${uiLabelMap.CommonId}:${quote.quoteId}]
> ${quote.description}</label>
> +                                    <label
> style="head1">[${uiLabelMap.CommonId}:${quote.quoteId}]
> ${quote.quoteName}</label>
>                                  </container>
>
>                                  <decorator-section-include name="body"/>
> @@ -99,7 +99,7 @@
>                                      <html><html-template
> location="component://order/webapp/ordermgr/quote/QuoteTabBar.ftl"/></html>
>                                  </platform-specific>
>                                  <container>
> -                                    <label
> style="head1">[${uiLabelMap.CommonId}:${quote.quoteId}]
> ${quote.description}</label>
> +                                    <label
> style="head1">[${uiLabelMap.CommonId}:${quote.quoteId}]
> ${quote.quoteName}</label>
>                                  </container>
>
>                                  <decorator-section-include name="body"/>
>
>
>
>
>
> > > 2) I need to add some "conditions" to the quote in the PDF report, to
> > > send it to the customer, like "VAT is included in prices" or "Travel
> > > Expenses not included". I didn´t find any specialized field so I added
> > > attributes to do it. Is it right to do this way? I also noticed that
> > > these attributes are not printed in the PDF report. Should them be
> > > included in the template creating an issue for this or should I
> > > customize it just for myself?
> >
> > You may submit a patch in Jira and we will see, don't you think so ?
>
> I think so.
>
>
> >
> > Jacques
> >
> > > Thanks
> > > --
> > > Guido Amarilla
> > > Córdoba, Argentina
> >
> >
>
>
> -- 
> Guido Amarilla

Reply via email to