Make WordPress Core

Opened 14 years ago

Closed 14 years ago

#13002 closed defect (bug) (duplicate)

header image upload doesn't in MS mode

Reported by: pbearne's profile pbearne Owned by: ryan's profile ryan
Milestone: Priority: high
Severity: normal Version:
Component: Rewrite Rules Keywords:
Focuses: Cc:



If you try to change the header image on sub site on multi site blog

If you select an image from your desktop and upload you don't get an saved and therefore can't change the header see screen-shot

the path looks good I think it is the save that doesn't work

Attachments (1)

uploadedheader.jpg (30.2 KB) - added by pbearne 14 years ago.
upload header missing

Download all attachments as: .zip

Change History (8)

14 years ago

upload header missing

#1 follow-up: @ocean90
14 years ago

Works for me.
Is the image listed in the media library?

#2 in reply to: ↑ 1 @pbearne
14 years ago

Replying to ocean90:

Works for me.
Is the image listed in the media library?

where you testing on the root site or sub site ?

#3 @ocean90
14 years ago

pbearne: iI tested it on both. Is it your first image for upload? Is the image on your server? Exists the upload dir? (wp-content/blogs.dir/{ID}/files/{year}/{month})

#4 @pbearne
14 years ago


I have solved this

I upgraded this site from 2.9 mu to 3.0 beta so I had extiting rewrite rules that pointed to blogs.php not the new ms-files.php

and I done as instructed and removed blogs.php

"The wp-content/blogs.php file is deprecated. Please remove it and update your server rewrite rules to use wp-includes/ms-files.php instead."

This not enough instuctions as to what I need to change in the rewrite rules.
We need to add some words of what to change in the rewrite or better still code this as part of the upgrade script

this is the change I made to .htaccess
#RewriteRule (.*/)?files/(.*) wp-content/blogs.php?file=$2 [L]
(.*/)?files/(.*) wp-includes/ms-files.php?file=$2 [L]

I have rasied a bug for this #13072 as the IIS rules are wrong (still have blogs.php)

this change needs to be scripted otherwise it will create broken sites.

#5 @pbearne
14 years ago

  • Cc pbearne added

#6 @ocean90
14 years ago

  • Component changed from Themes to Rewrite Rules
  • Keywords twentyten removed
  • Owner set to ryan
  • Severity changed from major to normal

Ok, I think we should close this ticket as invalid and use #11742 for the admin notice.

#7 @nacin
14 years ago

  • Milestone 3.0 deleted
  • Resolution set to duplicate
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.