Page MenuHomePhabricator

InviteSignup extension: Support PostgreSQL database
Closed, ResolvedPublic

Description

New file

extensions/InviteSignup/sql/invitesignup.pg.sql

-- Invite signup signups
CREATE TABLE /*$wgDBprefix*/invitesignup (
  is_inviter INTEGER  NOT NULL,
  is_invitee INTEGER,
  is_email   BYTEA    NOT NULL,
  is_when    BYTEA    NOT NULL,
  is_used    BYTEA,
  is_hash    BYTEA    NOT NULL,
  is_groups  BYTEA,

  PRIMARY KEY (is_hash)
) /*$wgDBTableOptions*/;

Update the hook for LoadExtensionSchemaUpdates in extensions/InviteSignup/InviteSignup.php

$wgHooks['LoadExtensionSchemaUpdates'][] = function ( DatabaseUpdater $updater ) {
	$dir = __DIR__ . '/sql';
	switch ( $updater->getDB()->getType() ) {
	case "mysql":
		$updater->addExtensionTable( 'invitesignup', "$dir/invitesignup.sql" );
		return true;
	case "postgres":
		$updater->addExtensionTable( 'invitesignup', "$dir/invitesignup.pg.sql" );
		return true;
	default:
		throw new MWException("InviteSignup does not support {$updater->getDB()->getType()} yet.");
	}

Related Objects

View Standalone Graph
This task is connected to more than 200 other tasks. Only direct parents and subtasks are shown here. Use View Standalone Graph to show more of the graph.

Event Timeline

Yoonghm created this task.Sep 14 2015, 1:15 AM
Yoonghm updated the task description. (Show Details)
Yoonghm raised the priority of this task from to Normal.
Yoonghm claimed this task.
Yoonghm added a subscriber: Yoonghm.
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptSep 14 2015, 1:15 AM

Thanks! It's easier if you can submit the patch yourself: https://www.mediawiki.org/wiki/Gerrit/Tutorial

In the worst case, you can also use a web interface: https://www.mediawiki.org/wiki/Gerrit_patch_uploader

Yoonghm added a comment.EditedSep 14 2015, 6:58 AM

I will add in the patch at home as my office blocks the port.

Aklapper renamed this task from To make InviteSignup extension to support PostgreSQL database to InviteSignup extension: Support PostgreSQL database.Sep 14 2015, 11:09 AM
Aklapper set Security to None.

I have updated the changes to feat-postgresql branch. I do not know if I need to push to master. Please advise.

Yes, you have to push to master.

Your patch is on master branch you just added a topic.

Change 238146 had a related patch set uploaded (by Paladox):
Changes to be committed: modified: InviteSignupHooks.php new file: sql/invitesignup.pg.sql

https://gerrit.wikimedia.org/r/238146

Change 238146 merged by jenkins-bot:
Add support for PostgreSQL database

https://gerrit.wikimedia.org/r/238146

Nikerabbit closed this task as Resolved.Feb 16 2016, 4:55 PM
Nikerabbit added a subscriber: Nikerabbit.
Jdforrester-WMF added a subscriber: Jdforrester-WMF.

Migrating from the old tracking task to a tag for PostgreSQL-related tasks.