google_sitemap , error on line 1 at column 10834: xmlParseEntityRef: no name

Post Reply

14 posts Page 1 of 1

google_sitemap , error on line 1 at column 10834: xmlParseEntityRef: no name

pson

New member

Posts

Joined

Wed Jan 18, 2012 2:06 am

Post by pson » Thu Jul 30, 2020 8:00 pm

Hi , this have probably been up several times , but if anyone could explain how I can edit this ?
It would be greatly appreciated

Mvh
Pson

pson

New member

Posts

Joined

Wed Jan 18, 2012 2:06 am

Re: google_sitemap , error on line 1 at column 10834: xmlParseEntityRef: no name

uksitebuilder

Guru Member

Posts

7716

Joined

Thu Jun 09, 2011 11:37 pm

Location

United Kindgom

Post by uksitebuilder » Thu Jul 30, 2020 8:23 pm

Your sitemap seems to take an age to load, probably due to the number of items in it.

uksitebuilder

Guru Member

Posts

7716

Joined

Thu Jun 09, 2011 11:37 pm

Location - United Kindgom

Re: google_sitemap , error on line 1 at column 10834: xmlParseEntityRef: no name

ADD Creative

Guru Member

Posts

5242

Joined

Sat Jan 14, 2012 1:02 am

Location

United Kingdom

Post by ADD Creative » Thu Jul 30, 2020 8:25 pm

What version of OpenCart? Any modifications?

It's probably an unescaped & in your product data somewhere.

www.add-creative.co.uk

ADD Creative

Guru Member

Posts

5242

Joined

Sat Jan 14, 2012 1:02 am

Location - United Kingdom

Re: google_sitemap , error on line 1 at column 10834: xmlParseEntityRef: no name

letxobnav

Expert Member

Posts

3264

Joined

Fri Aug 18, 2017 4:35 pm

Location

Taiwan

Post by letxobnav » Thu Jul 30, 2020 8:35 pm

since you have the & character in your product names, better use htmlspecialchars on that field when creating the xml.

Crystal Light Centrum Taiwan
Extensions: MailQueue | SUKHR | VBoces

“Data security is paramount at [...], and we are committed to protecting the privacy of anyone who is associated with our [...]. We’ve made a lot of improvements and will continue to make them.”
When you know your life savings are gone.

letxobnav

Expert Member

Posts

3264

Joined

Fri Aug 18, 2017 4:35 pm

Location - Taiwan

Re: google_sitemap , error on line 1 at column 10834: xmlParseEntityRef: no name

pson

New member

Posts

Joined

Wed Jan 18, 2012 2:06 am

Post by pson » Thu Jul 30, 2020 8:44 pm

uksitebuilder wrote: ↑
Thu Jul 30, 2020 8:23 pm
Your sitemap seems to take an age to load, probably due to the number of items in it.

Yes , I have 6000+ products
I am using OC Version 3.0.3.2

From the XML Validator it seem like it is removed :
<!DOCTYPE html>
<html style="height:100%">
<head>
<meta name="viewport" content="width=device-width, initial-scale=1, shrink-to-fit=no" >
<title> 301 Moved Permanently
</title></head>
<body style="color: #444; margin:0;font: normal 14px/20px Arial, Helvetica, sans-serif; height:100%; background-color: #fff;">
<div style="height:auto; min-height:100%; "> <div style="text-align: center; width:800px; margin-left: -400px; position:absolute; top: 30%; left:50%;">
<h1 style="margin:0; font-size:150px; line-height:150px; font-weight:bold;">301</h1>
<h2 style="margin-top:20px;font-size: 30px;">Moved Permanently
</h2>
<p>The document has been permanently moved.</p>
</div></div></body></html>

This page contains the following errors:
error on line 6 at column 16: Opening and ending tag mismatch: meta line 0 and head

Mvh
Pson

pson

New member

Posts

Joined

Wed Jan 18, 2012 2:06 am

Re: google_sitemap , error on line 1 at column 10834: xmlParseEntityRef: no name

pson

New member

Posts

Joined

Wed Jan 18, 2012 2:06 am

Post by pson » Thu Jul 30, 2020 8:48 pm

ADD Creative wrote: ↑
Thu Jul 30, 2020 8:25 pm
What version of OpenCart? Any modifications?

It's probably an unescaped & in your product data somewhere.

Thanks, I am using Version 3.0.3.2 , is there any easy way to find the & ?
I do have a lot of extensions , and I am using SEO generator, maybe the problem is from that extension

Mvh
Pson

pson

New member

Posts

Joined

Wed Jan 18, 2012 2:06 am

Re: google_sitemap , error on line 1 at column 10834: xmlParseEntityRef: no name

letxobnav

Expert Member

Posts

3264

Joined

Fri Aug 18, 2017 4:35 pm

Location

Taiwan

Post by letxobnav » Thu Jul 30, 2020 9:04 pm

well, you can just browse to:

Code: Select all

https://www.qprod.se/index.php?route=extension/feed/google_sitemap

and view source to see the xml, search that for the & characters.
you can also just use:

Code: Select all

$product['name'] = htmlspecialchars($product['name'], ENT_COMPAT | ENT_XML1);

when generating the sitemap.

and since you have about 29000 urls in there, many useless as they are covered by canonical urls and will never be indexed, you might want to cache that.

letxobnav

Expert Member

Posts

3264

Joined

Fri Aug 18, 2017 4:35 pm

Location - Taiwan

Re: google_sitemap , error on line 1 at column 10834: xmlParseEntityRef: no name

ADD Creative

Guru Member

Posts

5242

Joined

Sat Jan 14, 2012 1:02 am

Location

United Kingdom

Post by ADD Creative » Fri Jul 31, 2020 12:58 am

pson wrote: ↑
Thu Jul 30, 2020 8:48 pm

ADD Creative wrote: ↑
Thu Jul 30, 2020 8:25 pm
What version of OpenCart? Any modifications?

It's probably an unescaped & in your product data somewhere.
Thanks, I am using Version 3.0.3.2 , is there any easy way to find the & ?
I do have a lot of extensions , and I am using SEO generator, maybe the problem is from that extension

You could directly search your database product_description name column using phpMyAdmin. Use Like %...% and search for & with space each side. Or you could view source on your sitemap in a browser and search for any & in that. Note you also have some &amp without a ;.

I've guessing that some of your products have been imported into OpenCart. It's a common problem to forget to escape with htmlspecialchars before entering into the database.

www.add-creative.co.uk

ADD Creative

Guru Member

Posts

5242

Joined

Sat Jan 14, 2012 1:02 am

Location - United Kingdom

Re: google_sitemap , error on line 1 at column 10834: xmlParseEntityRef: no name

pson

New member

Posts

Joined

Wed Jan 18, 2012 2:06 am

Post by pson » Fri Jul 31, 2020 3:06 am

ADD Creative wrote: ↑
Fri Jul 31, 2020 12:58 am

pson wrote: ↑
Thu Jul 30, 2020 8:48 pm

ADD Creative wrote: ↑
Thu Jul 30, 2020 8:25 pm
What version of OpenCart? Any modifications?

It's probably an unescaped & in your product data somewhere.
Thanks, I am using Version 3.0.3.2 , is there any easy way to find the & ?
I do have a lot of extensions , and I am using SEO generator, maybe the problem is from that extension
You could directly search your database product_description name column using phpMyAdmin. Use Like %...% and search for & with space each side. Or you could view source on your sitemap in a browser and search for any & in that. Note you also have some &amp without a ;.

I've guessing that some of your products have been imported into OpenCart. It's a common problem to forget to escape with htmlspecialchars before entering into the database.

Thanks , ok, my products are "live" through a ftp feed so if I make the changes using phpMyAdmin i will probably have the problem later again .

Mvh
Pson

pson

New member

Posts

Joined

Wed Jan 18, 2012 2:06 am

Re: google_sitemap , error on line 1 at column 10834: xmlParseEntityRef: no name

pson

New member

Posts

Joined

Wed Jan 18, 2012 2:06 am

Post by pson » Fri Jul 31, 2020 3:12 am

letxobnav wrote: ↑
Thu Jul 30, 2020 9:04 pm
well, you can just browse to:
Code: Select all
https://www.qprod.se/index.php?route=extension/feed/google_sitemap
and view source to see the xml, search that for the & characters.
you can also just use:
Code: Select all
$product['name'] = htmlspecialchars($product['name'], ENT_COMPAT | ENT_XML1);
when generating the sitemap.

and since you have about 29000 urls in there, many useless as they are covered by canonical urls and will never be indexed, you might want to cache that.

Thanks , unfortunately this is not my strongest side

, do I add this

Code: Select all

$product['name'] = htmlspecialchars($product['name'], ENT_COMPAT | ENT_XML1);

in:
catalog/controller/extension/feed/google_sitemap.php ?

Mvh
Pson

pson

New member

Posts

Joined

Wed Jan 18, 2012 2:06 am

Re: google_sitemap , error on line 1 at column 10834: xmlParseEntityRef: no name

ADD Creative

Guru Member

Posts

5242

Joined

Sat Jan 14, 2012 1:02 am

Location

United Kingdom

Post by ADD Creative » Fri Jul 31, 2020 3:55 am

At the two line here.
https://github.com/opencart/opencart/bl ... hp#L22-L23

Replace.

Code: Select all

$product['name']

With.

Code: Select all

htmlspecialchars($product['name'], ENT_COMPAT | ENT_XML1, 'UTF-8')

However, you may be better fixing the code that imports into the database.

www.add-creative.co.uk

ADD Creative

Guru Member

Posts

5242

Joined

Sat Jan 14, 2012 1:02 am

Location - United Kingdom

Re: google_sitemap , error on line 1 at column 10834: xmlParseEntityRef: no name

pson

New member

Posts

Joined

Wed Jan 18, 2012 2:06 am

Post by pson » Fri Jul 31, 2020 4:42 am

ADD Creative wrote: ↑
Fri Jul 31, 2020 3:55 am
At the two line here.
https://github.com/opencart/opencart/bl ... hp#L22-L23

Replace.
Code: Select all
$product['name']
With.
Code: Select all
htmlspecialchars($product['name'], ENT_COMPAT | ENT_XML1, 'UTF-8')
However, you may be better fixing the code that imports into the database.

Many many thanks

, now it seems better

This XML file does not appear to have any style information associated with it. The document tree is shown below.
<urlset xmlns="http://www.sitemaps.org/schemas/sitemap/0.9" xmlns:image="http://www.google.com/schemas/sitemap-image/1.1">
<url>
<loc>https://www.qprod.se/</loc>
<changefreq>weekly</changefreq>
<priority>1.0</priority>
</url>
<url>
<loc>https://www.qprod.se/1-rca-1-rca-hane-hane-15m</loc>
<changefreq>weekly</changefreq>
<lastmod>2020-07-30T22:10:02+02:00</lastmod>
<priority>1.0</priority>
<image:image>
<image:loc>https://www.qprod.se/image/cache/catalo ... /image:loc>
<image:caption>1 RCA - 1 RCA Hane-Hane 1.5m</image:caption>
<image:title>1 RCA - 1 RCA Hane-Hane 1.5m</image:title>
</image:image>
</url>
<url>
<loc>https://www.qprod.se/1-rca-2-rca-hane-honor</loc>
<changefreq>weekly</changefreq>
<lastmod>2020-07-30T22:10:02+02:00</lastmod>
<priority>1.0</priority>
<image:image>
<image:loc>https://www.qprod.se/image/cache/catalo ... /image:loc>
<image:caption>1 RCA - 2 RCA Hane-Honor</image:caption>
<image:title>1 RCA - 2 RCA Hane-Honor</image:title>
</image:image>
</url>

Mvh
Pson

pson

New member

Posts

Joined

Wed Jan 18, 2012 2:06 am

Re: google_sitemap , error on line 1 at column 10834: xmlParseEntityRef: no name

letxobnav

Expert Member

Posts

3264

Joined

Fri Aug 18, 2017 4:35 pm

Location

Taiwan

Post by letxobnav » Fri Jul 31, 2020 7:18 am

better get rid of:

Code: Select all

				$products = $this->model_catalog_product->getProducts(array('filter_manufacturer_id' => $manufacturer['manufacturer_id']));

				foreach ($products as $product) {
					$output .= '<url>';
					$output .= '  <loc>' . $this->url->link('product/product', 'manufacturer_id=' . $manufacturer['manufacturer_id'] . '&product_id=' . $product['product_id']) . '</loc>';
					$output .= '  <changefreq>weekly</changefreq>';
					$output .= '  <priority>1.0</priority>';
					$output .= '</url>';
				}

and:

Code: Select all

			$products = $this->model_catalog_product->getProducts(array('filter_category_id' => $result['category_id']));

			foreach ($products as $product) {
				$output .= '<url>';
				$output .= '  <loc>' . $this->url->link('product/product', 'path=' . $new_path . '&product_id=' . $product['product_id']) . '</loc>';
				$output .= '  <changefreq>weekly</changefreq>';
				$output .= '  <priority>1.0</priority>';
				$output .= '</url>';
			}

as they will produce non-canonical urls to your product pages which have a canonical url tag.
In simple words, those urls are useless in your sitemap as they will be discarded and only make your sitemap unnecessary large.