Skip to content

Commit 1afdba9

Browse files
fixed suggestions
1 parent a59182e commit 1afdba9

File tree

3 files changed

+19
-19
lines changed

3 files changed

+19
-19
lines changed

includes/admin/feedzy-rss-feeds-admin.php

Lines changed: 6 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -2521,30 +2521,26 @@ public function add_black_friday_data( $configs ) {
25212521
*/
25222522
public function validate_feed() {
25232523
try {
2524-
if ( ! isset( $_POST['nonce'] ) || ! wp_verify_nonce( sanitize_text_field( wp_unslash( $_POST['nonce'] ) ), FEEDZY_BASEFILE ) ) {
2525-
wp_send_json_error( array( 'message' => __( 'Security check failed.', 'feedzy-rss-feeds' ) ) );
2526-
return;
2527-
}
2524+
if ( ! isset( $_POST['nonce'] ) ||
2525+
! wp_verify_nonce( sanitize_text_field( wp_unslash( $_POST['nonce'] ) ), FEEDZY_BASEFILE ) ) {
2526+
wp_send_json_error( array( 'message' => __( 'Security check failed.', 'feedzy-rss-feeds' ) ) );
2527+
}
25282528

25292529
$feed_urls = isset($_POST['feed_url']) ? sanitize_text_field( wp_unslash( $_POST['feed_url'] ) ) : '';
25302530

25312531
if (empty($feed_urls)) {
25322532
wp_send_json_error(array('message' => __('Feed URL cannot be empty.', 'feedzy-rss-feeds')));
2533-
return;
25342533
}
25352534

25362535
$urls = array_map('trim', explode(',', $feed_urls));
25372536
$urls = array_filter($urls);
25382537

25392538
if (empty($urls)) {
25402539
wp_send_json_error(array('message' => __('No valid URLs provided.', 'feedzy-rss-feeds')));
2541-
return;
25422540
}
25432541

25442542
$results = array();
25452543

2546-
$cache = isset($_POST['cache']) ? sanitize_text_field( wp_unslash( $_POST['cache'] ) ) : '1_mins';
2547-
25482544
foreach ($urls as $feed_url) {
25492545
$feed_url = esc_url_raw($feed_url);
25502546

@@ -2557,7 +2553,7 @@ public function validate_feed() {
25572553
continue;
25582554
}
25592555

2560-
$feed = $this->fetch_feed(array($feed_url), $cache, array());
2556+
$feed = $this->fetch_feed(array($feed_url), '1_mins', array());
25612557

25622558
if ( is_wp_error($feed) ) {
25632559
$results[] = array(
@@ -2630,7 +2626,7 @@ public function validate_feed() {
26302626
wp_send_json_success(array(
26312627
'results' => $results
26322628
));
2633-
} catch (Exception $e) {
2629+
} catch (Throwable $e) {
26342630
wp_send_json_error(array(
26352631
/* translators: %s is the error message */
26362632
'message' => sprintf(__('An error occurred: %s', 'feedzy-rss-feeds'), $e->getMessage())

includes/views/css/import-metabox-edit.css

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -208,14 +208,13 @@ span.feedzy-spinner {
208208
padding: 12px 15px;
209209
border-radius: 4px;
210210
font-size: 14px;
211-
animation: slideIn 0.3s ease-out;
212211
position: relative;
213212
z-index: 10;
214-
display: flex;
215-
flex-direction: column;
213+
display: flex;
214+
flex-direction: column;
216215
align-items: flex-start;
217216
gap: 8px;
218-
border: 1px solid #5555559c;
217+
border: 1px solid #5555559c;
219218
}
220219

221220
.feedzy-wrap .fz-validation-message button {

includes/views/js/import-metabox-edit.js

Lines changed: 10 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -172,7 +172,6 @@
172172
function showMessage( message, autoDismiss = true ) {
173173
const $container = $('.fz-validation-summary');
174174
if ( ! $container.length ) {
175-
$('body').append('<div class="fz-validation-summary"></div>');
176175
return;
177176
}
178177

@@ -187,8 +186,14 @@
187186
if ( autoDismiss ) {
188187
$message.delay(5000).fadeOut(300, () => $message.remove());
189188
} else {
190-
$message.append('<button type="button" class="button button-primary">✕</button>');
191-
$message.find('button').on('click', function(e) {
189+
const $closeButton = $('<button>', {
190+
type: 'button',
191+
class: 'button button-primary',
192+
text: '✕'
193+
});
194+
195+
$message.append($closeButton);
196+
$closeButton.on('click', function(e) {
192197
e.preventDefault();
193198
$(this).parent().fadeOut(300, () => $(this).remove());
194199
});
@@ -218,12 +223,12 @@
218223
$('[name="feedzy_meta_data[' + field_name + ']"]').focus();
219224
return false;
220225
}
221-
226+
222227
function remove_row() {
223228
$(this).closest(".key-value-item").remove();
224229
return false;
225230
}
226-
231+
227232
function new_row() {
228233
var html_row = "";
229234
html_row = $("#new_field_tpl").html();

0 commit comments

Comments
 (0)