Added cleanup records

[girishpanchal30]

Summary

Add a button Cleanup accounts to clean up the records and keep the last 1,000 records.

Screenshots

Check before Pull Request is ready:

• I have written a test and included it in this PR
• I have run all tests and they pass
• The code passes when running the PHP CodeSniffer
• Code meets WordPress Coding Standards for PHP, HTML, CSS and JS
• Security and Sanitization requirements have been followed
• I have assigned a reviewer or two to review this PR (if you're not sure who to assign, we can do this step for you)

Closes https://github.com/Codeinwp/tweet-old-post-pro/issues/431

[pirate-bot]

Plugin build for c90f11a is ready 🛎️!

• Download build

[stefan-cotitosu]

@girishpanchal30, could you please clarify on how to test this one?

I used a code snippet to add more than 1000 entries in the Sharing Logs and the Cleanup accounts button doesn't appear.

Below is the snippet I used:

add_action( 'init', function () {

    $option = 'rop_logs';

    $logs = get_option( $option, [] );

    // Safety: ensure it's an array
    if ( ! is_array( $logs ) ) {
        $logs = [];
    }

    $start = count( $logs );

    // Add dummy logs until we exceed 1000
    for ( $i = $start; $i < 1105; $i++ ) {
        $logs[] = [
            'time'    => time() + $i,
            'level'   => 'info',
            'message' => 'Dummy log entry #' . $i,
            'context' => 'test_cleanup',
        ];
    }

    update_option( $option, $logs, false );

    wp_die(
        'rop_logs now contains ' . count( $logs ) . ' entries. Remove this snippet.'
    );
});

[Soare-Robert-Daniel]

I used a code snippet to add more than 1000 entries in the Sharing Logs and the Cleanup accounts button doesn't appear.

I think this was a confusion. The implementation deletes the service accounts (twitter, Facebook) if they have more than 1000 registered. This should have been about the logs and not services.

[Soare-Robert-Daniel]

The issues should solve the problem with the logs and not services, based on the attached issue.

[girishpanchal30]

Sorry for the confusion. I’ve implemented logic to delete the logs once they have more than 1,000 logs. Please check it and let me know if any changes are needed.

[rodica-andronache]

@girishpanchal30, from what I've tried, it's working well.

However, I get this warning

[girishpanchal30]

@rodica-andronache Just to confirm, are you using the script provided by @stefan-cotitosu to add the dummy log? That script does not include the correct type attribute, which is why you might be seeing that notice.

If you’re not using that script, please share your instance details so I can check this further.

[rodica-andronache]

@girishpanchal30 I've used the same script provided by Stefan

[girishpanchal30]

@rodica-andronache You need to add 'type' => 'error' to the script. Please check again after adding it and let me know if you still see the notice.

[rodica-andronache]

thanks! Indeed, the notice is gone with that change in the code.