1

I'm sorry for my english... I am using the following code to create separate media directories for each author.. ( Thanks to https://wordpress.stackexchange.com/a/400352/220907 )

function per_user_upload_dir( $original ){
  $modified = $original;
  if ( is_user_logged_in() && current_user_can('instructor') ) {
      $current_user = wp_get_current_user();
      $subdir = 'user_docs/'.$current_user->user_login;
      $modified['subdir'] = $subdir;
      $modified['url'] = $original['baseurl'] . '/' . $subdir;
      $modified['path'] = $original['basedir'] . DIRECTORY_SEPARATOR . $subdir;
  }
  return $modified;
}
add_filter( 'upload_dir', 'per_user_upload_dir');

It works well, but the site dies with errors if we activate woocommerce

[03-Aug-2023 09:01:18 UTC] PHP Fatal error:  Uncaught Error: Call to undefined function is_user_logged_in() in /home/mydomain.com/data/www/mydomain/wp-content/plugins/!functions/my-functions.php:53
Stack trace:
#0 /home/mydomain.com/data/www/mydomain/wp-includes/class-wp-hook.php(308): per_user_upload_dir(Array)
#1 /home/mydomain.com/data/www/mydomain/wp-includes/plugin.php(205): WP_Hook->apply_filters(Array, Array)
#2 /home/mydomain.com/data/www/mydomain/wp-includes/functions.php(2363): apply_filters('upload_dir', Array)
#3 /home/mydomain.com/data/www/mydomain/wp-content/plugins/woocommerce/includes/class-woocommerce.php(305): wp_upload_dir(NULL, false)
#4 /home/mydomain.com/data/www/mydomain/wp-content/plugins/woocommerce/includes/class-woocommerce.php(183): WooCommerce->define_constants()
#5 /home/mydomain.com/data/www/mydomain/wp-content/plugins/woocommerce/includes/class-woocommerce.php(144): WooCommerce->__construct()
#6 /home/terraturis in /home/mydomain.com/data/www/mydomain/wp-content/plugins/!functions/my-functions.php on line 53

I would be grateful for any solution.

Thank you!

Improve this question
5
  • when you say include WooCommerce can you be more specific? Activating the plugin? Accessing a specific file? That function being undefined heavily implies that you're hitting a PHP file in a theme or plugin directly instead of using the normal methods. Have you asked this in a WooCommerce community? Questions about 3rd party plugins are offtopic here and not in this stacks scope
    – Tom J Nowell ♦
    Aug 4 at 13:42
  • For what it's worth, current_user_can('instructor') doesn't do anything. Even current_user_can('administrator') doesn't actually check if the user is an administrator. Admins can do anything. So if you put current_user_can('fight_a_gorilla_and_win') it will return true for administrators because they have permission to do anything. Check here: developer.wordpress.org/reference/functions/current_user_can and the list of capabilities is here: wordpress.org/documentation/article/roles-and-capabilities
    – Tony Djukic
    Aug 4 at 16:23
  • @TomJNowell This code works fine and completes the tasks, but there are problems after activating woocommerce. I am sorry for my English. I can't always translate thoughts correctly.
    – Andrei Ka
    Aug 4 at 18:23
  • @TonyDjukic current_user_can('instructor') doesn't matter, we can remove that. I gave an example from the original code. Error points to is_user_logged_in() I gave an example above Thanks
    – Andrei Ka
    Aug 4 at 18:28
  • are you making direct calls to PHP files in your theme/plugins using AJAX/forms/the browser? Have you raised this in a WooCommerce community? If this is related to WooCommerce then it's very likely the question will need to be closed, this isn't a place to get help with WooCommerce/3rd party plugins
    – Tom J Nowell ♦
    Aug 4 at 18:33

0

Reset to default

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.