dartiss/no-self-ping

WordPress plugin to prevent WordPress from sending pings to your own site

David Artiss - @dartiss

Hi, I'm David (he/him/his) 👋🏼 I'm a WordPress VIP Customer Success Support Lead.

WordPress VIP, part of Automattic, are enterprise WordPress and are the official digital experience platform from the makers of WordPress.com.

I'm a WordPress plugin developer, contributor and international speaker. I'm passionate about inclusive language and statistics. I'm also a regular, but average, gamer, having played poorly since the 1980s.

I'm also a massive geek - ask me about all things Apple, Marvel, Star Wars and Star Trek 🖖🏼

WordPress Plugins 🔌

Most of my repos are for my WordPress plugins, with many now archived and private. I work on these in my own time and take no donations for their continuation - I write and support them because I like doing so. For this reason, changes may be slow to be made but I will always try and provide support in the WordPress.org forums as quickly as I can. Indeed, that's where all support should be directed. Please use the repo's issues log to look for outstanding issues or to add new features that you'd like to see.

Thank you ♥️

WordPress Contributing 🤝

I contribute to WordPress, in varying degrees, to the following...

• Core: https://make.wordpress.org/core/
• Documentation: https://make.wordpress.org/docs/
• Polyglots: https://make.wordpress.org/polyglots/
• Support: https://make.wordpress.org/support/
• WordPress.TV: https://make.wordpress.org/tv/

Follow me 🕵

• All the socials: https://artiss.me
• Threads: @[email protected]
• Mastodon: @[email protected]
• LinkedIn: @[email protected]
• Personal site: https://artiss.blog
• Instagram: @[email protected]

Get in touch 🤙🏼

• Send a LinkedIn message: @[email protected]
• Use my Discord channel: https://discord.gg/GtwUUm
• DM me in the WordPress team Slack - I'm @dartiss

For ALL support questions about my plugins, please use the appropriate WordPress.org forum.

Gaming 🎮

• PSN user name: Darkstorm40
• Nintendo Switch friend code: SW-2993-1134-1615

no-self-ping's Issues

Validate my admin screen for accessibility

https://achecker.ca/checker/index.php

Add switch to settings

Add a switch for activating/deactivating no self pings.

On MU sites (is_multisite()), there should be an additional switch as to whether it applies to all sub-sites or just the current one.

Settings description not appearing

In Settings -> Discussion, I've added a new section named 'No Self Pings'. There should be a description below the title, as there is under the Comments section below it. But it's missing. Need to fix this.

Add link in plugin meta to settings

Move the settings

With regard to the settings - can I integrate it more fully into the existing settings screen, rather than at the bottom?

Here's the code to add the screen in... https://github.com/WordPress/WordPress/blob/master/wp-admin/options-discussion.php

Is there a way for me to hook into the right place?

Fix strict PHPCS issues

FILE: /Users/davidartiss/Documents/Github/no-self-ping/no-self-pings.php
--------------------------------------------------------------------------------
FOUND 4 ERRORS AND 1 WARNING AFFECTING 4 LINES
--------------------------------------------------------------------------------
  81 | ERROR   | Use Yoda Condition checks, you must.
 100 | ERROR   | The $text arg must be a single string literal, not "ucwords(
     |         | 'No Self Pings' )".
 102 | ERROR   | The $text arg must be a single string literal, not "ucwords(
     |         | 'Additional URLs' )".
 119 | WARNING | A gettext call containing placeholders was found, but was not
     |         | accompanied by a "translators:" comment on the line above to
     |         | clarify the meaning of the placeholders.
 119 | ERROR   | The $text arg must be a single string literal, not "sprintf(
     |         | 'By default, No Self Pings will exclude pings for this site
     |         | (%s) but you can supply additional URLs below. Separate
     |         | multiple URLs with line breaks.', esc_url( home_url() ) )".
--------------------------------------------------------------------------------

Automatically generate sub-site links

I the last release I added a settings box so that you could add extra URLs - this was intended for sub-site use.

Instead, I will generate this list automatically, removing this field.

Add a feature pointer

Consider adding this so, upon activation, it will point to where the settings are.

Update meta

Update the code for adding meta in the code. Plus also update the meta header on the primary plugin file.

Feature request: Filter incoming pings by domain (disallow-list)

Hi, as I understand this it is preventing outgoing pings to the listed domains from being emitted. However could it be added to reject / trash incoming pings that match a disallow list?

eg using this hook https://developer.wordpress.org/reference/hooks/pre_comment_approved/

Pingbacks slipping through

I run a self-hosted multi-site installation of WordPress and have activated this plugin network-wide for some years. Recently I noticed in one of the blogs that there are pingbacks to the same blog. While some of the links did ping, others didn't though – even within the same blog post.
As I'm not the author of this blog I can only speculate about what happened, so this is more of a question: What happens, if I'm linking to https resources while the home URL uses http? What about relative links?

Fix PHPCS issues

FILE: /Users/davidartiss/documents/github/no-self-pings/no-self-pings.php
--------------------------------------------------------------------------------
FOUND 2 ERRORS AND 2 WARNINGS AFFECTING 4 LINES
--------------------------------------------------------------------------------
  78 | WARNING | Unused variable $u.
  81 | WARNING | Found: !=. Use strict comparisons (=== or !==).
 119 | ERROR   | All output should be run through an escaping function (like
     |         | esc_html_e() or esc_attr_e()), found '_e'.
 135 | ERROR   | All output should be run through an escaping function (see the
     |         | Security sections in the WordPress Developer Handbooks), found
     |         | '$urls'.
--------------------------------------------------------------------------------


FILE: /Users/davidartiss/documents/github/no-self-pings/uninstall.php
--------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
--------------------------------------------------------------------------
 20 | ERROR | [x] A closing tag is not permitted at the end of a PHP file
--------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------