Skip to:

Opened 13 years ago

Closed 13 years ago

#3179 closed defect (bug) (fixed)

bp_blogs_record_existing_blogs() should run during MS upgrade routine

Reported by: boonebgorges's profile boonebgorges Owned by:
Milestone: 1.5 Priority: major
Severity: Version: 1.5
Component: Blogs Keywords:


It does not appear to do so at the moment.

Change History (3)

#1 @DJPaul
13 years ago

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?

#2 @boonebgorges
13 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.

#3 @boonebgorges
13 years ago

  • Resolution set to fixed
  • Status changed from new to closed

(In [4344]) Check to see that the blog setup routine has run on admin loads. Fixes #3179

Note: See TracTickets for help on using tickets.