Как изменить папку загрузки файлов в wordpress?

Question

[Даниил]

Доброго всем!
Хотел изменить стандартную папку загрузки файлов в wordpress.
Наваял такую вот функцию:

function change_upload_dir($args) {
	$id = (isset($_GET['post']) ? $_GET['post'] : '');

	if (get_post_type($id) == 'post_type') {
		$args['path'] = $args['basedir'] . '/post_type';
		$args['url'] = $args['baseurl'] . '/post_type';
		$args['subdir'] = '/post_type';
	}

	return $args;
}
add_filter('upload_dir', 'change_upload_dir');

Но, в таком виде не работает. Причем, если убрать условие или вставить в if true, то все работает. Причем, другие операторы, например, var_dump, отрабатывают с этим условием и даже создается подпапка для файлов, но файлы грузятся по дефолтному адресу. Как такое возможно вообще?

Answer 1

[Vitaliy Orlov]

судя по описанию, у тебя либо $id неправильно обрабатывается, либо post_type неверный

попробуй отладить так:

if (get_post_type(get_the_ID()) == 'post_type') {
    ...
  } else {
    die('POST TYPE: '.get_post_type(get_the_ID()));
  }

Comments

[Даниил]

Функция get_the_ID() здесь тоже не работает.
Сделал так:

$id = (isset($_GET['post']) ? $_GET['post'] : '');
	var_dump($id); //Выводит id поста
	var_dump(get_post_type($id)); //Выводит тип поста

	if ( $_GET['action'] == 'edit' ) {
		if (get_post_type($id) == 'post_type') {
			$args['path'] = $args['basedir'] . '/post_type';
			$args['url'] = $args['baseurl'] . '/post_type';
			$args['subdir'] = '/post_type';
			var_dump($id); //Выводит id поста
			var_dump(get_post_type($id)); //Выводит тип поста
		}else {
			die('POST TYPE: '.get_post_type(get_the_ID()));
		}
	}

Функция в таком виде не умирает, выводит корректные id и тип поста, создается папка с именем типа поста, но файлы туда не загружаются.

Answer 2

[Игорь Воротнёв]

В доке написано (болд мой):

Using this, in conjunction with the wp_handle_upload_prefilter, you can dynamically determine which directory to upload to, based on the files you upload.

Посмотрите примеры тут и тут.

Comments

[Даниил]

Сдклал так:

function change_upload_dir($args) {
	$id = (isset($_GET['post']) ? $_GET['post'] : '');
	var_dump(get_post_type($id)); //Не срабатывает

	if ( $_GET['action'] == 'edit' ) {
		if (get_post_type($id) == 'post_type') {
			$args['path'] = $args['basedir'] . '/post_type';
			$args['url'] = $args['baseurl'] . '/post_type';
			$args['subdir'] = '/post_type';	
			var_dump(get_post_type($id)); //Не срабатывает
		}else {
			die('POST TYPE: '.get_post_type(get_the_ID()));
		}
	}

	return $args;
}

add_filter('wp_handle_upload_prefilter', 'my_upload_prefilter');
add_filter('wp_handle_upload', 'my_handle_upload');

function my_upload_prefilter( $file ) {
	add_filter('upload_dir', 'change_upload_dir');
	return $file;
}

function my_handle_upload( $fileinfo ) {
	remove_filter('upload_dir', 'change_upload_dir');
	return $fileinfo;
}

Ничего не происходит. Т.е. файлы грузятся как обычно отладки не срабатывают.

[Игорь Воротнёв]

Даниил:
> var_dump(get_post_type($id)); //Не срабатывает
И не сработает, поскольку get_post_type() всегда здесь будет возвращать false. Вы работаете в контексте функции, $id внутри нее не определена.

[Даниил]

Игорь Воротнёв:
Мне кажется, что в таком виде функция вообще не работает, т.к. не происходит ничего, кок будто ее и нет.

[Даниил]

Хотя нет, функция работает, но опять при проверке либо на тип поста
if (get_post_type($id) == 'post_type')
либо на экшн
if ( $_GET['action'] == 'edit' )
файлы грузятся по стандартному пути.

[Даниил]

function change_upload_dir($args) {
	$current_page = $_SERVER['HTTP_REFERER'];
	$id          = $_REQUEST['post'];
	
	if (strpos( $current_page, 'post_type' )) {
		$args['path'] = $args['basedir'] . '/post_type';
		$args['url'] = $args['baseurl'] . '/post_type';
		$args['subdir'] = '/post_type';
		var_dump($id); //Выводит id поста
		
	}
	
	return $args;
}

Так тоже не работает, хотя var_dump выводить id.

[Игорь Воротнёв]

Даниил: стоп. У вас проверка в реальности идет на 'post_type', именно эту строку, дословно?

[Даниил]

Нет, нет. Предполагается несколько типов и для каждого нужна своя папка.