Theme check выдает ошибку wp_deregister_script как исправить?

Question

[Владислав]

Добрый день.

В вордпрес по умолчанию стоит не последняя версия jquery я подключил последнюю и Theme check пишет такую ошибку

WARNING: Found wp_deregister_script in functions.php. Themes must not deregister core scripts.
Line 384: wp_deregister_script( 'jquery' );

add_action( 'wp_enqueue_scripts', 'sky_scripts_method', 11 );
	function sky_scripts_method() {
		if (!is_admin()) {
			wp_deregister_script( 'jquery' );
			wp_register_script( 'jquery', get_template_directory_uri() . '/assets/libs/jquery/jquery-3.3.1.min.js', false, '3.3.1', true);
			wp_enqueue_script( 'jquery' );
		}
	}

Comments

[Орхан Гасанлы]

wp_deregister_script( 'jquery' );
wp_dequeue_script( 'jquery' );

попробуйте так...
а так в принципе, это не ошибка, а скорее предупреждение...

[Владислав]

wp_dequeue_script не решает проблему
там предупреждение именно на wp_deregister_script

Answer 1

[WP Panda]

для чего приоритет 11 ?

Comments

[Владислав]

если 11 убрать, то jquery не меняется. остается старый

[WP Panda]

И попробуйте вынести из условного тега, в свое время на themforest видел такое

add_action( 'wp_enqueue_scripts', 'sky_scripts_method' );
  function sky_scripts_method() {
    if ( is_admin() ) { 
return 
}
      wp_deregister_script( 'jquery' );
      wp_register_script( 'jquery', get_template_directory_uri() . '/assets/libs/jquery/jquery-3.3.1.min.js', false, '3.3.1', true);
      wp_enqueue_script( 'jquery' );

  }

Ну и трабл может быть в каком то плагине, надо смотреть по месту, вот подобное в поддержке enfold, там виноват JoomUnited

[Владислав]

WP Panda, SSMaker.ru/9523a487 так же.
он ругается на саму функцию wp_deregister_script

[Владислав]

WP Panda, плагинов нет. это тема на основе underscores. учусь темы создавать)

[WP Panda]

Vladddosss,
Он не ругается на саму функцию, он пишет следующее
Вы не должны отменять скрипты из ядра

[WP Panda]

Сейчас гляну код theme check

[WP Panda]

Vladddosss, Забейте, это чисто придурь theme check
https://github.com/WordPress/theme-check/blob/da79...
он видит, что переопределили файл ядра, он говорит, что не стоит

[Владислав]

WP Panda, Спасибо. Надеюсь модераторы темы это не сочтут ошибкой

[WP Panda]

Vladddosss, Ошибкой нет, но в темах на продажу переопределять скрипты не стоит

[Владислав]

WP Panda, а для репозитория вордпрес?

Answer 2

[Вова Дружаев]

Если вы это делаете для себя - не обращайте внимание. Вам этот гемор потом разгребать самому.

А если отдавать или продавать будете - то вас завернут.

Это плохая практика дерегистрировать скрипты ядра. Проблемы возникнут со сторонними плагинами. Опытный и хороший разработчик знает что работать надо не с хайповыми и последними версиями jquery, а я даже думаю вы и различий то между версиями не знаете, а со стабильными, теми что используем само ядро фреймворка.

Нужен вам видимо секс - вы его получите, как только куча плагинов вам в консоли покажет что jquery у вас не тот. ну и хендбуки - а кто их читает то. Да?