Issues in class-give-tools-delete-test-donors.php - New Issues - Inspection of "Merge pull request #1940 from WordImpress/issue/18..." - ravinderk/Give - Measure and Improve Code Quality continuously with Scrutinizer

Test Failed

Push — release/1.8.12 ( b58a2f...d255b1 )

by Ravinder

created 2018-08-27 05:28 UTC

tools/data/class-give-tools-delete-test-donors.php (13 issues)

Labels

Severity

Upgrade to new PHP Analysis Engine

These results are based on our legacy PHP analysis, consider migrating to our new PHP analysis engine instead. Learn more

<?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 array.
	 * @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 array.
	 * @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 array.
	 * @since 1.8.12
	 */
	var $step_key = 'give_temp_delete_step';

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

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

	/**
	 * Counting contain total number of step that completed.
	 * @var array.
	 * @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 integer
	 */
	public $per_step = 10;

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

	/**
	 * 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 ) );
			$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;

			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 );
			$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 ) {
				wp_delete_post( $item, true );
			}
		}



		// 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.12
	 *
	 * @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 array.
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 array.
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 array.
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 array.
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 array.
59		* @since 1.8.12
60		*/
61		var $total_step;
62
63		/**
64		* Counting contain total number of step that completed.
65		* @var array.
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 integer
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		* Get the Export Data
100		*
101		* @access public
102		* @since 1.8.12
103		* @global object $wpdb Used to query the database using the WordPress Database API
104		*
105		* @return array\|bool $data The data for the CSV file
106		*/
107		public function pre_fetch() {
108		$donation_ids = array();
109		$donor_ids = array();
110
111		// Check if the ajax request if running for the first time.
112		if ( 1 === (int) $this->step ) {
113		// Delete all the donation ids.
114		$this->delete_option( $this->donation_key );
115		// Delete all the donor ids.
116		$this->delete_option( $this->donor_key );
117
118		// Delete all the step and set to 'count' which if the first step in the process of deleting the donors.
119		$this->update_option( $this->step_key, 'count' );
120
121		// Delete tha page count of the step.
122		$this->update_option( $this->step_on_key, '0' );
123		} else {
124		// Get the old donors list.
125		$donor_ids = $this->get_option( $this->donor_key );
126
127		// Get the old donation list.
128		$donation_ids = $this->get_option( $this->donation_key );
129		}
130
131		// Get the step and check for it if it's on the first step( 'count' ) or not.
132		$step = (int) $this->get_step();
133		if ( 1 === $step ) {
134		/**
135		* Will add or update the donation and donor data by running wp query.
136		*/
137		$this->count( $step, $donation_ids, $donor_ids );
138		}
139		}
140
141		/**
142		* Will Update or Add the donation and donors ids in the with option table for there respected key.
143		*
144		* @param string $step On which the current ajax is running.
145		* @param array $donation_ids Contain the list of all the donation id's that has being add before this
146		* @param array $donor_ids Contain the list of all the donors id's that has being add before this
147		*/
148		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 Show Similar Issues like this 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...
149
150		// Get the Page count by default it's zero.
151		$paged = (int) $this->get_step_page();
152		// Incresed the page count by one.
153		++ $paged;
154
155		/**
156		* Filter add to alter the argument before the wp quest run
157		*/
158		$args = apply_filters( 'give_tools_reset_stats_total_args', array(
159		'post_type' => 'give_payment',
160		'post_status' => 'any',
161		'posts_per_page' => $this->per_step,
162		'paged' => $paged,
163		// ONLY TEST MODE TRANSACTIONS!!!
164		'meta_key' => '_give_payment_mode',
		0 ignored issues – show introduced 2017-08-01 08:32 UTC by Report Bug Copy Issue Report Show Similar Issues like this Detected usage of meta_key, possible slow query. Loading history...
165		'meta_value' => 'test'
		0 ignored issues – show introduced 2017-08-01 08:32 UTC by Report Bug Copy Issue Report Show Similar Issues like this Detected usage of meta_value, possible slow query. Loading history... introduced 2017-08-01 08:32 UTC by Report Bug Copy Issue Report Show Similar Issues like this Each line in an array declaration must end in a comma Loading history...
166		) );
167
168		// Reset the post data.
169		wp_reset_postdata();
170		// Getting the new donation.
171		$donation_posts = new WP_Query( $args );
172
173		// The Loop.
174		if ( $donation_posts->have_posts() ) {
175		while ( $donation_posts->have_posts() ) {
176		$donation_posts->the_post();
177		global $post;
178		// Add the donation id in side the array.
179		$donation_ids[] = $post->ID;
180
181		// Add the donor id in side the array.
182		$donor_ids[] = (int) $post->post_author;
183		}
184		/* Restore original Post Data */
185		}
186
187		// Get the total number of post found.
188		$total_donation = (int) $donation_posts->found_posts;
189
190		// Maximum number of page can be display
191		$max_num_pages = (int) $donation_posts->max_num_pages;
192
193		// Check current page is less then max number of page or not
194		if ( $paged < $max_num_pages ) {
195		// Update the curretn page virable for the next step
196		$this->update_option( $this->step_on_key, $paged );
197
198		// Calculating percentage.
199		$page_remain = $max_num_pages - $paged;
200		$this->total_step = (int) $max_num_pages + ( $total_donation / $this->per_step ) + ( ( $page_remain * 2 ) * count( $donor_ids ) );
201		$this->step_completed = $paged;
202		} else {
203		$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 Show Similar Issues like this `$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...
204		$this->update_option( $this->step_key, 'donation' );
205		$this->update_option( $this->step_on_key, '0' );
206		}
207
208		$donor_ids = array_unique( $donor_ids );
209		$this->update_option( $this->donor_key, $donor_ids );
210		$this->update_option( $this->donation_key, $donation_ids );
211
212		wp_reset_postdata();
213		}
214
215		/**
216		* Return the calculated completion percentage.
217		*
218		* @since 1.8.12
219		* @return int
220		*/
221		public function get_percentage_complete() {
222		return ceil( ( 100 * $this->step_completed ) / $this->total_step );
223		}
224
225	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 Show Similar Issues like this 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...
226
227		if ( ! $this->can_export() ) {
228		wp_die( __( 'You do not have permission to delete test transactions.', 'give' ), __( 'Error', 'give' ), array( 'response' => 403 ) );
229		}
230
231		$had_data = $this->get_data();
232
233		if ( $had_data ) {
234		$this->done = false;
235
236		return true;
237		} else {
238		update_option( 'give_earnings_total', give_get_total_earnings( true ) );
239		Give_Cache::delete( Give_Cache::get_key( 'give_estimated_monthly_stats' ) );
240
241		$this->delete_option( $this->donation_key );
242
243		// Reset the sequential order numbers
244		if ( give_get_option( 'enable_sequential' ) ) {
245		delete_option( 'give_last_payment_number' );
246		}
247
248		$this->done = true;
249		$this->message = __( 'Test donor and transactions successfully deleted.', 'give' );
250
251		return false;
252		}
253		}
254
255		/**
256		* Get the Export Data
257		*
258		* @access public
259		* @since 1.8.12
260		* @global object $wpdb Used to query the database using the WordPress Database API
261		*
262		* @return array\|bool $data The data for the CSV file
263		*/
264		public function get_data() {
265
266		// Get the donation id's.
267		$donation_ids = $this->get_option( $this->donation_key );
268
269		/**
270		* Return false id not test donation is found.
271		*/
272		if ( empty( $donation_ids ) ) {
273		$this->is_empty = true;
274		$this->total_step = 1;
275
276		return false;
277		}
278
279		// Get the current step.
280		$step = (int) $this->get_step();
281
282		// get teh donor ids.
283		$donor_ids = $this->get_option( $this->donor_key );
284
285		// In step to we delete all the donation in loop.
286		if ( 2 === $step ) {
287		$pass_to_donor = false;
288		$page = (int) $this->get_step_page();
289		$page ++;
290		$count = count( $donation_ids );
291
292		$this->total_step = ( ( count( $donation_ids ) / $this->per_step ) * 2 ) + count( $donor_ids );
293		$this->step_completed = $page;
294
		0 ignored issues – show Coding Style introduced 2017-08-01 08:32 UTC by Report Bug Copy Issue Report Show Similar Issues like this Functions must not contain multiple empty lines in a row; found 2 empty lines Loading history...
295
296		if ( $count > $this->per_step ) {
297
298		$this->update_option( $this->step_on_key, $page );
299		$donation_ids = $this->get_delete_ids( $donation_ids, $page );
300		$current_page = (int) ceil( $count / $this->per_step );
301
302		if ( $page === $current_page ) {
303		$pass_to_donor = true;
304		}
305		} else {
306		$pass_to_donor = true;
307		}
308
309		if ( true === $pass_to_donor ) {
310		$this->update_option( $this->step_key, 'donor' );
311		$this->update_option( $this->step_on_key, '0' );
312		}
313
314		foreach ( $donation_ids as $item ) {
315		wp_delete_post( $item, true );
316		}
317		}
318
		0 ignored issues – show Coding Style introduced 2017-08-01 08:32 UTC by Report Bug Copy Issue Report Show Similar Issues like this Functions must not contain multiple empty lines in a row; found 2 empty lines Loading history...
319
320		// Here we delete all the donor
321		if ( 3 === $step ) {
322		$page = (int) $this->get_step_page();
323		$count = count( $donor_ids );
324
325		$this->total_step = ( ( count( $donation_ids ) / $this->per_step ) * 2 ) + count( $donor_ids );
326		$this->step_completed = $page + ( count( $donation_ids ) / $this->per_step );
327
328		$args = apply_filters( 'give_tools_reset_stats_total_args', array(
329		'post_type' => 'give_payment',
330		'post_status' => 'any',
331		'posts_per_page' => 1,
332		'meta_key' => '_give_payment_mode',
		0 ignored issues – show introduced 2017-08-01 08:32 UTC by Report Bug Copy Issue Report Show Similar Issues like this Detected usage of meta_key, possible slow query. Loading history...
333		'meta_value' => 'live',
		0 ignored issues – show introduced 2017-08-01 08:32 UTC by Report Bug Copy Issue Report Show Similar Issues like this Detected usage of meta_value, possible slow query. Loading history...
334		'author' => $donor_ids[ $page ]
		0 ignored issues – show introduced 2017-08-01 08:32 UTC by Report Bug Copy Issue Report Show Similar Issues like this Each line in an array declaration must end in a comma Loading history...
335		) );
336
337		$donation_posts = get_posts( $args );
338		if ( empty( $donation_posts ) ) {
339		Give()->donors->delete_by_user_id( $donor_ids[ $page ] );
340		}
341
342		$page ++;
343		$this->update_option( $this->step_on_key, $page );
344		if ( $count === $page ) {
345		$this->is_empty = false;
346
347		return false;
348		}
349
350		return true;
351		}
352
353		return true;
354		}
355
356		public function get_delete_ids( $donation_ids, $page ) {
357		$index = $page --;
		0 ignored issues – show Unused Code introduced 2017-08-01 08:32 UTC by Report Bug Copy Issue Report Show Similar Issues like this `$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...
358		$count = count( $donation_ids );
		0 ignored issues – show Unused Code introduced 2017-08-01 08:32 UTC by Report Bug Copy Issue Report Show Similar Issues like this `$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...
359		$temp = 0;
360		$current_page = 0;
361		$post_delete = $this->per_step;
362		$page_donation_id = array();
363
364		foreach ( $donation_ids as $item ) {
365		$temp ++;
366		$page_donation_id[ $current_page ][] = $item;
367		if ( $temp === $post_delete ) {
368		$current_page ++;
369		$temp = 0;
370		}
371		}
372
373		return $page_donation_id[ $page ];
374		}
375
376		/**
377		* Given a key, get the information from the Database Directly
378		*
379		* @since 1.8.12
380		*
381		* @param string $key The option_name
382		*
383		* @return mixed Returns the data from the database
384		*/
385		public function get_option( $key, $defalut_value = false ) {
386		return get_option( $key, $defalut_value );
387		}
388
389		/**
390		* Give a key, store the value
391		*
392		* @since 1.8.12s
393		*
394		* @param string $key The option_name
395		* @param mixed $value The value to store
396		*
397		* @return void
398		*/
399		public function update_option( $key, $value ) {
400		update_option( $key, $value, false );
401		}
402
403		/**
404		* Delete an option
405		*
406		* @since 1.8.12
407		*
408		* @param string $key The option_name to delete
409		*
410		* @return void
411		*/
412		public function delete_option( $key ) {
413		delete_option( $key );
414		}
415
416		/**
417		* Get the current step in number.
418		*
419		* There are three step to delete the total donor first counting, second deleting donotion and third deleting donors.
420		*
421		* @return int\|string
422		*/
423		private function get_step() {
424		$step_key = (string) $this->get_option( $this->step_key, false );
425		if ( 'count' === $step_key ) {
426		return 1;
427		} elseif ( 'donation' === $step_key ) {
428		return 2;
429		} elseif ( 'donor' === $step_key ) {
430		return 3;
431		} else {
432		return $step_key;
433		}
434		}
435
436		/**
437		* Get the current $page value in the ajax.
438		*/
439		private function get_step_page() {
440		return $this->get_option( $this->step_on_key, false );
441		}
442		}
443

ravinderk / Give

Push — release/1.8.12 ( b58a2f...d255b1 )

tools/data/class-give-tools-delete-test-donors.php (13 issues)

Labels

Severity

Introduced By

Upgrade to new PHP Analysis Engine

Duplication Side-by-Side

Filter issues like