← All changes | includes/admin/views/html-bulk-edit-property.php +47 -10 1.4.49 → 2.3.1 View file →

		@@ -15,19 +15,21 @@
15	15	<?php do_action( 'propertyhive_property_bulk_edit_start' ); ?>
16	16
17	17	<div class="inline-edit-group">
18	18	<label class="alignleft">
19		- <span class="title"><?php _e( 'On Market', 'propertyhive' ); ?></span>
	19	+ <span class="title"><?php echo esc_html(__( 'On Market', 'propertyhive' )); ?></span>
20	20	<span class="input-text-wrap">
21	21	<select class="on_market" name="_on_market">
22	22	<?php
	23	+ // phpcs:ignore WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound -- Template-local variable in an admin view; PHPCS analyzes the view file standalone even though WordPress includes it inside a method/function scope.
23	24	$options = array(
24	25	'' => __( '— No Change —', 'propertyhive' ),
25	26	'yes' => __( 'Yes', 'propertyhive' ),
26	27	'no' => __( 'No', 'propertyhive' ),
27	28	);
	29	+ // phpcs:ignore WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound -- Template-local variable in an admin view; PHPCS analyzes the view file standalone even though WordPress includes it inside a method/function scope.
28	30	foreach ($options as $key => $value) {
29		- echo '<option value="' . esc_attr( $key ) . '">' . $value . '</option>';
	31	+ echo '<option value="' . esc_attr( $key ) . '">' . esc_html($value) . '</option>';
30	32	}
31	33	?>
32	34	</select>
33	35	</span>
		@@ -35,31 +37,59 @@
35	37	</div>
36	38
37	39	<div class="inline-edit-group">
38	40	<label class="alignleft">
39		- <span class="title"><?php _e( 'Availability', 'propertyhive' ); ?></span>
	41	+ <span class="title"><?php echo esc_html(__( 'Featured', 'propertyhive' )); ?></span>
40	42	<span class="input-text-wrap">
	43	+ <select class="featured" name="_featured">
	44	+ <?php
	45	+ // phpcs:ignore WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound -- Template-local variable in an admin view; PHPCS analyzes the view file standalone even though WordPress includes it inside a method/function scope.
	46	+ $options = array(
	47	+ '' => __( '— No Change —', 'propertyhive' ),
	48	+ 'yes' => __( 'Yes', 'propertyhive' ),
	49	+ 'no' => __( 'No', 'propertyhive' ),
	50	+ );
	51	+ // phpcs:ignore WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound -- Template-local variable in an admin view; PHPCS analyzes the view file standalone even though WordPress includes it inside a method/function scope.
	52	+ foreach ($options as $key => $value) {
	53	+ echo '<option value="' . esc_attr( $key ) . '">' . esc_html($value) . '</option>';
	54	+ }
	55	+ ?>
	56	+ </select>
	57	+ </span>
	58	+ </label>
	59	+ </div>
	60	+
	61	+ <div class="inline-edit-group">
	62	+ <label class="alignleft">
	63	+ <span class="title"><?php echo esc_html(__( 'Availability', 'propertyhive' )); ?></span>
	64	+ <span class="input-text-wrap">
41	65	<select class="availability" name="_availability">
42	66	<?php
43	67
	68	+ // phpcs:ignore WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound -- Template-local variable in an admin view; PHPCS analyzes the view file standalone even though WordPress includes it inside a method/function scope.
44	69	$options = array( '' => __( '— No Change —', 'propertyhive' ) );
	70	+ // phpcs:ignore WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound -- Template-local variable in an admin view; PHPCS analyzes the view file standalone even though WordPress includes it inside a method/function scope.
45	71	$args = array(
46	72	'hide_empty' => false,
47	73	'parent' => 0
48	74	);
49		- $terms = get_terms( 'availability', $args );
	75	+ // phpcs:ignore WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound -- Template-local variable in an admin view; PHPCS analyzes the view file standalone even though WordPress includes it inside a method/function scope.
	76	+ $terms = get_terms( array_merge( wp_parse_args( $args ), array( 'taxonomy' => 'availability' ) ) );
50	77
	78	+ // phpcs:ignore WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound -- Template-local variable in an admin view; PHPCS analyzes the view file standalone even though WordPress includes it inside a method/function scope.
51	79	$selected_value = '';
52	80	if ( !empty( $terms ) && !is_wp_error( $terms ) )
53	81	{
54	82	foreach ($terms as $term)
55	83	{
	84	+ // phpcs:ignore WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound -- Template-local variable in an admin view; PHPCS analyzes the view file standalone even though WordPress includes it inside a method/function scope.
56	85	$options[$term->term_id] = $term->name;
57	86	}
58	87	}
59	88
	89	+ // phpcs:ignore WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound -- Template-local variable in an admin view; PHPCS analyzes the view file standalone even though WordPress includes it inside a method/function scope.
60	90	foreach ($options as $key => $value) {
61		- echo '<option value="' . esc_attr( $key ) . '">' . $value . '</option>';
	91	+ echo '<option value="' . esc_attr( $key ) . '">' . esc_html($value) . '</option>';
62	92	}
63	93	?>
64	94	</select>
65	95	</span>
		@@ -67,16 +97,18 @@
67	97	</div>
68	98
69	99	<div class="inline-edit-group">
70	100	<label class="alignleft">
71		- <span class="title"><?php _e( 'Negotiator', 'propertyhive' ); ?></span>
	101	+ <span class="title"><?php echo esc_html(__( 'Negotiator', 'propertyhive' )); ?></span>
72	102	<span class="input-text-wrap">
73	103	<?php
	104	+ // phpcs:ignore WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound -- Template-local variable in an admin view; PHPCS analyzes the view file standalone even though WordPress includes it inside a method/function scope.
74	105	$args = array(
75	106	'name' => '_negotiator_id',
76	107	'id' => '_negotiator_id',
77	108	'show_option_none' => '— No Change —',
78		- 'role__not_in' => array('property_hive_contact')
	109	+ // phpcs:ignore WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedHooknameFound -- Legacy Property Negotiator compatibility filter; existing role filters depend on this exact public hook name.
	110	+ 'role__not_in' => apply_filters( 'property_negotiator_exclude_roles', array('property_hive_contact', 'subscriber') )
79	111	);
80	112	wp_dropdown_users($args);
81	113	?>
82	114	</span>
		@@ -84,14 +116,16 @@
84	116	</div>
85	117
86	118	<div class="inline-edit-group">
87	119	<label class="alignleft">
88		- <span class="title"><?php _e( 'Office', 'propertyhive' ); ?></span>
	120	+ <span class="title"><?php echo esc_html(__( 'Office', 'propertyhive' )); ?></span>
89	121	<span class="input-text-wrap">
90	122	<select class="office_id" name="_office_id">
91	123	<?php
92	124
	125	+ // phpcs:ignore WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound -- Template-local variable in an admin view; PHPCS analyzes the view file standalone even though WordPress includes it inside a method/function scope.
93	126	$options = array( '' => __( '— No Change —', 'propertyhive' ) );
	127	+ // phpcs:ignore WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound -- Template-local variable in an admin view; PHPCS analyzes the view file standalone even though WordPress includes it inside a method/function scope.
94	128	$args = array(
95	129	'post_type' => 'office',
96	130	'nopaging' => true,
97	131	'orderby' => 'title',
		@@ -96,8 +130,9 @@
96	130	'nopaging' => true,
97	131	'orderby' => 'title',
98	132	'order' => 'ASC'
99	133	);
	134	+ // phpcs:ignore WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound -- Template-local variable in an admin view; PHPCS analyzes the view file standalone even though WordPress includes it inside a method/function scope.
100	135	$office_query = new WP_Query($args);
101	136
102	137	if ($office_query->have_posts())
103	138	{
		@@ -104,8 +139,9 @@
104	139	while ($office_query->have_posts())
105	140	{
106	141	$office_query->the_post();
107	142
	143	+ // phpcs:ignore WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound -- Template-local variable in an admin view; PHPCS analyzes the view file standalone even though WordPress includes it inside a method/function scope.
108	144	$options[get_the_ID()] = get_the_title();
109	145	}
110	146	}
111	147
		@@ -110,10 +146,11 @@
110	146	}
111	147
112	148	$office_query->reset_postdata();
113	149
	150	+ // phpcs:ignore WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound -- Template-local variable in an admin view; PHPCS analyzes the view file standalone even though WordPress includes it inside a method/function scope.
114	151	foreach ($options as $key => $value) {
115		- echo '<option value="' . esc_attr( $key ) . '">' . $value . '</option>';
	152	+ echo '<option value="' . esc_attr( $key ) . '">' . esc_html($value) . '</option>';
116	153	}
117	154	?>
118	155	</select>
119	156	</span>
		@@ -122,7 +159,7 @@
122	159
123	160	<?php do_action( 'propertyhive_property_bulk_edit_end' ); ?>
124	161
125	162	<input type="hidden" name="propertyhive_bulk_edit" value="1" />
126		- <input type="hidden" name="propertyhive_bulk_edit_nonce" value="<?php echo wp_create_nonce( 'propertyhive_bulk_edit_nonce' ); ?>" />
	163	+ <input type="hidden" name="propertyhive_bulk_edit_nonce" value="<?php echo esc_attr(wp_create_nonce( 'propertyhive_bulk_edit_nonce' )); ?>" />
127	164	</div>
128	165	</fieldset>