Eliminate plugin name, stick with id and description

James Moger

2014-04-12 fad6b4a3d5f9a7fe7f7bced14e591478cf959bb3

Eliminate plugin name, stick with id and description

 src/main/java/com/gitblit/manager/GitblitManager.java

@@ -1270,12 +1270,12 @@
    }

    @Override
    public PluginRegistration lookupPlugin(String idOrName) {
        return pluginManager.lookupPlugin(idOrName);
    public PluginRegistration lookupPlugin(String pluginId) {
        return pluginManager.lookupPlugin(pluginId);
    }

    @Override
    public PluginRelease lookupRelease(String idOrName, String version) {
        return pluginManager.lookupRelease(idOrName, version);
    public PluginRelease lookupRelease(String pluginId, String version) {
        return pluginManager.lookupRelease(pluginId, version);
    }
}

 src/main/java/com/gitblit/manager/PluginManager.java

@@ -320,9 +320,9 @@
    }

    @Override
    public synchronized PluginRegistration lookupPlugin(String idOrName) {
    public synchronized PluginRegistration lookupPlugin(String pluginId) {
        for (PluginRegistration reg : getRegisteredPlugins()) {
            if (reg.id.equalsIgnoreCase(idOrName) || reg.name.equalsIgnoreCase(idOrName)) {
            if (reg.id.equalsIgnoreCase(pluginId)) {
                return reg;
            }
        }
@@ -330,8 +330,8 @@
    }

    @Override
    public synchronized PluginRelease lookupRelease(String idOrName, String version) {
        PluginRegistration reg = lookupPlugin(idOrName);
    public synchronized PluginRelease lookupRelease(String pluginId, String version) {
        PluginRegistration reg = lookupPlugin(pluginId);
        if (reg == null) {
            return null;
        }

 src/main/java/com/gitblit/models/PluginRegistry.java

@@ -47,10 +47,9 @@
        }

    }



    public PluginRegistration lookup(String idOrName) {

    public PluginRegistration lookup(String id) {

        for (PluginRegistration registration : registrations) {

            if (registration.id.equalsIgnoreCase(idOrName)

                    || registration.name.equalsIgnoreCase(idOrName)) {

            if (registration.id.equalsIgnoreCase(id)) {

                return registration;

            }

        }

@@ -74,8 +73,6 @@
        private static final long serialVersionUID = 1L;



        public final String id;



        public String name;



        public String description;




 src/main/java/com/gitblit/transport/ssh/commands/PluginDispatcher.java

@@ -73,7 +73,7 @@
        protected void asTable(List<PluginWrapper> list) {
            String[] headers;
            if (verbose) {
                String [] h = { "#", "Id", "Version", "State", "Path", "Provider"};
                String [] h = { "#", "Id", "Description", "Version", "Requires", "State", "Path", "Provider"};
                headers = h;
            } else {
                String [] h = { "#", "Id", "Version", "State", "Path"};
@@ -84,7 +84,7 @@
                PluginWrapper p = list.get(i);
                PluginDescriptor d = p.getDescriptor();
                if (verbose) {
                    data[i] = new Object[] { "" + (i + 1), d.getPluginId(), d.getVersion(), p.getPluginState(), p.getPluginPath(), d.getProvider() };
                    data[i] = new Object[] { "" + (i + 1), d.getPluginId(), null/*d.getDescription()*/, d.getVersion(), null/*d.getRequires()*/, p.getPluginState(), p.getPluginPath(), d.getProvider() };
                } else {
                    data[i] = new Object[] { "" + (i + 1), d.getPluginId(), d.getVersion(), p.getPluginState(), p.getPluginPath() };
                }
@@ -98,7 +98,7 @@
            for (PluginWrapper pw : list) {
                PluginDescriptor d = pw.getDescriptor();
                if (verbose) {
                    outTabbed(d.getPluginId(), d.getVersion(), pw.getPluginState(), pw.getPluginPath(), d.getProvider());
                    outTabbed(d.getPluginId(), null/*d.getDescription()*/, d.getVersion(), null/*d.getRequires()*/, pw.getPluginState(), pw.getPluginPath(), d.getProvider());
                } else {
                    outTabbed(d.getPluginId(), d.getVersion(), pw.getPluginState(), pw.getPluginPath());
                }
@@ -256,8 +256,9 @@

        protected String buildFieldTable(PluginWrapper pw, PluginRegistration reg) {
            final String id = pw == null ? reg.id : pw.getPluginId();
            final String name = reg == null ? "" : reg.name;
            final String version = pw == null ? "" : pw.getDescriptor().getVersion().toString();
            final String description = reg == null ? ""/*pw.getDescriptor().getDescription()*/ : reg.description;
            final String version = pw == null ? reg.getCurrentRelease().version : pw.getDescriptor().getVersion().toString();
            final String requires = pw == null ? reg.getCurrentRelease().requires : ""/*pw.getDescriptor().getRequires().toString()*/;
            final String provider = pw == null ? reg.provider : pw.getDescriptor().getProvider();
            final String registry = reg == null ? "" : reg.registry;
            final String path = pw == null ? "" : pw.getPluginPath();
@@ -277,10 +278,11 @@
            StringBuilder sb = new StringBuilder();
            sb.append("ID          : ").append(id).append('\n');
            sb.append("Version     : ").append(version).append('\n');
            sb.append("Requires    : ").append(requires).append('\n');
            sb.append("State       : ").append(state).append('\n');
            sb.append("Path        : ").append(path).append('\n');
            sb.append('\n');
            sb.append("Name        : ").append(name).append('\n');
            sb.append("Description : ").append(description).append('\n');
            sb.append("Provider    : ").append(provider).append('\n');
            sb.append("Project URL : ").append(projectUrl).append('\n');
            sb.append("Registry    : ").append(registry).append('\n');
@@ -451,17 +453,17 @@

        @Override
        protected boolean matches(String filter, PluginRegistration t) {
            return t.id.matches(filter) || t.name.matches(filter);
            return t.id.matches(filter) || (t.description != null && t.description.matches(filter));
        }

        @Override
        protected void asTable(List<PluginRegistration> list) {
            String[] headers;
            if (verbose) {
                String [] h = { "Id", "Name", "Description", "Installed", "Current", "Requires", "State", "Registry" };
                String [] h = { "Id", "Description", "Installed", "Current", "Requires", "State", "Registry" };
                headers = h;
            } else {
                String [] h = { "Id", "Name", "Installed", "Current", "Requires", "State" };
                String [] h = { "Id", "Installed", "Current", "Requires", "State" };
                headers = h;
            }
            Object[][] data = new Object[list.size()][];
@@ -469,9 +471,9 @@
                PluginRegistration p = list.get(i);
                PluginRelease curr = p.getCurrentRelease();
                if (verbose) {
                    data[i] = new Object[] {p.id, p.name, p.description, p.installedRelease, curr.version, curr.requires, p.getInstallState(), p.registry};
                    data[i] = new Object[] {p.id, p.description, p.installedRelease, curr.version, curr.requires, p.getInstallState(), p.registry};
                } else {
                    data[i] = new Object[] {p.id, p.name, p.installedRelease, curr.version, curr.requires, p.getInstallState()};
                    data[i] = new Object[] {p.id, p.installedRelease, curr.version, curr.requires, p.getInstallState()};
                }
            }

@@ -483,9 +485,9 @@
            for (PluginRegistration p : list) {
                PluginRelease curr = p.getCurrentRelease();
                if (verbose) {
                    outTabbed(p.id, p.name, p.description, p.installedRelease, curr.version, curr.requires, p.getInstallState(), p.provider, p.registry);
                    outTabbed(p.id, p.description, p.installedRelease, curr.version, curr.requires, p.getInstallState(), p.provider, p.registry);
                } else {
                    outTabbed(p.id, p.name, p.installedRelease, curr.version, curr.requires, p.getInstallState());
                    outTabbed(p.id, p.installedRelease, curr.version, curr.requires, p.getInstallState());
                }
            }
        }