Skip to content

Commit fb9c110

Browse files
fritzmgfritzmg
committed
add noopener noreferrer
1 parent 975d8a0 commit fb9c110

File tree

2 files changed

+20
-20
lines changed

2 files changed

+20
-20
lines changed

‎system/modules/sharebuttons/templates/sharebuttons_default.html5‎

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -4,28 +4,28 @@
44
<ul<?php if ($this->theme): ?> class="theme <?= $this->theme ?>"<?php endif ?>>
55
<?php foreach ($this->networks as $network): ?>
66
<?php if ($network == 'facebook'): ?>
7-
<li><a class="facebook" href="https://www.facebook.com/sharer/sharer.php?u=<?= $this->url ?>&amp;t=<?= $this->title ?>" rel="nofollow" data-skip-broken-link-checker title="<?= $this->lang['share_on_facebook'] ?>" onclick="return shd.open(this.href,600,300);">Facebook</a></li>
7+
<li><a class="facebook" href="https://www.facebook.com/sharer/sharer.php?u=<?= $this->url ?>&amp;t=<?= $this->title ?>" rel="noopener noreferrer nofollow" data-skip-broken-link-checker title="<?= $this->lang['share_on_facebook'] ?>" onclick="return shd.open(this.href,600,300);">Facebook</a></li>
88
<?php elseif ($network == 'twitter'): ?>
9-
<li><a class="twitter" href="https://twitter.com/intent/tweet?url=<?= $this->url ?>&amp;text=<?= $this->title ?>" rel="nofollow" data-skip-broken-link-checker title="<?= $this->lang['share_on_twitter'] ?>" onclick="return shd.open(this.href,500,260);">Twitter</a></li>
9+
<li><a class="twitter" href="https://twitter.com/intent/tweet?url=<?= $this->url ?>&amp;text=<?= $this->title ?>" rel="noopener noreferrer nofollow" data-skip-broken-link-checker title="<?= $this->lang['share_on_twitter'] ?>" onclick="return shd.open(this.href,500,260);">Twitter</a></li>
1010
<?php elseif ($network == 'linkedin'): ?>
11-
<li><a class="linkedin" href="http://www.linkedin.com/shareArticle?mini=true&amp;url=<?= $this->url ?>&amp;title=<?= $this->title ?>" rel="nofollow" data-skip-broken-link-checker title="<?= $this->lang['share_on_linkedin'] ?>" onclick="return shd.open(this.href,520,570);">LinkedIn</a></li>
11+
<li><a class="linkedin" href="http://www.linkedin.com/shareArticle?mini=true&amp;url=<?= $this->url ?>&amp;title=<?= $this->title ?>" rel="noopener noreferrer nofollow" data-skip-broken-link-checker title="<?= $this->lang['share_on_linkedin'] ?>" onclick="return shd.open(this.href,520,570);">LinkedIn</a></li>
1212
<?php elseif ($network == 'xing'): ?>
13-
<li><a class="xing" href="https://www.xing.com/social_plugins/share/new?sc_p=xing-share&amp;h=1&amp;url=<?= $this->url ?>" rel="nofollow" data-skip-broken-link-checker title="<?= $this->lang['share_on_xing'] ?>" onclick="return shd.open(this.href,460,460);">Xing</a></li>
13+
<li><a class="xing" href="https://www.xing.com/social_plugins/share/new?sc_p=xing-share&amp;h=1&amp;url=<?= $this->url ?>" rel="noopener noreferrer nofollow" data-skip-broken-link-checker title="<?= $this->lang['share_on_xing'] ?>" onclick="return shd.open(this.href,460,460);">Xing</a></li>
1414
<?php elseif ($network == 'mail'): ?>
1515
<li><a class="mail" href="mailto:?subject=<?= $this->lang['mail_subject'] ?>:%20<?= $this->title ?>&amp;body=<?= $this->url ?>">Mail</a></li>
1616
<?php elseif ($network == 'tumblr'): ?>
17-
<li><a class="tumblr" href="http://www.tumblr.com/share?v=3&amp;u=<?= $this->url ?>&amp;t=<?= $this->title ?>&amp;s=<?= $this->description ?>" rel="nofollow" data-skip-broken-link-checker title="<?= $this->lang['share_on_tumblr'] ?>" onclick="return shd.open(this.href,800,450);">tumblr</a></li>
17+
<li><a class="tumblr" href="http://www.tumblr.com/share?v=3&amp;u=<?= $this->url ?>&amp;t=<?= $this->title ?>&amp;s=<?= $this->description ?>" rel="noopener noreferrer nofollow" data-skip-broken-link-checker title="<?= $this->lang['share_on_tumblr'] ?>" onclick="return shd.open(this.href,800,450);">tumblr</a></li>
1818
<?php elseif ($network == 'pinterest' && $this->image): ?>
19-
<li><a class="pinterest" href="http://pinterest.com/pin/create/button/?url=<?= $this->url ?>&amp;media=<?= $this->image ?>&amp;description=<?= $this->description ?>" rel="nofollow" data-skip-broken-link-checker title="<?= $this->lang['share_on_pinterest'] ?>" onclick="return shd.open(this.href,460,460);">Pinterest</a></li>
19+
<li><a class="pinterest" href="http://pinterest.com/pin/create/button/?url=<?= $this->url ?>&amp;media=<?= $this->image ?>&amp;description=<?= $this->description ?>" rel="noopener noreferrer nofollow" data-skip-broken-link-checker title="<?= $this->lang['share_on_pinterest'] ?>" onclick="return shd.open(this.href,460,460);">Pinterest</a></li>
2020
<?php endif ;?>
2121
<?php if ($network == 'reddit'): ?>
22-
<li><a class="reddit" href="http://www.reddit.com/submit?url=<?= $this->url ?>&amp;title=<?= $this->title ?>" rel="nofollow" data-skip-broken-link-checker title="<?= $this->lang['share_on_reddit'] ?>" onclick="return shd.open(this.href,855,900);">Reddit</a></li>
22+
<li><a class="reddit" href="http://www.reddit.com/submit?url=<?= $this->url ?>&amp;title=<?= $this->title ?>" rel="noopener noreferrer nofollow" data-skip-broken-link-checker title="<?= $this->lang['share_on_reddit'] ?>" onclick="return shd.open(this.href,855,900);">Reddit</a></li>
2323
<?php elseif ($network == 'whatsapp' && \Environment::get('agent')->mobile): ?>
24-
<li><a class="whatsapp" href="whatsapp://send?text=<?= $this->description ? $this->description . '%0A%0A' : '' ?><?= $this->url ?>" data-action="share/whatsapp/share" rel="nofollow" data-skip-broken-link-checker title="<?= $this->lang['share_on_whatsapp'] ?>">WhatsApp</a></li>
24+
<li><a class="whatsapp" href="whatsapp://send?text=<?= $this->description ? $this->description . '%0A%0A' : '' ?><?= $this->url ?>" data-action="share/whatsapp/share" rel="noopener noreferrer nofollow" data-skip-broken-link-checker title="<?= $this->lang['share_on_whatsapp'] ?>">WhatsApp</a></li>
2525
<?php elseif ($network == 'print'): ?>
26-
<li><a class="print" href="#" rel="nofollow" data-skip-broken-link-checker title="<?= $this->lang['print_page'] ?>" onclick="window.print();return false"><?= $this->lang['print_page'] ?></a></li>
26+
<li><a class="print" href="#" rel="noopener noreferrer nofollow" data-skip-broken-link-checker title="<?= $this->lang['print_page'] ?>" onclick="window.print();return false"><?= $this->lang['print_page'] ?></a></li>
2727
<?php elseif ($network == 'pdf' && $this->pdfLink): ?>
28-
<li><a class="pdf" href="<?= $this->pdfLink ?>" rel="nofollow" data-skip-broken-link-checker title="<?= $this->lang['create_pdf'] ?>"><?= $this->lang['create_pdf'] ?></a></li>
28+
<li><a class="pdf" href="<?= $this->pdfLink ?>" rel="noopener noreferrer nofollow" data-skip-broken-link-checker title="<?= $this->lang['create_pdf'] ?>"><?= $this->lang['create_pdf'] ?></a></li>
2929
<?php endif; ?>
3030
<?php endforeach; ?>
3131
</ul>

‎system/modules/sharebuttons/templates/sharebuttons_fontawesome.html5‎

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -4,28 +4,28 @@
44
<ul<?php if ($this->theme): ?> class="theme <?= $this->theme ?>"<?php endif ?>>
55
<?php foreach ($this->networks as $network): ?>
66
<?php if ($network == 'facebook'): ?>
7-
<li><a class="facebook fa fa-facebook-official" href="https://www.facebook.com/sharer/sharer.php?u=<?= $this->url ?>&amp;t=<?= $this->title ?>" rel="nofollow" data-skip-broken-link-checker title="<?= $this->lang['share_on_facebook'] ?>" onclick="return shd.open(this.href,600,300);"></a></li>
7+
<li><a class="facebook fa fa-facebook-official" href="https://www.facebook.com/sharer/sharer.php?u=<?= $this->url ?>&amp;t=<?= $this->title ?>" rel="noopener noreferrer nofollow" data-skip-broken-link-checker title="<?= $this->lang['share_on_facebook'] ?>" onclick="return shd.open(this.href,600,300);"></a></li>
88
<?php elseif ($network == 'twitter'): ?>
9-
<li><a class="twitter fa fa-twitter" href="https://twitter.com/intent/tweet?url=<?= $this->url ?>&amp;text=<?= $this->title ?>" rel="nofollow" data-skip-broken-link-checker title="<?= $this->lang['share_on_twitter'] ?>" onclick="return shd.open(this.href,500,260);"></a></li>
9+
<li><a class="twitter fa fa-twitter" href="https://twitter.com/intent/tweet?url=<?= $this->url ?>&amp;text=<?= $this->title ?>" rel="noopener noreferrer nofollow" data-skip-broken-link-checker title="<?= $this->lang['share_on_twitter'] ?>" onclick="return shd.open(this.href,500,260);"></a></li>
1010
<?php elseif ($network == 'linkedin'): ?>
11-
<li><a class="linkedin fa fa-linkedin" href="http://www.linkedin.com/shareArticle?mini=true&amp;url=<?= $this->url ?>&amp;title=<?= $this->title ?>" rel="nofollow" data-skip-broken-link-checker title="<?= $this->lang['share_on_linkedin'] ?>" onclick="return shd.open(this.href,520,570);"></a></li>
11+
<li><a class="linkedin fa fa-linkedin" href="http://www.linkedin.com/shareArticle?mini=true&amp;url=<?= $this->url ?>&amp;title=<?= $this->title ?>" rel="noopener noreferrer nofollow" data-skip-broken-link-checker title="<?= $this->lang['share_on_linkedin'] ?>" onclick="return shd.open(this.href,520,570);"></a></li>
1212
<?php elseif ($network == 'xing'): ?>
13-
<li><a class="xing fa fa-xing" href="https://www.xing.com/social_plugins/share/new?sc_p=xing-share&amp;h=1&amp;url=<?= $this->url ?>" rel="nofollow" data-skip-broken-link-checker title="<?= $this->lang['share_on_xing'] ?>" onclick="return shd.open(this.href,460,460);"></a></li>
13+
<li><a class="xing fa fa-xing" href="https://www.xing.com/social_plugins/share/new?sc_p=xing-share&amp;h=1&amp;url=<?= $this->url ?>" rel="noopener noreferrer nofollow" data-skip-broken-link-checker title="<?= $this->lang['share_on_xing'] ?>" onclick="return shd.open(this.href,460,460);"></a></li>
1414
<?php elseif ($network == 'mail'): ?>
1515
<li><a class="mail fa fa-envelope" href="mailto:?subject=<?= $this->lang['mail_subject'] ?>:%20<?= $this->title ?>&amp;body=<?= $this->url ?>"></a></li>
1616
<?php elseif ($network == 'tumblr'): ?>
17-
<li><a class="tumblr fa fa-tumblr" href="http://www.tumblr.com/share?v=3&amp;u=<?= $this->url ?>&amp;t=<?= $this->title ?>&amp;s=<?= $this->description ?>" rel="nofollow" data-skip-broken-link-checker title="<?= $this->lang['share_on_tumblr'] ?>" onclick="return shd.open(this.href,800,450);"></a></li>
17+
<li><a class="tumblr fa fa-tumblr" href="http://www.tumblr.com/share?v=3&amp;u=<?= $this->url ?>&amp;t=<?= $this->title ?>&amp;s=<?= $this->description ?>" rel="noopener noreferrer nofollow" data-skip-broken-link-checker title="<?= $this->lang['share_on_tumblr'] ?>" onclick="return shd.open(this.href,800,450);"></a></li>
1818
<?php elseif ($network == 'pinterest' && $this->image): ?>
19-
<li><a class="pinterest fa fa-pinterest" href="http://pinterest.com/pin/create/button/?url=<?= $this->url ?>&amp;media=<?= $this->image ?>&amp;description=<?= $this->description ?>" rel="nofollow" data-skip-broken-link-checker title="<?= $this->lang['share_on_pinterest'] ?>" onclick="return shd.open(this.href,460,460);"></a></li>
19+
<li><a class="pinterest fa fa-pinterest" href="http://pinterest.com/pin/create/button/?url=<?= $this->url ?>&amp;media=<?= $this->image ?>&amp;description=<?= $this->description ?>" rel="noopener noreferrer nofollow" data-skip-broken-link-checker title="<?= $this->lang['share_on_pinterest'] ?>" onclick="return shd.open(this.href,460,460);"></a></li>
2020
<?php endif ;?>
2121
<?php if ($network == 'reddit'): ?>
22-
<li><a class="reddit fa fa-reddit" href="http://www.reddit.com/submit?url=<?= $this->url ?>&amp;title=<?= $this->title ?>" rel="nofollow" data-skip-broken-link-checker title="<?= $this->lang['share_on_reddit'] ?>" onclick="return shd.open(this.href,855,900);"></a></li>
22+
<li><a class="reddit fa fa-reddit" href="http://www.reddit.com/submit?url=<?= $this->url ?>&amp;title=<?= $this->title ?>" rel="noopener noreferrer nofollow" data-skip-broken-link-checker title="<?= $this->lang['share_on_reddit'] ?>" onclick="return shd.open(this.href,855,900);"></a></li>
2323
<?php elseif ($network == 'whatsapp' && \Environment::get('agent')->mobile): ?>
24-
<li><a class="whatsapp fa fa-whatsapp" href="whatsapp://send?text=<?= $this->description ? $this->description . '%0A%0A' : '' ?><?= $this->url ?>" data-action="share/whatsapp/share" rel="nofollow" data-skip-broken-link-checker title="<?= $this->lang['share_on_whatsapp'] ?>"></a></li>
24+
<li><a class="whatsapp fa fa-whatsapp" href="whatsapp://send?text=<?= $this->description ? $this->description . '%0A%0A' : '' ?><?= $this->url ?>" data-action="share/whatsapp/share" rel="noopener noreferrer nofollow" data-skip-broken-link-checker title="<?= $this->lang['share_on_whatsapp'] ?>"></a></li>
2525
<?php elseif ($network == 'print'): ?>
26-
<li><a class="print fa fa-print" href="#" rel="nofollow" data-skip-broken-link-checker title="<?= $this->lang['print_page'] ?>" onclick="window.print();return false"></a></li>
26+
<li><a class="print fa fa-print" href="#" rel="noopener noreferrer nofollow" data-skip-broken-link-checker title="<?= $this->lang['print_page'] ?>" onclick="window.print();return false"></a></li>
2727
<?php elseif ($network == 'pdf' && $this->pdfLink): ?>
28-
<li><a class="pdf fa fa-file-pdf-o fa-file-pdf" href="<?= $this->pdfLink ?>" rel="nofollow" data-skip-broken-link-checker title="<?= $this->lang['create_pdf'] ?>"></a></li>
28+
<li><a class="pdf fa fa-file-pdf-o fa-file-pdf" href="<?= $this->pdfLink ?>" rel="noopener noreferrer nofollow" data-skip-broken-link-checker title="<?= $this->lang['create_pdf'] ?>"></a></li>
2929
<?php endif; ?>
3030
<?php endforeach; ?>
3131
</ul>

0 commit comments

Comments
 (0)