Opened 14 years ago
Closed 14 years ago
#3179 closed defect (bug) (fixed)
bp_blogs_record_existing_blogs() should run during MS upgrade routine
Reported by: | boonebgorges | Owned by: | |
---|---|---|---|
Milestone: | 1.5 | Priority: | major |
Severity: | Version: | 1.5 | |
Component: | Blogs | Keywords: | |
Cc: |
Description
It does not appear to do so at the moment.
Change History (3)
#2
@
14 years ago
It seems like it would be easy, but I just spent 90 minutes trying every possible permutation, and I simply could not get it to work. First, you need to have the whole of BP loaded in order for bp_blogs_record_existing_blogs() to work. But just loading this is not enough, because the blogs component has to be activated, which is not the case until after the wizard is totally finished.
I'm going to commit a fix that does a small check on admin loads to see whether the blogs table is empty (which it should never be). I know that this is not ideal. If anyone has a better idea, please feel free to reopen this ticket.
This looks like it should be as simple as adding bp_blogs_record_existing_blogs() in to the bottom of step_ms_update_save(). I'm not sure if the ticket is for any MS upgrade, or for a non-MS to MS upgrade; if the latter, I think checking if $this->is_network_activate is set would work?