Changeset 3475895

Timestamp:

03/05/2026 08:03:50 PM (3 weeks ago)

Author:

apos37

Message:

1.3.7

• Update: Changed storage of broken links from custom post type to custom database table
• Update: Add option to cleanup on uninstall
• Fix: Error when blnotifier_link_before_prechecks hook returns status

Location:

broken-link-notifier

Files:

• tags/1.2.0 (deleted)
• tags/1.2.1 (deleted)
• tags/1.2.2 (deleted)
• tags/1.2.3 (deleted)
• tags/1.2.4 (deleted)
• tags/1.2.4.1 (deleted)
• tags/1.2.4.2 (deleted)
• tags/1.2.5 (deleted)
• tags/1.2.5.1 (deleted)
• tags/1.2.5.2 (deleted)
• tags/1.2.5.3 (deleted)
• tags/1.2.6 (deleted)
• tags/1.2.6.1 (deleted)
• tags/1.3.1 (deleted)
• tags/1.3.7 (added)
• tags/1.3.7/broken-link-notifier.php (added)
• tags/1.3.7/includes (added)
• tags/1.3.7/includes/cache.php (added)
• tags/1.3.7/includes/css (added)
• tags/1.3.7/includes/css/results-front.css (added)
• tags/1.3.7/includes/css/results-front.min.css (added)
• tags/1.3.7/includes/discord.php (added)
• tags/1.3.7/includes/example-hook.php (added)
• tags/1.3.7/includes/export.php (added)
• tags/1.3.7/includes/helpers.php (added)
• tags/1.3.7/includes/img (added)
• tags/1.3.7/includes/img/admin-help-docs.png (added)
• tags/1.3.7/includes/img/clear-cache-everywhere.png (added)
• tags/1.3.7/includes/img/dev-debug-tools.png (added)
• tags/1.3.7/includes/img/devconsole.gif (added)
• tags/1.3.7/includes/img/discord.png (added)
• tags/1.3.7/includes/img/eri-file-library.png (added)
• tags/1.3.7/includes/img/gf-discord.png (added)
• tags/1.3.7/includes/img/gf-msteams.png (added)
• tags/1.3.7/includes/img/gf-tools.png (added)
• tags/1.3.7/includes/img/gravity-zwr.png (added)
• tags/1.3.7/includes/img/logo-teal.png (added)
• tags/1.3.7/includes/img/logo-transparent.png (added)
• tags/1.3.7/includes/img/screen_options.png (added)
• tags/1.3.7/includes/index.php (added)
• tags/1.3.7/includes/integrations.php (added)
• tags/1.3.7/includes/js (added)
• tags/1.3.7/includes/js/index.php (added)
• tags/1.3.7/includes/js/omits.js (added)
• tags/1.3.7/includes/js/results-back.js (added)
• tags/1.3.7/includes/js/results-front.js (added)
• tags/1.3.7/includes/js/results-front.min.js (added)
• tags/1.3.7/includes/js/scan-multi.js (added)
• tags/1.3.7/includes/js/scan-single.js (added)
• tags/1.3.7/includes/js/settings.js (added)
• tags/1.3.7/includes/loader.php (added)
• tags/1.3.7/includes/menu.php (added)
• tags/1.3.7/includes/msteams.php (added)
• tags/1.3.7/includes/omits.php (added)
• tags/1.3.7/includes/page-export.php (added)
• tags/1.3.7/includes/page-help.php (added)
• tags/1.3.7/includes/page-link-search.php (added)
• tags/1.3.7/includes/page-results.php (added)
• tags/1.3.7/includes/page-scan-multi.php (added)
• tags/1.3.7/includes/page-scan-single.php (added)
• tags/1.3.7/includes/page-settings.php (added)
• tags/1.3.7/includes/page.php (added)
• tags/1.3.7/includes/results.php (added)
• tags/1.3.7/includes/scan-multi.php (added)
• tags/1.3.7/includes/scan.php (added)
• tags/1.3.7/includes/test_files (added)
• tags/1.3.7/includes/test_files/Jubiläum-A-und-B.pdf (added)
• tags/1.3.7/index.php (added)
• tags/1.3.7/readme.txt (added)
• tags/1.3.7/uninstall.php (added)
• trunk/broken-link-notifier.php (modified) (1 diff)
• trunk/includes/cache.php (modified) (1 diff)
• trunk/includes/example-hook.php (modified) (1 diff)
• trunk/includes/helpers.php (modified) (5 diffs)
• trunk/includes/js/omits.js (modified) (1 diff)
• trunk/includes/js/results-back.js (modified) (21 diffs)
• trunk/includes/menu.php (modified) (4 diffs)
• trunk/includes/omits.php (modified) (3 diffs)
• trunk/includes/page-results.php (added)
• trunk/includes/results.php (modified) (24 diffs)
• trunk/readme.txt (modified) (2 diffs)
• trunk/uninstall.php (added)

broken-link-notifier/trunk/broken-link-notifier.php

@@ -4 +4 @@
  * Plugin URI:          https://pluginrx.com/plugin/broken-link-notifier/
  * Description:         Get notified when someone loads a page with a broken link
- * Version:             1.3.6
+ * Version:             1.3.7
  * Requires at least:   5.9
  * Tested up to:        6.9

broken-link-notifier/trunk/includes/cache.php

@@ -18 +18 @@
 
 /**
- * Main plugin class.
+ * Cache class.
  */
 class BLNOTIFIER_CACHE {

broken-link-notifier/trunk/includes/example-hook.php

@@ -3 +3 @@
 
 /**
- * An example of how to filter a link before it's checked for validity
+ * Skip links containing a specific query string before link checks.
  *
- * @param string $link
- * @return string|false
+ * @param string|array $link The original link to be checked.
+ * @return string|array Modified link or structured status array.
  */
-function blnotifier_link( $link ) {
-    // Filter ajax calls
-    if ( strpos( $link, '/admin-ajax.php' ) !== false ) {
-
-        // Parse the url (ie. https://mydomain.com/wp-admin/admin-ajax.php?action=my_action&post_id=12345&nonce=1234567890)
-        $url_components = wp_parse_url( $link );
-        $query_strings = explode( '&', $url_components[ 'query' ] );
-        $action = false;
-        $post_id = false;
-        foreach ( $query_strings as $query_string ) {
-            $query_string = str_replace( 'amp;', '', $query_string );
-            if ( substr( $query_string, 0, strlen( 'action=' ) ) == 'action=' ) {
-                $action = substr( $query_string, strlen( 'action=' ) );
-            } 
-            if ( substr( $query_string, 0, strlen( 'post_id=' ) ) == 'post_id=' ) {
-                $post_id = substr( $query_string, strlen( 'post_id=' ) );
-            }
-        }
-
-        // Check the action and make sure we have a post id
-        if ( $action == 'my_action' && $post_id ) {
-
-            // Check the post it's associated with
-            $meta_value = get_post_meta( $post_id, 'meta_key', true );
-
-            // New url to pass through all the checks
-            if ( $meta_value && $meta_value == 'some keyword' ) {
-                $link = 'https://somewebsite.com/'.$meta_value;
-            }
-
-        // Or simply return a status
-        } elseif ( $action == 'my_other_action' ) {
-
-            // Condition
-            if ( get_post( $post_id ) ) {
-                $status = [
-                    'type' => 'good',
-                    'code' => 200,
-                    'text' => 'Post exists: '.get_the_title( $post_id ),
-                    'link' => $link
-                ];
-            } else {
-                $status = [
-                    'type' => 'broken',
-                    'code' => 666, // We use 666 as an alternative to 0 in case warnings are disabled
-                    'text' => 'Post does not exist',
-                    'link' => $link
-                ];
-            }
-            return $status;
-        }
+add_filter( 'blnotifier_link_before_prechecks', function( $link ) {
+    // Skip UTM tracking links
+    if ( is_string( $link ) && strpos( $link, 'utm_' ) !== false ) {
+        return [
+            'type' => 'good',
+            'code' => 200,
+            'text' => 'Skipped due to tracking params',
+            'link' => $link
+        ];
     }
 
-    // Always return link
     return $link;
-} // End blnotifier_link()
-
-add_filter( 'blnotifier_link_before_prechecks', 'blnotifier_link' );
+}, 10 );

broken-link-notifier/trunk/includes/helpers.php

@@ -200 +200 @@
             return [];
         }
-
-        unset( $post_types[ ( new BLNOTIFIER_RESULTS )->post_type ] );
 
         if ( isset( $post_types[ 'help-docs' ] ) ) {
@@ -405 +403 @@
      */
     public function count_broken_links() {
-        $broken_links = get_posts( [
-            'posts_per_page'    => -1,
-            'post_status'       => 'publish',
-            'post_type'         => 'blnotifier-results',
-            'meta_query' => [ // phpcs:ignore WordPress.DB.SlowDBQuery.slow_db_query_meta_query
-                [
-                    'key'   => 'type',
-                    'value' => 'broken',
-                ]
-            ],
-            'fields' => 'ids'
-        ] );
-        return count( $broken_links );
+        global $wpdb;
+        $table = $wpdb->prefix . 'blnotifier_results';
+
+        $count = $wpdb->get_var(
+            $wpdb->prepare(
+                "SELECT COUNT(*) FROM {$table} WHERE type = %s",
+                'broken'
+            )
+        );
+
+        return absint( $count );
     } // End count_broken_links()
 
@@ -1563 +1559 @@
 
         // String replace
-        $link = $this->str_replace_on_link( $link );
+        if ( is_array( $link ) && isset( $link[ 'link' ] ) ) {
+            $link[ 'link' ] = $this->str_replace_on_link( $link[ 'link' ] );
+        } elseif ( is_string( $link ) ) {
+            $link = $this->str_replace_on_link( $link );
+        }
 
         // Check for cached link only if the link is not an array
@@ -1800 +1800 @@
      */
     public function get_broken_links() {
+        global $wpdb;
+        $table = $wpdb->prefix . 'blnotifier_results';
+
+        $rows = $wpdb->get_results(
+            $wpdb->prepare(
+                "SELECT link, text, type, code, source, location, method, author_id, created_at FROM {$table} WHERE type = %s ORDER BY created_at DESC",
+                'broken'
+            ),
+            ARRAY_A
+        );
+
         $links = [];
-
-        $results = new BLNOTIFIER_RESULTS();
-        $post_type = $results->post_type;
-
-        $posts = get_posts( [
-            'post_type'      => $post_type,
-            'posts_per_page' => -1,
-            'post_status'    => 'publish',
-            'orderby'        => 'date',
-            'order'          => 'DESC',
-        ] );
+        $source_cache = []; // stores [ 'post_id' => ..., 'title' => ... ] for each source URL
 
         $methods = [
-            'visit'   => __( 'Front-End Visit', 'broken-link-notifier' ),
-            'multi'   => __( 'Multi-Scan', 'broken-link-notifier' ),
-            'single'  => __( 'Page Scan', 'broken-link-notifier' ),
+            'visit'  => __( 'Front-End Visit', 'broken-link-notifier' ),
+            'multi'  => __( 'Multi-Scan', 'broken-link-notifier' ),
+            'single' => __( 'Page Scan', 'broken-link-notifier' ),
         ];
 
-        foreach ( $posts as $post ) {
-            $source_link = sanitize_url( get_post_meta( $post->ID, 'source', true ) );
-            $location = sanitize_key( get_post_meta( $post->ID, 'location', true ) );
-            $type_meta = sanitize_key( get_post_meta( $post->ID, 'type', true ) );
-            $code = absint( get_post_meta( $post->ID, 'code', true ) );
-            $user_id = $post->post_author;
-            $user = $user_id ? get_userdata( $user_id ) : null;
-            $method = sanitize_key( get_post_meta( $post->ID, 'method', true ) );
+        foreach ( $rows as $row ) {
+
+            $source_url = $row[ 'source' ];
+
+            if ( isset( $source_cache[ $source_url ] ) ) {
+                $source_post_id = $source_cache[ $source_url ][ 'post_id' ];
+                $source_title   = $source_cache[ $source_url ][ 'title' ];
+            } else {
+                $source_post_id = url_to_postid( $source_url );
+                $source_title   = $source_post_id ? get_the_title( $source_post_id ) : __( 'Unknown', 'broken-link-notifier' );
+                $source_cache[ $source_url ] = [
+                    'post_id' => $source_post_id,
+                    'title'   => $source_title,
+                ];
+            }
+
+            $user = $row[ 'author_id' ] ? get_userdata( $row[ 'author_id' ] ) : null;
 
             $links[] = [
-                'date'        => gmdate( 'Y-m-d H:i:s', strtotime( $post->post_date ) ),
-                'type'        => ucwords( $type_meta ),
-                'code'        => $code,
-                'message'     => $post->post_content,
-                'link'        => $post->post_title,
-                'source_name' => $source_link ? get_the_title( url_to_postid( $source_link ) ) : 'Unknown',
-                'source_link' => $source_link,
-                'location'    => ucwords( $location ),
-                'user'        => $user ? $user->display_name : 'Guest',
-                'method'      => isset( $methods[ $method ] ) ? $methods[ $method ] : __( 'Unknown', 'broken-link-notifier' ),
+                'date'        => mysql2date( 'Y-m-d H:i:s', $row[ 'created_at' ] ),
+                'type'        => ucwords( sanitize_text_field( $row[ 'type' ] ) ),
+                'code'        => absint( $row[ 'code' ] ),
+                'message'     => sanitize_text_field( $row[ 'text' ] ),
+                'link'        => esc_url( $row[ 'link' ] ),
+                'source_name' => $source_title,
+                'source_link' => esc_url( $source_url ),
+                'location'    => ucwords( sanitize_key( $row[ 'location' ] ) ),
+                'user'        => $user ? $user->display_name : __( 'Guest', 'broken-link-notifier' ),
+                'method'      => isset( $methods[ $row[ 'method' ] ] ) ? $methods[ $row[ 'method' ] ] : __( 'Unknown', 'broken-link-notifier' ),
             ];
         }
@@ -1844 +1854 @@
         usort( $links, function( $a, $b ) {
             $date_compare = strtotime( $b[ 'date' ] ) <=> strtotime( $a[ 'date' ] );
-        
             if ( $date_compare === 0 ) {
                 return strcasecmp( $a[ 'link' ], $b[ 'link' ] );
             }
-        
             return $date_compare;
-        } );
+        });
 
         return $links;

broken-link-notifier/trunk/includes/js/omits.js

@@ -7 +7 @@
     // Scan type
     const scanType = blnotifier_omit.scan_type;
+    const omitEl = scanType == 'scan-results' ? '.omit-link a' : '.omit-link';
 
     // Listen for omitting links
-    $( '.omit-link' ).on( 'click', function( e ) {
+    $( omitEl ).on( 'click', function( e ) {
         e.preventDefault();
         var row;
         var link;
         if ( scanType == 'scan-results' ) {
-            const postID = $( this ).data( 'post-id' );
-            $( `#post-${postID}` ).addClass( 'omitted' );
+            const linkID = $( this ).closest( 'tr' ).data( 'link-id' );
+            $( `#link-${linkID}` ).addClass( 'omitted' );
             $( this ).replaceWith( 'Omitted' );
             link = $( this ).data( 'link' );

broken-link-notifier/trunk/includes/js/results-back.js

@@ -1 +1 @@
 jQuery( $ => {
     // console.log( 'Broken Link Notifier JS Loaded...' );
-
-    // Add target _blank to title links
-    $( 'a.row-title' ).attr( 'target', '_blank' );
-
-    // Clear filters
-    $( '#link-type-filter' ).on( 'change', function( e ) {
-        $( '#code-filter' ).val( '' );
-    } );
-    $( '#code-filter' ).on( 'change', function( e ) {
-        $( '#link-type-filter' ).val( '' );
-    } );
 
     // Nonces
@@ -24 +13 @@
    
     // Scan an individual link
-    const scanLink = async ( link, postID, code, type, sourceID, method ) => {
+    const scanLink = async ( link, linkID, code, type, sourceID, method ) => {
         console.log( `Scanning link (${link})...` );
 
         // Say it started
-        var span = $( `#bln-verify-${postID}` );
+        var span = $( `#bln-verify-${linkID}` );
         span.addClass( 'scanning' ).html( `<em>Verifying</em>` );
 
@@ -40 +29 @@
                 nonce: nonceRescan,
                 link: link,
-                postID: postID,
+                linkID: linkID,
                 code: code,
                 type: type,
@@ -58 +47 @@
         for ( const linkSpan of linkSpans ) {
             const link = linkSpan.dataset.link;
-            const postID = linkSpan.dataset.postId;
+            const linkID = linkSpan.dataset.linkId;
             const code = linkSpan.dataset.code;
             const type = linkSpan.dataset.type;
@@ -65 +54 @@
 
             // Scan it
-            const data = await scanLink( link, postID, code, type, sourceID, method );
+            const data = await scanLink( link, linkID, code, type, sourceID, method );
             console.log( data );
 
@@ -91 +80 @@
                 }
                 
-                $( `#post-${postID}` ).addClass( 'omitted' );
-                $( `#post-${postID} .bln-type` ).addClass( statusType ).text( statusType );
-                $( `#post-${postID} .bln_type code` ).html( 'Code: ' + statusCode );
-                $( `#post-${postID} .bln_type .message` ).text( statusText );
-                $( `#post-${postID} .title .row-actions` ).remove();
-                $( `#post-${postID} .bln_source .row-actions` ).remove();
+                $( `#link-${linkID}` ).addClass( 'omitted' );
+                $( `#link-${linkID} .bln-type` ).addClass( statusType ).text( statusType );
+                $( `#link-${linkID} .bln_type code` ).html( 'Code: ' + statusCode );
+                $( `#link-${linkID} .bln_type .message` ).text( statusText );
+                $( `#link-${linkID} .link .row-actions` ).remove();
+                $( `#link-${linkID} .source .row-actions` ).remove();
 
                 // Also reduce count in admin bar
-                reduceAdminBarCount();
+                reduceCount();
 
             } else if ( code != statusCode || type != statusType ) {
@@ -109 +98 @@
                     text = `Link is still bad, but showing a different type. Old type was ${type}; new type is ${statusType}.`;
                 }
-                $( `#post-${postID} .bln-type` ).attr( 'class', `bln-type ${statusType}`).text( statusType );
+                $( `#link-${linkID} .bln-type` ).attr( 'class', `bln-type ${statusType}`).text( statusType );
                 var codeLink = 'Code: ' + statusCode;
                 if ( statusCode != 0 && statusCode != 666 ) {
                     codeLink = `<a href="https://http.dev/${statusCode}" target="_blank">Code: ${statusCode}</a>`;
                 }
-                $( `#post-${postID} .bln_type code` ).html( codeLink );
-                $( `#post-${postID} .bln_type message` ).text( statusText );
+                $( `#link-${linkID} .bln_type code` ).html( codeLink );
+                $( `#link-${linkID} .bln_type .message` ).text( statusText );
             } else {
                 text = `Still showing ${statusType}.`;
@@ -121 +110 @@
 
             // Update the page
-            $( `#bln-verify-${postID}` ).removeClass( 'scanning' ).addClass( statusType ).html( text );
+            $( `#bln-verify-${linkID}` ).removeClass( 'scanning' ).addClass( statusType ).html( text );
         }
 
@@ -137 +126 @@
      */
 
-    $( document ).on( 'click', '.replace-link', function ( e ) {
+    $( document ).on( 'click', '.replace-link a', function ( e ) {
         e.preventDefault();
     
-        let linkElement = $( this ).closest( 'td' ).find( '.row-title' );
-        let oldLink = linkElement.attr( 'href' );
-        let postID = $( this ).data( 'post-id' );
-        let sourceID = $( this ).data( 'source-id' );
+        let linkElement = $( this ).closest( 'td' ).find( '.link-url' );
+        let oldLink = $( this ).data( 'link' );
+        let linkID = $( this ).closest( 'tr' ).data( 'link-id' );
+        let sourceID = $( this ).closest( 'tr' ).find( '.source' ).data( 'source-id' );
     
         // Get the current link text
@@ -159 +148 @@
         // Handle input field blur (when user clicks outside)
         inputField.on( 'blur', function () {
-            saveLink( $(this), oldLink, sourceID, postID );
+            saveLink( $(this), oldLink, sourceID, linkID );
         } );
     
@@ -171 +160 @@
     
     // Function to save the link and replace the input field with a new link
-    function saveLink( inputField, oldLink, sourceID, postID ) {
+    function saveLink( inputField, oldLink, sourceID, linkID ) {
         let newLink = inputField.val().trim();
     
         // If the new link is empty, revert to the original
         if ( newLink === '' || newLink === oldLink || !sourceID ) {
-            inputField.replaceWith( `<a class="row-title" href="${oldLink}" target="_blank">${oldLink}</a>` );
+            inputField.replaceWith( `<a href="${oldLink}" class="link-url" target="_blank" rel="noopener">${oldLink}</a>` );
             return;
         }
-        console.log( oldLink, newLink );
     
         // Create a new link element with the updated text
-        let newLinkElement = $( `<a class="row-title" href="${newLink}" target="_blank">${newLink}</a>` );
+        let newLinkElement = $( `<a href="${newLink}" class="link-url" target="_blank" rel="noopener">${newLink}</a>` );
     
         // Replace the input field with the new link
@@ -195 +183 @@
                 action: 'blnotifier_replace_link', 
                 nonce: nonceReplace,
-                resultID: postID,
                 oldLink: oldLink,
                 newLink: newLink,
@@ -204 +191 @@
                     console.log( 'Link updated successfully.' );
 
+                    // Replace the old data-link in the Replace Link attribute
+                    $( `#link-${linkID} .link .row-actions .replace-link a` ).data( 'link', newLink ).attr( 'data-link', newLink );
+
                     // Replace the source blink
-                    let viewPageLink = $( `tr#post-${postID} .column-bln_source .row-actions .view a` );
+                    let viewPageLink = $( `tr#link-${linkID} .source .row-actions .view a` );
                     let currentHref = viewPageLink.attr( 'href' );
                     let newBlinkUrl = encodeURIComponent( newLink );
@@ -212 +202 @@
 
                     // Update the type
-                    $( `#post-${postID} .bln-type` ).addClass( 'fixed' ).text( 'Replaced' );
-                    $( `#post-${postID} .bln_type code` ).remove();
-                    $( `#post-${postID} .bln_type .message` ).html( `The old link has been replaced. Result will be removed.<br>Old link: ${oldLink}` );
+                    $( `#link-${linkID} .bln-type` ).addClass( 'fixed' ).text( 'Replaced' );
+                    $( `#link-${linkID} .bln_type code` ).remove();
+                    $( `#link-${linkID} .bln_type .message` ).html( `The old link has been replaced. Result will be removed.<br>Old link: ${oldLink}` );
 
                     // Remove omit link action
-                    $( `#post-${postID} .column-title .row-actions .clear` ).remove();
-                    $( `#post-${postID} .column-title .row-actions .omit` ).remove();
+                    $( `#link-${linkID} .link .row-actions .clear-result` ).remove();
+                    $( `#link-${linkID} .link .row-actions .omit-link` ).remove();
 
                     // Update the Verify column
-                    $( `#bln-verify-${postID}` ).text( `N/A` );
-
-                    let rowActions = $( `#post-${postID} .column-title .row-actions` );
+                    $( `#bln-verify-${linkID}` ).text( `N/A` );
+
+                    let rowActions = $( `#link-${linkID} .link .row-actions` );
                     if ( rowActions.children().length === 1 ) {
                         rowActions.html( rowActions.html().replace(' | ', '' ) );
@@ -229 +219 @@
 
                     // Reduce admin bar count
-                    reduceAdminBarCount();
+                    reduceCount();
                     
                 } else {
@@ -246 +236 @@
      */
 
-    $( document ).on( 'click', '.clear-result', function ( e ) {
+    $( document ).on( 'click', '.clear-result a', function ( e ) {
         e.preventDefault();
 
         let button = $( this );
-        let postID = button.data( 'post-id' );
+        let link = button.data( 'link' );
 
         $.ajax( {
@@ -259 +249 @@
                 action: 'blnotifier_delete_result',
                 nonce: nonceDelete,
-                postID: postID
+                link: link
             },
             success: function ( response ) {
@@ -265 +255 @@
                     button.closest( 'tr' ).fadeOut( 'fast', function () {
                         $( this ).remove();
+                        reduceCount();
                     } );
-
-                    // Reduce admin bar count
-                    reduceAdminBarCount();
 
                 } else {
@@ -290 +278 @@
         let button = $( this );
         let sourceID = button.data( 'source-id' );
-        console.log( sourceID );
         let postTitle = button.data( 'source-title' );
 
@@ -311 +298 @@
                     $( 'tr' ).each( function () {
                         let row = $( this );
-                        let rowSourceID = row.find( '.row-actions[data-source-id]' ).attr( 'data-source-id' );
+                        let rowSourceID = row.find( '.source[data-source-id]' ).attr( 'data-source-id' );
     
                         if ( rowSourceID == sourceID ) {
                             row.fadeOut( 'fast', function () {
                                 $( this ).remove();
+                                reduceCount();
                             } );
-
-                            // Reduce admin bar count
-                            reduceAdminBarCount();
                         }
                     } );
@@ -337 +322 @@
      */
 
-    function reduceAdminBarCount() {
+    function reduceCount() {
+        // Count broken links currently on the page
+        var countBroken = $( 'tr .bln-type.broken' ).length;
+        var countAll = $( 'tr .bln-type' ).length;
+
+        // Update admin bar
         var adminBarEl = $( '#wp-admin-bar-blnotifier-notify' );
         if ( adminBarEl.length ) {
-            var adminBarCountEl = adminBarEl.find( '.awaiting-mod' );
-            var adminBarCount = parseInt( adminBarCountEl.text(), 10 );
-    
-            if ( !isNaN( adminBarCount ) && adminBarCount > 0 ) {
-                adminBarCountEl.text( adminBarCount - 1 );
-            }
-        }
-    }
-    
+            adminBarEl.find( '.blnotifier-count-indicator' ).text( countBroken );
+        }
+
+        // Update admin menu
+        var adminMenuEl = $( 'li.toplevel_page_broken-link-notifier' );
+        if ( adminMenuEl.length ) {
+            adminMenuEl.find( '.awaiting-mod' ).text( countBroken );
+        }
+
+        // Update page total counter
+        var pageCountEl = $( '#bln-total-broken-links' );
+        if ( pageCountEl.length ) {
+            pageCountEl.text( countAll );
+        }
+    }
+
+
+    /**
+     * TRASH SELECTED BUTTON
+     */
+    function updateDeleteButton() {
+        const checkedCount = $( '.bln-row-checkbox:checked' ).length;
+        $( '#bln-delete-selected' ).prop( 'disabled', checkedCount === 0 );
+    }
+
+    $( document ).on( 'change', '.bln-row-checkbox', function() {
+        updateDeleteButton();
+    } );
+
+    $( '#cb-select-all-1' ).on( 'change', function() {
+        const checked = $( this ).prop( 'checked' );
+        $( '.bln-row-checkbox' ).prop( 'checked', checked );
+        updateDeleteButton();
+    } );
 } )

broken-link-notifier/trunk/includes/menu.php

@@ -56 +56 @@
         // The menu items
         $this->menu_items = [
-            'results'     => [ __( 'Results', 'broken-link-notifier' ), 'edit.php?post_type=blnotifier-results' ],
-            'omit-links'  => [ __( 'Omitted Links', 'broken-link-notifier' ), 'edit-tags.php?taxonomy=omit-links&post_type=blnotifier-results' ],
-            'omit-pages'  => [ __( 'Omitted Pages', 'broken-link-notifier' ), 'edit-tags.php?taxonomy=omit-pages&post_type=blnotifier-results' ],
+            'results'     => [ __( 'Results', 'broken-link-notifier' ) ],
+            'omit-links'  => [ __( 'Omitted Links', 'broken-link-notifier' ), 'edit-tags.php?taxonomy=omit-links' ],
+            'omit-pages'  => [ __( 'Omitted Pages', 'broken-link-notifier' ), 'edit-tags.php?taxonomy=omit-pages' ],
             'scan-single' => [ __( 'Page Scan', 'broken-link-notifier' ) ],
             'scan-multi'  => [ __( 'Multi-Scan', 'broken-link-notifier' ) ],
@@ -206 +206 @@
         // Taxonomies first
         } elseif ( $current_screen->id == 'edit-omit-links' ) {
-            $submenu_file = 'edit-tags.php?taxonomy=omit-links&post_type=blnotifier-results';
+            $submenu_file = 'edit-tags.php?taxonomy=omit-links';
             $parent_file = $this->get_plugin_page_short_path( null );
         } elseif ( $current_screen->id == 'edit-omit-pages' ) {
-            $submenu_file = 'edit-tags.php?taxonomy=omit-pages&post_type=blnotifier-results';
+            $submenu_file = 'edit-tags.php?taxonomy=omit-pages';
             $parent_file = $this->get_plugin_page_short_path( null );
-        
-        // Post Type
-        } elseif ( $current_screen->post_type == 'blnotifier-results' ) {
-            $submenu_file = 'edit.php?post_type=blnotifier-results';
-            $parent_file = $this->get_plugin_page_short_path();
         }
 
@@ -582 +577 @@
             ]
         );   
+
+        // Uninstall database cleanup
+        $uninstall_cleanup_option_name = 'blnotifier_uninstall_cleanup';
+        register_setting( $this->page_slug, $uninstall_cleanup_option_name, [ $this, 'sanitize_checkbox' ] );
+        add_settings_field(
+            $uninstall_cleanup_option_name,
+            'Remove Data on Uninstall',
+            [ $this, 'field_checkbox' ],
+            $this->page_slug,
+            'general',
+            [
+                'class'    => $uninstall_cleanup_option_name,
+                'name'     => $uninstall_cleanup_option_name,
+                'default'  => false,
+                'comments' => 'Enable this option to automatically remove the database table that the links are stored in and all options when the plugin is uninstalled.'
+            ]
+        );
     } // End settings_fields()
 
@@ -981 +993 @@
      */
     public function get_plugin_page( $tab = 'settings' ) {
-        if ( $tab == 'results' ) {
-            return admin_url( 'edit.php?post_type=blnotifier-results' );
-        } elseif ( $tab == 'omit-links' || $tab == 'omit-pages' ) {
-            return admin_url( 'edit-tags.php?taxonomy='.$tab.'&post_type=blnotifier-results' );
+        if ( $tab == 'omit-links' || $tab == 'omit-pages' ) {
+            return admin_url( 'edit-tags.php?taxonomy='.$tab );
         } else {
             return admin_url( 'admin.php?page='.BLNOTIFIER_TEXTDOMAIN ).'&tab='.sanitize_key( $tab );

broken-link-notifier/trunk/includes/omits.php

@@ -123 +123 @@
 
         // Register it as a new taxonomy
-        register_taxonomy( $taxonomy, 'blnotifier-results', [
+        register_taxonomy( $taxonomy, [], [
             // 'hierarchical'       => false,
             'labels'             => $labels,
@@ -292 +292 @@
             // Also delete it from results
             if ( $page && $page == 'scan-results' ) {
-                if ( $post_id = post_exists( $link ) ) {
-                    if ( get_post_type( $post_id ) == (new BLNOTIFIER_RESULTS())->post_type ) {
-                        wp_delete_post( $post_id, true );
-                    }
-                }
+                (new BLNOTIFIER_RESULTS)->remove( $link );
             }
             return true;
@@ -419 +415 @@
         $options_page = 'toplevel_page_'.BLNOTIFIER_TEXTDOMAIN;
         $tab = (new BLNOTIFIER_HELPERS)->get_tab();
-        $post_type = get_post_type();
-        if ( ( $screen == $options_page && $tab == 'scan-single' ) || ( $screen == 'edit.php' && ( isset( $_REQUEST[ '_wpnonce' ] ) && wp_verify_nonce( sanitize_text_field( wp_unslash ( $_REQUEST[ '_wpnonce' ] ) ), 'blnotifier_blinks' ) && isset( $_GET[ 'blinks' ] ) && sanitize_key( $_GET[ 'blinks' ] ) == 'true' )  || $post_type == 'blnotifier-results' ) ) {
-            if ( !$tab && $post_type == 'blnotifier-results' ) {
+
+        if ( 
+            ( $screen == $options_page && $tab == 'scan-single' ) || 
+            ( $screen == $options_page && $tab == 'results' ) ||
+            ( $screen == 'edit.php' && 
+                ( 
+                    isset( $_REQUEST[ '_wpnonce' ] ) && 
+                    wp_verify_nonce( sanitize_text_field( wp_unslash ( $_REQUEST[ '_wpnonce' ] ) ), 'blnotifier_blinks' ) && 
+                    isset( $_GET[ 'blinks' ] ) && 
+                    sanitize_key( $_GET[ 'blinks' ] ) == 'true' 
+                ) 
+            ) 
+        ) {
+        
+        if ( $tab == 'results' ) {
                 $tab = 'scan-results';
             } elseif ( !$tab ) {

broken-link-notifier/trunk/includes/results.php

@@ -21 +21 @@
 
 /**
- * Main plugin class.
+ * Results class.
  */
 class BLNOTIFIER_RESULTS {
@@ -30 +30 @@
      * @var string
      */ 
-    public $post_type = 'blnotifier-results';
+    public $table_name = 'blnotifier_results';
 
 
@@ -62 +62 @@
     public function init() {
 
-        // Register the post type
-        $this->register_post_type();
+        // Maybe create the database table
+        $this->maybe_create_db();
 
         // Add the header to the top of the admin list page
         add_action( 'load-edit.php', [ $this, 'add_header' ] );
         add_action( 'load-edit-tags.php', [ $this, 'add_header' ] );
-
-        // Add instructions to top of page
-        add_action( 'admin_notices', [ $this, 'description_notice' ] );
-
-        // Redirect
-        add_filter( 'get_edit_post_link', [ $this, 'redirect' ], 10, 3 );
-
-        // Remove Edit from Bulk Actions
-        add_filter( 'bulk_actions-edit-'.$this->post_type, [ $this, 'remove_from_bulk_actions' ] );
-
-        // Remove post states
-        add_filter( 'display_post_states', [ $this, 'remove_post_states' ], 999, 2 );
-
-        // Remove Edit and Quick Edit links, and add ignore link
-        add_action( 'post_row_actions', [ $this, 'row_actions' ], 10, 2 );
-
-        // Skip trash and auto delete
-        add_action( 'trashed_post', [ $this, 'skip_trash' ] );
-
-        // Add a type filter
-        add_action( 'restrict_manage_posts', [ $this, 'admin_filters' ], 10, 2 );
-        add_action( 'pre_get_posts', [ $this, 'admin_filters_query' ] );
-
-        // Add admin columns
-        add_filter( 'manage_'.$this->post_type.'_posts_columns', [ $this, 'admin_columns' ] );
-        add_action( 'manage_'.$this->post_type.'_posts_custom_column', [ $this, 'admin_column_content' ], 10, 2 );
-
-        // Make admin columns sortable
-        add_filter( 'manage_edit-'.$this->post_type.'_sortable_columns', [ $this, 'sort_columns' ] );
-        add_action( 'pre_get_posts', [ $this, 'sort_columns_query' ] );
 
         // Add notifications to admin bar
@@ -118 +88 @@
 
     } // End init()
-
-    
-    /**
-     * Register the post type
-     */
-    public function register_post_type() {
-        // Set the labels
-        $labels = [
-            'name'                  => _x( 'Links', 'Post Type General Name', 'broken-link-notifier' ),
-            'singular_name'         => _x( 'Link', 'Post Type Singular Name', 'broken-link-notifier' ),
-            'menu_name'             => __( 'Links', 'broken-link-notifier' ),
-            'name_admin_bar'        => __( 'Links', 'broken-link-notifier' ),
-            'search_items'          => __( 'Search links', 'broken-link-notifier' ),
-            'not_found'             => __( 'Not found', 'broken-link-notifier' ),
-            'not_found_in_trash'    => __( 'Not found in Trash', 'broken-link-notifier' ),
-            'filter_items_list'     => __( 'Filter link list', 'broken-link-notifier' ),
-        ];
-    
-        // Set the CPT args
-        $args = [
-            'label'                 => __( 'Links', 'broken-link-notifier' ),
-            'description'           => __( 'Links', 'broken-link-notifier' ),
-            'labels'                => $labels,
-            'supports'              => [],
-            'taxonomies'            => [],
-            'public'                => false,
-            'show_ui'               => true,
-            'show_in_menu'          => false,
-            'show_in_admin_bar'     => false,
-            'show_in_nav_menus'     => false,
-            'can_export'            => true,
-            'has_archive'           => false,
-            'exclude_from_search'   => true,
-            'publicly_queryable'    => false,
-            'query_var'             => $this->post_type,
-            'capability_type'       => 'post',
-            'capabilities'          => [
-                'create_posts'      => 'do_not_allow',
-            ],
-            'map_meta_cap'          => true, 
-            'show_in_rest'          => true,
-        ];
-    
-        // Register the CPT
-        register_post_type( $this->post_type, $args );
-    } // End register_post_type()
 
 
@@ -235 +159 @@
 
     /**
-     * Add a description notice
-     * 
-     * @return void
-     */
-    public function description_notice() {
-        global $current_screen;
-        if ( 'edit-'.$this->post_type === $current_screen->id ) {
-            echo '<div class="notice notice-info" >
-                <p>' . esc_html__( 'This page shows the results of your scans. It automatically rechecks the links themselves to see if they are still broken, but it does not remove the links from the pages or rescan the pages to see if broken links have been fixed. After fixing a broken link, you will need to clear the result below. Then when you rescan the page it should not show up here again. Note that the plugin will still find broken links if you simply hide them on the page.', 'broken-link-notifier' ) . '</p>
-            </div>';
-        }
-    } // End description_notice()
-
-
-    /**
-     * Redirect the edit link
-     *
-     * @param string $url
-     * @param int $post_id
-     * @param [type] $context
-     * @return string
-     */
-    public function redirect( $url, $post_id, $context ) {
-        if ( get_post_type( $post_id ) == $this->post_type ) {
-            return sanitize_text_field( get_the_title( $post_id ) );
-        } else {
-            return $url;
-        }
-    } // End redirect()
-  
-
-    /**
-     * Remove Edit from Bulk Actions
-     *
-     * @param array $actions
-     * @return array
-     */
-    public function remove_from_bulk_actions( $actions ) {
-        unset( $actions[ 'edit' ] );
-        return $actions;
-    } // End remove_from_bulk_actions()
-
-
-    /**
-     * Remove states
-     *
-     * @param array $states
-     * @param WP_Post $post
-     * @return void
-     */
-    public function remove_post_states( $states, $post ) {
-        if ( get_post_type( $post ) == $this->post_type ) {
-            return false;
-        }
-        return $states;
-    } // End remove_post_states()
-
-
-    /**
-     * Action links
-     *
-     * @param array $actions
-     * @param object $post
-     * @return array
-     */
-    public function row_actions( $actions, $post ) {
-        if ( $this->post_type == $post->post_type ) {
-            $actions = [];
-
-            $actions[ 'clear' ] = '<a class="clear-result" href="#" data-post-id="' . $post->ID . '">' . __( 'Clear Result', 'broken-link-notifier' ) . '</a></span>';
-
-            $permalink = get_the_title( $post );
-            if ( !(new BLNOTIFIER_OMITS)->is_omitted( $permalink, 'links' ) ) {
-                $actions[ 'omit' ] = '<a class="omit-link" href="#" data-link="' . $permalink . '" data-post-id="' . $post->ID . '">' . __( 'Omit Link', 'broken-link-notifier' ) . '</a>';
-            }
-
-            $source_url = get_post_meta( $post->ID, 'source', true );
-            if ( $source_url ) {
-                $source_url = filter_var( $source_url, FILTER_SANITIZE_URL );
-                
-                if ( $source_id = url_to_postid( $source_url ) ) {
-                    $actions[ 'replace' ] = '<a class="replace-link" href="#" data-link="' . $permalink . '" data-post-id="' . $post->ID . '" data-source-id="' . $source_id . '">' . __( 'Replace Link', 'broken-link-notifier' ) . '</a></span>';
-                }
-            }
-        }
-        return $actions;
-    } // End row_actions()
-
-
-    /**
-     * Skip trash and auto delete
-     *
-     * @param int $post_id
-     * @return void
-     */
-    public function skip_trash( $post_id ) {
-        if ( get_post_type( $post_id ) == $this->post_type ) {
-            wp_delete_post( $post_id, true );
-        }
-    } // End skip_trash()
-
-
-    /**
-     * Type filter
-     *
-     * @return void
-     */
-    public function admin_filters( $post_type, $which ) {
-        $screen = get_current_screen();
-        if ( $screen && $screen->id == 'edit-'.$this->post_type ) {
-
-            // Link type
-            if ( isset( $_REQUEST[ 'link-type' ] ) ) {  // phpcs:ignore
-                $s = sanitize_key( $_REQUEST[ 'link-type' ] ); // phpcs:ignore
-            } else {
-                $s = '';
-            }
-            $HELPERS = new BLNOTIFIER_HELPERS;
-            echo '<select id="link-type-filter" name="link-type">
-                <option value=""'.esc_html( $HELPERS->is_selected( $s, '' ) ).'>All Link Types</option>
-                <option value="broken"'.esc_html( $HELPERS->is_selected( $s, 'broken' ) ).'>Broken Links</option>
-                <option value="warning"'.esc_html( $HELPERS->is_selected( $s, 'warning' ) ).'>Warning Links</option>
-            </select>';
-
-            // Status code
-            $bad_codes = $HELPERS->get_bad_status_codes();
-            $warning_codes = $HELPERS->get_warning_status_codes();
-            $all_codes = array_unique( array_merge( $bad_codes, $warning_codes ) );
-            sort( $all_codes );
-
-            if ( isset( $_GET[ 'code' ] ) && sanitize_text_field( $_GET[ 'code' ] ) != '' ) { // phpcs:ignore
-                $s = absint( $_GET[ 'code' ] ); // phpcs:ignore
-            } else {
-                $s = '';
-            }
-            $HELPERS = new BLNOTIFIER_HELPERS;
-            echo '<select id="code-filter" name="code">
-                <option value=""'.esc_html( $HELPERS->is_selected( $s, '' ) ).'>All Status Codes</option>';
-
-                foreach ( $all_codes as $code ) {
-                    echo '<option value="'.esc_attr( $code ).'"'.esc_html( $HELPERS->is_selected( $s, $code ) ).'>'.esc_attr( $code ).'</option>';
-                }
-
-            echo '</select>';
-        }
-    } // End admin_filters()
-
-
-    /**
-     * The type filter query
-     *
-     * @param object $query
-     * @return void
-     */
-    public function admin_filters_query( $query ) {
-        $post_type = $query->get( 'post_type' );
-        if ( $post_type == $this->post_type ) {
-
-            // Link type
-            if ( isset( $_REQUEST[ 'link-type' ] ) && sanitize_text_field( $_REQUEST[ 'link-type' ] ) != '' ) { // phpcs:ignore
-                $type = sanitize_key( $_REQUEST[ 'link-type' ] ); // phpcs:ignore
-                if ( $type != '' ) {
-                    $meta_query[] = [
-                        'key'     => 'type',
-                        'value'   => $type,
-                    ];
-                    $query->set( 'meta_query', $meta_query );
-                }
-
-            // Status code
-            } elseif ( isset( $_REQUEST[ 'code' ] ) && sanitize_text_field( $_REQUEST[ 'code' ] ) != '' ) { // phpcs:ignore
-                $code = absint( $_REQUEST[ 'code' ] ); // phpcs:ignore
-                if ( $code != '' ) {
-                    $meta_query[] = [
-                        'key'     => 'code',
-                        'value'   => $code,
-                    ];
-                    $query->set( 'meta_query', $meta_query );
-                }
-            }
-        }
-    } // End admin_filters_query()
-
-    
-    /**
-     * Admin columns
-     *
-     * @param array $columns
-     * @return array
-     */
-    public function admin_columns( $columns ) {
-        return [
-            'cb'            => '<input type="checkbox"/>',
-            'bln_type'      => __( 'Type', 'broken-link-notifier' ),
-            'title'         => __( 'Link', 'broken-link-notifier' ),
-            'bln_source'    => __( 'Source', 'broken-link-notifier' ),
-            'bln_source_pt' => __( 'Source Post Type', 'broken-link-notifier' ),
-            'bln_date'      => __( 'Date', 'broken-link-notifier' ),
-            'bln_author'    => __( 'User', 'broken-link-notifier' ),
-            'bln_method'    => __( 'Method', 'broken-link-notifier' ),
-            'bln_verify'    => __( 'Verify', 'broken-link-notifier' ),
-        ];
-    } // End admin_columns()
-
-
-    /**
-     * Admin column content
-     *
-     * @param string $column
-     * @param int $post_id
-     * @return void
-     */
-    public function admin_column_content( $column, $post_id ) {
-        // Type
-        if ( 'bln_type' === $column ) {
-            $post = get_post( $post_id );
-            if ( $post->type == 'broken' ) {
-                echo '<div class="bln-type broken">' . esc_html( __( 'Broken', 'broken-link-notifier' ) ) . '</div>';
-            } elseif ( $post->type == 'warning' ) {
-                echo '<div class="bln-type warning">' . esc_html( __( 'Warning', 'broken-link-notifier' ) ) . '</div>';
-            } elseif ( $post->type == 'good' ) {
-                echo '<div class="bln-type good">' . esc_html( __( 'Good', 'broken-link-notifier' ) ) . '</div>';
-            }
-            $code = $post->code;
-            if ( $code != 0 && $code != 666 ) {
-                $code = '<a href="https://http.dev/'.$code.'" target="_blank">'.$code.'</a>';
-            }
-            if ( $code == 666 ) {
-                $incl_title = ' title="' . __( 'A status code of 666 is a code we use to force invalid URL code 0 to be a broken link. It is not an official status code.', 'broken-link-notifier' ) . '"';
-            } elseif ( $code == 0 ) {
-                $incl_title = ' title="' . __( 'A status code of 0 means there was no response and it can occur for various reasons, like request time outs. It almost always means something is randomly interfering with the user\'s connection, like a proxy server / firewall / load balancer / laggy connection / network congestion, etc.', 'broken-link-notifier' ) . '"';
-            } else {
-                $incl_title = '';
-            }
-            echo '<code'.wp_kses_post( $incl_title ).'>Code: '.wp_kses_post( $code ).'</code> <span class="message">'.esc_html( $post->post_content ).'</span>';
-        }
-
-        // Source
-        if ( 'bln_source' === $column ) {
-            $source_url = get_post_meta( $post_id, 'source', true );
-            if ( $source_url ) {
-                $source_url = filter_var( $source_url, FILTER_SANITIZE_URL );
-                $source_url = remove_query_arg( (new BLNOTIFIER_HELPERS)->get_qs_to_remove_from_source(), $source_url );
-                $actions = [
-                    '<span class="view"><a href="'.add_query_arg( 'blink', urlencode( strtok( get_the_title( $post_id ), '?' ) ), $source_url ).'" target="_blank">' . __( 'View Page', 'broken-link-notifier' ) . '</a></span>'
-                ];
-                if ( $source_id = url_to_postid( $source_url ) ) {
-                    if ( !(new BLNOTIFIER_OMITS)->is_omitted( $source_url, 'pages' ) ) {
-                        $actions[] = '<span class="omit"><a class="omit-page" href="#" data-link="'.$source_url.'">' . __( 'Omit Page', 'broken-link-notifier' ) . '</a></span>';
-                    }
-                    $scan_nonce = wp_create_nonce( 'blnotifier_scan_single' );
-                    $actions[] = '<span class="scan"><a class="scan-page" href="'.(new BLNOTIFIER_MENU)->get_plugin_page( 'scan-single' ).'&scan='.$source_url.'&_wpnonce='.$scan_nonce.'" target="_blank">Scan Page</a></span>';
-                    $actions[] = '<span class="edit"><a href="'.admin_url( 'post.php' ).'?post='.$source_id.'&action=edit">' . __( 'Edit Page', 'broken-link-notifier' ) . '</a></span>';
-                    if ( is_plugin_active( 'cornerstone/cornerstone.php' ) ) {
-                        $actions[] = '<span class="edit-in-cornerstone"><a href="'.home_url( '/cornerstone/edit/'.$source_id ).'">' . __( 'Edit in Cornerstone', 'broken-link-notifier' ) . '</a></span>';
-                    }
-
-                    $title = get_the_title( $source_id );
-                    $source_url = '<strong><a class="source-url" href="'.$source_url.'">'.$title.'</a></strong>';
-
-                    if ( get_option( 'blnotifier_enable_delete_source' ) ) {
-                        $actions[] = '<span class="delete"><a class="delete-source" href="#" data-source-title="' . $title . '" data-source-id="' . $source_id . '">' . __( 'Trash Page', 'broken-link-notifier' ) . '</a></span>';
-                    }
-                }
-
-                echo wp_kses_post( $source_url ).'<div class="row-actions" data-source-id="' . esc_attr( $source_id ) . '">'.wp_kses_post( implode( ' | ', $actions ) ).'</div>';
-            } else {
-                echo esc_html__( 'No source found...', 'broken-link-notifier' );
-            }
-        }
-
-        // Source URL
-        if ( 'bln_source_pt' === $column ) {
-            $url = get_post_meta( $post_id, 'source', true );
-            if ( $url ) {
-                $url = filter_var( $url, FILTER_SANITIZE_URL );
-                if ( $source_id = url_to_postid( $url ) ) {
-                    $post_type = get_post_type( $source_id );
-                    $post_type_name = (new BLNOTIFIER_HELPERS)->get_post_type_name( $post_type, true );
-                } else {
-                    $post_type_name = '--';
-                }
-                echo esc_html( $post_type_name );
-            }
-        }
-
-        // Date
-        if ( 'bln_date' === $column ) {
-            $date = get_the_date( 'F j, Y g:i A', $post_id );
-            $location = get_post_meta( $post_id, 'location', true );
-            echo 'Discovered in '.esc_html( ucwords( $location ) ).'<br>'.esc_html( $date );
-        }
-
-        // Author
-        if ( 'bln_author' === $column ) {
-            $post = get_post( $post_id );
-            if ( isset( $post->guest ) && $post->guest ) {
-                $display_name = 'Guest';
-            } elseif ( $author = $post->post_author ) {
-                $user = get_user_by( 'ID', $author );
-                $display_name = $user->display_name;
-            } else {
-                $display_name = 'Guest';
-            }
-            echo esc_html( $display_name );
-        }
-
-        // Method
-        if ( 'bln_method' === $column ) {
-            $post = get_post( $post_id );
-            if ( isset( $post->method ) && $post->method ) {
-
-                switch ( sanitize_key( $post->method ) ) {
-                    case 'visit':
-                        $method = __( 'Front-End Visit', 'broken-link-notifier' );
-                        break;
-                    case 'multi':
-                        $method = __( 'Multi-Scan', 'broken-link-notifier' );
-                        break;
-                    case 'single':
-                        $method = __( 'Page Scan', 'broken-link-notifier' );
-                        break;
-                    default:
-                        $method = __( 'Unknown', 'broken-link-notifier' );
-                }
-            } else {
-                $method = __( 'Unknown', 'broken-link-notifier' );
-            }
-            echo esc_html( $method );
-        }
-
-        // Verify
-        if ( 'bln_verify' === $column ) {
-
-            if ( !(new BLNOTIFIER_HELPERS ())->is_results_verification_paused() ) {
-                $link = get_the_title( $post_id );
-                $post = get_post( $post_id );
-                $code = absint( $post->code );
-                $type = sanitize_text_field( $post->type );
-                $source_url = filter_var( $post->source, FILTER_SANITIZE_URL );
-                $source_url = remove_query_arg( (new BLNOTIFIER_HELPERS)->get_qs_to_remove_from_source(), $source_url );
-                $source_id = url_to_postid( $source_url );
-                $method = $post->method ? sanitize_key( $post->method ) : 'unknown';
-
-                echo '<span id="bln-verify-'.esc_attr( $post_id ).'" class="bln-verify" data-type="' . esc_attr( $type ) . '" data-post-id="'.esc_attr( $post_id ).'" data-link="'.esc_html( $link ).'" data-code="'.esc_attr( $code ).'" data-source-id="'.esc_attr( $source_id ).'" data-method="'.esc_attr( $method ).'">' . esc_html__( 'Pending', 'broken-link-notifier' ) . '</span>';
-
-            } else {
-                echo esc_html__( 'Auto-verification is paused in settings.', 'broken-link-notifier' );
-            }
-
-        }
-    } // End admin_column_content()
-    
-
-    /**
-     * Make admin columns sortable
-     *
-     * @param array $columns
-     * @return array
-     */
-    public function sort_columns( $columns ) {
-        $columns[ 'bln_type' ]      = 'bln_type';
-        $columns[ 'bln_source' ]    = 'bln_source';
-        $columns[ 'bln_source_pt' ] = 'bln_source_pt';
-        $columns[ 'bln_date' ]      = 'bln_date';
-        return $columns;
-    } // End sort_columns()
-
-
-    /**
-     * Sort the order column properly
-     *
-     * @param object $query
-     * @return void
-     */
-    public function sort_columns_query( $query ) {
-        global $current_screen;
-        if ( is_admin() && isset( $current_screen ) && $current_screen->id === 'edit-'.$this->post_type ) {
-            $orderby = $query->get( 'orderby' );
-            if ( 'bln_type' == $orderby ) {
-                $query->set( 'meta_key', 'type' );
-                $query->set( 'orderby', 'meta_value' );
-            } elseif ( 'bln_source' == $orderby ) {
-                $query->set( 'meta_key', 'bln_source' );
-                $query->set( 'orderby', 'meta_value' );
-            } elseif ( 'bln_date' == $orderby ) {
-                // $query->set( 'meta_key', 'date' );
-                $query->set( 'orderby', 'date' );
-            }
-        }
-    } // End sort_columns_query()
-
-
-    /**
      * Add an online user count to the admin bar
      *
@@ -681 +211 @@
 
     /**
+     * Create the database table if it doesn't exist.
+     *
+     * @return void
+     */
+    public function maybe_create_db() {
+        global $wpdb;
+
+        $table_name = $wpdb->prefix . $this->table_name;
+        
+        if ( $wpdb->get_var( "SHOW TABLES LIKE '{$table_name}'" ) !== $table_name ) {
+            require_once ABSPATH . 'wp-admin/includes/upgrade.php';
+            $charset_collate = $wpdb->get_charset_collate();
+
+            $sql = "CREATE TABLE $table_name (
+                id bigint(20) unsigned NOT NULL AUTO_INCREMENT,
+                link varchar(2048) NOT NULL,
+                link_hash char(32) NOT NULL,
+                text varchar(255) NOT NULL,
+                type varchar(20) NOT NULL,
+                code smallint(5) unsigned NOT NULL,
+                source varchar(2048) NOT NULL,
+                location varchar(50) NOT NULL,
+                method varchar(20) NOT NULL,
+                guest tinyint(1) NOT NULL DEFAULT 0,
+                author_id bigint(20) unsigned NOT NULL DEFAULT 0,
+                created_at datetime NOT NULL,
+                PRIMARY KEY (id),
+                UNIQUE KEY link_hash (link_hash),
+                KEY code (code),
+                KEY created_at (created_at)
+            ) $charset_collate;";
+
+            dbDelta( $sql );
+
+            // Migrate old posts if they exist
+            $old_posts = get_posts( [
+                'post_type'      => 'blnotifier-results',
+                'posts_per_page' => -1,
+                'post_status'    => 'publish',
+                'fields'         => 'ids',
+            ] );
+
+            if ( ! empty( $old_posts ) ) {
+                foreach ( $old_posts as $post_id ) {
+                    $code     = absint( get_post_meta( $post_id, 'code', true ) );
+                    $location = get_post_meta( $post_id, 'location', true );
+                    $method   = get_post_meta( $post_id, 'method', true );
+                    $source   = get_post_meta( $post_id, 'source', true );
+                    $type     = get_post_meta( $post_id, 'type', true );
+
+                    $post    = get_post( $post_id );
+                    $link    = $post ? $post->post_title : '';
+                    $text    = $post ? $post->post_content : '';
+                    $author  = $post ? $post->post_author : 0;
+                    $created = $post ? $post->post_date : current_time( 'mysql' );
+
+                    $this->add( [
+                        'link'   => $link,
+                        'text'   => $text,
+                        'type'   => $type,
+                        'code'   => $code,
+                        'source' => $source,
+                        'location' => $location,
+                        'method' => $method,
+                        'author' => $author,
+                        'created_at' => $created,
+                    ] );
+
+                    // Delete the old post
+                    wp_delete_post( $post_id, true );
+                }
+            }
+        }
+    } // End maybe_create_db()
+
+
+    /**
      * Check if the link has already been added
      *
@@ -687 +294 @@
      */
     public function already_added( $link ) {
-        return post_exists( sanitize_text_field( $link ), '', '', $this->post_type );
+        global $wpdb;
+
+        $table_name = $wpdb->prefix . $this->table_name;
+        $link_clean = sanitize_text_field( $link );
+        $link_hash = md5( strtolower( untrailingslashit( $link_clean ) ) );
+
+        $exists = $wpdb->get_var(
+            $wpdb->prepare(
+                "SELECT id FROM $table_name WHERE link_hash = %s LIMIT 1",
+                $link_hash
+            )
+        );
+
+        return !empty( $exists );
     } // End already_added()
 
@@ -693 +313 @@
     /**
      * Add a new broken or warning link
-     * $args = [
-     *      'broken'   => true,
-     *      'warning'  => false,
-     *      'code'     => 404,
-     *      'text'     => 'Not found',
-     *      'link'     => 'https://brokenlink.com',
-     *      'source'   => 'https://source-url.com',
-     *      'author'   => 1,
-     *      'location' => 'content'
-     * ]
      *
      * @param array $args
@@ -708 +318 @@
      */
     public function add( $args ) {
-        // If already added, no need to add again
-        if ( $this->already_added( $args[ 'link' ] ) ) {
+        global $wpdb;
+
+        $table_name = $wpdb->prefix . $this->table_name;
+
+        $link = sanitize_text_field( $args[ 'link' ] );
+        $link_normalized = strtolower( untrailingslashit( $link ) );
+        $link_hash = md5( $link_normalized );
+
+        if ( $this->already_added( $link ) ) {
             return 'Link already added';
         }
 
-        // Validate source
-        $source_url = remove_query_arg( (new BLNOTIFIER_HELPERS)->get_qs_to_remove_from_source(), filter_var( $args[ 'source' ], FILTER_SANITIZE_URL ) );
+        $source_url = remove_query_arg(
+            ( new BLNOTIFIER_HELPERS )->get_qs_to_remove_from_source(),
+            filter_var( $args[ 'source' ], FILTER_SANITIZE_URL )
+        );
+
         if ( !$source_url ) {
             return __( 'Invalid source:', 'broken-link-notifier' ) . ' ' . $source_url;
         }
 
-        // Args
-        $data = [
-            'post_title'    => sanitize_text_field( $args[ 'link' ] ),
-            'post_content'  => sanitize_text_field( $args[ 'text' ] ),
-            'post_status'   => 'publish',
-            'post_type'     => $this->post_type,
-            'meta_input'    => [
-                'type'     => sanitize_key( $args[ 'type' ] ),
-                'code'     => absint( $args[ 'code' ] ),
-                'source'   => $source_url,
-                'location' => sanitize_key( $args[ 'location' ] ),
-                'guest'    => false,
-                'method'   => sanitize_key( $args[ 'method' ] )
+        $inserted = $wpdb->insert(
+            $table_name,
+            [
+                'link'       => $link,
+                'link_hash'  => $link_hash,
+                'text'       => sanitize_text_field( $args[ 'text' ] ),
+                'type'       => sanitize_key( $args[ 'type' ] ),
+                'code'       => absint( $args[ 'code' ] ),
+                'source'     => esc_url_raw( $source_url ),
+                'location'   => sanitize_key( $args[ 'location' ] ),
+                'method'     => sanitize_key( $args[ 'method' ] ),
+                'guest'      => ( absint( $args[ 'author' ] ) === 0 ) ? 1 : 0,
+                'author_id'  => absint( $args[ 'author' ] ),
+                'created_at' => current_time( 'mysql' ),
             ],
-        ];
-
-        // Guest or not?
-        if ( $args[ 'author' ] == 0 ) {
-            $data[ 'meta_input' ][ 'guest' ] = true;
-        } else {
-            $data[ 'post_author' ] = absint( $args[ 'author' ] );
-        }
-
-        // Add it
-        $link_id = wp_insert_post( $data );
-        if ( !is_wp_error( $link_id ) ) {
-            return $link_id;
-        } else {
-            $error = $link_id->get_error_message();
-            error_log( $error ); // phpcs:ignore 
-            return $error;
-        }
+            [ '%s','%s','%s','%s','%d','%s','%s','%s','%d','%d','%s' ]
+        );
+
+        if ( $inserted ) {
+            return $wpdb->insert_id;
+        }
+
+        return 'Insert failed';
     } // End add()
 
@@ -760 +371 @@
      * @return boolean
      */
-    public function remove( $link, $link_id = false ) {
-        if ( !$link_id ) {
-            $link_id = post_exists( $link, '', '', $this->post_type );
-        }
-        if ( $link_id ) {
-            if ( get_post_type( $link_id ) == $this->post_type ) {
-                if ( wp_delete_post( $link_id ) ) {
-                    return true;
-                }
-            }
-        }
-        return false;
+    public function remove( $link ) {
+        global $wpdb;
+
+        $table_name = $wpdb->prefix . $this->table_name;
+        $link_hash = md5( strtolower( untrailingslashit( sanitize_text_field( $link ) ) ) );
+
+        $deleted = $wpdb->delete(
+            $table_name,
+            [ 'link_hash' => $link_hash ],
+            [ '%s' ]
+        );
+
+        return ( $deleted > 0 );
     } // End remove()
 
@@ -928 +540 @@
 
     /**
-     * Ajax call for front end
-     *
-     * @return void
+     * Public AJAX endpoint used by the front-end scanner.
+     *
+     * This endpoint intentionally allows unauthenticated requests because the
+     * plugin scans links from publicly accessible pages visited by guests.
+     *
+     * Security protections implemented:
+     * - Nonce verification to prevent CSRF
+     * - Rate limiting per IP via transient
+     * - Maximum links per scan enforced
+     * - URLs sanitized before processing
+     * - Only HTTP/HTTPS sources allowed
+     *
+     * No privileged actions are performed. The endpoint only scans links and
+     * records results in a custom table that is not publicly accessible.
      */
     public function ajax_blinks() {
@@ -937 +560 @@
             exit( 'No naughty business please.' );
         }
+
+        // Public endpoint: allow guests, but validate capability for logged-in users.
+        if ( is_user_logged_in() && ! current_user_can( 'read' ) ) {
+            wp_send_json_error( 'Permission denied' );
+        }
     
-        // Get the ID
+        // Get the source and links
         $source_url    = isset( $_REQUEST[ 'source_url' ] ) ? filter_var( wp_unslash( $_REQUEST[ 'source_url' ] ), FILTER_SANITIZE_URL ) : '';
         $header_links  = isset( $_REQUEST[ 'header_links' ] ) ? wp_unslash( $_REQUEST[ 'header_links' ] ) : []; // phpcs:ignore WordPress.Security.ValidatedSanitizedInput.InputNotSanitized
@@ -1116 +744 @@
 
         // Check permissions
-        if ( !(new BLNOTIFIER_HELPERS)->user_can_manage_broken_links() ) {
+        $HELPERS = new BLNOTIFIER_HELPERS;
+        if ( !$HELPERS->user_can_manage_broken_links() ) {
             exit( 'Unauthorized access.' );
         }
     
-        // Get the ID
+        // Get the data
         $link      = isset( $_REQUEST[ 'link' ] ) ? sanitize_text_field( wp_unslash( $_REQUEST[ 'link' ] ) ) : false;
-        $post_id   = isset( $_REQUEST[ 'postID' ] ) ? absint( wp_unslash( $_REQUEST[ 'postID' ] ) ) : false;
+        $link_id   = isset( $_REQUEST[ 'linkID' ] ) ? absint( wp_unslash( $_REQUEST[ 'linkID' ] ) ) : false;
         $code      = isset( $_REQUEST[ 'code' ] ) ? absint( wp_unslash( $_REQUEST[ 'code' ] ) ) : false;
         $type      = isset( $_REQUEST[ 'type' ] ) ? sanitize_key( wp_unslash( $_REQUEST[ 'type' ] ) ) : false;
@@ -1131 +760 @@
         if ( $link ) {
 
-            // Initiate helpers
-            $HELPERS = new BLNOTIFIER_HELPERS;
-
             // If the source no longer exists, auto remove it
             if ( !$source_id || !get_post( $source_id ) ) {
-                $remove = $this->remove( $HELPERS->str_replace_on_link( $link ), $post_id );
+                $remove = $this->remove( $HELPERS->str_replace_on_link( $link ) );
                 $status = [
                     'type' => 'n/a',
@@ -1148 +774 @@
                     $result[ 'status' ] = $status;
                     $result[ 'link' ] = $link;
-                    $result[ 'post_id' ] = $post_id;
+                    $result[ 'link_id' ] = $link_id;
                 } else {
                     $result[ 'type' ] = 'error';
@@ -1162 +788 @@
                 // If it's good now, remove the old post
                 if ( $status[ 'type' ] == 'good' || $status[ 'type' ] == 'omitted' ) {
-                    $remove = $this->remove( $HELPERS->str_replace_on_link( $link ), $post_id );
+                    $remove = $this->remove( $HELPERS->str_replace_on_link( $link ) );
                     if ( $remove ) {
                         $result[ 'type' ] = 'success';
                         $result[ 'status' ] = $status;
                         $result[ 'link' ] = $link;
-                        $result[ 'post_id' ] = $post_id;
+                        $result[ 'link_id' ] = $link_id;
                     } else {
                         $result[ 'type' ] = 'error';
@@ -1178 +804 @@
                 // If it's still not good, but doesn't have the same code or type, update it
                 } elseif ( $code !== $status[ 'code' ] || $type !== $status[ 'type' ] ) {
-                    $remove = $this->remove( $HELPERS->str_replace_on_link( $link ), $post_id );
+                    $remove = $this->remove( $HELPERS->str_replace_on_link( $link ) );
                     if ( $remove ) {
                         $result[ 'type' ] = 'success';
                         $result[ 'status' ] = $status;
                         $result[ 'link' ] = $link;
-                        $result[ 'post_id' ] = $post_id;
+                        $result[ 'link_id' ] = $link_id;
     
                         // Re-add it with new data
@@ -1191 +817 @@
                             'text'     => $status[ 'text' ],
                             'link'     => $status[ 'link' ],
-                            'source'   => get_the_permalink( $post_id ),
+                            'source'   => get_the_permalink( $source_id ),
                             'author'   => get_current_user_id(),
                             'location' => 'content',
@@ -1204 +830 @@
                     $result[ 'status' ] = $status;
                     $result[ 'link' ] = $link;
-                    $result[ 'post_id' ] = $post_id;
+                    $result[ 'link_id' ] = $link_id;
                 }
             }
@@ -1230 +856 @@
         }
 
-        if ( !(new BLNOTIFIER_HELPERS)->user_can_manage_broken_links() ) {
+        $HELPERS = new BLNOTIFIER_HELPERS;
+        if ( !$HELPERS->user_can_manage_broken_links() ) {
             exit( 'Unauthorized access.' );
         }
     
         // Get the vars
-        $result_id  = isset( $_REQUEST[ 'resultID' ] ) ? absint( wp_unslash( $_REQUEST[ 'resultID' ] ) ) : false;
         $oldLink    = isset( $_REQUEST[ 'oldLink' ] ) ? sanitize_text_field( wp_unslash( $_REQUEST[ 'oldLink' ] ) ) : false;
         $newLink    = isset( $_REQUEST[ 'newLink' ] ) ? sanitize_text_field( wp_unslash( $_REQUEST[ 'newLink' ] ) ) : false;
         $source_id  = isset( $_REQUEST[ 'sourceID' ] ) ? absint( wp_unslash( $_REQUEST[ 'sourceID' ] ) ) : false;
 
-        if ( $result_id && $oldLink && $newLink && $source_id && get_post( $source_id ) ) {
+        if ( $oldLink && $newLink && $source_id && get_post( $source_id ) ) {
 
             // Get the current post content
@@ -1259 +885 @@
 
                 // Let's also delete the result
-                if ( get_post_type( $result_id ) == $this->post_type ) {
-                    wp_delete_post( $result_id );
-                }
+                $this->remove( $HELPERS->str_replace_on_link( $oldLink ) );
 
                 // Respond
@@ -1286 +910 @@
         }
 
-        if ( !(new BLNOTIFIER_HELPERS)->user_can_manage_broken_links() ) {
+        $HELPERS = new BLNOTIFIER_HELPERS;
+        if ( !$HELPERS->user_can_manage_broken_links() ) {
             exit( 'Unauthorized access.' );
         }
     
-        // Get the ID
-        $post_id = isset( $_REQUEST[ 'postID' ] ) ? absint( $_REQUEST[ 'postID' ] ) : false;
-        if ( $post_id ) {
-
-            // Delete it
-            if ( get_post_type( $post_id ) == $this->post_type ) {
-                if ( wp_delete_post( $post_id ) ) {
-                    wp_send_json_success();
-                }
-            }
+        // Remove the link
+        $link = isset( $_REQUEST[ 'link' ] ) ? sanitize_text_field( wp_unslash( $_REQUEST[ 'link' ] ) ) : false;
+        if ( $link ) {
+            $this->remove( $HELPERS->str_replace_on_link( $link ) );
+            wp_send_json_success();
         }
 
@@ -1333 +953 @@
 
             // Delete all links with this source
+            global $wpdb;
+
             $source_url = (new BLNOTIFIER_HELPERS)->get_clean_permalink( $source_id );
             if ( $source_url ) {
-                $query = new WP_Query( [
-                    'post_type'      => $this->post_type,
-                    'meta_query'     => [ // phpcs:ignore 
-                        [
-                            'key'   => 'source',
-                            'value' => $source_url,
-                        ]
-                    ],
-                    'fields'         => 'ids',
-                    'posts_per_page' => -1,
-                ] );
-    
-                if ( !empty( $query->posts ) ) {
-                    foreach ( $query->posts as $post_id ) {
-                        if ( get_post_type( $post_id ) == $this->post_type ) {
-                            wp_delete_post( $post_id, true );
-                        }
-                    }
-                }
+                $table_name = $wpdb->prefix . $this->table_name;
+
+                $wpdb->delete(
+                    $table_name,
+                    [ 'source' => $source_url ],
+                    [ '%s' ]
+                );
             }
 
@@ -1427 +1037 @@
      */
     public function back_script_enqueuer( $screen ) {
-        $post_type = get_post_type();
-        if ( $screen == 'edit.php' && $post_type == 'blnotifier-results' ) {
+        if ( $screen == 'toplevel_page_' . BLNOTIFIER_TEXTDOMAIN ) {
             $handle = 'blnotifier_results_back_end_script';
-            wp_register_script( $handle, BLNOTIFIER_PLUGIN_JS_PATH.'results-back.js', [ 'jquery' ], BLNOTIFIER_VERSION, true );
+            wp_register_script( $handle, BLNOTIFIER_PLUGIN_JS_PATH.'results-back.js', [ 'jquery' ], time(), true );
             wp_localize_script( $handle, 'blnotifier_back_end', [
                 'verifying'     => !(new BLNOTIFIER_HELPERS())->is_results_verification_paused(),

broken-link-notifier/trunk/readme.txt

@@ -5 +5 @@
 Tested up to: 6.9
 Requires PHP: 7.4
-Stable tag: 1.3.6
+Stable tag: 1.3.7
 License: GPLv2 or later
 License URI: http://www.gnu.org/licenses/gpl-2.0.txt
@@ -127 +127 @@
 
 == Changelog ==
+= 1.3.7 =
+* Update: Changed storage of broken links from custom post type to custom database table
+* Update: Add option to cleanup on uninstall
+* Fix: Error when `blnotifier_link_before_prechecks` hook returns status
+
 = 1.3.6 =
 * Update: Added option for Max Links Per Page to help prevent attacks and timeouts