Quote screens issues

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
5 messages Options
Reply | Threaded
Open this post in threaded view
|

Quote screens issues

Guido Amarilla
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?

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?

Thanks
--
Guido Amarilla
Córdoba, Argentina
Reply | Threaded
Open this post in threaded view
|

Re: Quote screens issues

Jacques Le Roux
Administrator
Guido,

From: "Guido Amarilla" <[hidden email]>
> 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.

> 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 ?

Jacques

> Thanks
> --
> Guido Amarilla
> Córdoba, Argentina

Reply | Threaded
Open this post in threaded view
|

Re: Quote screens issues

Guido Amarilla
2007/1/13, Jacques Le Roux <[hidden email]>:

> Guido,
>
> From: "Guido Amarilla" <[hidden email]>
> > 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 | Threaded
Open this post in threaded view
|

Re: Quote screens issues

Guido Amarilla
Something similar happens to Categories and Catalogs. In Catalog
Manager the Category Description and Cataog description are shown in
the "Browse..." left menus instead of the Category Name. And in this
case Descriptions appears as multiline texts. It is not consistent to
allow a field to be entered as a long (multiline) text and at the same
time use it as a menu item link. Shouldn't it be the Name?

2007/1/13, Guido Amarilla <[hidden email]>:

> 2007/1/13, Jacques Le Roux <[hidden email]>:
> > Guido,
> >
> > From: "Guido Amarilla" <[hidden email]>
> > > 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
>


--
Guido Amarilla
Reply | Threaded
Open this post in threaded view
|

Re: Quote screens issues

Jacques Le Roux
Administrator
In reply to this post by Guido Amarilla
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" <[hidden email]>
To: <[hidden email]>
Sent: Saturday, January 13, 2007 4:47 PM
Subject: Re: Quote screens issues


> 2007/1/13, Jacques Le Roux <[hidden email]>:
> > Guido,
> >
> > From: "Guido Amarilla" <[hidden email]>
> > > 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