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