Working on error handling.

James Moger

2011-04-29 bc9d4a0f2266e5ac1a018f0b691c76d53479b9e5

Working on error handling.

 src/com/gitblit/Constants.java

@@ -19,7 +19,7 @@
    }



    public static String getJGitVersion() {

        return "JGit 0.11.3";

        return "JGit 0.12-stable";

    }



    public static String getRunningVersion() {


 src/com/gitblit/GitBlit.java

@@ -105,8 +105,7 @@
            r = repositoryResolver.open(null, repositoryName);

        } catch (RepositoryNotFoundException e) {

            r = null;

            logger.error("Failed to find repository " + repositoryName);

            e.printStackTrace();

            logger.error("GitBlit.getRepository(String) failed to find repository " + repositoryName);

        } catch (ServiceNotEnabledException e) {

            r = null;

            e.printStackTrace();

@@ -128,6 +127,7 @@
        Repository r = getRepository(repositoryName);

        RepositoryModel model = new RepositoryModel();

        model.name = repositoryName;

        model.hasCommits = JGitUtils.hasCommits(r);

        model.lastChange = JGitUtils.getLastChange(r);

        StoredConfig config = JGitUtils.readConfig(r);

        if (config != null) {


 src/com/gitblit/utils/JGitUtils.java

@@ -94,10 +94,10 @@
                File gitFolder = new File(file, Constants.DOT_GIT);

                boolean isGitRepository = gitFolder.exists() && gitFolder.isDirectory();

                

                // then look for folder.git/HEAD or folder/HEAD and folder/config

                // then look for folder.git/HEAD or folder/HEAD and

                // folder/config

                if (!isGitRepository) {

                    if ((file.getName().endsWith(Constants.DOT_GIT_EXT) && new File(file, Constants.HEAD).exists())

                            || (new File(file, "config").exists() && new File(file, Constants.HEAD).exists())) {

                    if ((file.getName().endsWith(Constants.DOT_GIT_EXT) && new File(file, Constants.HEAD).exists()) || (new File(file, "config").exists() && new File(file, Constants.HEAD).exists())) {

                        gitFolder = file;

                        isGitRepository = true;

                    }

@@ -124,6 +124,9 @@
    }



    public static RevCommit getFirstCommit(Repository r, String branch) {

        if (!hasCommits(r)) {

            return null;

        }

        if (StringUtils.isEmpty(branch)) {

            branch = Constants.HEAD;

        }

@@ -155,17 +158,24 @@
        return null;

    }



    public static boolean hasCommits(Repository r) {

        return new File(r.getDirectory(), Constants.R_HEADS).list().length > 0;

    }



    public static Date getLastChange(Repository r) {        

        RevCommit commit = getCommit(r, Constants.HEAD);

        if (commit == null) {

        if (!hasCommits(r)) {

            // fresh repository

            return new Date(r.getDirectory().lastModified());            

        }

        RevCommit commit = getCommit(r, Constants.HEAD);

        return getCommitDate(commit);

    }



    public static RevCommit getCommit(Repository r, String objectId) {

        RevCommit commit = null;

        if (!hasCommits(r)) {

            return null;

        }

        try {

            if (objectId == null || objectId.trim().length() == 0) {

                objectId = Constants.HEAD;

@@ -176,7 +186,7 @@
            commit = rev;

            walk.dispose();

        } catch (Throwable t) {

            LOGGER.error("Failed to determine last change", t);

            LOGGER.error("Failed to get commit " + objectId, t);

        }

        return commit;

    }

@@ -328,6 +338,10 @@


    public static List<PathChangeModel> getFilesInCommit(Repository r, RevCommit commit) {

        List<PathChangeModel> list = new ArrayList<PathChangeModel>();

        if (commit == null) {

            LOGGER.warn("getFilesInCommit for NULL commit");

            return list;

        }

        try {

            final RevWalk rw = new RevWalk(r);

            RevCommit parent = rw.parseCommit(commit.getParent(0).getId());

@@ -606,6 +620,9 @@


    public static List<RevCommit> getRevLog(Repository r, String objectId, String path, int offset, int maxCount) {

        List<RevCommit> list = new ArrayList<RevCommit>();

        if (!hasCommits(r)) {

            return list;

        }

        try {

            if (objectId == null || objectId.trim().length() == 0) {

                objectId = Constants.HEAD;

@@ -664,6 +681,9 @@
    public static List<RevCommit> searchRevlogs(Repository r, String objectId, String value, final SearchType type, int offset, int maxCount) {

        final String lcValue = value.toLowerCase();

        List<RevCommit> list = new ArrayList<RevCommit>();

        if (!hasCommits(r)) {

            return list;

        }

        try {

            if (objectId == null || objectId.trim().length() == 0) {

                objectId = Constants.HEAD;

@@ -792,13 +812,15 @@
    }



    public static List<Metric> getDateMetrics(Repository r) {

        Metric total = new Metric("TOTAL");

        final Map<String, Metric> metricMap = new HashMap<String, Metric>();

		
        if (hasCommits(r)) {

        final List<RefModel> tags = getTags(r, -1);

        final Map<ObjectId, RefModel> tagMap = new HashMap<ObjectId, RefModel>();

        for (RefModel tag : tags) {

            tagMap.put(tag.getCommitId(), tag);

        }

        Metric total = new Metric("TOTAL");

        final Map<String, Metric> metricMap = new HashMap<String, Metric>();

        try {

            RevWalk walk = new RevWalk(r);

            ObjectId object = r.resolve(Constants.HEAD);

@@ -837,6 +859,7 @@
        } catch (Throwable t) {

            LOGGER.error("Failed to mine log history for metrics", t);

        }

        }

        List<String> keys = new ArrayList<String>(metricMap.keySet());

        Collections.sort(keys);

        List<Metric> metrics = new ArrayList<Metric>();


 src/com/gitblit/wicket/BasePage.html

@@ -24,7 +24,6 @@
            <span>

                <a href="/"><span wicket:id="siteName">[site name]</span></a> / <span wicket:id="repositoryName">[repository name]</span> <span wicket:id="pageName">[page name]</span>

            </span>    

            <span wicket:id="feedback">[Feedback Panel]</span>

        </div>



        <!-- page content -->


 src/com/gitblit/wicket/BasePage.java

@@ -5,6 +5,7 @@
import javax.servlet.http.HttpServletRequest;



import org.apache.wicket.PageParameters;

import org.apache.wicket.RestartResponseAtInterceptPageException;

import org.apache.wicket.markup.html.WebPage;

import org.apache.wicket.markup.html.basic.Label;

import org.apache.wicket.markup.html.panel.FeedbackPanel;

@@ -50,8 +51,7 @@
        add(new FeedbackPanel("feedback"));



        // footer

        if (GitBlit.self().settings().getBoolean(Keys.web.authenticateViewPages, true)

                || GitBlit.self().settings().getBoolean(Keys.web.authenticateAdminPages, true)) {

        if (GitBlit.self().settings().getBoolean(Keys.web.authenticateViewPages, true) || GitBlit.self().settings().getBoolean(Keys.web.authenticateAdminPages, true)) {

            if (GitBlitWebSession.get().isLoggedIn()) {

                // logout

                add(new LinkPanel("userPanel", null, getString("gb.logout") + " " + GitBlitWebSession.get().getUser().toString(), LogoutPage.class));

@@ -78,8 +78,23 @@
        return req.getServerName();

    }



    public void error(String message, Throwable t) {

    public void error(String message, boolean redirect) {

        logger.error(message);

        if (redirect) {

            GitBlitWebSession.get().cacheErrorMessage(message);

            throw new RestartResponseAtInterceptPageException(getApplication().getHomePage());

        } else {

        super.error(message);

        }

    }



    public void error(String message, Throwable t, boolean redirect) {

        logger.error(message, t);

        if (redirect) {

            GitBlitWebSession.get().cacheErrorMessage(message);

            throw new RestartResponseAtInterceptPageException(getApplication().getHomePage());

        } else {

            super.error(message);

        }

    }

}


 src/com/gitblit/wicket/GitBlitWebApp.java

@@ -63,7 +63,7 @@
        mount(new MixedParamUrlCodingStrategy("/commitdiff", CommitDiffPage.class, new String[] { "r", "h" }));

        mount(new MixedParamUrlCodingStrategy("/patch", PatchPage.class, new String[] { "r", "h", "f" }));

        mount(new MixedParamUrlCodingStrategy("/history", HistoryPage.class, new String[] { "r", "h", "f" }));

        mount(new MixedParamUrlCodingStrategy("/search", SearchPage.class, new String[] { "r", "h", "a", "c" }));

        mount(new MixedParamUrlCodingStrategy("/search", SearchPage.class, new String[] { }));



        // setup ticket urls

        mount(new MixedParamUrlCodingStrategy("/tickets", TicketsPage.class, new String[] { "r" }));


 src/com/gitblit/wicket/GitBlitWebApp.properties

@@ -76,3 +76,4 @@
gb.description = description

gb.enableTickets = enable tickets

gb.enableDocs = enable docs
gb.save = save

 src/com/gitblit/wicket/GitBlitWebSession.java

@@ -15,6 +15,8 @@


    private User user = null;



    private String errorMessage = null;



    public GitBlitWebSession(Request request) {

        super(request);

    }

@@ -54,6 +56,16 @@
        return timezone;

    }



    public void cacheErrorMessage(String message) {

        this.errorMessage = message;

    }

	
    public String clearErrorMessage() {

        String msg = errorMessage;

        errorMessage = null;

        return msg;

    }



    public static GitBlitWebSession get() {

        return (GitBlitWebSession) Session.get();

    }


 src/com/gitblit/wicket/LoginPage.html

@@ -32,7 +32,7 @@
                    <input type="password"  wicket:id="password" value=""/>

                    <p/>

                    <input type="submit" value="Login" wicket:message="value:gb.login" />

                    <div style="background-color:#c7c7c7" wicket:id="feedback"></div>

                    <div style="padding-top:10px;" wicket:id="feedback"></div>

                </div>

            </form>

        </div>


 src/com/gitblit/wicket/RepositoryPage.html

@@ -22,6 +22,8 @@
            </div>

        </div>

    

        <div style="text-align:center;" wicket:id="feedback">[Feedback Panel]</div>

	
        <!-- page content -->

        <wicket:child />

        


 src/com/gitblit/wicket/RepositoryPage.java

@@ -1,5 +1,6 @@
package com.gitblit.wicket;



import java.text.MessageFormat;

import java.util.ArrayList;

import java.util.Arrays;

import java.util.HashMap;

@@ -70,14 +71,17 @@


    public RepositoryPage(PageParameters params) {

        super(params);

        if (!params.containsKey("r")) {

            error("Repository not specified!");

            redirectToInterceptPage(new RepositoriesPage());

        }

        repositoryName = WicketUtils.getRepositoryName(params);

        objectId = WicketUtils.getObject(params);



        if (StringUtils.isEmpty(repositoryName)) {

            error(MessageFormat.format("Repository not specified for {0}!", getPageName()), true);

        }



        Repository r = getRepository();

        if (r == null) {

            error(MessageFormat.format("Failed to open repository {0} for {1}!", repositoryName, getPageName()), true);

        }



        // standard page links

        add(new BookmarkablePageLink<Void>("summary", SummaryPage.class, WicketUtils.newRepositoryParameter(repositoryName)));

@@ -162,6 +166,14 @@
        return m;

    }



    protected RevCommit getCommit() {

        RevCommit commit = JGitUtils.getCommit(r, objectId);

        if (commit == null) {

            error(MessageFormat.format("Failed to find commit \"{0}\" in {1} for {2} page!", objectId, repositoryName, getPageName()), true);

        }

        return commit;

    }



    protected void addRefs(Repository r, RevCommit c) {

        add(new RefsPanel("refsPanel", repositoryName, c, JGitUtils.getAllRefs(r)));

    }


 src/com/gitblit/wicket/models/RepositoryModel.java

@@ -11,6 +11,7 @@
    public String owner;

    public String group;

    public Date lastChange;

    public boolean hasCommits;

    public boolean useTickets;

    public boolean useDocs;

    public boolean useRestrictedAccess;


 src/com/gitblit/wicket/pages/BlobDiffPage.java

@@ -25,7 +25,7 @@
        final String baseObjectId = WicketUtils.getBaseObjectId(params);



        Repository r = getRepository();

        RevCommit commit = JGitUtils.getCommit(r, objectId);

        RevCommit commit = getCommit();

        

        DiffOutputType diffType = DiffOutputType.forName(GitBlit.self().settings().getString(Keys.web.diffStyle, DiffOutputType.GITBLIT.name()));




 src/com/gitblit/wicket/pages/BlobPage.java

@@ -41,7 +41,7 @@
        

        // standard blob view

        Repository r = getRepository();

        RevCommit commit = JGitUtils.getCommit(r, objectId);

        RevCommit commit = getCommit();



        // blob page links

        add(new Label("blameLink", getString("gb.blame")));


 src/com/gitblit/wicket/pages/CommitDiffPage.java

@@ -29,7 +29,7 @@
        super(params);



        Repository r = getRepository();

        RevCommit commit = JGitUtils.getCommit(r, objectId);

        RevCommit commit = getCommit();

        DiffOutputType diffType = DiffOutputType.forName(GitBlit.self().settings().getString(Keys.web.diffStyle, DiffOutputType.GITBLIT.name()));

        String diff = JGitUtils.getCommitDiff(r, commit, diffType);




 src/com/gitblit/wicket/pages/CommitPage.java

@@ -28,7 +28,7 @@
        super(params);



        Repository r = getRepository();

        RevCommit c = JGitUtils.getCommit(r, objectId);

        RevCommit c = getCommit();



        List<String> parents = new ArrayList<String>();

        if (c.getParentCount() > 0) {


 src/com/gitblit/wicket/pages/EditRepositoryPage.html

@@ -9,6 +9,8 @@
    <!-- Push content down to preserve header image -->

    <div style="padding-top:20px"></div>

    

    <div style="text-align:center;" wicket:id="feedback">[Feedback Panel]</div>	


    <!-- Repository Table -->

    <form wicket:id="editForm">

        <table class="plain">

@@ -19,10 +21,11 @@
                <tr><th><wicket:message key="gb.group"></wicket:message></th><td class="edit"><input type="text" wicket:id="group" size="30" tabindex="4" /></td></tr>

                <tr><th><wicket:message key="gb.enableTickets"></wicket:message></th><td class="edit"><input type="checkbox" wicket:id="useTickets" tabindex="5" /> &nbsp;<i>distributed Ticgit issues</i></td></tr>

                <tr><th><wicket:message key="gb.enableDocs"></wicket:message></th><td class="edit"><input type="checkbox" wicket:id="useDocs" tabindex="6" /> &nbsp;<i>enumerates repository Markdown documentation</i></td></tr>

                <tr><td class="edit" colspan="2"><input type="submit" value="Submit" tabindex="7" /></td></tr>

                <tr><td class="edit" colspan="2"><input type="submit" value="Save" wicket:message="value:gb.save" tabindex="7" /></td></tr>

            </tbody>

        </table>

    </form>    



</wicket:extend>

</body>

</html>

 src/com/gitblit/wicket/pages/PatchPage.java

@@ -9,6 +9,7 @@
import com.gitblit.GitBlit;

import com.gitblit.utils.JGitUtils;

import com.gitblit.utils.StringUtils;

import com.gitblit.wicket.GitBlitWebSession;

import com.gitblit.wicket.WicketUtils;



public class PatchPage extends WebPage {

@@ -17,9 +18,11 @@
        super(params);



        if (!params.containsKey("r")) {

            error("Repository not specified!");

            GitBlitWebSession.get().cacheErrorMessage("Repository not specified!");

            redirectToInterceptPage(new RepositoriesPage());

            return;

        }

		
        final String repositoryName = WicketUtils.getRepositoryName(params);

        final String baseObjectId = WicketUtils.getBaseObjectId(params);

        final String objectId = WicketUtils.getObject(params);

@@ -27,12 +30,18 @@


        Repository r = GitBlit.self().getRepository(repositoryName);

        if (r == null) {

            error("Can not load repository " + repositoryName);

            GitBlitWebSession.get().cacheErrorMessage("Can not load repository " + repositoryName);

            redirectToInterceptPage(new RepositoriesPage());

            return;

        }



        RevCommit commit = JGitUtils.getCommit(r, objectId);

        if (commit == null) {

            GitBlitWebSession.get().cacheErrorMessage("Commit is null");

            redirectToInterceptPage(new RepositoriesPage());

            return;

        }

		
        String patch;

        if (StringUtils.isEmpty(baseObjectId)) {

            patch = JGitUtils.getCommitPatch(r, commit, blobPath);


 src/com/gitblit/wicket/pages/RepositoriesPage.html

@@ -13,6 +13,8 @@
<wicket:extend>

    <div wicket:id="adminPanel">[admin links]</div>

    

    <div style="text-align:center;padding-top:20px;" wicket:id="feedback">[Feedback Panel]</div>

	
    <div class="markdown" wicket:id="repositoriesMessage">[repositories message]</div>

        

    <table class="repositories">

@@ -46,6 +48,7 @@
    <wicket:fragment wicket:id="repositoryAdminLinks">

        <span class="link"><a wicket:id="editRepository"><wicket:message key="gb.edit">[edit]</wicket:message></a> | <a wicket:id="renameRepository"><wicket:message key="gb.rename">[rename]</wicket:message></a> | <a wicket:id="deleteRepository"><wicket:message key="gb.delete">[delete]</wicket:message></a></span>

    </wicket:fragment>

	
</wicket:extend>

</body>

</html>

 src/com/gitblit/wicket/pages/RepositoriesPage.java

@@ -52,6 +52,13 @@
        adminLinks.add(new BookmarkablePageLink<Void>("newUser", RepositoriesPage.class));

        add(adminLinks.setVisible(showAdmin));



        // display an error message cached from a redirect

        String cachedMessage = GitBlitWebSession.get().clearErrorMessage();

        if (!StringUtils.isEmpty(cachedMessage)) {

            error(cachedMessage);

            System.out.println("displayed message");

        }

		
        // Load the markdown welcome message

        String messageSource = GitBlit.self().settings().getString(Keys.web.repositoriesMessage, "gitblit");

        String message = "";

@@ -64,7 +71,7 @@
                message = StringUtils.transformMarkdown(reader);

            } catch (Throwable t) {

                message = "Failed to read default welcome message!";

                error(message, t);

                error(message, t, false);

            }

        } else {

            // Read user-supplied welcome message

@@ -76,7 +83,7 @@
                        message = StringUtils.transformMarkdown(reader);

                    } catch (Throwable t) {

                        message = "Failed to read " + file;

                        error(message, t);

                        error(message, t, false);

                    }

                } else {

                    message = messageSource + " is not a valid file.";

@@ -97,9 +104,16 @@


            public void populateItem(final Item<RepositoryModel> item) {

                final RepositoryModel entry = item.getModelObject();

                if (entry.hasCommits) {

                    // Existing repository

                PageParameters pp = WicketUtils.newRepositoryParameter(entry.name);

                item.add(new LinkPanel("repositoryName", "list", entry.name, SummaryPage.class, pp));

                item.add(new LinkPanel("repositoryDescription", "list", entry.description, SummaryPage.class, pp));

                } else {

                    // New repository

                    item.add(new Label("repositoryName", entry.name + "<span class='empty'>(empty)</span>").setEscapeModelStrings(false));

                    item.add(new Label("repositoryDescription", entry.description));					
                }



                if (entry.useTickets) {

                    item.add(WicketUtils.newImage("ticketsIcon", "bug_16x16.png", getString("gb.tickets")));


 src/com/gitblit/wicket/pages/TagPage.java

@@ -19,8 +19,8 @@
        super(params);



        Repository r = getRepository();

        RevCommit c = getCommit();

        List<RefModel> tags = JGitUtils.getTags(r, -1);

        RevCommit c = JGitUtils.getCommit(r, objectId);



        RefModel tagRef = null;

        // determine tag


 src/com/gitblit/wicket/pages/TreePage.java

@@ -30,7 +30,7 @@
        final String path = WicketUtils.getPath(params);



        Repository r = getRepository();

        RevCommit commit = JGitUtils.getCommit(r, objectId);

        RevCommit commit = getCommit();

        List<PathModel> paths = JGitUtils.getFilesInPath(r, path, commit);



        // tree page links


 src/com/gitblit/wicket/panels/CommitHeaderPanel.java

@@ -1,7 +1,5 @@
package com.gitblit.wicket.panels;



import java.util.Date;



import org.apache.wicket.markup.html.basic.Label;

import org.eclipse.jgit.revwalk.RevCommit;



@@ -15,9 +13,9 @@


    public CommitHeaderPanel(String id, String repositoryName, RevCommit c) {

        super(id);

        add(new LinkPanel("shortmessage", "title", c == null ? "" : c.getShortMessage(), CommitPage.class, WicketUtils.newObjectParameter(repositoryName, c == null ? "" : c.getName())));

        add(new LinkPanel("shortmessage", "title", c.getShortMessage(), CommitPage.class, WicketUtils.newObjectParameter(repositoryName, c.getName())));

        add(new Label("commitid", "(" + c.getName().substring(0, 8) + ")"));        

        add(new Label("author", c == null ? "" : c.getAuthorIdent().getName()));

        add(WicketUtils.createDateLabel("date", c == null ? new Date(0) : c.getAuthorIdent().getWhen(), getTimeZone()));

        add(new Label("author", c.getAuthorIdent().getName()));

        add(WicketUtils.createDateLabel("date", c.getAuthorIdent().getWhen(), getTimeZone()));

    }

}

 src/com/gitblit/wicket/panels/SearchPanel.java

@@ -55,7 +55,7 @@
        hasMore = commits.size() >= itemsPerPage;



        // header

        add(new LinkPanel("header", "title", commit.getShortMessage(), CommitPage.class, WicketUtils.newObjectParameter(repositoryName, commit.getName())));

        add(new LinkPanel("header", "title", commit == null ? "":commit.getShortMessage(), CommitPage.class, WicketUtils.newObjectParameter(repositoryName, commit == null ? "":commit.getName())));



        ListDataProvider<RevCommit> dp = new ListDataProvider<RevCommit>(commits);

        DataView<RevCommit> searchView = new DataView<RevCommit>("commit", dp) {


 src/com/gitblit/wicket/resources/gitblit.css

@@ -100,6 +100,13 @@
    padding: 0px 0px 15px 5px;

}



span.empty {

    font-size: 0.9em;

    font-style: italic;

    padding-left:10px;

    color: #008000;

}



span.link, span.link a {

    font-family: sans-serif;

    font-size: 11px;

@@ -634,7 +641,6 @@
    color: red;

    list-style-image: url(bullet_error.png);

    font-weight: bold;

    text-align:left;

    vertical-align: top;

    padding:0;    

    margin:0;    

@@ -644,7 +650,6 @@
    color: green;

    list-style: none;

    font-weight: bold;

    text-align:center;

    padding:0;

    margin:0;

}