← All changes
|
includes/admin/views/html-admin-matching-applicants.php
+62
-14
2.2.5
→
2.3.1
View file →
| @@ -14,17 +14,20 @@ | ||
| 14 | 14 | |
| 15 | 15 | <?php |
| 16 | 16 | if ( !empty($applicants) ) |
| 17 | 17 | { |
| 18 | + // 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. | |
| 18 | 19 | $select_all_actions = array( |
| 19 | 20 | 'email' => __( 'Email', 'propertyhive' ), |
| 20 | 21 | 'not_interested' => __( 'Not Suitable', 'propertyhive' ) |
| 21 | 22 | ); |
| 22 | 23 | |
| 24 | + // 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 | 25 | $select_all_actions = apply_filters( 'propertyhive_matching_select_all_actions', $select_all_actions ); |
| 24 | 26 | ?> |
| 25 | 27 | <div class="select-actions" style="padding-bottom:15px"> |
| 26 | 28 | <span style="display:inline-block; vertical-align:middle;"><?php echo esc_html(__( 'Select', 'propertyhive' )); ?>:</span> <?php |
| 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. | |
| 27 | 30 | foreach ( $select_all_actions as $key => $value ) |
| 28 | 31 | { |
| 29 | 32 | echo '<a href="javascript:;" class="button" id="select_all_' . esc_attr(sanitize_title($key)) . '" style="display:inline-block; vertical-align:middle;">All - ' . esc_html($value) . '</a> '; |
| 30 | 33 | } |
| @@ -33,8 +36,9 @@ | ||
| 33 | 36 | </div> |
| 34 | 37 | <?php |
| 35 | 38 | echo '<table width="100%">'; |
| 36 | 39 | |
| 40 | + // 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. | |
| 37 | 41 | $columns = array( |
| 38 | 42 | 'name' => 'Name', |
| 39 | 43 | 'contact_details' => 'Contact Details', |
| 40 | 44 | 'requirements' => 'Requirements' |
| @@ -39,12 +43,14 @@ | ||
| 39 | 43 | 'contact_details' => 'Contact Details', |
| 40 | 44 | 'requirements' => 'Requirements' |
| 41 | 45 | ); |
| 42 | 46 | |
| 47 | + // 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. | |
| 43 | 48 | $columns = apply_filters( 'propertyhive_matching_applicants_row_headings', $columns ); |
| 44 | 49 | |
| 45 | 50 | echo '<thead> |
| 46 | 51 | <tr>'; |
| 52 | + // 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. | |
| 47 | 53 | foreach ( $columns as $key => $column ) |
| 48 | 54 | { |
| 49 | 55 | echo '<th style="text-align:left">' . esc_html($column) . '</th>'; |
| 50 | 56 | } |
| @@ -54,40 +60,55 @@ | ||
| 54 | 60 | </thead> |
| 55 | 61 | <tbody> |
| 56 | 62 | '; |
| 57 | 63 | |
| 64 | + // 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. | |
| 58 | 65 | $percentage_lower = get_option( 'propertyhive_applicant_match_price_range_percentage_lower', '' ); |
| 66 | + // 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. | |
| 59 | 67 | $percentage_higher = get_option( 'propertyhive_applicant_match_price_range_percentage_higher', '' ); |
| 60 | 68 | |
| 69 | + // 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. | |
| 61 | 70 | foreach ( $applicants as $applicant ) |
| 62 | 71 | { |
| 72 | + // 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. | |
| 63 | 73 | $previously_sent = array(); |
| 64 | 74 | |
| 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. | |
| 65 | 76 | $currency = '£'; |
| 66 | 77 | if ( isset($applicant['applicant_profile']['currency']) && !empty($applicant['applicant_profile']['currency']) ) |
| 67 | 78 | { |
| 79 | + // 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. | |
| 68 | 80 | $PH_Countries = new PH_Countries(); |
| 81 | + // 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. | |
| 69 | 82 | $selected_currency = $PH_Countries->get_currency($applicant['applicant_profile']['currency']); |
| 70 | 83 | if ( $selected_currency !== false ) |
| 71 | 84 | { |
| 85 | + // 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. | |
| 72 | 86 | $currency = $selected_currency['currency_symbol']; |
| 73 | 87 | } |
| 74 | 88 | } |
| 75 | 89 | |
| 90 | + // 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 | 91 | $applicant_profile_match_history = get_post_meta( $applicant['contact_id'], '_applicant_profile_' . $applicant['applicant_profile']['applicant_profile_id'] . '_match_history', TRUE ); |
| 77 | 92 | if ( isset($applicant_profile_match_history[$property->id]) && is_array($applicant_profile_match_history[$property->id]) && !empty($applicant_profile_match_history[$property->id]) ) |
| 78 | 93 | { |
| 94 | + // 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. | |
| 79 | 95 | $previously_sent = $applicant_profile_match_history[$property->id]; |
| 80 | 96 | } |
| 81 | 97 | |
| 98 | + // 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. | |
| 82 | 99 | $email_address = get_post_meta( $applicant['contact_id'], '_email_address', TRUE ); |
| 100 | + // 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. | |
| 83 | 101 | $forbidden_contact_methods = get_post_meta( $applicant['contact_id'], '_forbidden_contact_methods', TRUE ); |
| 102 | + // 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. | |
| 84 | 103 | $do_not_email = false; |
| 85 | 104 | if ( is_array($forbidden_contact_methods) && in_array('email', $forbidden_contact_methods) ) |
| 86 | 105 | { |
| 106 | + // 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. | |
| 87 | 107 | $do_not_email = true; |
| 88 | 108 | } |
| 89 | 109 | |
| 110 | + // 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. | |
| 90 | 111 | $requirements = array(); |
| 91 | 112 | if ( |
| 92 | 113 | isset($applicant['applicant_profile']['department']) && |
| 93 | 114 | ( $applicant['applicant_profile']['department'] == 'residential-sales' || ph_get_custom_department_based_on($applicant['applicant_profile']['department']) == 'residential-sales' |
| @@ -95,15 +116,17 @@ | ||
| 95 | 116 | ) |
| 96 | 117 | { |
| 97 | 118 | if ( isset($applicant['applicant_profile']['max_price']) && $applicant['applicant_profile']['max_price'] != '' ) |
| 98 | 119 | { |
| 120 | + // 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. | |
| 99 | 121 | $requirements[] = array( |
| 100 | 122 | 'label' => __( 'Maximum Price', 'propertyhive' ), |
| 101 | - 'value' => $currency . ph_display_price_field($applicant['applicant_profile']['max_price']), | |
| 123 | + 'value' => $currency . esc_html( ph_display_price_field($applicant['applicant_profile']['max_price']) ), | |
| 102 | 124 | ); |
| 103 | 125 | } |
| 104 | 126 | if ( $percentage_lower != '' && $percentage_higher != '' ) |
| 105 | 127 | { |
| 128 | + // 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. | |
| 106 | 129 | $match_price_range_lower = ''; |
| 107 | 130 | if ( |
| 108 | 131 | !isset($applicant['applicant_profile']['match_price_range_lower_actual']) || |
| 109 | 132 | ( isset($applicant['applicant_profile']['match_price_range_lower_actual']) && $applicant['applicant_profile']['match_price_range_lower_actual'] == '' ) |
| @@ -110,16 +133,19 @@ | ||
| 110 | 133 | ) |
| 111 | 134 | { |
| 112 | 135 | if ( isset($applicant['applicant_profile']['max_price']) && $applicant['applicant_profile']['max_price'] != '' ) |
| 113 | 136 | { |
| 114 | - $match_price_range_lower = $applicant['applicant_profile']['max_price'] - ( $applicant['applicant_profile']['max_price'] * ( $percentage_lower / 100 ) ); | |
| 137 | + // 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. | |
| 138 | + $match_price_range_lower = (float) $applicant['applicant_profile']['max_price'] - ( (float) $applicant['applicant_profile']['max_price'] * ( (float) $percentage_lower / 100 ) ); | |
| 115 | 139 | } |
| 116 | 140 | } |
| 117 | 141 | else |
| 118 | 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. | |
| 119 | 144 | $match_price_range_lower = $applicant['applicant_profile']['match_price_range_lower']; |
| 120 | 145 | } |
| 121 | 146 | |
| 147 | + // 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. | |
| 122 | 148 | $match_price_range_higher = ''; |
| 123 | 149 | if ( |
| 124 | 150 | !isset($applicant['applicant_profile']['match_price_range_higher_actual']) || |
| 125 | 151 | ( isset($applicant['applicant_profile']['match_price_range_higher_actual']) && $applicant['applicant_profile']['match_price_range_higher_actual'] == '' ) |
| @@ -126,13 +152,15 @@ | ||
| 126 | 152 | ) |
| 127 | 153 | { |
| 128 | 154 | if ( isset($applicant['applicant_profile']['max_price']) && $applicant['applicant_profile']['max_price'] != '' ) |
| 129 | 155 | { |
| 130 | - $match_price_range_higher = $applicant['applicant_profile']['max_price'] + ( $applicant['applicant_profile']['max_price'] * ( $percentage_higher / 100 ) ); | |
| 156 | + // 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. | |
| 157 | + $match_price_range_higher = (float) $applicant['applicant_profile']['max_price'] + ( (float) $applicant['applicant_profile']['max_price'] * ( (float) $percentage_higher / 100 ) ); | |
| 131 | 158 | } |
| 132 | 159 | } |
| 133 | 160 | else |
| 134 | 161 | { |
| 162 | + // 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. | |
| 135 | 163 | $match_price_range_higher = $applicant['applicant_profile']['match_price_range_higher']; |
| 136 | 164 | } |
| 137 | 165 | |
| 138 | 166 | if ( |
| @@ -138,11 +166,12 @@ | ||
| 138 | 166 | if ( |
| 139 | 167 | $match_price_range_lower != '' && $match_price_range_higher != '' |
| 140 | 168 | ) |
| 141 | 169 | { |
| 170 | + // 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. | |
| 142 | 171 | $requirements[] = array( |
| 143 | 172 | 'label' => __( 'Maximum Price Range', 'propertyhive' ), |
| 144 | - 'value' => $currency . ph_display_price_field($match_price_range_lower) . ' to ' . $currency . ph_display_price_field($match_price_range_higher), | |
| 173 | + 'value' => $currency . esc_html( ph_display_price_field($match_price_range_lower) ) . ' to ' . $currency . esc_html( ph_display_price_field($match_price_range_higher) ), | |
| 145 | 174 | ); |
| 146 | 175 | } |
| 147 | 176 | } |
| 148 | 177 | } |
| @@ -154,11 +183,12 @@ | ||
| 154 | 183 | ) |
| 155 | 184 | { |
| 156 | 185 | if ( isset($applicant['applicant_profile']['max_rent']) && $applicant['applicant_profile']['max_rent'] != '' ) |
| 157 | 186 | { |
| 187 | + // 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. | |
| 158 | 188 | $requirements[] = array( |
| 159 | 189 | 'label' => __( 'Maximum Rent', 'propertyhive' ), |
| 160 | - 'value' => $currency . ph_display_price_field($applicant['applicant_profile']['max_rent']) . ' ' . $applicant['applicant_profile']['rent_frequency'] | |
| 190 | + 'value' => $currency . esc_html( ph_display_price_field($applicant['applicant_profile']['max_rent']) ) . ' ' . esc_html( $applicant['applicant_profile']['rent_frequency'] ) | |
| 161 | 191 | ); |
| 162 | 192 | } |
| 163 | 193 | } |
| 164 | 194 | |
| @@ -163,11 +193,12 @@ | ||
| 163 | 193 | } |
| 164 | 194 | |
| 165 | 195 | if ( isset($applicant['applicant_profile']['min_beds']) && $applicant['applicant_profile']['min_beds'] != '' ) |
| 166 | 196 | { |
| 197 | + // 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. | |
| 167 | 198 | $requirements[] = array( |
| 168 | 199 | 'label' => __( 'Minimum Beds', 'propertyhive' ), |
| 169 | - 'value' => $applicant['applicant_profile']['min_beds'], | |
| 200 | + 'value' => esc_html( $applicant['applicant_profile']['min_beds'] ), | |
| 170 | 201 | ); |
| 171 | 202 | } |
| 172 | 203 | |
| 173 | 204 | if ( isset($applicant['applicant_profile']['property_types']) && is_array($applicant['applicant_profile']['property_types']) && !empty($applicant['applicant_profile']['property_types']) ) |
| @@ -176,44 +207,54 @@ | ||
| 176 | 207 | } |
| 177 | 208 | |
| 178 | 209 | if ( get_option('propertyhive_applicant_locations_type') == 'text' && isset($applicant['applicant_profile']['location_text']) && $applicant['applicant_profile']['location_text'] != '') |
| 179 | 210 | { |
| 211 | + // 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. | |
| 180 | 212 | $location_value = $applicant['applicant_profile']['location_text']; |
| 181 | 213 | |
| 182 | 214 | if ( isset($applicant['applicant_profile']['location_radius']) && $applicant['applicant_profile']['location_radius'] != '' ) |
| 183 | 215 | { |
| 216 | + // 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. | |
| 184 | 217 | $location_value .= ' (Within '. $applicant['applicant_profile']['location_radius'] .' Miles)'; |
| 185 | 218 | } |
| 186 | 219 | |
| 220 | + // 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. | |
| 187 | 221 | $requirements[] = array( |
| 188 | 222 | 'label' => __( 'Location', 'propertyhive' ), |
| 189 | - 'value' => $location_value, | |
| 223 | + 'value' => esc_html( $location_value ), | |
| 190 | 224 | ); |
| 191 | 225 | } |
| 192 | 226 | |
| 227 | + // 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. | |
| 193 | 228 | $requirements = apply_filters( 'propertyhive_applicant_requirements_display', $requirements, $applicant['contact_id'], $applicant['applicant_profile'] ); |
| 194 | 229 | |
| 195 | 230 | if ( isset($applicant['applicant_profile']['notes']) && $applicant['applicant_profile']['notes'] != '' ) |
| 196 | 231 | { |
| 232 | + // 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. | |
| 197 | 233 | $requirements[] = array( |
| 198 | 234 | 'label' => __( 'Notes', 'propertyhive' ), |
| 199 | - 'value' => nl2br( strip_tags($applicant['applicant_profile']['notes']) ), | |
| 235 | + 'value' => nl2br( esc_html( wp_strip_all_tags($applicant['applicant_profile']['notes']) ) ), | |
| 200 | 236 | ); |
| 201 | 237 | } |
| 202 | 238 | |
| 239 | + // 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. | |
| 203 | 240 | $requirements_output = ''; |
| 204 | 241 | if ( empty($requirements) ) |
| 205 | 242 | { |
| 243 | + // 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. | |
| 206 | 244 | $requirements_output = '-'; |
| 207 | 245 | } |
| 208 | 246 | else |
| 209 | 247 | { |
| 248 | + // 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. | |
| 210 | 249 | foreach ( $requirements as $requirement ) |
| 211 | 250 | { |
| 212 | - $requirements_output .= $requirement['label'] . ': ' . $requirement['value'] . '<br>'; | |
| 251 | + // 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. | |
| 252 | + $requirements_output .= esc_html( $requirement['label'] ) . ': ' . $requirement['value'] . '<br>'; | |
| 213 | 253 | } |
| 214 | 254 | } |
| 215 | 255 | |
| 256 | + // 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. | |
| 216 | 257 | $columns = array( |
| 217 | 258 | 'name' => '<strong><a href="' . esc_url(get_edit_post_link($applicant['contact_id'])) . '" target="_blank">' . esc_html(get_the_title($applicant['contact_id'])) . '</a>' . ( ( isset($applicant['applicant_profile']['grading']) && $applicant['applicant_profile']['grading'] == 'hot' ) ? '<br><span style="color:#C00;">('. esc_html(__( 'Hot Applicant', 'propertyhive' )) . ')</span>' : '' ) . '</strong>', |
| 218 | 259 | 'contact_details' => 'T: ' . esc_html(get_post_meta( $applicant['contact_id'], '_telephone_number', TRUE )) . '<br>E: ' . esc_html($email_address), |
| 219 | 260 | 'requirements' => $requirements_output, |
| @@ -218,13 +259,16 @@ | ||
| 218 | 259 | 'contact_details' => 'T: ' . esc_html(get_post_meta( $applicant['contact_id'], '_telephone_number', TRUE )) . '<br>E: ' . esc_html($email_address), |
| 219 | 260 | 'requirements' => $requirements_output, |
| 220 | 261 | ); |
| 221 | 262 | |
| 263 | + // 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. | |
| 222 | 264 | $columns = apply_filters( 'propertyhive_matching_applicants_row_data', $columns, $applicant, $property->id ); |
| 223 | 265 | |
| 224 | 266 | echo '<tr id="matching_contact_' . (int)$applicant['contact_id'] . '_applicant_profile_' . (int)$applicant['applicant_profile']['applicant_profile_id'] . '">'; |
| 267 | + // 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. | |
| 225 | 268 | foreach ( $columns as $key => $column ) |
| 226 | 269 | { |
| 270 | + // phpcs:ignore WordPress.Security.EscapeOutput.OutputNotEscaped -- Built-in row text and requirement labels/values are escaped above; preserve trusted currency, requirement and propertyhive_matching_applicants_row_data HTML filters. | |
| 227 | 271 | echo '<td style="border-bottom:1px solid #CCC; padding:5px 0;">' . $column . '</td>'; |
| 228 | 272 | } |
| 229 | 273 | echo '<td style="border-bottom:1px solid #CCC; padding:5px 0;">'; |
| 230 | 274 | |
| @@ -229,8 +273,9 @@ | ||
| 229 | 273 | echo '<td style="border-bottom:1px solid #CCC; padding:5px 0;">'; |
| 230 | 274 | |
| 231 | 275 | echo '<label><input type="checkbox" name="email_contact_applicant_profile_id[]" value="' . (int)$applicant['contact_id'] . '|' . (int)$applicant['applicant_profile']['applicant_profile_id'] . '" '; |
| 232 | 276 | |
| 277 | + // 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. | |
| 233 | 278 | $post_tip = ''; |
| 234 | 279 | if ( strpos($email_address, '@') === FALSE ) |
| 235 | 280 | { |
| 236 | 281 | echo ' disabled title="Invalid email address: ' . esc_attr($email_address) . '"'; |
| @@ -244,9 +289,10 @@ | ||
| 244 | 289 | else |
| 245 | 290 | { |
| 246 | 291 | if ( !empty($previously_sent) ) |
| 247 | 292 | { |
| 248 | - $post_tip = 'Sent previously via ' . $previously_sent[count($previously_sent) - 1]['method'] . ' on ' . date("jS F Y", strtotime($previously_sent[count($previously_sent) - 1]['date'])); | |
| 293 | + // 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. | |
| 294 | + $post_tip = 'Sent previously via ' . $previously_sent[count($previously_sent) - 1]['method'] . ' on ' . gmdate("jS F Y", strtotime($previously_sent[count($previously_sent) - 1]['date'])); | |
| 249 | 295 | |
| 250 | 296 | if ( |
| 251 | 297 | $on_market_change_date > $previously_sent[count($previously_sent) - 1]['date'] || |
| 252 | 298 | $price_change_date > $previously_sent[count($previously_sent) - 1]['date'] |
| @@ -253,13 +299,15 @@ | ||
| 253 | 299 | ) |
| 254 | 300 | { |
| 255 | 301 | if ( $price_change_date > $previously_sent[count($previously_sent) - 1]['date'] ) |
| 256 | 302 | { |
| 257 | - $post_tip .= ', however a price change occurred on ' . date("jS F Y", strtotime($price_change_date)); | |
| 303 | + // 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. | |
| 304 | + $post_tip .= ', however a price change occurred on ' . gmdate("jS F Y", strtotime($price_change_date)); | |
| 258 | 305 | } |
| 259 | 306 | elseif ( $on_market_change_date > $previously_sent[count($previously_sent) - 1]['date'] ) |
| 260 | 307 | { |
| 261 | - $post_tip .= ', however a change to the on market status occurred on ' . date("jS F Y", strtotime($on_market_change_date)); | |
| 308 | + // 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. | |
| 309 | + $post_tip .= ', however a change to the on market status occurred on ' . gmdate("jS F Y", strtotime($on_market_change_date)); | |
| 262 | 310 | } |
| 263 | 311 | echo ' checked'; |
| 264 | 312 | } |
| 265 | 313 | } |
| @@ -287,9 +335,9 @@ | ||
| 287 | 335 | '; |
| 288 | 336 | } |
| 289 | 337 | else |
| 290 | 338 | { |
| 291 | - echo __( 'No matching applicants found', 'propertyhive' ); | |
| 339 | + echo esc_html__( 'No matching applicants found', 'propertyhive' ); | |
| 292 | 340 | } |
| 293 | 341 | ?> |
| 294 | 342 | |
| 295 | 343 | <p class="submit"> |
| @@ -295,9 +343,9 @@ | ||
| 295 | 343 | <p class="submit"> |
| 296 | 344 | |
| 297 | 345 | <input name="save" class="button-primary" type="submit" value="<?php echo esc_html(__( 'Continue', 'propertyhive' )); ?>" /> |
| 298 | 346 | |
| 299 | - <a href="<?php echo esc_url(get_edit_post_link((int)$_GET['property_id'])); ?>" class="button"><?php echo esc_html(__( 'Cancel', 'propertyhive' )); ?></a> | |
| 347 | + <a href="<?php echo esc_url(get_edit_post_link( (int) $property_id )); ?>" class="button"><?php echo esc_html(__( 'Cancel', 'propertyhive' )); ?></a> | |
| 300 | 348 | |
| 301 | 349 | <input type="hidden" name="step" value="one" /> |
| 302 | 350 | <?php wp_nonce_field( 'propertyhive-matching-applicants' ); ?> |
| 303 | 351 | |