OFBiz › OFBiz - Dev

Google Base Code is broken after commit r808410 - Please provide your suggestion to solve the problem

_‹ Previous Topic Next Topic _›

Classic

List

Threaded

6 messages Options

Ashish Vijaywargiya

Google Base Code is broken after commit r808410 - Please provide your suggestion to solve the problem

Hello Hans,

I am working on Google Base.
The code was working fine till revision r808332.

But it stopped working after your commit r808410 (Big one in which you have
added docbook related things)
The reason is the addition of
xalan-2.7.1.jar<http://svn.apache.org/viewvc/ofbiz/trunk/framework/base/lib/xalan-2.7.1.jar?limit_changes=0&view=markup&pathrev=808410>in
the trunk.
Can you please let me know the purpose for the addition of this file - this
will help me to find some alternate to solve this issue?

Here is the String generated on r808332 (Till this revision code was working
fine).

<?xml version="1.0" encoding="UTF-8"?>
<feed xmlns="http://www.w3.org/2005/Atom" xmlns:app="
http://purl.org/atom/app#" xmlns:batch="
http://schemas.google.com/gdata/batch" xmlns:g="
http://base.google.com/ns/1.0" xmlns:gm="
http://base.google.com/ns-metadata/1.0" xmlns:openSearch="
http://a9.com/-/spec/opensearchrss/1.0/">
<entry>
<batch:operation type="insert"/>
<app:control>
<gm:publishing_priority>high</gm:publishing_priority>
</app:control>
<title>Tiny Gizmo</title>
<content type="xhtml">The smallest gizmo in town.</content>
<g:id>
http://demo.ofbiz.org/ecommerce/control/product/~product_id=GZ-1000</g:id>
<link href="
http://demo.ofbiz.org/ecommerce/control/product/~product_id=GZ-1000"
rel="alternate" type="text/html"/>
<g:item_type>products</g:item_type>
<g:product_type>Electronics</g:product_type>
<g:product_type>Business & Industrial</g:product_type>
<g:price>15.990</g:price>
<g:currency>USD</g:currency>
<g:target_country>US</g:target_country>
<g:ean>12345</g:ean>
<g:condition>new</g:condition>
<g:image_link>
http://demo.ofbiz.org/images/products/large/web_browser.png</g:image_link>
</entry>
</feed>

After your changes done at r808410 the string looks like - NamespaceURL is
being added in each generated tag.
Google Base server don't like the generated string format in which we have
namespace tag so the Google Base stopped working.

<?xml version="1.0" encoding="UTF-8"?><feed xmlns:openSearch="
http://a9.com/-/spec/opensearchrss/1.0/">
<entry>
<batch:operation xmlns:batch="http://schemas.google.com/gdata/batch"
type="insert"/>
<app:control xmlns:app="http://purl.org/atom/app#">
<gm:publishing_priority xmlns:gm="
http://base.google.com/ns-metadata/1.0">high</gm:publishing_priority>
</app:control>
<title>Square Gizmo</title>
<content type="xhtml">A square gizmo</content>
<g:id xmlns:g="http://base.google.com/ns/1.0">
http://demo.ofbiz.org/ecommerce/control/product/~product_id=GZ-2002</g:id>
<link href="
http://demo.ofbiz.org/ecommerce/control/product/~product_id=GZ-2002"
rel="alternate" type="text/html"/>
<g:item_type xmlns:g="http://base.google.com/ns/1.0
">products</g:item_type>
<g:product_type xmlns:g="http://base.google.com/ns/1.0
">Electronics</g:product_type>
<g:product_type xmlns:g="http://base.google.com/ns/1.0">Business
& Industrial</g:product_type>
<g:price xmlns:g="http://base.google.com/ns/1.0">47.990</g:price>
<g:currency xmlns:g="http://base.google.com/ns/1.0">USD</g:currency>
<g:target_country xmlns:g="http://base.google.com/ns/1.0
">US</g:target_country>
<g:condition xmlns:g="http://base.google.com/ns/1.0
">new</g:condition>
<g:image_link xmlns:g="http://base.google.com/ns/1.0">
http://demo.ofbiz.org/images/products/large/server_mimooh.png</g:image_link>
</entry>
</feed>

--
Ashish

hans_bakker

Re: Google Base Code is broken after commit r808410 - Please provide your suggestion to solve the problem

Hi Ashish,

xalan is required for the generation of html documents from the xml/xsl
combination files.

It could be that that the xalan overrides the xercess libraries.

with xalan you can set the namespace awareness the following way:

SAXParserFactory pfactory= SAXParserFactory.newInstance();
pfactory.setNamespaceAware(true);

an example you can find in the XslTransform.java file where i added the
code.

let me know which xml processing code Google base is using, i will have
a look at it...

Regards,
Hans

On Mon, 2009-08-31 at 21:26 +0530, Ashish Vijaywargiya wrote:

> Hello Hans,
>
> I am working on Google Base.
> The code was working fine till revision r808332.
>
> But it stopped working after your commit r808410 (Big one in which you have
> added docbook related things)
> The reason is the addition of
> xalan-2.7.1.jar<http://svn.apache.org/viewvc/ofbiz/trunk/framework/base/lib/xalan-2.7.1.jar?limit_changes=0&view=markup&pathrev=808410>in
> the trunk.
> Can you please let me know the purpose for the addition of this file - this
> will help me to find some alternate to solve this issue?
>
> Here is the String generated on r808332 (Till this revision code was working
> fine).
>
> <?xml version="1.0" encoding="UTF-8"?>
> <feed xmlns="http://www.w3.org/2005/Atom" xmlns:app="
> http://purl.org/atom/app#" xmlns:batch="
> http://schemas.google.com/gdata/batch" xmlns:g="
> http://base.google.com/ns/1.0" xmlns:gm="
> http://base.google.com/ns-metadata/1.0" xmlns:openSearch="
> http://a9.com/-/spec/opensearchrss/1.0/">
> <entry>
> <batch:operation type="insert"/>
> <app:control>
> <gm:publishing_priority>high</gm:publishing_priority>
> </app:control>
> <title>Tiny Gizmo</title>
> <content type="xhtml">The smallest gizmo in town.</content>
> <g:id>
> http://demo.ofbiz.org/ecommerce/control/product/~product_id=GZ-1000</g:id>
> <link href="
> http://demo.ofbiz.org/ecommerce/control/product/~product_id=GZ-1000"
> rel="alternate" type="text/html"/>
> <g:item_type>products</g:item_type>
> <g:product_type>Electronics</g:product_type>
> <g:product_type>Business & Industrial</g:product_type>
> <g:price>15.990</g:price>
> <g:currency>USD</g:currency>
> <g:target_country>US</g:target_country>
> <g:ean>12345</g:ean>
> <g:condition>new</g:condition>
> <g:image_link>
> http://demo.ofbiz.org/images/products/large/web_browser.png</g:image_link>
> </entry>
> </feed>
>
> After your changes done at r808410 the string looks like - NamespaceURL is
> being added in each generated tag.
> Google Base server don't like the generated string format in which we have
> namespace tag so the Google Base stopped working.
>
> <?xml version="1.0" encoding="UTF-8"?><feed xmlns:openSearch="
> http://a9.com/-/spec/opensearchrss/1.0/">
> <entry>
> <batch:operation xmlns:batch="http://schemas.google.com/gdata/batch"
> type="insert"/>
> <app:control xmlns:app="http://purl.org/atom/app#">
> <gm:publishing_priority xmlns:gm="
> http://base.google.com/ns-metadata/1.0">high</gm:publishing_priority>
> </app:control>
> <title>Square Gizmo</title>
> <content type="xhtml">A square gizmo</content>
> <g:id xmlns:g="http://base.google.com/ns/1.0">
> http://demo.ofbiz.org/ecommerce/control/product/~product_id=GZ-2002</g:id>
> <link href="
> http://demo.ofbiz.org/ecommerce/control/product/~product_id=GZ-2002"
> rel="alternate" type="text/html"/>
> <g:item_type xmlns:g="http://base.google.com/ns/1.0
> ">products</g:item_type>
> <g:product_type xmlns:g="http://base.google.com/ns/1.0
> ">Electronics</g:product_type>
> <g:product_type xmlns:g="http://base.google.com/ns/1.0">Business
> & Industrial</g:product_type>
> <g:price xmlns:g="http://base.google.com/ns/1.0">47.990</g:price>
> <g:currency xmlns:g="http://base.google.com/ns/1.0">USD</g:currency>
> <g:target_country xmlns:g="http://base.google.com/ns/1.0
> ">US</g:target_country>
> <g:condition xmlns:g="http://base.google.com/ns/1.0
> ">new</g:condition>
> <g:image_link xmlns:g="http://base.google.com/ns/1.0">
> http://demo.ofbiz.org/images/products/large/server_mimooh.png</g:image_link>
> </entry>
> </feed>
>
> --
> Ashish

--
Antwebsystems.com: Quality OFBiz services for competitive rates

Ashish Vijaywargiya

Re: Google Base Code is broken after commit r808410 - Please provide your suggestion to solve the problem

Thanks Hans for your comment on this - I will explore it and will get back
to you if needed.
Rest comments Inline:

--
Ashish

On Tue, Sep 1, 2009 at 5:15 AM, Hans Bakker
<[hidden email]>wrote:

> Hi Ashish,
>
> xalan is required for the generation of html documents from the xml/xsl
> combination files.
>
> It could be that that the xalan overrides the xercess libraries.
>
> with xalan you can set the namespace awareness the following way:
>
> SAXParserFactory pfactory= SAXParserFactory.newInstance();
> pfactory.setNamespaceAware(true);
>
> an example you can find in the XslTransform.java file where i added the
> code.
>
> let me know which xml processing code Google base is using, i will have
> a look at it...
>

You can refer method "buildDataItemsXml" present inside
ProductsExportToGoogle.java file present inside GoogleBase component.

>
> Regards,
> Hans
>
>
> On Mon, 2009-08-31 at 21:26 +0530, Ashish Vijaywargiya wrote:
> > Hello Hans,
> >
> > I am working on Google Base.
> > The code was working fine till revision r808332.
> >
> > But it stopped working after your commit r808410 (Big one in which you
> have
> > added docbook related things)
> > The reason is the addition of
> > xalan-2.7.1.jar<
> http://svn.apache.org/viewvc/ofbiz/trunk/framework/base/lib/xalan-2.7.1.jar?limit_changes=0&view=markup&pathrev=808410
> >in
> > the trunk.
> > Can you please let me know the purpose for the addition of this file -
> this
> > will help me to find some alternate to solve this issue?
> >
> > Here is the String generated on r808332 (Till this revision code was
> working
> > fine).
> >
> > <?xml version="1.0" encoding="UTF-8"?>
> > <feed xmlns="http://www.w3.org/2005/Atom" xmlns:app="
> > http://purl.org/atom/app#" xmlns:batch="
> > http://schemas.google.com/gdata/batch" xmlns:g="
> > http://base.google.com/ns/1.0" xmlns:gm="
> > http://base.google.com/ns-metadata/1.0" xmlns:openSearch="
> > http://a9.com/-/spec/opensearchrss/1.0/">
> > <entry>
> > <batch:operation type="insert"/>
> > <app:control>
> > <gm:publishing_priority>high</gm:publishing_priority>
> > </app:control>
> > <title>Tiny Gizmo</title>
> > <content type="xhtml">The smallest gizmo in town.</content>
> > <g:id>
> > http://demo.ofbiz.org/ecommerce/control/product/~product_id=GZ-1000<http://demo.ofbiz.org/ecommerce/control/product/%7Eproduct_id=GZ-1000>
> </g:id>
> > <link href="
> > http://demo.ofbiz.org/ecommerce/control/product/~product_id=GZ-1000<http://demo.ofbiz.org/ecommerce/control/product/%7Eproduct_id=GZ-1000>
> "
> > rel="alternate" type="text/html"/>
> > <g:item_type>products</g:item_type>
> > <g:product_type>Electronics</g:product_type>
> > <g:product_type>Business & Industrial</g:product_type>
> > <g:price>15.990</g:price>
> > <g:currency>USD</g:currency>
> > <g:target_country>US</g:target_country>
> > <g:ean>12345</g:ean>
> > <g:condition>new</g:condition>
> > <g:image_link>
> > http://demo.ofbiz.org/images/products/large/web_browser.png
> </g:image_link>
> > </entry>
> > </feed>
> >
> > After your changes done at r808410 the string looks like - NamespaceURL
> is
> > being added in each generated tag.
> > Google Base server don't like the generated string format in which we
> have
> > namespace tag so the Google Base stopped working.
> >
> > <?xml version="1.0" encoding="UTF-8"?><feed xmlns:openSearch="
> > http://a9.com/-/spec/opensearchrss/1.0/">
> > <entry>
> > <batch:operation xmlns:batch="
> http://schemas.google.com/gdata/batch"
> > type="insert"/>
> > <app:control xmlns:app="http://purl.org/atom/app#">
> > <gm:publishing_priority xmlns:gm="
> > http://base.google.com/ns-metadata/1.0">high</gm:publishing_priority>
> > </app:control>
> > <title>Square Gizmo</title>
> > <content type="xhtml">A square gizmo</content>
> > <g:id xmlns:g="http://base.google.com/ns/1.0">
> > http://demo.ofbiz.org/ecommerce/control/product/~product_id=GZ-2002<http://demo.ofbiz.org/ecommerce/control/product/%7Eproduct_id=GZ-2002>
> </g:id>
> > <link href="
> > http://demo.ofbiz.org/ecommerce/control/product/~product_id=GZ-2002<http://demo.ofbiz.org/ecommerce/control/product/%7Eproduct_id=GZ-2002>
> "
> > rel="alternate" type="text/html"/>
> > <g:item_type xmlns:g="http://base.google.com/ns/1.0
> > ">products</g:item_type>
> > <g:product_type xmlns:g="http://base.google.com/ns/1.0
> > ">Electronics</g:product_type>
> > <g:product_type xmlns:g="http://base.google.com/ns/1.0">Business
> > & Industrial</g:product_type>
> > <g:price xmlns:g="http://base.google.com/ns/1.0
> ">47.990</g:price>
> > <g:currency xmlns:g="http://base.google.com/ns/1.0
> ">USD</g:currency>
> > <g:target_country xmlns:g="http://base.google.com/ns/1.0
> > ">US</g:target_country>
> > <g:condition xmlns:g="http://base.google.com/ns/1.0
> > ">new</g:condition>
> > <g:image_link xmlns:g="http://base.google.com/ns/1.0">
> > http://demo.ofbiz.org/images/products/large/server_mimooh.png
> </g:image_link>
> > </entry>
> > </feed>
> >
> > --
> > Ashish
> --
> Antwebsystems.com: Quality OFBiz services for competitive rates
>
>

hans_bakker

Re: Google Base Code is broken after commit r808410 - Please provide your suggestion to solve the problem

Hi Ashish

i had a quick look at the code but could not find any xalan dependable
code there....It uses however UtilXml.java.

This file you should scan with the string: setNamespaceAware(true)
and see what is used within the google base component.

Regards, Hans

On Tue, 2009-09-01 at 10:39 +0530, Ashish Vijaywargiya wrote:

> Thanks Hans for your comment on this - I will explore it and will get back
> to you if needed.
> Rest comments Inline:
>
> --
> Ashish
>
> On Tue, Sep 1, 2009 at 5:15 AM, Hans Bakker
> <[hidden email]>wrote:
>
> > Hi Ashish,
> >
> > xalan is required for the generation of html documents from the xml/xsl
> > combination files.
> >
> > It could be that that the xalan overrides the xercess libraries.
> >
> > with xalan you can set the namespace awareness the following way:
> >
> > SAXParserFactory pfactory= SAXParserFactory.newInstance();
> > pfactory.setNamespaceAware(true);
> >
> > an example you can find in the XslTransform.java file where i added the
> > code.
> >
> > let me know which xml processing code Google base is using, i will have
> > a look at it...
> >
>
> You can refer method "buildDataItemsXml" present inside
> ProductsExportToGoogle.java file present inside GoogleBase component.
>
>
>
> >
> > Regards,
> > Hans
> >
> >
> > On Mon, 2009-08-31 at 21:26 +0530, Ashish Vijaywargiya wrote:
> > > Hello Hans,
> > >
> > > I am working on Google Base.
> > > The code was working fine till revision r808332.
> > >
> > > But it stopped working after your commit r808410 (Big one in which you
> > have
> > > added docbook related things)
> > > The reason is the addition of
> > > xalan-2.7.1.jar<
> > http://svn.apache.org/viewvc/ofbiz/trunk/framework/base/lib/xalan-2.7.1.jar?limit_changes=0&view=markup&pathrev=808410
> > >in
> > > the trunk.
> > > Can you please let me know the purpose for the addition of this file -
> > this
> > > will help me to find some alternate to solve this issue?
> > >
> > > Here is the String generated on r808332 (Till this revision code was
> > working
> > > fine).
> > >
> > > <?xml version="1.0" encoding="UTF-8"?>
> > > <feed xmlns="http://www.w3.org/2005/Atom" xmlns:app="
> > > http://purl.org/atom/app#" xmlns:batch="
> > > http://schemas.google.com/gdata/batch" xmlns:g="
> > > http://base.google.com/ns/1.0" xmlns:gm="
> > > http://base.google.com/ns-metadata/1.0" xmlns:openSearch="
> > > http://a9.com/-/spec/opensearchrss/1.0/">
> > > <entry>
> > > <batch:operation type="insert"/>
> > > <app:control>
> > > <gm:publishing_priority>high</gm:publishing_priority>
> > > </app:control>
> > > <title>Tiny Gizmo</title>
> > > <content type="xhtml">The smallest gizmo in town.</content>
> > > <g:id>
> > > http://demo.ofbiz.org/ecommerce/control/product/~product_id=GZ-1000<http://demo.ofbiz.org/ecommerce/control/product/%7Eproduct_id=GZ-1000>
> > </g:id>
> > > <link href="
> > > http://demo.ofbiz.org/ecommerce/control/product/~product_id=GZ-1000<http://demo.ofbiz.org/ecommerce/control/product/%7Eproduct_id=GZ-1000>
> > "
> > > rel="alternate" type="text/html"/>
> > > <g:item_type>products</g:item_type>
> > > <g:product_type>Electronics</g:product_type>
> > > <g:product_type>Business & Industrial</g:product_type>
> > > <g:price>15.990</g:price>
> > > <g:currency>USD</g:currency>
> > > <g:target_country>US</g:target_country>
> > > <g:ean>12345</g:ean>
> > > <g:condition>new</g:condition>
> > > <g:image_link>
> > > http://demo.ofbiz.org/images/products/large/web_browser.png
> > </g:image_link>
> > > </entry>
> > > </feed>
> > >
> > > After your changes done at r808410 the string looks like - NamespaceURL
> > is
> > > being added in each generated tag.
> > > Google Base server don't like the generated string format in which we
> > have
> > > namespace tag so the Google Base stopped working.
> > >
> > > <?xml version="1.0" encoding="UTF-8"?><feed xmlns:openSearch="
> > > http://a9.com/-/spec/opensearchrss/1.0/">
> > > <entry>
> > > <batch:operation xmlns:batch="
> > http://schemas.google.com/gdata/batch"
> > > type="insert"/>
> > > <app:control xmlns:app="http://purl.org/atom/app#">
> > > <gm:publishing_priority xmlns:gm="
> > > http://base.google.com/ns-metadata/1.0">high</gm:publishing_priority>
> > > </app:control>
> > > <title>Square Gizmo</title>
> > > <content type="xhtml">A square gizmo</content>
> > > <g:id xmlns:g="http://base.google.com/ns/1.0">
> > > http://demo.ofbiz.org/ecommerce/control/product/~product_id=GZ-2002<http://demo.ofbiz.org/ecommerce/control/product/%7Eproduct_id=GZ-2002>
> > </g:id>
> > > <link href="
> > > http://demo.ofbiz.org/ecommerce/control/product/~product_id=GZ-2002<http://demo.ofbiz.org/ecommerce/control/product/%7Eproduct_id=GZ-2002>
> > "
> > > rel="alternate" type="text/html"/>
> > > <g:item_type xmlns:g="http://base.google.com/ns/1.0
> > > ">products</g:item_type>
> > > <g:product_type xmlns:g="http://base.google.com/ns/1.0
> > > ">Electronics</g:product_type>
> > > <g:product_type xmlns:g="http://base.google.com/ns/1.0">Business
> > > & Industrial</g:product_type>
> > > <g:price xmlns:g="http://base.google.com/ns/1.0
> > ">47.990</g:price>
> > > <g:currency xmlns:g="http://base.google.com/ns/1.0
> > ">USD</g:currency>
> > > <g:target_country xmlns:g="http://base.google.com/ns/1.0
> > > ">US</g:target_country>
> > > <g:condition xmlns:g="http://base.google.com/ns/1.0
> > > ">new</g:condition>
> > > <g:image_link xmlns:g="http://base.google.com/ns/1.0">
> > > http://demo.ofbiz.org/images/products/large/server_mimooh.png
> > </g:image_link>
> > > </entry>
> > > </feed>
> > >
> > > --
> > > Ashish
> > --
> > Antwebsystems.com: Quality OFBiz services for competitive rates
> >
> >

--
Antwebsystems.com: Quality OFBiz services for competitive rates

Ashish Vijaywargiya

Re: Google Base Code is broken after commit r808410 - Please provide your suggestion to solve the problem

The code uses makeEmptyXmlDocument() method of UtilXml.java file.
The default value of namespace aware is kept commented - which is good in
case of Google Base.
// factory.setNamespaceAware(true);

If I remove Xalan*.jar then Google Base code works fine.
I am exploring the root cause of this problem.

Thanks for your comment on this.
--
Ashish

On Tue, Sep 1, 2009 at 11:26 AM, Hans Bakker
<[hidden email]>wrote:

> Hi Ashish
>
> i had a quick look at the code but could not find any xalan dependable
> code there....It uses however UtilXml.java.
>
> This file you should scan with the string: setNamespaceAware(true)
> and see what is used within the google base component.
>
> Regards, Hans
>
>
> On Tue, 2009-09-01 at 10:39 +0530, Ashish Vijaywargiya wrote:
> > Thanks Hans for your comment on this - I will explore it and will get
> back
> > to you if needed.
> > Rest comments Inline:
> >
> > --
> > Ashish
> >
> > On Tue, Sep 1, 2009 at 5:15 AM, Hans Bakker
> > <[hidden email]>wrote:
> >
> > > Hi Ashish,
> > >
> > > xalan is required for the generation of html documents from the xml/xsl
> > > combination files.
> > >
> > > It could be that that the xalan overrides the xercess libraries.
> > >
> > > with xalan you can set the namespace awareness the following way:
> > >
> > > SAXParserFactory pfactory= SAXParserFactory.newInstance();
> > > pfactory.setNamespaceAware(true);
> > >
> > > an example you can find in the XslTransform.java file where i added the
> > > code.
> > >
> > > let me know which xml processing code Google base is using, i will
> have
> > > a look at it...
> > >
> >
> > You can refer method "buildDataItemsXml" present inside
> > ProductsExportToGoogle.java file present inside GoogleBase component.
> >
> >
> >
> > >
> > > Regards,
> > > Hans
> > >
> > >
> > > On Mon, 2009-08-31 at 21:26 +0530, Ashish Vijaywargiya wrote:
> > > > Hello Hans,
> > > >
> > > > I am working on Google Base.
> > > > The code was working fine till revision r808332.
> > > >
> > > > But it stopped working after your commit r808410 (Big one in which
> you
> > > have
> > > > added docbook related things)
> > > > The reason is the addition of
> > > > xalan-2.7.1.jar<
> > >
> http://svn.apache.org/viewvc/ofbiz/trunk/framework/base/lib/xalan-2.7.1.jar?limit_changes=0&view=markup&pathrev=808410
> > > >in
> > > > the trunk.
> > > > Can you please let me know the purpose for the addition of this file
> -
> > > this
> > > > will help me to find some alternate to solve this issue?
> > > >
> > > > Here is the String generated on r808332 (Till this revision code was
> > > working
> > > > fine).
> > > >
> > > > <?xml version="1.0" encoding="UTF-8"?>
> > > > <feed xmlns="http://www.w3.org/2005/Atom" xmlns:app="
> > > > http://purl.org/atom/app#" xmlns:batch="
> > > > http://schemas.google.com/gdata/batch" xmlns:g="
> > > > http://base.google.com/ns/1.0" xmlns:gm="
> > > > http://base.google.com/ns-metadata/1.0" xmlns:openSearch="
> > > > http://a9.com/-/spec/opensearchrss/1.0/">
> > > > <entry>
> > > > <batch:operation type="insert"/>
> > > > <app:control>
> > > > <gm:publishing_priority>high</gm:publishing_priority>
> > > > </app:control>
> > > > <title>Tiny Gizmo</title>
> > > > <content type="xhtml">The smallest gizmo in town.</content>
> > > > <g:id>
> > > > http://demo.ofbiz.org/ecommerce/control/product/~product_id=GZ-1000<http://demo.ofbiz.org/ecommerce/control/product/%7Eproduct_id=GZ-1000>
> <http://demo.ofbiz.org/ecommerce/control/product/%7Eproduct_id=GZ-1000>
> > > </g:id>
> > > > <link href="
> > > > http://demo.ofbiz.org/ecommerce/control/product/~product_id=GZ-1000<http://demo.ofbiz.org/ecommerce/control/product/%7Eproduct_id=GZ-1000>
> <http://demo.ofbiz.org/ecommerce/control/product/%7Eproduct_id=GZ-1000>
> > > "
> > > > rel="alternate" type="text/html"/>
> > > > <g:item_type>products</g:item_type>
> > > > <g:product_type>Electronics</g:product_type>
> > > > <g:product_type>Business & Industrial</g:product_type>
> > > > <g:price>15.990</g:price>
> > > > <g:currency>USD</g:currency>
> > > > <g:target_country>US</g:target_country>
> > > > <g:ean>12345</g:ean>
> > > > <g:condition>new</g:condition>
> > > > <g:image_link>
> > > > http://demo.ofbiz.org/images/products/large/web_browser.png
> > > </g:image_link>
> > > > </entry>
> > > > </feed>
> > > >
> > > > After your changes done at r808410 the string looks like -
> NamespaceURL
> > > is
> > > > being added in each generated tag.
> > > > Google Base server don't like the generated string format in which we
> > > have
> > > > namespace tag so the Google Base stopped working.
> > > >
> > > > <?xml version="1.0" encoding="UTF-8"?><feed xmlns:openSearch="
> > > > http://a9.com/-/spec/opensearchrss/1.0/">
> > > > <entry>
> > > > <batch:operation xmlns:batch="
> > > http://schemas.google.com/gdata/batch"
> > > > type="insert"/>
> > > > <app:control xmlns:app="http://purl.org/atom/app#">
> > > > <gm:publishing_priority xmlns:gm="
> > > > http://base.google.com/ns-metadata/1.0
> ">high</gm:publishing_priority>
> > > > </app:control>
> > > > <title>Square Gizmo</title>
> > > > <content type="xhtml">A square gizmo</content>
> > > > <g:id xmlns:g="http://base.google.com/ns/1.0">
> > > > http://demo.ofbiz.org/ecommerce/control/product/~product_id=GZ-2002<http://demo.ofbiz.org/ecommerce/control/product/%7Eproduct_id=GZ-2002>
> <http://demo.ofbiz.org/ecommerce/control/product/%7Eproduct_id=GZ-2002>
> > > </g:id>
> > > > <link href="
> > > > http://demo.ofbiz.org/ecommerce/control/product/~product_id=GZ-2002<http://demo.ofbiz.org/ecommerce/control/product/%7Eproduct_id=GZ-2002>
> <http://demo.ofbiz.org/ecommerce/control/product/%7Eproduct_id=GZ-2002>
> > > "
> > > > rel="alternate" type="text/html"/>
> > > > <g:item_type xmlns:g="http://base.google.com/ns/1.0
> > > > ">products</g:item_type>
> > > > <g:product_type xmlns:g="http://base.google.com/ns/1.0
> > > > ">Electronics</g:product_type>
> > > > <g:product_type xmlns:g="http://base.google.com/ns/1.0
> ">Business
> > > > & Industrial</g:product_type>
> > > > <g:price xmlns:g="http://base.google.com/ns/1.0
> > > ">47.990</g:price>
> > > > <g:currency xmlns:g="http://base.google.com/ns/1.0
> > > ">USD</g:currency>
> > > > <g:target_country xmlns:g="http://base.google.com/ns/1.0
> > > > ">US</g:target_country>
> > > > <g:condition xmlns:g="http://base.google.com/ns/1.0
> > > > ">new</g:condition>
> > > > <g:image_link xmlns:g="http://base.google.com/ns/1.0">
> > > > http://demo.ofbiz.org/images/products/large/server_mimooh.png
> > > </g:image_link>
> > > > </entry>
> > > > </feed>
> > > >
> > > > --
> > > > Ashish
> > > --
> > > Antwebsystems.com: Quality OFBiz services for competitive rates
> > >
> > >
> --
> Antwebsystems.com: Quality OFBiz services for competitive rates
>
>

Jacopo Cappellato-4

Re: Google Base Code is broken after commit r808410 - Please provide your suggestion to solve the problem

It is now fixed in rev. 810857

Jacopo

On Sep 1, 2009, at 8:19 AM, Ashish Vijaywargiya wrote:

> The code uses makeEmptyXmlDocument() method of UtilXml.java file.
> The default value of namespace aware is kept commented - which is
> good in
> case of Google Base.
> // factory.setNamespaceAware(true);
>
> If I remove Xalan*.jar then Google Base code works fine.
> I am exploring the root cause of this problem.
>
> Thanks for your comment on this.
> --
> Ashish
>
> On Tue, Sep 1, 2009 at 11:26 AM, Hans Bakker
> <[hidden email]>wrote:
>
>> Hi Ashish
>>
>> i had a quick look at the code but could not find any xalan
>> dependable
>> code there....It uses however UtilXml.java.
>>
>> This file you should scan with the string: setNamespaceAware(true)
>> and see what is used within the google base component.
>>
>> Regards, Hans
>>
>>
>> On Tue, 2009-09-01 at 10:39 +0530, Ashish Vijaywargiya wrote:
>>> Thanks Hans for your comment on this - I will explore it and will
>>> get
>> back
>>> to you if needed.
>>> Rest comments Inline:
>>>
>>> --
>>> Ashish
>>>
>>> On Tue, Sep 1, 2009 at 5:15 AM, Hans Bakker
>>> <[hidden email]>wrote:
>>>
>>>> Hi Ashish,
>>>>
>>>> xalan is required for the generation of html documents from the
>>>> xml/xsl
>>>> combination files.
>>>>
>>>> It could be that that the xalan overrides the xercess libraries.
>>>>
>>>> with xalan you can set the namespace awareness the following way:
>>>>
>>>> SAXParserFactory pfactory= SAXParserFactory.newInstance();
>>>> pfactory.setNamespaceAware(true);
>>>>
>>>> an example you can find in the XslTransform.java file where i
>>>> added the
>>>> code.
>>>>
>>>> let me know which xml processing code Google base is using, i will
>> have
>>>> a look at it...
>>>>
>>>
>>> You can refer method "buildDataItemsXml" present inside
>>> ProductsExportToGoogle.java file present inside GoogleBase
>>> component.
>>>
>>>
>>>
>>>>
>>>> Regards,
>>>> Hans
>>>>
>>>>
>>>> On Mon, 2009-08-31 at 21:26 +0530, Ashish Vijaywargiya wrote:
>>>>> Hello Hans,
>>>>>
>>>>> I am working on Google Base.
>>>>> The code was working fine till revision r808332.
>>>>>
>>>>> But it stopped working after your commit r808410 (Big one in which
>> you
>>>> have
>>>>> added docbook related things)
>>>>> The reason is the addition of
>>>>> xalan-2.7.1.jar<
>>>>
>> http://svn.apache.org/viewvc/ofbiz/trunk/framework/base/lib/xalan-2.7.1.jar?limit_changes=0&view=markup&pathrev=808410
>>>>> in
>>>>> the trunk.
>>>>> Can you please let me know the purpose for the addition of this
>>>>> file
>> -
>>>> this
>>>>> will help me to find some alternate to solve this issue?
>>>>>
>>>>> Here is the String generated on r808332 (Till this revision code
>>>>> was
>>>> working
>>>>> fine).
>>>>>
>>>>> <?xml version="1.0" encoding="UTF-8"?>
>>>>> <feed xmlns="http://www.w3.org/2005/Atom" xmlns:app="
>>>>> http://purl.org/atom/app#" xmlns:batch="
>>>>> http://schemas.google.com/gdata/batch" xmlns:g="
>>>>> http://base.google.com/ns/1.0" xmlns:gm="
>>>>> http://base.google.com/ns-metadata/1.0" xmlns:openSearch="
>>>>> http://a9.com/-/spec/opensearchrss/1.0/">
>>>>> <entry>
>>>>> <batch:operation type="insert"/>
>>>>> <app:control>
>>>>> <gm:publishing_priority>high</gm:publishing_priority>
>>>>> </app:control>
>>>>> <title>Tiny Gizmo</title>
>>>>> <content type="xhtml">The smallest gizmo in town.</content>
>>>>> <g:id>
>>>>> http://demo.ofbiz.org/ecommerce/control/product/
>>>>> ~product_id=GZ-1000<http://demo.ofbiz.org/ecommerce/control/product/%7Eproduct_id=GZ-1000
>>>>> >
>> <http://demo.ofbiz.org/ecommerce/control/product/%7Eproduct_id=GZ-1000
>> >
>>>> </g:id>
>>>>> <link href="
>>>>> http://demo.ofbiz.org/ecommerce/control/product/
>>>>> ~product_id=GZ-1000<http://demo.ofbiz.org/ecommerce/control/product/%7Eproduct_id=GZ-1000
>>>>> >
>> <http://demo.ofbiz.org/ecommerce/control/product/%7Eproduct_id=GZ-1000
>> >
>>>> "
>>>>> rel="alternate" type="text/html"/>
>>>>> <g:item_type>products</g:item_type>
>>>>> <g:product_type>Electronics</g:product_type>
>>>>> <g:product_type>Business & Industrial</g:product_type>
>>>>> <g:price>15.990</g:price>
>>>>> <g:currency>USD</g:currency>
>>>>> <g:target_country>US</g:target_country>
>>>>> <g:ean>12345</g:ean>
>>>>> <g:condition>new</g:condition>
>>>>> <g:image_link>
>>>>> http://demo.ofbiz.org/images/products/large/web_browser.png
>>>> </g:image_link>
>>>>> </entry>
>>>>> </feed>
>>>>>
>>>>> After your changes done at r808410 the string looks like -
>> NamespaceURL
>>>> is
>>>>> being added in each generated tag.
>>>>> Google Base server don't like the generated string format in
>>>>> which we
>>>> have
>>>>> namespace tag so the Google Base stopped working.
>>>>>
>>>>> <?xml version="1.0" encoding="UTF-8"?><feed xmlns:openSearch="
>>>>> http://a9.com/-/spec/opensearchrss/1.0/">
>>>>> <entry>
>>>>> <batch:operation xmlns:batch="
>>>> http://schemas.google.com/gdata/batch"
>>>>> type="insert"/>
>>>>> <app:control xmlns:app="http://purl.org/atom/app#">
>>>>> <gm:publishing_priority xmlns:gm="
>>>>> http://base.google.com/ns-metadata/1.0
>> ">high</gm:publishing_priority>
>>>>> </app:control>
>>>>> <title>Square Gizmo</title>
>>>>> <content type="xhtml">A square gizmo</content>
>>>>> <g:id xmlns:g="http://base.google.com/ns/1.0">
>>>>> http://demo.ofbiz.org/ecommerce/control/product/
>>>>> ~product_id=GZ-2002<http://demo.ofbiz.org/ecommerce/control/product/%7Eproduct_id=GZ-2002
>>>>> >
>> <http://demo.ofbiz.org/ecommerce/control/product/%7Eproduct_id=GZ-2002
>> >
>>>> </g:id>
>>>>> <link href="
>>>>> http://demo.ofbiz.org/ecommerce/control/product/
>>>>> ~product_id=GZ-2002<http://demo.ofbiz.org/ecommerce/control/product/%7Eproduct_id=GZ-2002
>>>>> >
>> <http://demo.ofbiz.org/ecommerce/control/product/%7Eproduct_id=GZ-2002
>> >
>>>> "
>>>>> rel="alternate" type="text/html"/>
>>>>> <g:item_type xmlns:g="http://base.google.com/ns/1.0
>>>>> ">products</g:item_type>
>>>>> <g:product_type xmlns:g="http://base.google.com/ns/1.0
>>>>> ">Electronics</g:product_type>
>>>>> <g:product_type xmlns:g="http://base.google.com/ns/1.0
>> ">Business
>>>>> & Industrial</g:product_type>
>>>>> <g:price xmlns:g="http://base.google.com/ns/1.0
>>>> ">47.990</g:price>
>>>>> <g:currency xmlns:g="http://base.google.com/ns/1.0
>>>> ">USD</g:currency>
>>>>> <g:target_country xmlns:g="http://base.google.com/ns/1.0
>>>>> ">US</g:target_country>
>>>>> <g:condition xmlns:g="http://base.google.com/ns/1.0
>>>>> ">new</g:condition>
>>>>> <g:image_link xmlns:g="http://base.google.com/ns/1.0">
>>>>> http://demo.ofbiz.org/images/products/large/server_mimooh.png
>>>> </g:image_link>
>>>>> </entry>
>>>>> </feed>
>>>>>
>>>>> --
>>>>> Ashish
>>>> --
>>>> Antwebsystems.com: Quality OFBiz services for competitive rates
>>>>
>>>>
>> --
>> Antwebsystems.com: Quality OFBiz services for competitive rates
>>
>>