Issues in class-give-graph.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

includes/admin/reports/class-give-graph.php (45 issues)

Severity

Unknown 45

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
/**
 * Graphs
 *
 * This class handles building pretty report graphs
 *
 * @package     Give
 * @subpackage  Admin/Reports
 * @copyright   Copyright (c) 2012, WordImpress
 * @license     https://opensource.org/licenses/gpl-license GNU Public License
 * @since       1.0
 */

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

/**
 * Give_Graph Class
 *
 * @since 1.0
 */
class Give_Graph {

	/*

	Simple example:

	data format for each point: array( location on x, location on y )

	$data = array(

		'Label' => array(
			array( 1, 5 ),
			array( 3, 8 ),
			array( 10, 2 )
		),

		'Second Label' => array(
			array( 1, 7 ),
			array( 4, 5 ),
			array( 12, 8 )
		)
	);

	$graph = new Give_Graph( $data );
	$graph->display();

	*/

	/**
	 * Data to graph
	 *
	 * @var array
	 * @since 1.0
	 */
	private $data;

	/**
	 * Unique ID for the graph
	 *
	 * @var string
	 * @since 1.0
	 */
	private $id = '';

	/**
	 * Graph options
	 *
	 * @var array
	 * @since 1.0
	 */
	private $options = array();

	/**
	 * Get things started
	 *
	 * @since 1.0
	 *
	 * @param array $_data
	 * @param array $options
	 */
	public function __construct( $_data, $options = array() ) {

		$this->data      = $_data;

		// Generate unique ID
		$this->id = md5( rand() );

		// Setup default options;
		$this->options = apply_filters( 'give_graph_args', array(
			'y_mode'          => null,
			'x_mode'          => null,
			'y_decimals'      => 0,
			'x_decimals'      => 0,
			'y_position'      => 'right',
			'time_format'     => '%d/%b',
			'ticksize_unit'   => 'day',
			'ticksize_num'    => 1,
			'multiple_y_axes' => false,
			'bgcolor'         => '#f9f9f9',
			'bordercolor'     => '#eee',
			'color'           => '#bbb',
			'borderwidth'     => 1,
			'bars'            => true,
			'lines'           => false,
			'points'          => true,
			'dataType'        => array()
		) );

		$this->options = wp_parse_args( $options, $this->options );
	}

	/**
	 * Set an option
	 *
	 * @param $key   The option key to set
	 * @param $value The value to assign to the key
	 *
	 * @since 1.0
	 */
	public function set( $key, $value ) {
		$this->options[ $key ] = $value;
	}

	/**
	 * Get an option
	 *
	 * @param $key The option key to get
	 *
	 * @since 1.0
	 */
	public function get( $key ) {
		return isset( $this->options[ $key ] ) ? $this->options[ $key ] : false;
	}

	/**
	 * Get graph data
	 *
	 * @since 1.0
	 */
	public function get_data() {
		return apply_filters( 'give_get_graph_data', $this->data, $this );
	}

	/**
	 * Load the graphing library script
	 *
	 * @since 1.0
	 */
	public function load_scripts() {
		// Use minified libraries if SCRIPT_DEBUG is turned off
		$suffix = ( defined( 'SCRIPT_DEBUG' ) && SCRIPT_DEBUG ) ? '' : '.min';

		wp_register_script( 'jquery-flot-orderbars', GIVE_PLUGIN_URL . 'assets/js/plugins/jquery.flot.orderBars' . $suffix . '.js', array('jquery-flot'), GIVE_VERSION );

		wp_enqueue_script( 'jquery-flot-orderbars' );

		wp_register_script( 'jquery-flot-time', GIVE_PLUGIN_URL . 'assets/js/plugins/jquery.flot.time' . $suffix . '.js', array('jquery-flot'), GIVE_VERSION );

		wp_enqueue_script( 'jquery-flot-time' );

		wp_register_script( 'jquery-flot-resize', GIVE_PLUGIN_URL . 'assets/js/plugins/jquery.flot.resize' . $suffix . '.js', array('jquery-flot'), GIVE_VERSION );

		wp_enqueue_script( 'jquery-flot-resize' );

		wp_register_script( 'jquery-flot', GIVE_PLUGIN_URL . 'assets/js/plugins/jquery.flot' . $suffix . '.js', false, GIVE_VERSION );
		wp_enqueue_script( 'jquery-flot' );

	}

	/**
	 * Build the graph and return it as a string
	 *
	 * @var array
	 * @since 1.0
	 * @return string
	 */
	public function build_graph() {

		$yaxis_count = 1;

		$this->load_scripts();

		ob_start();
		?>
		<script type="text/javascript">

			jQuery( document ).ready( function ( $ ) {
				$.plot(
					$( "#give-graph-<?php echo $this->id; ?>" ),

					[
						<?php
							$order = 0;
							foreach( $this->get_data() as $label => $data ) :

						?>
						{
							label : "<?php echo esc_attr( $label ); ?>",
							id    : "<?php echo sanitize_key( $label ); ?>",
							dataType  : '<?php echo ( ! empty( $this->options['dataType'][$order] ) ? $this->options['dataType'][$order] : 'count' ); ?>',

							// data format is: [ point on x, value on y ]
							data  : [<?php foreach( $data as $point ) { echo '[' . implode( ',', $point ) . '],'; } ?>],

							points: {
								show: <?php echo $this->options['points'] ? 'true' : 'false'; ?>,

							},
							bars  : {
								show    : <?php echo $this->options['bars'] ? 'true' : 'false'; ?>,

								barWidth: 100,
				                order: <?php echo $order++; ?>,

								align   : 'center'
							},
							lines : {
								show     : <?php echo $this->options['lines'] ? 'true' : 'false'; ?>,

								fill     : true,
								fillColor: {colors: [{opacity: 0.4}, {opacity: 0.1}]}
							},
							<?php if( $this->options[ 'multiple_y_axes' ] ) : ?>

							yaxis : <?php echo $yaxis_count; ?>

							<?php endif; ?>

						},

						<?php $yaxis_count++; endforeach; ?>

					],
					{
						// Options
						grid: {
							show           : true,
							aboveData      : false,
							color          : "<?php echo $this->options[ 'color' ]; ?>",

							backgroundColor: "<?php echo $this->options[ 'bgcolor' ]; ?>",

							borderColor    : "<?php echo $this->options[ 'bordercolor' ]; ?>",

							borderWidth    : <?php echo absint( $this->options[ 'borderwidth' ] ); ?>,

							clickable      : false,
							hoverable      : true
						},

						colors: ["#66bb6a", "#546e7a"], //Give Colors

						xaxis: {
							mode        : "<?php echo $this->options['x_mode']; ?>",

							timeFormat  : "<?php echo $this->options['x_mode'] == 'time' ? $this->options['time_format'] : ''; ?>",

							tickSize    : "<?php echo $this->options['x_mode'] == 'time' ? '' : 1; ?>",

							<?php if( $this->options['x_mode'] != 'time' ) : ?>

							tickDecimals: <?php echo $this->options['x_decimals']; ?>

							<?php endif; ?>
						},
						yaxis: {
							position    : 'right',
							min         : 0,
							mode        : "<?php echo $this->options['y_mode']; ?>",

							timeFormat  : "<?php echo $this->options['y_mode'] == 'time' ? $this->options['time_format'] : ''; ?>",

							<?php if( $this->options['y_mode'] != 'time' ) : ?>

							tickDecimals: <?php echo $this->options['y_decimals']; ?>

							<?php endif; ?>
						}
					}
				);

				function give_flot_tooltip( x, y, contents ) {
					$( '<div id="give-flot-tooltip">' + contents + '</div>' ).css( {
						position          : 'absolute',
						display           : 'none',
						top               : y + 5,
						left              : x + 5,
						border            : '1px solid #fdd',
						padding           : '2px',
						'background-color': '#fee',
						opacity           : 0.80
					} ).appendTo( "body" ).fadeIn( 200 );
				}

				var previousPoint = null;
				$( "#give-graph-<?php echo $this->id; ?>" ).bind( "plothover", function ( event, pos, item ) {


					$( "#x" ).text( pos.x.toFixed( 2 ) );
					$( "#y" ).text( pos.y.toFixed( 2 ) );
					if ( item ) {
						if ( previousPoint !== item.dataIndex ) {
							previousPoint = item.dataIndex;
							$( "#give-flot-tooltip" ).remove();
							var x = item.datapoint[0].toFixed( 2 ),
                                y = accounting.formatMoney( item.datapoint[1].toFixed( give_vars.currency_decimals ), '', give_vars.currency_decimals, give_vars.thousands_separator, give_vars.decimal_separator );

							if ( item.series.dataType.length &&  item.series.dataType === 'amount' ) {

								if ( give_vars.currency_pos === 'before' ) {

									give_flot_tooltip( item.pageX, item.pageY, item.series.label + ' ' + give_vars.currency_sign + y );
								} else {
									give_flot_tooltip( item.pageX, item.pageY, item.series.label + ' ' + y + give_vars.currency_sign );
								}
							} else {
								give_flot_tooltip( item.pageX, item.pageY, item.series.label + ' ' + parseInt( y ) );
							}
						}
					} else {
						$( "#give-flot-tooltip" ).remove();
						previousPoint = null;
					}
				} );

			} );

		</script>
		<div id="give-graph-<?php echo $this->id; ?>" class="give-graph" style="height: 300px;"></div>

		<?php
		return ob_get_clean();
	}

	/**
	 * Output the final graph
	 *
	 * @since 1.0
	 */
	public function display() {
		/**
		 * Fires before displaying the final graph.
		 *
		 * @since 1.0
		 *
		 * @param Give_Graph $this Graph object.
		 */
		do_action( 'give_before_graph', $this );

		// Build the graph.
		echo $this->build_graph();


		/**
		 * Fires after displaying the final graph.
		 *
		 * @since 1.0
		 *
		 * @param Give_Graph $this Graph object.
		 */
		do_action( 'give_after_graph', $this );
	}

}


1			<?php
2			/**
3			* Graphs
4			*
5			* This class handles building pretty report graphs
6			*
7			* @package Give
8			* @subpackage Admin/Reports
9			* @copyright Copyright (c) 2012, WordImpress
10			* @license https://opensource.org/licenses/gpl-license GNU Public License
11			* @since 1.0
12			*/
13
14			// Exit if accessed directly.
15			if ( ! defined( 'ABSPATH' ) ) {
16			exit;
17			}
18
19			/**
20			* Give_Graph Class
21			*
22			* @since 1.0
23			*/
24			class Give_Graph {
25
26			/*
27
28			Simple example:
29
30			data format for each point: array( location on x, location on y )
31
32			$data = array(
33
34			'Label' => array(
35			array( 1, 5 ),
36			array( 3, 8 ),
37			array( 10, 2 )
38			),
39
40			'Second Label' => array(
41			array( 1, 7 ),
42			array( 4, 5 ),
43			array( 12, 8 )
44			)
45			);
46
47			$graph = new Give_Graph( $data );
48			$graph->display();
49
50			*/
51
52			/**
53			* Data to graph
54			*
55			* @var array
56			* @since 1.0
57			*/
58			private $data;
59
60			/**
61			* Unique ID for the graph
62			*
63			* @var string
64			* @since 1.0
65			*/
66			private $id = '';
67
68			/**
69			* Graph options
70			*
71			* @var array
72			* @since 1.0
73			*/
74			private $options = array();
75
76			/**
77			* Get things started
78			*
79			* @since 1.0
80			*
81			* @param array $_data
82			* @param array $options
83			*/
84			public function __construct( $_data, $options = array() ) {
85
86			$this->data = $_data;
87
88			// Generate unique ID
89			$this->id = md5( rand() );
90
91			// Setup default options;
92			$this->options = apply_filters( 'give_graph_args', array(
93			'y_mode' => null,
94			'x_mode' => null,
95			'y_decimals' => 0,
96			'x_decimals' => 0,
97			'y_position' => 'right',
98			'time_format' => '%d/%b',
99			'ticksize_unit' => 'day',
100			'ticksize_num' => 1,
101			'multiple_y_axes' => false,
102			'bgcolor' => '#f9f9f9',
103			'bordercolor' => '#eee',
104			'color' => '#bbb',
105			'borderwidth' => 1,
106			'bars' => true,
107			'lines' => false,
108			'points' => true,
109			'dataType' => array()
110			) );
111
112			$this->options = wp_parse_args( $options, $this->options );
113			}
114
115			/**
116			* Set an option
117			*
118			* @param $key The option key to set
119			* @param $value The value to assign to the key
120			*
121			* @since 1.0
122			*/
123			public function set( $key, $value ) {
124			$this->options[ $key ] = $value;
125			}
126
127			/**
128			* Get an option
129			*
130			* @param $key The option key to get
131			*
132			* @since 1.0
133			*/
134			public function get( $key ) {
135			return isset( $this->options[ $key ] ) ? $this->options[ $key ] : false;
136			}
137
138			/**
139			* Get graph data
140			*
141			* @since 1.0
142			*/
143			public function get_data() {
144			return apply_filters( 'give_get_graph_data', $this->data, $this );
145			}
146
147			/**
148			* Load the graphing library script
149			*
150			* @since 1.0
151			*/
152			public function load_scripts() {
153			// Use minified libraries if SCRIPT_DEBUG is turned off
154			$suffix = ( defined( 'SCRIPT_DEBUG' ) && SCRIPT_DEBUG ) ? '' : '.min';
155
156			wp_register_script( 'jquery-flot-orderbars', GIVE_PLUGIN_URL . 'assets/js/plugins/jquery.flot.orderBars' . $suffix . '.js', array('jquery-flot'), GIVE_VERSION );
			0 ignored issues – show introduced 2017-08-01 08:32 UTC by Report Bug Copy Issue Report Show Similar Issues like this No space after opening parenthesis of array is bad style Loading history... introduced 2017-08-01 08:32 UTC by Report Bug Copy Issue Report Show Similar Issues like this No space before closing parenthesis of array is bad style Loading history...
157			wp_enqueue_script( 'jquery-flot-orderbars' );
158
159			wp_register_script( 'jquery-flot-time', GIVE_PLUGIN_URL . 'assets/js/plugins/jquery.flot.time' . $suffix . '.js', array('jquery-flot'), GIVE_VERSION );
			0 ignored issues – show introduced 2017-08-01 08:32 UTC by Report Bug Copy Issue Report Show Similar Issues like this No space after opening parenthesis of array is bad style Loading history... introduced 2017-08-01 08:32 UTC by Report Bug Copy Issue Report Show Similar Issues like this No space before closing parenthesis of array is bad style Loading history...
160			wp_enqueue_script( 'jquery-flot-time' );
161
162			wp_register_script( 'jquery-flot-resize', GIVE_PLUGIN_URL . 'assets/js/plugins/jquery.flot.resize' . $suffix . '.js', array('jquery-flot'), GIVE_VERSION );
			0 ignored issues – show introduced 2017-08-01 08:32 UTC by Report Bug Copy Issue Report Show Similar Issues like this No space after opening parenthesis of array is bad style Loading history... introduced 2017-08-01 08:32 UTC by Report Bug Copy Issue Report Show Similar Issues like this No space before closing parenthesis of array is bad style Loading history...
163			wp_enqueue_script( 'jquery-flot-resize' );
164
165			wp_register_script( 'jquery-flot', GIVE_PLUGIN_URL . 'assets/js/plugins/jquery.flot' . $suffix . '.js', false, GIVE_VERSION );
166			wp_enqueue_script( 'jquery-flot' );
167
168			}
169
170			/**
171			* Build the graph and return it as a string
172			*
173			* @var array
174			* @since 1.0
175			* @return string
176			*/
177			public function build_graph() {
178
179			$yaxis_count = 1;
180
181			$this->load_scripts();
182
183			ob_start();
184			?>
185			<script type="text/javascript">
186
187			jQuery( document ).ready( function ( $ ) {
188			$.plot(
189			$( "#give-graph-<?php echo $this->id; ?>" ),
			0 ignored issues – show introduced 2017-08-01 08:32 UTC by Report Bug Copy Issue Report Show Similar Issues like this Expected next thing to be a escaping function, not '$this' Loading history...
190			[
191			<?php
192			$order = 0;
193			foreach( $this->get_data() as $label => $data ) :
			0 ignored issues – show introduced 2017-08-01 08:32 UTC by Report Bug Copy Issue Report Show Similar Issues like this Space after opening control structure is required Loading history... introduced 2017-08-01 08:32 UTC by Report Bug Copy Issue Report Show Similar Issues like this No space before opening parenthesis is prohibited Loading history...
194			?>
195			{
196			label : "<?php echo esc_attr( $label ); ?>",
197			id : "<?php echo sanitize_key( $label ); ?>",
198			dataType : '<?php echo ( ! empty( $this->options['dataType'][$order] ) ? $this->options['dataType'][$order] : 'count' ); ?>',
			0 ignored issues – show introduced 2017-08-01 08:32 UTC by Report Bug Copy Issue Report Show Similar Issues like this Expected next thing to be a escaping function, not '(' Loading history... introduced 2017-08-01 08:32 UTC by Report Bug Copy Issue Report Show Similar Issues like this Array keys should be surrounded by spaces unless they contain a string or an integer. Loading history...
199			// data format is: [ point on x, value on y ]
200			data : [<?php foreach( $data as $point ) { echo '[' . implode( ',', $point ) . '],'; } ?>],
			0 ignored issues – show introduced 2017-08-01 08:32 UTC by Report Bug Copy Issue Report Show Similar Issues like this Space after opening control structure is required Loading history... introduced 2017-08-01 08:32 UTC by Report Bug Copy Issue Report Show Similar Issues like this No space before opening parenthesis is prohibited Loading history... introduced 2017-08-01 08:32 UTC by Report Bug Copy Issue Report Show Similar Issues like this Expected a sanitizing function (see Codex for 'Data Validation'), but instead saw 'implode' Loading history...
201			points: {
202			show: <?php echo $this->options['points'] ? 'true' : 'false'; ?>,
			0 ignored issues – show introduced 2017-08-01 08:32 UTC by Report Bug Copy Issue Report Show Similar Issues like this Expected next thing to be a escaping function, not '$this' Loading history...
203			},
204			bars : {
205			show : <?php echo $this->options['bars'] ? 'true' : 'false'; ?>,
			0 ignored issues – show introduced 2017-08-01 08:32 UTC by Report Bug Copy Issue Report Show Similar Issues like this Expected next thing to be a escaping function, not '$this' Loading history...
206			barWidth: 100,
207			order: <?php echo $order++; ?>,
			0 ignored issues – show introduced 2017-08-01 08:32 UTC by Report Bug Copy Issue Report Show Similar Issues like this Expected next thing to be a escaping function, not '$order' Loading history...
208			align : 'center'
209			},
210			lines : {
211			show : <?php echo $this->options['lines'] ? 'true' : 'false'; ?>,
			0 ignored issues – show introduced 2017-08-01 08:32 UTC by Report Bug Copy Issue Report Show Similar Issues like this Expected next thing to be a escaping function, not '$this' Loading history...
212			fill : true,
213			fillColor: {colors: [{opacity: 0.4}, {opacity: 0.1}]}
214			},
215			<?php if( $this->options[ 'multiple_y_axes' ] ) : ?>
			0 ignored issues – show introduced 2017-08-01 08:32 UTC by Report Bug Copy Issue Report Show Similar Issues like this Space after opening control structure is required Loading history... introduced 2017-08-01 08:32 UTC by Report Bug Copy Issue Report Show Similar Issues like this No space before opening parenthesis is prohibited Loading history... introduced 2017-08-01 08:32 UTC by Report Bug Copy Issue Report Show Similar Issues like this Array keys should NOT be surrounded by spaces if they only contain a string or an integer. Loading history...
216			yaxis : <?php echo $yaxis_count; ?>
			0 ignored issues – show introduced 2017-08-01 08:32 UTC by Report Bug Copy Issue Report Show Similar Issues like this Expected next thing to be a escaping function, not '$yaxis_count' Loading history...
217			<?php endif; ?>
218
219			},
220
221			<?php $yaxis_count++; endforeach; ?>
222
223			],
224			{
225			// Options
226			grid: {
227			show : true,
228			aboveData : false,
229			color : "<?php echo $this->options[ 'color' ]; ?>",
			0 ignored issues – show introduced 2017-08-01 08:32 UTC by Report Bug Copy Issue Report Show Similar Issues like this Expected next thing to be a escaping function, not '$this' Loading history... introduced 2017-08-01 08:32 UTC by Report Bug Copy Issue Report Show Similar Issues like this Array keys should NOT be surrounded by spaces if they only contain a string or an integer. Loading history...
230			backgroundColor: "<?php echo $this->options[ 'bgcolor' ]; ?>",
			0 ignored issues – show introduced 2017-08-01 08:32 UTC by Report Bug Copy Issue Report Show Similar Issues like this Expected next thing to be a escaping function, not '$this' Loading history... introduced 2017-08-01 08:32 UTC by Report Bug Copy Issue Report Show Similar Issues like this Array keys should NOT be surrounded by spaces if they only contain a string or an integer. Loading history...
231			borderColor : "<?php echo $this->options[ 'bordercolor' ]; ?>",
			0 ignored issues – show introduced 2017-08-01 08:32 UTC by Report Bug Copy Issue Report Show Similar Issues like this Expected next thing to be a escaping function, not '$this' Loading history... introduced 2017-08-01 08:32 UTC by Report Bug Copy Issue Report Show Similar Issues like this Array keys should NOT be surrounded by spaces if they only contain a string or an integer. Loading history...
232			borderWidth : <?php echo absint( $this->options[ 'borderwidth' ] ); ?>,
			0 ignored issues – show introduced 2017-08-01 08:32 UTC by Report Bug Copy Issue Report Show Similar Issues like this Array keys should NOT be surrounded by spaces if they only contain a string or an integer. Loading history...
233			clickable : false,
234			hoverable : true
235			},
236
237			colors: ["#66bb6a", "#546e7a"], //Give Colors
238
239			xaxis: {
240			mode : "<?php echo $this->options['x_mode']; ?>",
			0 ignored issues – show introduced 2017-08-01 08:32 UTC by Report Bug Copy Issue Report Show Similar Issues like this Expected next thing to be a escaping function, not '$this' Loading history...
241			timeFormat : "<?php echo $this->options['x_mode'] == 'time' ? $this->options['time_format'] : ''; ?>",
			0 ignored issues – show introduced 2017-08-01 08:32 UTC by Report Bug Copy Issue Report Show Similar Issues like this Expected next thing to be a escaping function, not '$this' Loading history...
242			tickSize : "<?php echo $this->options['x_mode'] == 'time' ? '' : 1; ?>",
			0 ignored issues – show introduced 2017-08-01 08:32 UTC by Report Bug Copy Issue Report Show Similar Issues like this Expected next thing to be a escaping function, not '$this' Loading history...
243			<?php if( $this->options['x_mode'] != 'time' ) : ?>
			0 ignored issues – show introduced 2017-08-01 08:32 UTC by Report Bug Copy Issue Report Show Similar Issues like this Space after opening control structure is required Loading history... introduced 2017-08-01 08:32 UTC by Report Bug Copy Issue Report Show Similar Issues like this No space before opening parenthesis is prohibited Loading history... introduced 2017-08-01 08:32 UTC by Report Bug Copy Issue Report Show Similar Issues like this Found "!= '". Use Yoda Condition checks, you must Loading history...
244			tickDecimals: <?php echo $this->options['x_decimals']; ?>
			0 ignored issues – show introduced 2017-08-01 08:32 UTC by Report Bug Copy Issue Report Show Similar Issues like this Expected next thing to be a escaping function, not '$this' Loading history...
245			<?php endif; ?>
246			},
247			yaxis: {
248			position : 'right',
249			min : 0,
250			mode : "<?php echo $this->options['y_mode']; ?>",
			0 ignored issues – show introduced 2017-08-01 08:32 UTC by Report Bug Copy Issue Report Show Similar Issues like this Expected next thing to be a escaping function, not '$this' Loading history...
251			timeFormat : "<?php echo $this->options['y_mode'] == 'time' ? $this->options['time_format'] : ''; ?>",
			0 ignored issues – show introduced 2017-08-01 08:32 UTC by Report Bug Copy Issue Report Show Similar Issues like this Expected next thing to be a escaping function, not '$this' Loading history...
252			<?php if( $this->options['y_mode'] != 'time' ) : ?>
			0 ignored issues – show introduced 2017-08-01 08:32 UTC by Report Bug Copy Issue Report Show Similar Issues like this Space after opening control structure is required Loading history... introduced 2017-08-01 08:32 UTC by Report Bug Copy Issue Report Show Similar Issues like this No space before opening parenthesis is prohibited Loading history... introduced 2017-08-01 08:32 UTC by Report Bug Copy Issue Report Show Similar Issues like this Found "!= '". Use Yoda Condition checks, you must Loading history...
253			tickDecimals: <?php echo $this->options['y_decimals']; ?>
			0 ignored issues – show introduced 2017-08-01 08:32 UTC by Report Bug Copy Issue Report Show Similar Issues like this Expected next thing to be a escaping function, not '$this' Loading history...
254			<?php endif; ?>
255			}
256			}
257			);
258
259			function give_flot_tooltip( x, y, contents ) {
260			$( '<div id="give-flot-tooltip">' + contents + '</div>' ).css( {
261			position : 'absolute',
262			display : 'none',
263			top : y + 5,
264			left : x + 5,
265			border : '1px solid #fdd',
266			padding : '2px',
267			'background-color': '#fee',
268			opacity : 0.80
269			} ).appendTo( "body" ).fadeIn( 200 );
270			}
271
272			var previousPoint = null;
273			$( "#give-graph-<?php echo $this->id; ?>" ).bind( "plothover", function ( event, pos, item ) {
			0 ignored issues – show introduced 2017-08-01 08:32 UTC by Report Bug Copy Issue Report Show Similar Issues like this Expected next thing to be a escaping function, not '$this' Loading history...
274
275			$( "#x" ).text( pos.x.toFixed( 2 ) );
276			$( "#y" ).text( pos.y.toFixed( 2 ) );
277			if ( item ) {
278			if ( previousPoint !== item.dataIndex ) {
279			previousPoint = item.dataIndex;
280			$( "#give-flot-tooltip" ).remove();
281			var x = item.datapoint[0].toFixed( 2 ),
282			y = accounting.formatMoney( item.datapoint[1].toFixed( give_vars.currency_decimals ), '', give_vars.currency_decimals, give_vars.thousands_separator, give_vars.decimal_separator );
283
284			if ( item.series.dataType.length && item.series.dataType === 'amount' ) {
285
286			if ( give_vars.currency_pos === 'before' ) {
287
288			give_flot_tooltip( item.pageX, item.pageY, item.series.label + ' ' + give_vars.currency_sign + y );
289			} else {
290			give_flot_tooltip( item.pageX, item.pageY, item.series.label + ' ' + y + give_vars.currency_sign );
291			}
292			} else {
293			give_flot_tooltip( item.pageX, item.pageY, item.series.label + ' ' + parseInt( y ) );
294			}
295			}
296			} else {
297			$( "#give-flot-tooltip" ).remove();
298			previousPoint = null;
299			}
300			} );
301
302			} );
303
304			</script>
305			<div id="give-graph-<?php echo $this->id; ?>" class="give-graph" style="height: 300px;"></div>
			0 ignored issues – show introduced 2017-08-01 08:32 UTC by Report Bug Copy Issue Report Show Similar Issues like this Expected next thing to be a escaping function, not '$this' Loading history...
306			<?php
307			return ob_get_clean();
308			}
309
310			/**
311			* Output the final graph
312			*
313			* @since 1.0
314			*/
315			public function display() {
316			/**
317			* Fires before displaying the final graph.
318			*
319			* @since 1.0
320			*
321			* @param Give_Graph $this Graph object.
322			*/
323			do_action( 'give_before_graph', $this );
324
325			// Build the graph.
326			echo $this->build_graph();
			0 ignored issues – show introduced 2017-08-01 08:32 UTC by Report Bug Copy Issue Report Show Similar Issues like this Expected next thing to be a escaping function, not '$this' Loading history...
327
328			/**
329			* Fires after displaying the final graph.
330			*
331			* @since 1.0
332			*
333			* @param Give_Graph $this Graph object.
334			*/
335			do_action( 'give_after_graph', $this );
336			}
337
338			}
339

ravinderk / Give

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

includes/admin/reports/class-give-graph.php (45 issues)

Severity

Introduced By

Upgrade to new PHP Analysis Engine

Duplication Side-by-Side

Filter issues like