Give_Customer_Reports_Table::reports_data() - Code Metrics - Inspection of "Show correct donor count" - ravinderk/Give - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — issues/1486 ( 0cf53f )

by Ravinder

created 2017-02-14 09:53 UTC

Give_Customer_Reports_Table::reports_data() B

↳ Parent: Give_Customer_Reports_Table

Complexity

Conditions	4
Paths	2

Size

Total Lines	29
Code Lines	17

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	4
eloc	17
nc	2
nop	0
dl	0
loc	29
rs	8.5806
c	0
b	0
f	0

<?php

/**
 * Customer (Donor) Reports Table Class.
 *
 * @package     Give
 * @subpackage  Admin/Reports
 * @copyright   Copyright (c) 2016, WordImpress
 * @license     https://opensource.org/licenses/gpl-license GNU Public License
 * @since       1.0
 */

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

// Load WP_List_Table if not loaded
if ( ! class_exists( 'WP_List_Table' ) ) {
	require_once ABSPATH . 'wp-admin/includes/class-wp-list-table.php';
}

/**
 * Give_Customer_Reports_Table Class.
 *
 * Renders the Customer Reports table.
 *
 * @since 1.0
 */
class Give_Customer_Reports_Table extends WP_List_Table {

	/**
	 * Number of items per page.
	 *
	 * @var int
	 * @since 1.0
	 */
	public $per_page = 30;

	/**
	 * Number of donors found.
	 *
	 * @var int
	 * @since 1.0
	 */
	public $count = 0;

	/**
	 * Total donors.
	 *
	 * @var int
	 * @since 1.0
	 */
	public $total = 0;

	/**
	 * Get things started.
	 *
	 * @since 1.0
	 * @see   WP_List_Table::__construct()
	 */
	public function __construct() {

		// Set parent defaults
		parent::__construct( array(
			'singular' => esc_html__( 'Donor', 'give' ),     // Singular name of the listed records
			'plural'   => esc_html__( 'Donors', 'give' ),    // Plural name of the listed records
			'ajax'     => false                       // Does this table support ajax?
		) );

	}

	/**
	 * Show the search field.
	 *
	 * @since  1.0
	 * @access public
	 *
	 * @param string $text Label for the search box.
	 * @param string $input_id ID of the search box.
	 *
	 * @return void
	 */
	public function search_box( $text, $input_id ) {
		$input_id = $input_id . '-search-input';

		if ( ! empty( $_REQUEST['orderby'] ) ) {
			echo '<input type="hidden" name="orderby" value="' . esc_attr( $_REQUEST['orderby'] ) . '" />';
		}
		if ( ! empty( $_REQUEST['order'] ) ) {
			echo '<input type="hidden" name="order" value="' . esc_attr( $_REQUEST['order'] ) . '" />';
		}
		?>
		<p class="search-box" role="search">
			<label class="screen-reader-text" for="<?php echo $input_id ?>"><?php echo $text; ?>:</label>
			<input type="search" id="<?php echo $input_id ?>" name="s" value="<?php _admin_search_query(); ?>"/>
			<?php submit_button( $text, 'button', false, false, array( 'ID' => 'search-submit' ) ); ?>
		</p>
		<?php
	}

	/**
	 * This function renders most of the columns in the list table.
	 *
	 * @access public
	 * @since  1.0
	 *
	 * @param array  $item Contains all the data of the customers.
	 * @param string $column_name The name of the column.
	 *
	 * @return string Column Name.
	 */
	public function column_default( $item, $column_name ) {
		switch ( $column_name ) {

			case 'num_purchases' :
				$value = '<a href="' .
				         admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&user=' . urlencode( $item['email'] )
				         ) . '">' . esc_html( $item['num_purchases'] ) . '</a>';
				break;

			case 'amount_spent' :
				$value = give_currency_filter( give_format_amount( $item[ $column_name ] ) );
				break;

			case 'date_created' :
				$value = date_i18n( give_date_format(), strtotime( $item['date_created'] ) );
				break;

			default:
				$value = isset( $item[ $column_name ] ) ? $item[ $column_name ] : null;
				break;
		}

		return apply_filters( "give_report_column_{$column_name}", $value, $item['id'] );

	}

	/**
	 * Column name.
	 *
	 * @param $item
	 *
	 * @return string
	 */
	public function column_name( $item ) {
		$name = '#' . $item['id'] . ' ';
		$name .= ! empty( $item['name'] ) ? $item['name'] : '<em>' . esc_html__( 'Unnamed Donor', 'give' ) . '</em>';
		$view_url = admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $item['id'] );
		$actions  = $this->get_row_actions( $item );

		return '<a href="' . esc_url( $view_url ) . '">' . $name . '</a>' . $this->row_actions( $actions );
	}

	/**
	 * Retrieve the table columns.
	 *
	 * @access public
	 * @since  1.0
	 * @return array $columns Array of all the list table columns.
	 */
	public function get_columns() {
		$columns = array(
			'name'          => esc_html__( 'Name', 'give' ),
			'email'         => esc_html__( 'Email', 'give' ),
			'num_purchases' => esc_html__( 'Donations', 'give' ),
			'amount_spent'  => esc_html__( 'Total Donated', 'give' ),
			'date_created'  => esc_html__( 'Date Created', 'give' )
		);

		return apply_filters( 'give_report_customer_columns', $columns );

	}

	/**
	 * Get the sortable columns.
	 *
	 * @access public
	 * @since  2.1
	 * @return array Array of all the sortable columns.
	 */
	public function get_sortable_columns() {
		return array(
			'date_created'  => array( 'date_created', true ),
			'name'          => array( 'name', true ),
			'num_purchases' => array( 'purchase_count', false ),
			'amount_spent'  => array( 'purchase_value', false ),
		);
	}

	/**
	 * Retrieve row actions.
	 *
	 * @since  1.7
	 * @access public
	 *
	 * @return array An array of action links.
	 */
	public function get_row_actions( $item ) {

		$actions = array(

			'view' => sprintf(
				'<a href="%1$s" aria-label="%2$s">%3$s</a>',
				admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $item['id'] ),
				sprintf( esc_attr__( 'View "%s"', 'give' ), $item['name'] ),
				esc_html__( 'View Donor', 'give' )
			),

			'notes' => sprintf(
				'<a href="%1$s" aria-label="%2$s">%3$s</a>',
				admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=notes&id=' . $item['id'] ),
				sprintf( esc_attr__( 'Notes for "%s"', 'give' ), $item['name'] ),
				esc_html__( 'Notes', 'give' )
			),

			'delete' => sprintf(
				'<a href="%1$s" aria-label="%2$s">%3$s</a>',
				admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=delete&id=' . $item['id'] ),
				sprintf( esc_attr__( 'Delete "%s"', 'give' ), $item['name'] ),
				esc_html__( 'Delete', 'give' )
			)

		);

		return apply_filters( 'give_donor_row_actions', $actions, $item );

	}

	/**
	 * Outputs the reporting views.
	 *
	 * @access public
	 * @since  1.0
	 * @return void
	 */
	public function bulk_actions( $which = '' ) {
		// These aren't really bulk actions but this outputs the markup in the right place.
	}

	/**
	 * Retrieve the current page number.
	 *
	 * @access public
	 * @since  1.0
	 * @return int Current page number.
	 */
	public function get_paged() {
		return isset( $_GET['paged'] ) ? absint( $_GET['paged'] ) : 1;
	}

	/**
	 * Retrieves the search query string.
	 *
	 * @access public
	 * @since  1.0
	 * @return mixed string If search is present, false otherwise.
	 */
	public function get_search() {
		return ! empty( $_GET['s'] ) ? urldecode( trim( $_GET['s'] ) ) : false;
	}

	/**
	 * Build all the reports data.
	 *
	 * @access public
	 * @since  1.0
	 * @global object $wpdb Used to query the database using the WordPress.
	 *                      Database API
	 * @return array $reports_data All the data for customer reports.
	 */
	public function reports_data() {
		global $wpdb;
function myFunction($a, $b) {
    // Do something
}

		$data = array();

		// Get donor query.
		$args      = $this->get_donor_query();
		$customers = Give()->customers->get_customers( $args );

		if ( $customers ) {

			foreach ( $customers as $customer ) {

				$user_id = ! empty( $customer->user_id ) ? intval( $customer->user_id ) : 0;

				$data[] = array(
					'id'            => $customer->id,
					'user_id'       => $user_id,
					'name'          => $customer->name,
					'email'         => $customer->email,
					'num_purchases' => $customer->purchase_count,
					'amount_spent'  => $customer->purchase_value,
					'date_created'  => $customer->date_created,
				);
			}
		}

		return $data;
	}

	/**
	 * Get donor count.
	 *
	 * @since 1.8.1
	 * @access private
	 */
	private function get_donor_count() {
		// Get donor query.
		$_donor_query = $this->get_donor_query();

		$_donor_query['number'] = -1;
		$donors = Give()->customers->get_customers( $_donor_query );

		return count( $donors );
	}

	/**
	 * Get donor query.
	 *
	 * @since  1.8.1
	 * @access public
	 * @return array
	 */
	public function get_donor_query() {
		$paged   = $this->get_paged();
		$offset  = $this->per_page * ( $paged - 1 );
		$search  = $this->get_search();
		$order   = isset( $_GET['order'] ) ? sanitize_text_field( $_GET['order'] ) : 'DESC';
		$orderby = isset( $_GET['orderby'] ) ? sanitize_text_field( $_GET['orderby'] ) : 'id';

		$args = array(
			'number'  => $this->per_page,
			'offset'  => $offset,
			'order'   => $order,
			'orderby' => $orderby,
		);

		if( $search ) {
''   == false // true
''   == null  // true
'ab' == false // false
'ab' == null  // false

// It is often better to use strict comparison
'' === false // false
'' === null  // false
			if ( is_email( $search ) ) {
				$args['email'] = $search;
			} elseif ( is_numeric( $search ) ) {
				$args['id'] = $search;
			} else {
				$args['name'] = $search;
			}
		}

		return $args;
	}

	/**
	 * Setup the final data for the table.
	 *
	 * @access public
	 * @since  1.0
	 * @uses   Give_Customer_Reports_Table::get_columns()
	 * @uses   WP_List_Table::get_sortable_columns()
	 * @uses   Give_Customer_Reports_Table::get_pagenum()
	 * @uses   Give_Customer_Reports_Table::get_total_customers()
	 * @return void
	 */
	public function prepare_items() {

		$columns  = $this->get_columns();
		$hidden   = array(); // No hidden columns
		$sortable = $this->get_sortable_columns();

		$this->_column_headers = array( $columns, $hidden, $sortable );

		$this->items = $this->reports_data();

		$this->total = $this->get_donor_count();

		$this->set_pagination_args( array(
			'total_items' => $this->total,
			'per_page'    => $this->per_page,
			'total_pages' => ceil( $this->total / $this->per_page )
		) );
	}
}

Push — issues/1486 ( 0cf53f )

Give_Customer_Reports_Table::reports_data() B

Complexity

Size

Duplication

Importance

1. Pass all data via parameters

2. Create a class that maintains your state

1			<?php
			0 ignored issues – show Coding Style Compatibility introduced 2016-12-08 11:11 UTC by Report Bug Copy Issue Report For compatibility and reusability of your code, PSR1 recommends that a file should introduce either new symbols (like classes, functions, etc.) or have side-effects (like outputting something, or including other files), but not both at the same time. The first symbol is defined on line 29 and the first side effect is on line 14. The PSR-1: Basic Coding Standard recommends that a file should either introduce new symbols, that is classes, functions, constants or similar, or have side effects. Side effects are anything that executes logic, like for example printing output, changing ini settings or writing to a file. The idea behind this recommendation is that merely auto-loading a class should not change the state of an application. It also promotes a cleaner style of programming and makes your code less prone to errors, because the logic is not spread out all over the place. To learn more about the PSR-1, please see the PHP-FIG site on the PSR-1. Loading history...
2			/**
3			* Customer (Donor) Reports Table Class.
4			*
5			* @package Give
6			* @subpackage Admin/Reports
7			* @copyright Copyright (c) 2016, WordImpress
8			* @license https://opensource.org/licenses/gpl-license GNU Public License
9			* @since 1.0
10			*/
11
12			// Exit if accessed directly.
13			if ( ! defined( 'ABSPATH' ) ) {
14			exit;
15			}
16
17			// Load WP_List_Table if not loaded
18			if ( ! class_exists( 'WP_List_Table' ) ) {
19			require_once ABSPATH . 'wp-admin/includes/class-wp-list-table.php';
20			}
21
22			/**
23			* Give_Customer_Reports_Table Class.
24			*
25			* Renders the Customer Reports table.
26			*
27			* @since 1.0
28			*/
29			class Give_Customer_Reports_Table extends WP_List_Table {
30
31			/**
32			* Number of items per page.
33			*
34			* @var int
35			* @since 1.0
36			*/
37			public $per_page = 30;
38
39			/**
40			* Number of donors found.
41			*
42			* @var int
43			* @since 1.0
44			*/
45			public $count = 0;
46
47			/**
48			* Total donors.
49			*
50			* @var int
51			* @since 1.0
52			*/
53			public $total = 0;
54
55			/**
56			* Get things started.
57			*
58			* @since 1.0
59			* @see WP_List_Table::__construct()
60			*/
61			public function __construct() {
62
63			// Set parent defaults
64			parent::__construct( array(
65			'singular' => esc_html__( 'Donor', 'give' ), // Singular name of the listed records
66			'plural' => esc_html__( 'Donors', 'give' ), // Plural name of the listed records
67			'ajax' => false // Does this table support ajax?
68			) );
69
70			}
71
72			/**
73			* Show the search field.
74			*
75			* @since 1.0
76			* @access public
77			*
78			* @param string $text Label for the search box.
79			* @param string $input_id ID of the search box.
80			*
81			* @return void
82			*/
83			public function search_box( $text, $input_id ) {
84			$input_id = $input_id . '-search-input';
85
86			if ( ! empty( $_REQUEST['orderby'] ) ) {
87			echo '<input type="hidden" name="orderby" value="' . esc_attr( $_REQUEST['orderby'] ) . '" />';
88			}
89			if ( ! empty( $_REQUEST['order'] ) ) {
90			echo '<input type="hidden" name="order" value="' . esc_attr( $_REQUEST['order'] ) . '" />';
91			}
92			?>
93			<p class="search-box" role="search">
94			<label class="screen-reader-text" for="<?php echo $input_id ?>"><?php echo $text; ?>:</label>
95			<input type="search" id="<?php echo $input_id ?>" name="s" value="<?php _admin_search_query(); ?>"/>
96			<?php submit_button( $text, 'button', false, false, array( 'ID' => 'search-submit' ) ); ?>
97			</p>
98			<?php
99			}
100
101			/**
102			* This function renders most of the columns in the list table.
103			*
104			* @access public
105			* @since 1.0
106			*
107			* @param array $item Contains all the data of the customers.
108			* @param string $column_name The name of the column.
109			*
110			* @return string Column Name.
111			*/
112			public function column_default( $item, $column_name ) {
113			switch ( $column_name ) {
114
115			case 'num_purchases' :
116			$value = '<a href="' .
117			admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&user=' . urlencode( $item['email'] )
118			) . '">' . esc_html( $item['num_purchases'] ) . '</a>';
119			break;
120
121			case 'amount_spent' :
122			$value = give_currency_filter( give_format_amount( $item[ $column_name ] ) );
123			break;
124
125			case 'date_created' :
126			$value = date_i18n( give_date_format(), strtotime( $item['date_created'] ) );
127			break;
128
129			default:
130			$value = isset( $item[ $column_name ] ) ? $item[ $column_name ] : null;
131			break;
132			}
133
134			return apply_filters( "give_report_column_{$column_name}", $value, $item['id'] );
135
136			}
137
138			/**
139			* Column name.
140			*
141			* @param $item
142			*
143			* @return string
144			*/
145			public function column_name( $item ) {
146			$name = '#' . $item['id'] . ' ';
147			$name .= ! empty( $item['name'] ) ? $item['name'] : '<em>' . esc_html__( 'Unnamed Donor', 'give' ) . '</em>';
148			$view_url = admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $item['id'] );
149			$actions = $this->get_row_actions( $item );
150
151			return '<a href="' . esc_url( $view_url ) . '">' . $name . '</a>' . $this->row_actions( $actions );
152			}
153
154			/**
155			* Retrieve the table columns.
156			*
157			* @access public
158			* @since 1.0
159			* @return array $columns Array of all the list table columns.
160			*/
161			public function get_columns() {
162			$columns = array(
163			'name' => esc_html__( 'Name', 'give' ),
164			'email' => esc_html__( 'Email', 'give' ),
165			'num_purchases' => esc_html__( 'Donations', 'give' ),
166			'amount_spent' => esc_html__( 'Total Donated', 'give' ),
167			'date_created' => esc_html__( 'Date Created', 'give' )
168			);
169
170			return apply_filters( 'give_report_customer_columns', $columns );
171
172			}
173
174			/**
175			* Get the sortable columns.
176			*
177			* @access public
178			* @since 2.1
179			* @return array Array of all the sortable columns.
180			*/
181			public function get_sortable_columns() {
182			return array(
183			'date_created' => array( 'date_created', true ),
184			'name' => array( 'name', true ),
185			'num_purchases' => array( 'purchase_count', false ),
186			'amount_spent' => array( 'purchase_value', false ),
187			);
188			}
189
190			/**
191			* Retrieve row actions.
192			*
193			* @since 1.7
194			* @access public
195			*
196			* @return array An array of action links.
197			*/
198			public function get_row_actions( $item ) {
199
200			$actions = array(
201
202			'view' => sprintf(
203			'<a href="%1$s" aria-label="%2$s">%3$s</a>',
204			admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $item['id'] ),
205			sprintf( esc_attr__( 'View "%s"', 'give' ), $item['name'] ),
206			esc_html__( 'View Donor', 'give' )
207			),
208
209			'notes' => sprintf(
210			'<a href="%1$s" aria-label="%2$s">%3$s</a>',
211			admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=notes&id=' . $item['id'] ),
212			sprintf( esc_attr__( 'Notes for "%s"', 'give' ), $item['name'] ),
213			esc_html__( 'Notes', 'give' )
214			),
215
216			'delete' => sprintf(
217			'<a href="%1$s" aria-label="%2$s">%3$s</a>',
218			admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=delete&id=' . $item['id'] ),
219			sprintf( esc_attr__( 'Delete "%s"', 'give' ), $item['name'] ),
220			esc_html__( 'Delete', 'give' )
221			)
222
223			);
224
225			return apply_filters( 'give_donor_row_actions', $actions, $item );
226
227			}
228
229			/**
230			* Outputs the reporting views.
231			*
232			* @access public
233			* @since 1.0
234			* @return void
235			*/
236			public function bulk_actions( $which = '' ) {
237			// These aren't really bulk actions but this outputs the markup in the right place.
238			}
239
240			/**
241			* Retrieve the current page number.
242			*
243			* @access public
244			* @since 1.0
245			* @return int Current page number.
246			*/
247			public function get_paged() {
248			return isset( $_GET['paged'] ) ? absint( $_GET['paged'] ) : 1;
249			}
250
251			/**
252			* Retrieves the search query string.
253			*
254			* @access public
255			* @since 1.0
256			* @return mixed string If search is present, false otherwise.
257			*/
258			public function get_search() {
259			return ! empty( $_GET['s'] ) ? urldecode( trim( $_GET['s'] ) ) : false;
260			}
261
262			/**
263			* Build all the reports data.
264			*
265			* @access public
266			* @since 1.0
267			* @global object $wpdb Used to query the database using the WordPress.
268			* Database API
269			* @return array $reports_data All the data for customer reports.
270			*/
271			public function reports_data() {
272			global $wpdb;
			0 ignored issues – show Compatibility Best Practice introduced 2016-12-08 11:11 UTC by Report Bug Copy Issue Report Use of `global` functionality is not recommended; it makes your code harder to test, and less reusable. Instead of relying on `global` state, we recommend one of these alternatives: 1. Pass all data via parameters function myFunction($a, $b) { // Do something } 2. Create a class that maintains your state class MyClass { private $a; private $b; public function __construct($a, $b) { $this->a = $a; $this->b = $b; } public function myFunction() { // Do something } } Loading history...
273
274			$data = array();
275
276			// Get donor query.
277			$args = $this->get_donor_query();
278			$customers = Give()->customers->get_customers( $args );
279
280			if ( $customers ) {
281
282			foreach ( $customers as $customer ) {
283
284			$user_id = ! empty( $customer->user_id ) ? intval( $customer->user_id ) : 0;
285
286			$data[] = array(
287			'id' => $customer->id,
288			'user_id' => $user_id,
289			'name' => $customer->name,
290			'email' => $customer->email,
291			'num_purchases' => $customer->purchase_count,
292			'amount_spent' => $customer->purchase_value,
293			'date_created' => $customer->date_created,
294			);
295			}
296			}
297
298			return $data;
299			}
300
301			/**
302			* Get donor count.
303			*
304			* @since 1.8.1
305			* @access private
306			*/
307			private function get_donor_count() {
308			// Get donor query.
309			$_donor_query = $this->get_donor_query();
310
311			$_donor_query['number'] = -1;
312			$donors = Give()->customers->get_customers( $_donor_query );
313
314			return count( $donors );
315			}
316
317			/**
318			* Get donor query.
319			*
320			* @since 1.8.1
321			* @access public
322			* @return array
323			*/
324			public function get_donor_query() {
325			$paged = $this->get_paged();
326			$offset = $this->per_page * ( $paged - 1 );
327			$search = $this->get_search();
328			$order = isset( $_GET['order'] ) ? sanitize_text_field( $_GET['order'] ) : 'DESC';
329			$orderby = isset( $_GET['orderby'] ) ? sanitize_text_field( $_GET['orderby'] ) : 'id';
330
331			$args = array(
332			'number' => $this->per_page,
333			'offset' => $offset,
334			'order' => $order,
335			'orderby' => $orderby,
336			);
337
338			if( $search ) {
			0 ignored issues – show Bug Best Practice introduced 2017-02-14 10:11 UTC by Report Bug Copy Issue Report The expression `$search` of type `string\|false` is loosely compared to `true`; this is ambiguous if the string can be empty. You might want to explicitly use `!== false` instead. In PHP, under loose comparison (like `==`, or `!=`, or `switch` conditions), values of different types might be equal. For `string` values, the empty string `''` is a special case, in particular the following results might be unexpected: '' == false // true '' == null // true 'ab' == false // false 'ab' == null // false // It is often better to use strict comparison '' === false // false '' === null // false Loading history...
339			if ( is_email( $search ) ) {
340			$args['email'] = $search;
341			} elseif ( is_numeric( $search ) ) {
342			$args['id'] = $search;
343			} else {
344			$args['name'] = $search;
345			}
346			}
347
348			return $args;
349			}
350
351			/**
352			* Setup the final data for the table.
353			*
354			* @access public
355			* @since 1.0
356			* @uses Give_Customer_Reports_Table::get_columns()
357			* @uses WP_List_Table::get_sortable_columns()
358			* @uses Give_Customer_Reports_Table::get_pagenum()
359			* @uses Give_Customer_Reports_Table::get_total_customers()
360			* @return void
361			*/
362			public function prepare_items() {
363
364			$columns = $this->get_columns();
365			$hidden = array(); // No hidden columns
366			$sortable = $this->get_sortable_columns();
367
368			$this->_column_headers = array( $columns, $hidden, $sortable );
369
370			$this->items = $this->reports_data();
371
372			$this->total = $this->get_donor_count();
373
374			$this->set_pagination_args( array(
375			'total_items' => $this->total,
376			'per_page' => $this->per_page,
377			'total_pages' => ceil( $this->total / $this->per_page )
378			) );
379			}
380			}

ravinderk / Give

Push — issues/1486 ( 0cf53f )

Give_Customer_Reports_Table::reports_data() B

Complexity

Size

Duplication

Importance

1. Pass all data via parameters

2. Create a class that maintains your state

Duplication Side-by-Side

Filter issues like