Feature Request: support custom WP_CONTENT_DIR

  • Hello, thanks for your nice plugin – looks good, but unfortunately I cannot start using it, because custom WP_CONTENT_DIR and WP_CONTENT_URL is not supported.
    The WP_CONTENT_DIR constant via wp-config.php allows to rename the wp-content directory for security reasons (that of course must be really renamed in file system, too).
    Your plugins seems to use it partially, so the constant WPPA_CONTENT_PATH is already correct. But the constants WPPA_UPLOAD_PATH, WPPA_UPLOAD_URL, WPPA_DEPOT_PATH and WPPA_DEPOT_URL still use the hardcoded “wp-content” slug instead.
    Could you please take the path given by the function wp_upload_dir() via ['basedir'] (you already are listing in your CONSTANTS list) to generate your path constants and the ['baseurl'] for your url constants?

Viewing 10 replies - 1 through 10 (of 10 total)
  • Plugin Author Jacob N. Breetvelt

    (@opajaap)

    I will get into it. The reason is historically. About 7 years ago or so, wp_upload_dir():[basedir] just returned a wp option and contained garbage on older sites even when updated to the current wp rev.

    Are you willing to test version 8.1.00 pre-prelease? If so:
    1. it will be available tomorrow and i will put it in there.
    2. You will get priority in replying to questions.

    Thread Starter Fee

    (@wdfee)

    Great! Yes I can test the pre-release and give you feedback.

    Plugin Author Jacob N. Breetvelt

    (@opajaap)

    Is it a single site or a multi site?
    And: are the locations below ABSPATH?

    Thread Starter Fee

    (@wdfee)

    single site, yes all below ABSPATH

    Plugin Author Jacob N. Breetvelt

    (@opajaap)

    Download https://downloads.www.ads-software.com/plugin/wp-photo-album-plus.8.1.00.002.zip and install by dashboard menu -> Plugins -> Add New -> Upload plugin, and click the button that asks if you want to replace the current installed version.

    After install, go to Advanced settings -> System -> I -> Item 34: Use wp_upload_dir() and tick the checkbox.

    This should do it

    Thread Starter Fee

    (@wdfee)

    ok… I had to de- and reactivate the plugin again after changing this option, for the folders creation. Sure that the depot folder should be on the same level as uploads and not inside of it?

    Plugin Author Jacob N. Breetvelt

    (@opajaap)

    Yes, that is normal. The depot folder is used by ftp rather than http uploads, so its easyer to find with your ftp program like filezilla.

    Plugin Author Jacob N. Breetvelt

    (@opajaap)

    Pls go to Advanced settings -》Misc II item 8 And look if there are no errors or warnings saying Unsafe path detected

    Thread Starter Fee

    (@wdfee)

    I guess this one is from before the patch:

    Recursive log detected: Err Could not create /[…]/wp-content/uploads/wppa

    After deleting it, de- and reactivating the plugin, there are no error messages.

    Plugin Author Jacob N. Breetvelt

    (@opajaap)

    This message is because you changed the file locations. No issue.

    Please be aware of the following issue i had in the past (many years ago) and i do not know if that could be a potential issue:

    When your hosting provider changes the location of your installation, check if wp_upload_dir() : [basedir] returns the changed location.

Viewing 10 replies - 1 through 10 (of 10 total)
  • The topic ‘Feature Request: support custom WP_CONTENT_DIR’ is closed to new replies.