/*
 * Copyright (C) 2003 EBI, GRL
 * 
 * This library is free software; you can redistribute it and/or modify it under the terms of the GNU Lesser General Public License as published by
 * the Free Software Foundation; either version 2.1 of the License, or (at your option) any later version.
 * 
 * This library is distributed in the hope that it will be useful, but WITHOUT ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
 * FITNESS FOR A PARTICULAR PURPOSE. See the GNU Lesser General Public License for more details.
 * 
 * You should have received a copy of the GNU Lesser General Public License along with this library; if not, write to the Free Software Foundation,
 * Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
 */
package org.ensembl.healthcheck.testcase.generic;

import java.sql.Connection;
import java.sql.ResultSet;
import java.sql.SQLException;
import java.sql.Statement;

import org.ensembl.healthcheck.DatabaseRegistryEntry;
import org.ensembl.healthcheck.ReportManager;
import org.ensembl.healthcheck.testcase.Priority;
import org.ensembl.healthcheck.testcase.SingleDatabaseTestCase;

/**
 * Check that not all xrefs have the same identifier.
 */

public class XrefIdentifiers extends SingleDatabaseTestCase {

	/**
	 * Create a new XrefIdentifiers testcase.
	 */
	public XrefIdentifiers() {

		addToGroup("post_genebuild");
		addToGroup("release");
		addToGroup("core_xrefs");
		setDescription("Check that not all xrefs have the same identifier.");
		setPriority(Priority.AMBER);
		setEffect("Web display and all other uses of xrefs are broken");
		setFix("Re-import or recalculate xrefs");

	}

	/**
	 * Run the test.
	 * 
	 * @param dbre
	 *          The database to use.
	 * @return true if the test passed.
	 * 
	 */
	public boolean run(DatabaseRegistryEntry dbre) {

		boolean result = true;

		Connection con = dbre.getConnection();

		// TODO - do this on a per-xref-type basis
		try {

			Statement stmt = con.createStatement();

			ResultSet rs = stmt.executeQuery("SELECT COUNT(DISTINCT(dbprimary_acc)) FROM xref");

			rs.next();
			
			int count = rs.getInt(1);
			if (count == 1) {
				ReportManager.problem(this, con, "All xrefs appear to have the same identifier.");
				result = false;
			} else {
				ReportManager.correct(this, con, "Not all xrefs have the same identifier.");
			}

			stmt.close();

		} catch (SQLException e) {
			e.printStackTrace();
		}

		if (result) {

			ReportManager.correct(this, con, "All external dbs are only associated with one Ensembl object type");

		}

		return result;

	} // run

	// ----------------------------------------------------------------------

} // XrefIdentifiers

@@ -1191,17 +1191,76 @@
 ntifier.
+ Also check that there are%0A * no blank/null display_labels.
 %0A
-
  */%0A%0Apub
@@ -1949,16 +1949,252 @@
 ion();%0A%0A
+%09%09result &= checkSame(con);%0A%0A%09%09result &= checkBlank(con);%0A%0A%09%09return result;%0A%0A%09%7D // run%0A%0A%09// ----------------------------------------------------------------------%0A%0A%09private boolean checkSame(Connection con) %7B%0A%0A%09%09boolean result = true;%0A%0A
 %09%09// TOD
@@ -2390,12 +2390,9 @@
 ();%0A
-%09%09%09
 %0A
+
 %09%09%09i
@@ -2864,32 +2864,634 @@
  result;%0A%0A%09%7D
- // run%0A
+%0A%0A%09// ----------------------------------------------------------------------%0A%0A%09private boolean checkBlank(Connection con) %7B%0A%0A%09%09boolean result = true;%0A%0A%09%09String%5B%5D columns = %7B %22dbprimary_acc%22, %22display_label%22 %7D;%0A%0A%09%09for (String column : columns) %7B%0A%0A%09%09%09int rows = getRowCount(con, %22SELECT COUNT(*) FROM xref WHERE %22 + column + %22='' OR %22 + column + %22 IS NULL%22);%0A%0A%09%09%09if (rows %3E 0) %7B%0A%0A%09%09%09%09ReportManager.problem(this, con, rows + %22 xrefs have blank or null %22 + column + %22s%22);%0A%09%09%09%09result = false;%0A%0A%09%09%09%7D else %7B%0A%0A%09%09%09%09ReportManager.correct(this, con, %22No blank xref %22 + column + %22s%22);%0A%09%09%09%7D%0A%0A%09%09%7D%0A%09%09%0A%09%09return result;%0A%09%09%0A%09%7D%0A%09
 %0A%09// -------

Add check for xrefs that have null or blank dbprimary_acc or display_label.