package com.conveyal.gtfs.error;

import com.conveyal.gtfs.model.Entity;
import com.conveyal.gtfs.validator.model.Priority;

import java.io.Serializable;

/**
 * Represents an error encountered while loading or validating a GFTS feed.
 *
 * TODO de-implement Comparable and Serializable
 * TODO parameterize with the type of affected entity <T extends Entity>, with pseudo-entity for lines?
 */
public abstract class GTFSError implements Comparable<GTFSError>, Serializable {

    public final String file; // TODO GTFSTable enum? Or simply use class objects.
    public final long   line;
    public final String field;
    public final String affectedEntityId;
    public final String errorType;
    public final Priority priority;

    public GTFSError(String file, long line, String field, Priority priority) {
        this(file, line, field, priority, null);
    }

    public GTFSError(String file, long line, String field, Priority priority, String affectedEntityId) {
        this.file  = file;
        this.line  = line;
        this.field = field;
        this.affectedEntityId = affectedEntityId;
        this.errorType = this.getClass().getSimpleName();
        this.priority = priority;
    }

    /**
     * New constructor for SQL storage.
     */
    public GTFSError (String entityId) {
        this.file = null;
        this.line = -1;
        this.field = null;
        this.errorType = null;
        this.affectedEntityId = entityId;
        this.priority = Priority.UNKNOWN;
    }

    /**
     * @return the name of this class without a package, to be used as a unique identifier for the kind of error.
     * By putting all these classes in the same package we guarantee they have unique names.
     */
    public final String getErrorCode () {
        return this.getClass().getSimpleName();
    }

    /**
     * @return a Class object for the class of GTFS entity in which the error was found,
     *         which also implies a table in the GTFS feed.
     */
    public Class<? extends Entity> getAffectedEntityType () {
        return null;
    }

    public String getMessage() {
        return "no message";
    }

    public String getMessageWithContext() {
        StringBuilder sb = new StringBuilder();
        sb.append(file);
        sb.append(' ');
        if (line >= 0) {
            sb.append("line ");
            sb.append(line);
        } else {
            sb.append("(no line)");
        }
        if (field != null) {
            sb.append(", field '");
            sb.append(field);
            sb.append('\'');
        }
        sb.append(": ");
        sb.append(getMessage());
        return sb.toString();
    }

    /** must be comparable to put into mapdb */
    public int compareTo (GTFSError o) {
        if (this.file == null && o.file != null) return -1;
        else if (this.file != null && o.file == null) return 1;

        int file = this.file == null && o.file == null ? 0 : String.CASE_INSENSITIVE_ORDER.compare(this.file, o.file);
        if (file != 0) return file;
        int errorType = String.CASE_INSENSITIVE_ORDER.compare(this.errorType, o.errorType);
        if (errorType != 0) return errorType;
        int affectedEntityId = this.affectedEntityId == null && o.affectedEntityId == null ? 0 : String.CASE_INSENSITIVE_ORDER.compare(this.affectedEntityId, o.affectedEntityId);
        if (affectedEntityId != 0) return affectedEntityId;
        else return Long.compare(this.line, o.line);
    }

    @Override
    public String toString() {
        return "GTFSError: " + getMessageWithContext();
    }



}

@@ -687,24 +687,254 @@
  errorType;%0A
+    // NOTE: Do not remove this field. Though this field is somewhat redundant (since every instance of each class has%0A    // the same priority) we have old MapDB files around that contain serialized errors. They would all break.%0A
     public f

Add comment to explain reverted changes.