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.
StatusSubtypeAssignedTask
InvalidNone
ResolvedYoonghm

Event Timeline

Yoonghm claimed this task.
Yoonghm raised the priority of this task from to Medium.
Yoonghm updated the task description. (Show Details)
Yoonghm subscribed.

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

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

Jdforrester-WMF subscribed.

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