search

fritzmg / contao-sharebuttons

Simple extension to provide share buttons as a module and content element in Contao
GNU Lesser General Public License v3.0
18 stars 2 forks source link

URL shows domain twice #31

Closed thmsklngr closed 7 years ago

thmsklngr commented 7 years ago

I just added this module to my webseite and added <?= $this->sharebuttons ?> to my template. The share buttons are shown as configured in the news archive, however, the links for sharing are built a little bit odd.

Example: http://www.breitseitcomics.at/share/index.php?p=facebook&u=http%3A%2F%2Fwww.breitseitcomics.at%2Fhttp%3A%2F%2Fwww.breitseitcartoons.at%2Fhome%2Fkunst-und-geschichte.html&t=Kunst%20und%20Geschichte

Notice the double domain name in the URL, which is provided through the template vars.

Any ideas?

Regards, Thomas

fritzmg commented 7 years ago

Hm, I will have a look at it. URL generation has changed in later Contao versions, so this might be why this is happening.

Are only the news URLs affected or also the URLs when you use the Share Buttons content element?

thmsklngr commented 7 years ago

Hallo Fritz,

bin Österreicher. :) Ob das jetzt nur die URLs in den Neuigkeiten betrifft oder auch andere, kann ich leider nicht sagen. Ich habe es jetzt gerade mal in einem News-Template eingebaut, dort ist mir das aufgefallen.

Beste Grüße,

--  Thomas Klinger Dr. Karl Sablik-Straße 22 A-2104 Spillern M: +43-699-11894386 E: thomas.klinger@me.com

Am 15. März 2017 um 19:49:27, Fritz Michael Gschwantner (notifications@github.com) schrieb:

Hm, I will have a look at it. URL generation has changed in later Contao versions, so this might be why this is happening.

Are only the news URLs affected or also the URLs when you use the Share Buttons content element?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or mute the thread.

fritzmg commented 7 years ago

Should be fixed in c4f34ac