Does it negatively affect indexing?

Question

Search Engine Optimization

Hello. There is one very exciting question.

In the product category, all products are displayed as expected. For each product, the name is a link to the product's internal page (site.ru/pizza/syrnaya).

In fact - I don't need that when clicking on the name - a person goes to the product page. It is necessary only for the link mass of the site. To view product information, I use modal windows.
Therefore, when clicking on the title - I kept the href parameter, but added onclick. Now it looks like this:

<a itemprop="url" href="https://site.ru/pizza/syrnaya" onclick="quickview(97); return false" class="product-title">Сырная</a>

When you click on the name, a modal window opens and there is no transition.
Question: Will this negatively affect indexing?

Even in the sitemap file, each product has 2 entries.

1 entry

<url>
<loc>https://test.ru/rolly/dve-tortili-s-ovoshchami</loc>
<changefreq>weekly</changefreq>
<lastmod>2019-06-18T22:22:33+03:00</lastmod>
<priority>1.0</priority>
<image:image>
<image:loc>https://test.ru/image/cache/catalog/demo/demo-cover/2432fdsfs-260x200.jpg</image:loc>
<image:caption>Две тортильи с овощами</image:caption>
<image:title>Две тортильи с овощами</image:title>
</image:image>
</url>

2 entry

<url>
<loc>https://test.ru/rolly/dve-tortili-s-ovoshchami</loc>
<changefreq>weekly</changefreq>
<priority>1.0</priority>
</url>

What's the point of a duplicate entry? Is that how it should be? Might be worth removing?

Thank you in advance for your help :-)

Answer 1

Yes, he will see everything through the
sitemap