attachments should store the WP uploads path that was configured when they were uploaded

[Denis-de-Bernardy]

When you upload an image, currently, the uploads path (defaults to wp-content/uploads) is not stored.

If you change this later on to something else, previously inserted galleries no longer work, among multitudes of other problems.

[azaozz]

You mean change the uploads directory without moving the previously uploaded files so there are two different upload directories?

It seems the more common user case is to move the uploads directory to a new location. In that case galleries continue to work properly (as long as the sub-dirs are kept intact) but the inserted images in older posts need to be corrected.

[Denis-de-Bernardy]

Replying to azaozz:

You mean change the uploads directory without moving the previously uploaded files so there are two different upload directories?

Pretty much so, yeah. Site is currently using wp-content/uploads, and I was looking into changing that to media. So one old upload directory, and a new one. If we store the "current" uploads directory (e.g. wp-content/uploads) as a meta, we fix this.

It seems the more common user case is to move the uploads directory to a new location. In that case galleries continue to work properly (as long as the sub-dirs are kept intact) but the inserted images in older posts need to be corrected.

Right. Hence the suggestion, which would kill two birds in one stone.

[dd32]

-1 to the ability to have 2 upload directories..

+1 (Like another ticket somewhere in WP) to change the url's in posts retroactively

[Denis-de-Bernardy]

I think everyone is totally agreeing on that. :-)

[Denis-de-Bernardy]

Or rather, what needs to be done is, we store the current value of get_option(upload_dir) or whatever the useful function is, over in post_meta for that particular attachment.

That way, if the folder is changed later on, we still have the correct url -- through wp_get_attachment_url() or whatever it is, which would then use the meta instead of the generic function.

[azaozz]

Replying to dd32:

+1 (Like another ticket somewhere in WP) to change the url's in posts retroactively

To do that properly at the moment a db backup has to be made, URLs replaced, then some old posts tested to confirm success and eventually the db restored on failure. Think that's best left for a plugin or for the user to do manually as it would be needed in relatively few cases.

Replying to Denis-de-Bernardy:

Or rather, what needs to be done is, we store the current value of get_option(upload_dir) ... That way, if the folder is changed later on, we still have the correct url

But that will return the wrong URL when the folder has been moved or renamed which seem to be the most common cases.

Perhaps we need a good "canonical" plugin that can handle the different cases when moving, copying, renaming, etc. the uploads directory.

[azaozz]

No patch.

[ericlewis]

This sounds like plugin (or WordPress systems administrator) territory. As this hasn't been a whisper on this in five years, I'm closing this out.

[Denis-de-Bernardy]

Replying to ericlewis:

Reopening because:

It seems the more common user case is to move the uploads directory to a new location. In that case galleries continue to work properly (as long as the sub-dirs are kept intact) but the inserted images in older posts need to be corrected.

Right. Hence the suggestion, which would kill two birds in one stone.

[ericlewis]

So the proposed solution is retroactive URL replacements of the old uploads directory to the new uploads directory?

Right now, we don't store the upload dir path in the database at all (unless you manually set the upload_path option, the UPLOADS constant, or utilize the upload_dir filter. So we would need to set that somewhere, like _upload_path, which would need a comparison on every load to trigger the content migration.

Something like

<?php
add_action( 'init', 'check_if_uploads_folder_moved' );
function check_if_uploads_folder_moved() {
        if ( wp_upload_dir() !=  get_option( '_upload_dir' ) ) {
                /* ... migrate post content ... */
                update_option( '_upload_dir', wp_upload_dir() );
        }
}

[azaozz]

Thinking this is best resolved with a rewrite rule rather than changes in the DB. In theory we can go through all existing posts and replace URLs to the old uploads directory. However that leaves out post_meta, options, etc. where plugins may have stored such URLs.

[chriscct7]

This seems toward plugin territory in terms of migration, but it could be trivially handled with a rewrite rule like @azaozz suggested instead of separate meta for each media item