Give_Tools_Delete_Donors - Code Metrics - Inspection of "Merge pull request #2753 from WordImpress/release/..." - WordImpress/Give - Measure and Improve Code Quality continuously with Scrutinizer

Test Failed

Push — master ( cd44a0...d8bbfb )

by Devin

created 2018-01-31 19:39 UTC

Give_Tools_Delete_Donors A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	430
Duplicated Lines	27.44 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes

Metric	Value
dl	118
loc	430
rs	9.2
c	0
b	0
f	0
wmc	34
lcom	1
cbo	4

12 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
B	pre_fetch()	33	33	3
B	count()	13	66	4
A	get_percentage_complete()	0	3	1
B	process_step()	29	29	4
C	get_data()	12	93	10
A	get_delete_ids()	19	19	3
A	get_option()	0	3	1
A	update_option()	0	3	1
A	delete_option()	0	3	1
A	get_step()	12	12	4
A	get_step_page()	0	3	1

How to fix Duplicated Code

<?php
/**
 * Delete Donors.
 *
 * This class handles batch processing of deleting donor data.
 *
 * @subpackage  Admin/Tools/Give_Tools_Delete_Donors
 * @copyright   Copyright (c) 2016, WordImpress
 * @license     https://opensource.org/licenses/gpl-license GNU Public License
 * @since       1.8.12
 */

// Exit if accessed directly.
if ( ! defined( 'ABSPATH' ) ) {
	exit;
}


/**
 * Give_Tools_Delete_Donors Class
 *
 * @since 1.8.12
 */
class Give_Tools_Delete_Donors extends Give_Batch_Export {

	var $request;

	/**
	 * Used to store donation id's that are going to get deleted.
	 * @var string
	 * @since 1.8.12
	 */
	var $donation_key = 'give_temp_delete_donation_ids';

	/**
	 * Used to store donors id's that are going to get deleted.
	 * @var string
	 * @since 1.8.12
	 */
	var $donor_key = 'give_temp_delete_donor_ids';

	/**
	 * Used to store the step where the step will be. ( 'count', 'donations', 'donors' ).
	 * @var string
	 * @since 1.8.12
	 */
	var $step_key = 'give_temp_delete_step';

	/**
	 * Used to store to get the page count in the loop.
	 * @var string
	 * @since 1.8.12
	 */
	var $step_on_key = 'give_temp_delete_step_on';

	/**
	 * Contain total number of step .
	 * @var string
	 * @since 1.8.12
	 */
	var $total_step;

	/**
	 * Counting contain total number of step that completed.
	 * @var int
	 * @since 1.8.12
	 */
	var $step_completed;

	/**
	 * Our export type. Used for export-type specific filters/actions
	 * @var string
	 * @since 1.8.12
	 */
	public $export_type = '';

	/**
	 * Allows for a non-form batch processing to be run.
	 * @since  1.8.12
	 * @var boolean
	 */
	public $is_void = true;

	/**
	 * Sets the number of items to pull on each step
	 * @since  1.8.12
	 * @var int
	 */
	public $per_step = 10;

	/**
	 * Set's all the donors id's
	 * @since  1.8.12
	 * @var array
	 */
	public $donor_ids = array();

	/**
	 * Constructor.
	 */
	public function __construct( $_step = 1 ) {
		parent::__construct( $_step );

		$this->is_writable = true;
	}

	/**
	 * Get the Export Data
	 *
	 * @access public
	 * @since  1.8.12
	 * @global object $wpdb Used to query the database using the WordPress Database API
	 *
	 * @return array|bool $data The data for the CSV file
	 */
	public function pre_fetch() {

		$donation_ids = array();
		$donor_ids    = array();

		// Check if the ajax request if running for the first time.
		if ( 1 === (int) $this->step ) {
			// Delete all the donation ids.
			$this->delete_option( $this->donation_key );
			// Delete all the donor ids.
			$this->delete_option( $this->donor_key );

			// Delete all the step and set to 'count' which if the first step in the process of deleting the donors.
			$this->update_option( $this->step_key, 'count' );

			// Delete tha page count of the step.
			$this->update_option( $this->step_on_key, '0' );
		} else {
			// Get the old donors list.
			$donor_ids = $this->get_option( $this->donor_key );

			// Get the old donation list.
			$donation_ids = $this->get_option( $this->donation_key );
		}

		// Get the step and check for it if it's on the first step( 'count' ) or not.
		$step = (int) $this->get_step();
		if ( 1 === $step ) {
			/**
			 * Will add or update the donation and donor data by running wp query.
			 */
			$this->count( $step, $donation_ids, $donor_ids );
		}
	}

	/**
	 * Will Update or Add the donation and donors ids in the with option table for there respected key.
	 *
	 * @param string $step On which the current ajax is running.
	 * @param array $donation_ids Contain the list of all the donation id's that has being add before this
	 * @param array $donor_ids Contain the list of all the donors id's that has being add before this
	 */
	private function count( $step, $donation_ids = array(), $donor_ids = array() ) {


		// Get the Page count by default it's zero.
		$paged = (int) $this->get_step_page();
		// Incresed the page count by one.
		++ $paged;

		/**
		 * Filter add to alter the argument before the wp quest run
		 */
		$args = apply_filters( 'give_tools_reset_stats_total_args', array(
			'post_type'      => 'give_payment',
			'post_status'    => 'any',
			'posts_per_page' => $this->per_step,
			'paged'          => $paged,
			// ONLY TEST MODE TRANSACTIONS!!!
			'meta_key'       => '_give_payment_mode',

			'meta_value'     => 'test',

		) );

		// Reset the post data.
		wp_reset_postdata();
		// Getting the new donation.
		$donation_posts = new WP_Query( $args );

		// The Loop.
		if ( $donation_posts->have_posts() ) {
			while ( $donation_posts->have_posts() ) {
				$donation_posts->the_post();
				global $post;
				// Add the donation id in side the array.
				$donation_ids[] = $post->ID;

				// Add the donor id in side the array.
				$donor_ids[] = (int) $post->post_author;
			}
			/* Restore original Post Data */
		}

		// Get the total number of post found.
		$total_donation = (int) $donation_posts->found_posts;

		// Maximum number of page can be display
		$max_num_pages = (int) $donation_posts->max_num_pages;

		// Check current page is less then max number of page or not
		if ( $paged < $max_num_pages ) {

			// Update the curretn page virable for the next step
			$this->update_option( $this->step_on_key, $paged );

			// Calculating percentage.
			$page_remain          = $max_num_pages - $paged;
			$this->total_step     = (int) $max_num_pages + ( $total_donation / $this->per_step ) + ( ( $page_remain * 2 ) * count( $donor_ids ) );
$answer = 42;

$correct = false;

$correct = (bool) $answer;
			$this->step_completed = $paged;
		} else {
			$donation_ids_count = count( $donor_ids );
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}
			$this->update_option( $this->step_key, 'donation' );
			$this->update_option( $this->step_on_key, '0' );
		}

		$donor_ids = array_unique( $donor_ids );
		$this->update_option( $this->donor_key, $donor_ids );
		$this->update_option( $this->donation_key, $donation_ids );

		wp_reset_postdata();
	}

	/**
	 * Return the calculated completion percentage.
	 *
	 * @since 1.8.12
	 * @return int
	 */
	public function get_percentage_complete() {
		return ceil( ( 100 * $this->step_completed ) / $this->total_step );
	}

	public function process_step() {


		if ( ! $this->can_export() ) {
			wp_die( __( 'You do not have permission to delete test transactions.', 'give' ), __( 'Error', 'give' ), array( 'response' => 403 ) );
		}

		$had_data = $this->get_data();

		if ( $had_data ) {
			$this->done = false;

			return true;
		} else {
			update_option( 'give_earnings_total', give_get_total_earnings( true ) );
			Give_Cache::delete( Give_Cache::get_key( 'give_estimated_monthly_stats' ) );

			$this->delete_option( $this->donation_key );

			// Reset the sequential order numbers
			if ( give_get_option( 'enable_sequential' ) ) {
				delete_option( 'give_last_payment_number' );
			}

			$this->done    = true;
			$this->message = __( 'Test donor and transactions successfully deleted.', 'give' );

			return false;
		}
	}

	/**
	 * Get the Export Data
	 *
	 * @access public
	 * @since  1.8.12
	 * @global object $wpdb Used to query the database using the WordPress Database API
	 *
	 * @return array|bool $data The data for the CSV file
	 */
	public function get_data() {

		// Get the donation id's.
		$donation_ids = $this->get_option( $this->donation_key );

		/**
		 * Return false id not test donation is found.
		 */
		if ( empty( $donation_ids ) ) {
			$this->is_empty   = true;
			$this->total_step = 1;
$answer = 42;

$correct = false;

$correct = (bool) $answer;

			return false;
		}

		// Get the current step.
		$step = (int) $this->get_step();

		// get teh donor ids.
		$donor_ids = $this->get_option( $this->donor_key );

		// In step to we delete all the donation in loop.
		if ( 2 === $step ) {
			$pass_to_donor = false;
			$page          = (int) $this->get_step_page();
			$page ++;
			$count = count( $donation_ids );

			$this->total_step     = ( ( count( $donation_ids ) / $this->per_step ) * 2 ) + count( $donor_ids );
$answer = 42;

$correct = false;

$correct = (bool) $answer;
			$this->step_completed = $page;



			if ( $count > $this->per_step ) {


				$this->update_option( $this->step_on_key, $page );
				$donation_ids = $this->get_delete_ids( $donation_ids, $page );
				$current_page = (int) ceil( $count / $this->per_step );

				if ( $page === $current_page ) {
					$pass_to_donor = true;
				}
			} else {
				$pass_to_donor = true;
			}

			if ( true === $pass_to_donor ) {
				$this->update_option( $this->step_key, 'donor' );
				$this->update_option( $this->step_on_key, '0' );
			}

			foreach ( $donation_ids as $item ) {
				// Delete the main payment.
				give_delete_donation( absint( $item ) );
			}
			do_action( 'give_delete_log_cache' );
		}



		// Here we delete all the donor
		if ( 3 === $step ) {
			$page  = (int) $this->get_step_page();
			$count = count( $donor_ids );

			$this->total_step     = ( ( count( $donation_ids ) / $this->per_step ) * 2 ) + count( $donor_ids );
			$this->step_completed = $page + ( count( $donation_ids ) / $this->per_step );

			$args = apply_filters( 'give_tools_reset_stats_total_args', array(
				'post_type'      => 'give_payment',
				'post_status'    => 'any',
				'posts_per_page' => 1,
				'meta_key'       => '_give_payment_mode',

				'meta_value'     => 'live',

				'author'         => $donor_ids[ $page ],
			) );

			$donation_posts = get_posts( $args );
			if ( empty( $donation_posts ) ) {
				Give()->donors->delete_by_user_id( $donor_ids[ $page ] );
			}

			$page ++;
			$this->update_option( $this->step_on_key, $page );
			if ( $count === $page ) {
				$this->is_empty = false;

				return false;
			}

			return true;
		}

		return true;
	}

	public function get_delete_ids( $donation_ids, $page ) {

		$index            = $page --;
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}
		$count            = count( $donation_ids );
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}
		$temp             = 0;
		$current_page     = 0;
		$post_delete      = $this->per_step;
		$page_donation_id = array();

		foreach ( $donation_ids as $item ) {
			$temp ++;
			$page_donation_id[ $current_page ][] = $item;
			if ( $temp === $post_delete ) {
				$current_page ++;
				$temp = 0;
			}
		}

		return $page_donation_id[ $page ];
	}

	/**
	 * Given a key, get the information from the Database Directly
	 *
	 * @since  1.8.13
	 *
	 * @param  string $key The option_name
	 *
	 * @return mixed       Returns the data from the database
	 */
	public function get_option( $key, $defalut_value = false ) {
		return get_option( $key, $defalut_value );
	}

	/**
	 * Give a key, store the value
	 *
	 * @since  1.8.12s
	 *
	 * @param  string $key The option_name
	 * @param  mixed $value The value to store
	 *
	 * @return void
	 */
	public function update_option( $key, $value ) {
		update_option( $key, $value, false );
	}

	/**
	 * Delete an option
	 *
	 * @since  1.8.12
	 *
	 * @param  string $key The option_name to delete
	 *
	 * @return void
	 */
	public function delete_option( $key ) {
		delete_option( $key );
	}

	/**
	 * Get the current step in number.
	 *
	 * There are three step to delete the total donor first counting, second deleting donotion and third deleting donors.
	 *
	 * @return int|string
	 */
	private function get_step() {

		$step_key = (string) $this->get_option( $this->step_key, false );
		if ( 'count' === $step_key ) {
			return 1;
		} elseif ( 'donation' === $step_key ) {
			return 2;
		} elseif ( 'donor' === $step_key ) {
			return 3;
		} else {
			return $step_key;
		}
	}

	/**
	 * Get the current $page value in the ajax.
	 */
	private function get_step_page() {
		return $this->get_option( $this->step_on_key, false );
	}
}


1		<?php
2		/**
3		* Delete Donors.
4		*
5		* This class handles batch processing of deleting donor data.
6		*
7		* @subpackage Admin/Tools/Give_Tools_Delete_Donors
8		* @copyright Copyright (c) 2016, WordImpress
9		* @license https://opensource.org/licenses/gpl-license GNU Public License
10		* @since 1.8.12
11		*/
12
13		// Exit if accessed directly.
14		if ( ! defined( 'ABSPATH' ) ) {
15		exit;
16		}
17
18
19		/**
20		* Give_Tools_Delete_Donors Class
21		*
22		* @since 1.8.12
23		*/
24		class Give_Tools_Delete_Donors extends Give_Batch_Export {
25
26		var $request;
27
28		/**
29		* Used to store donation id's that are going to get deleted.
30		* @var string
31		* @since 1.8.12
32		*/
33		var $donation_key = 'give_temp_delete_donation_ids';
34
35		/**
36		* Used to store donors id's that are going to get deleted.
37		* @var string
38		* @since 1.8.12
39		*/
40		var $donor_key = 'give_temp_delete_donor_ids';
41
42		/**
43		* Used to store the step where the step will be. ( 'count', 'donations', 'donors' ).
44		* @var string
45		* @since 1.8.12
46		*/
47		var $step_key = 'give_temp_delete_step';
48
49		/**
50		* Used to store to get the page count in the loop.
51		* @var string
52		* @since 1.8.12
53		*/
54		var $step_on_key = 'give_temp_delete_step_on';
55
56		/**
57		* Contain total number of step .
58		* @var string
59		* @since 1.8.12
60		*/
61		var $total_step;
62
63		/**
64		* Counting contain total number of step that completed.
65		* @var int
66		* @since 1.8.12
67		*/
68		var $step_completed;
69
70		/**
71		* Our export type. Used for export-type specific filters/actions
72		* @var string
73		* @since 1.8.12
74		*/
75		public $export_type = '';
76
77		/**
78		* Allows for a non-form batch processing to be run.
79		* @since 1.8.12
80		* @var boolean
81		*/
82		public $is_void = true;
83
84		/**
85		* Sets the number of items to pull on each step
86		* @since 1.8.12
87		* @var int
88		*/
89		public $per_step = 10;
90
91		/**
92		* Set's all the donors id's
93		* @since 1.8.12
94		* @var array
95		*/
96		public $donor_ids = array();
97
98		/**
99		* Constructor.
100		*/
101		public function __construct( $_step = 1 ) {
102		parent::__construct( $_step );
103
104		$this->is_writable = true;
105		}
106
107		/**
108		* Get the Export Data
109		*
110		* @access public
111		* @since 1.8.12
112		* @global object $wpdb Used to query the database using the WordPress Database API
113		*
114		* @return array\|bool $data The data for the CSV file
115		*/
116	View Code Duplication	public function pre_fetch() {
		0 ignored issues – show Duplication introduced 2017-09-07 23:03 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
117		$donation_ids = array();
118		$donor_ids = array();
119
120		// Check if the ajax request if running for the first time.
121		if ( 1 === (int) $this->step ) {
122		// Delete all the donation ids.
123		$this->delete_option( $this->donation_key );
124		// Delete all the donor ids.
125		$this->delete_option( $this->donor_key );
126
127		// Delete all the step and set to 'count' which if the first step in the process of deleting the donors.
128		$this->update_option( $this->step_key, 'count' );
129
130		// Delete tha page count of the step.
131		$this->update_option( $this->step_on_key, '0' );
132		} else {
133		// Get the old donors list.
134		$donor_ids = $this->get_option( $this->donor_key );
135
136		// Get the old donation list.
137		$donation_ids = $this->get_option( $this->donation_key );
138		}
139
140		// Get the step and check for it if it's on the first step( 'count' ) or not.
141		$step = (int) $this->get_step();
142		if ( 1 === $step ) {
143		/**
144		* Will add or update the donation and donor data by running wp query.
145		*/
146		$this->count( $step, $donation_ids, $donor_ids );
147		}
148		}
149
150		/**
151		* Will Update or Add the donation and donors ids in the with option table for there respected key.
152		*
153		* @param string $step On which the current ajax is running.
154		* @param array $donation_ids Contain the list of all the donation id's that has being add before this
155		* @param array $donor_ids Contain the list of all the donors id's that has being add before this
156		*/
157		private function count( $step, $donation_ids = array(), $donor_ids = array() ) {
		0 ignored issues – show Unused Code introduced 2017-08-01 08:32 UTC by Report Bug Copy Issue Report The parameter `$step` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history...
158
159		// Get the Page count by default it's zero.
160		$paged = (int) $this->get_step_page();
161		// Incresed the page count by one.
162		++ $paged;
163
164		/**
165		* Filter add to alter the argument before the wp quest run
166		*/
167		$args = apply_filters( 'give_tools_reset_stats_total_args', array(
168		'post_type' => 'give_payment',
169		'post_status' => 'any',
170		'posts_per_page' => $this->per_step,
171		'paged' => $paged,
172		// ONLY TEST MODE TRANSACTIONS!!!
173		'meta_key' => '_give_payment_mode',
		0 ignored issues – show introduced 2017-08-01 08:32 UTC by Report Bug Copy Issue Report Detected usage of meta_key, possible slow query. Loading history...
174		'meta_value' => 'test',
		0 ignored issues – show introduced 2017-08-01 08:32 UTC by Report Bug Copy Issue Report Detected usage of meta_value, possible slow query. Loading history...
175		) );
176
177		// Reset the post data.
178		wp_reset_postdata();
179		// Getting the new donation.
180		$donation_posts = new WP_Query( $args );
181
182		// The Loop.
183		if ( $donation_posts->have_posts() ) {
184		while ( $donation_posts->have_posts() ) {
185		$donation_posts->the_post();
186		global $post;
187		// Add the donation id in side the array.
188		$donation_ids[] = $post->ID;
189
190		// Add the donor id in side the array.
191		$donor_ids[] = (int) $post->post_author;
192		}
193		/* Restore original Post Data */
194		}
195
196		// Get the total number of post found.
197		$total_donation = (int) $donation_posts->found_posts;
198
199		// Maximum number of page can be display
200		$max_num_pages = (int) $donation_posts->max_num_pages;
201
202		// Check current page is less then max number of page or not
203	View Code Duplication	if ( $paged < $max_num_pages ) {
		0 ignored issues – show Duplication introduced 2017-09-07 23:03 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
204		// Update the curretn page virable for the next step
205		$this->update_option( $this->step_on_key, $paged );
206
207		// Calculating percentage.
208		$page_remain = $max_num_pages - $paged;
209		$this->total_step = (int) $max_num_pages + ( $total_donation / $this->per_step ) + ( ( $page_remain * 2 ) * count( $donor_ids ) );
		0 ignored issues – show Documentation Bug introduced 2017-08-14 10:27 UTC by Report Bug Copy Issue Report The property `$total_step` was declared of type `string`, but `(int) $max_num_pages + $...* 2 * count($donor_ids)` is of type `integer`. Maybe add a type cast? This check looks for assignments to scalar types that may be of the wrong type. To ensure the code behaves as expected, it may be a good idea to add an explicit type cast. $answer = 42; $correct = false; $correct = (bool) $answer; Loading history...
210		$this->step_completed = $paged;
211		} else {
212		$donation_ids_count = count( $donor_ids );
		0 ignored issues – show Unused Code introduced 2017-08-01 08:32 UTC by Report Bug Copy Issue Report `$donation_ids_count` is not used, you could remove the assignment. This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently. $myVar = 'Value'; $higher = false; if (rand(1, 6) > 3) { $higher = true; } else { $higher = false; } Both the `$myVar` assignment in line 1 and the `$higher` assignment in line 2 are dead. The first because `$myVar` is never used and the second because `$higher` is always overwritten for every possible time line. Loading history...
213		$this->update_option( $this->step_key, 'donation' );
214		$this->update_option( $this->step_on_key, '0' );
215		}
216
217		$donor_ids = array_unique( $donor_ids );
218		$this->update_option( $this->donor_key, $donor_ids );
219		$this->update_option( $this->donation_key, $donation_ids );
220
221		wp_reset_postdata();
222		}
223
224		/**
225		* Return the calculated completion percentage.
226		*
227		* @since 1.8.12
228		* @return int
229		*/
230		public function get_percentage_complete() {
231		return ceil( ( 100 * $this->step_completed ) / $this->total_step );
232		}
233
234	View Code Duplication	public function process_step() {
		0 ignored issues – show Duplication introduced 2017-08-01 08:32 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
235
236		if ( ! $this->can_export() ) {
237		wp_die( __( 'You do not have permission to delete test transactions.', 'give' ), __( 'Error', 'give' ), array( 'response' => 403 ) );
238		}
239
240		$had_data = $this->get_data();
241
242		if ( $had_data ) {
243		$this->done = false;
244
245		return true;
246		} else {
247		update_option( 'give_earnings_total', give_get_total_earnings( true ) );
248		Give_Cache::delete( Give_Cache::get_key( 'give_estimated_monthly_stats' ) );
249
250		$this->delete_option( $this->donation_key );
251
252		// Reset the sequential order numbers
253		if ( give_get_option( 'enable_sequential' ) ) {
254		delete_option( 'give_last_payment_number' );
255		}
256
257		$this->done = true;
258		$this->message = __( 'Test donor and transactions successfully deleted.', 'give' );
259
260		return false;
261		}
262		}
263
264		/**
265		* Get the Export Data
266		*
267		* @access public
268		* @since 1.8.12
269		* @global object $wpdb Used to query the database using the WordPress Database API
270		*
271		* @return array\|bool $data The data for the CSV file
272		*/
273		public function get_data() {
274
275		// Get the donation id's.
276		$donation_ids = $this->get_option( $this->donation_key );
277
278		/**
279		* Return false id not test donation is found.
280		*/
281		if ( empty( $donation_ids ) ) {
282		$this->is_empty = true;
283		$this->total_step = 1;
		0 ignored issues – show Documentation Bug introduced 2017-08-14 10:27 UTC by Report Bug Copy Issue Report The property `$total_step` was declared of type `string`, but `1` is of type `integer`. Maybe add a type cast? This check looks for assignments to scalar types that may be of the wrong type. To ensure the code behaves as expected, it may be a good idea to add an explicit type cast. $answer = 42; $correct = false; $correct = (bool) $answer; Loading history...
284
285		return false;
286		}
287
288		// Get the current step.
289		$step = (int) $this->get_step();
290
291		// get teh donor ids.
292		$donor_ids = $this->get_option( $this->donor_key );
293
294		// In step to we delete all the donation in loop.
295		if ( 2 === $step ) {
296		$pass_to_donor = false;
297		$page = (int) $this->get_step_page();
298		$page ++;
299		$count = count( $donation_ids );
300
301		$this->total_step = ( ( count( $donation_ids ) / $this->per_step ) * 2 ) + count( $donor_ids );
		0 ignored issues – show Documentation Bug introduced 2017-08-14 10:27 UTC by Report Bug Copy Issue Report The property `$total_step` was declared of type `string`, but `count($donation_ids) / $...* 2 + count($donor_ids)` is of type `integer`. Maybe add a type cast? This check looks for assignments to scalar types that may be of the wrong type. To ensure the code behaves as expected, it may be a good idea to add an explicit type cast. $answer = 42; $correct = false; $correct = (bool) $answer; Loading history...
302		$this->step_completed = $page;
303
		0 ignored issues – show Coding Style introduced 2017-08-01 08:32 UTC by Report Bug Copy Issue Report Functions must not contain multiple empty lines in a row; found 2 empty lines Loading history...
304
305	View Code Duplication	if ( $count > $this->per_step ) {
		0 ignored issues – show Duplication introduced 2017-09-07 23:03 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
306
307		$this->update_option( $this->step_on_key, $page );
308		$donation_ids = $this->get_delete_ids( $donation_ids, $page );
309		$current_page = (int) ceil( $count / $this->per_step );
310
311		if ( $page === $current_page ) {
312		$pass_to_donor = true;
313		}
314		} else {
315		$pass_to_donor = true;
316		}
317
318		if ( true === $pass_to_donor ) {
319		$this->update_option( $this->step_key, 'donor' );
320		$this->update_option( $this->step_on_key, '0' );
321		}
322
323		foreach ( $donation_ids as $item ) {
324		// Delete the main payment.
325		give_delete_donation( absint( $item ) );
326		}
327		do_action( 'give_delete_log_cache' );
328		}
329
		0 ignored issues – show Coding Style introduced 2017-08-13 16:10 UTC by Report Bug Copy Issue Report Functions must not contain multiple empty lines in a row; found 2 empty lines Loading history...
330
331		// Here we delete all the donor
332		if ( 3 === $step ) {
333		$page = (int) $this->get_step_page();
334		$count = count( $donor_ids );
335
336		$this->total_step = ( ( count( $donation_ids ) / $this->per_step ) * 2 ) + count( $donor_ids );
337		$this->step_completed = $page + ( count( $donation_ids ) / $this->per_step );
338
339		$args = apply_filters( 'give_tools_reset_stats_total_args', array(
340		'post_type' => 'give_payment',
341		'post_status' => 'any',
342		'posts_per_page' => 1,
343		'meta_key' => '_give_payment_mode',
		0 ignored issues – show introduced 2017-08-01 08:32 UTC by Report Bug Copy Issue Report Detected usage of meta_key, possible slow query. Loading history...
344		'meta_value' => 'live',
		0 ignored issues – show introduced 2017-08-01 08:32 UTC by Report Bug Copy Issue Report Detected usage of meta_value, possible slow query. Loading history...
345		'author' => $donor_ids[ $page ],
346		) );
347
348		$donation_posts = get_posts( $args );
349		if ( empty( $donation_posts ) ) {
350		Give()->donors->delete_by_user_id( $donor_ids[ $page ] );
351		}
352
353		$page ++;
354		$this->update_option( $this->step_on_key, $page );
355		if ( $count === $page ) {
356		$this->is_empty = false;
357
358		return false;
359		}
360
361		return true;
362		}
363
364		return true;
365		}
366
367	View Code Duplication	public function get_delete_ids( $donation_ids, $page ) {
		0 ignored issues – show Duplication introduced 2017-09-07 23:03 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
368		$index = $page --;
		0 ignored issues – show Unused Code introduced 2017-09-07 23:03 UTC by Report Bug Copy Issue Report `$index` is not used, you could remove the assignment. This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently. $myVar = 'Value'; $higher = false; if (rand(1, 6) > 3) { $higher = true; } else { $higher = false; } Both the `$myVar` assignment in line 1 and the `$higher` assignment in line 2 are dead. The first because `$myVar` is never used and the second because `$higher` is always overwritten for every possible time line. Loading history...
369		$count = count( $donation_ids );
		0 ignored issues – show Unused Code introduced 2017-09-07 23:03 UTC by Report Bug Copy Issue Report `$count` is not used, you could remove the assignment. This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently. $myVar = 'Value'; $higher = false; if (rand(1, 6) > 3) { $higher = true; } else { $higher = false; } Both the `$myVar` assignment in line 1 and the `$higher` assignment in line 2 are dead. The first because `$myVar` is never used and the second because `$higher` is always overwritten for every possible time line. Loading history...
370		$temp = 0;
371		$current_page = 0;
372		$post_delete = $this->per_step;
373		$page_donation_id = array();
374
375		foreach ( $donation_ids as $item ) {
376		$temp ++;
377		$page_donation_id[ $current_page ][] = $item;
378		if ( $temp === $post_delete ) {
379		$current_page ++;
380		$temp = 0;
381		}
382		}
383
384		return $page_donation_id[ $page ];
385		}
386
387		/**
388		* Given a key, get the information from the Database Directly
389		*
390		* @since 1.8.13
391		*
392		* @param string $key The option_name
393		*
394		* @return mixed Returns the data from the database
395		*/
396		public function get_option( $key, $defalut_value = false ) {
397		return get_option( $key, $defalut_value );
398		}
399
400		/**
401		* Give a key, store the value
402		*
403		* @since 1.8.12s
404		*
405		* @param string $key The option_name
406		* @param mixed $value The value to store
407		*
408		* @return void
409		*/
410		public function update_option( $key, $value ) {
411		update_option( $key, $value, false );
412		}
413
414		/**
415		* Delete an option
416		*
417		* @since 1.8.12
418		*
419		* @param string $key The option_name to delete
420		*
421		* @return void
422		*/
423		public function delete_option( $key ) {
424		delete_option( $key );
425		}
426
427		/**
428		* Get the current step in number.
429		*
430		* There are three step to delete the total donor first counting, second deleting donotion and third deleting donors.
431		*
432		* @return int\|string
433		*/
434	View Code Duplication	private function get_step() {
		0 ignored issues – show Duplication introduced 2017-09-07 23:03 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
435		$step_key = (string) $this->get_option( $this->step_key, false );
436		if ( 'count' === $step_key ) {
437		return 1;
438		} elseif ( 'donation' === $step_key ) {
439		return 2;
440		} elseif ( 'donor' === $step_key ) {
441		return 3;
442		} else {
443		return $step_key;
444		}
445		}
446
447		/**
448		* Get the current $page value in the ajax.
449		*/
450		private function get_step_page() {
451		return $this->get_option( $this->step_on_key, false );
452		}
453		}
454

WordImpress / Give

Push — master ( cd44a0...d8bbfb )

Give_Tools_Delete_Donors A

Complexity

Size/Duplication

Coupling/Cohesion

Importance

12 Methods

How to fix Duplicated Code

Duplicated Code

Duplication Side-by-Side

Filter issues like