Page MenuHomeMiraheze

Introduce CDB for CreateWiki/ManageWiki
Closed, DeclinedPublic

Description

Could introduce a CreateWikiCDB class which has shared code logic for implementing CDB across multiple code cases. Allowing us to have a base CDB set up for what cw_wikis, permissions, namespaces and any other modular code we want in the future.

Event Timeline

John triaged this task as Low priority.Sep 20 2018, 23:41
John created this task.
Paladox added a subscriber: Paladox.Sep 20 2018, 23:55

Working on this now, and the potential for this is immense.

It's allowing me to refractor a lot of my code to be more efficient, allows us to increase the efficiency of things like RemoteWiki and other helper functions we use.

Also it allows me to make CreateWiki work more out of the box.

This is definitely now a goal task for me.

John added a comment.Nov 13 2018, 05:30

https://git.io/fpmju is the code for this on a basic level.

Need to do;

  • utilise it!
  • remove permissions caching from MW after CreateWiki uses it's CDB version
  • move all.dblist processing to per wiki CDB
  • introduce a script that allows us to keep foreachwikiindblist functionality
  • then kill *.dblist after everything works.

The above is a requirement for ManageWikiNamespaces, it wasn't a requirement for ManageWikiPermissions though for a third party view now, it is. The old DBListGenerator method should remain supported - though potentially should look at killing MW support for it. Which makes this a requirement to use ManageWiki. Thoughts on the last part form anyone?

Paladox added a comment.Nov 13 2018, 12:47

@John what do you mean by killing DBListGenerator at some point?

What will replace it?

John added a comment.EditedNov 13 2018, 13:46

@John what do you mean by killing DBListGenerator at some point?
What will replace it?

I never said kill DBListGenerator.

And even if I did, this. This would replace it.

Paladox added a comment.Nov 13 2018, 14:40

@John you said “The old DBListGenerator method should remain supported - though potentially should look at killing MW support for it.” though which sounds like killing to me at some point?

John added a comment.Nov 13 2018, 16:20

Yes so it goes back to its old format, so in order to use the ManageWiki extension, you must be using CreateWiki’s CDB storage.

John closed this task as Declined.Nov 27 2018, 17:24

After testing, the implementation I've done is increasing page loads and isn't working with several extensions that are being super picky about the order things are loaded in. Therefore, this isn't going to happen and I'm going to instead focus on working on other tasks.